commit 4a07e80c6de85e2479fed21284c4500663745a54
author Amy Zhang <amyjojo@google.com> Thu Jan 21 17:10:21 2021 -0800
committer Amy Zhang <amyjojo@google.com> Fri Jan 22 10:20:26 2021 -0800
tree 8a49a5399909b24ae652a534e1e58995d8eeb49e
parent 9eeba43cfd8a4aa660d790f232ab7adf5abe39bd

Connect existed TunerFilter/TunerDemux AIDL apis to TunerClient

This CL also resolved some TODO to handle aidl error messages

Test: make
Bug: 159067322
Change-Id: I6da66d1fa3989a501ec58bf1e0553f85a13e18db

media/jni/tuner/FrontendClient.cpp

diff --git a/media/jni/tuner/FrontendClient.cpp b/media/jni/tuner/FrontendClient.cpp
index d6d64f6..ef8f57f 100644
--- a/media/jni/tuner/FrontendClient.cpp
+++ b/media/jni/tuner/FrontendClient.cpp
@@ -89,9 +89,8 @@
         // TODO: parse hidl settings to aidl settings
         // TODO: aidl frontend settings to include Tuner HAL 1.1 settings
         TunerFrontendSettings settings;
-        // TODO: handle error message.
-        mTunerFrontend->tune(settings);
-        return Result::SUCCESS;
+        Status s = mTunerFrontend->tune(settings);
+        return ClientHelper::getServiceSpecificErrorCode(s);
     }
 
     Result result;
@@ -110,9 +109,8 @@
 
 Result FrontendClient::stopTune() {
     if (mTunerFrontend != NULL) {
-        // TODO: handle error message.
-        mTunerFrontend->stopTune();
-        return Result::SUCCESS;
+        Status s = mTunerFrontend->stopTune();
+        return ClientHelper::getServiceSpecificErrorCode(s);
     }
 
     if (mFrontend != NULL) {
@@ -129,9 +127,8 @@
         // TODO: parse hidl settings to aidl settings
         // TODO: aidl frontend settings to include Tuner HAL 1.1 settings
         TunerFrontendSettings settings;
-        // TODO: handle error message.
-        mTunerFrontend->scan(settings, (int)type);
-        return Result::SUCCESS;
+        Status s = mTunerFrontend->scan(settings, (int)type);
+        return ClientHelper::getServiceSpecificErrorCode(s);
     }
 
     Result result;
@@ -150,9 +147,8 @@
 
 Result FrontendClient::stopScan() {
     if (mTunerFrontend != NULL) {
-        // TODO: handle error message.
-        mTunerFrontend->stopScan();
-        return Result::SUCCESS;
+        Status s = mTunerFrontend->stopScan();
+        return ClientHelper::getServiceSpecificErrorCode(s);
     }
 
     if (mFrontend != NULL) {
@@ -284,8 +280,8 @@
 Result FrontendClient::close() {
     if (mTunerFrontend != NULL) {
         // TODO: handle error message.
-        mTunerFrontend->close();
-        return Result::SUCCESS;
+        Status s = mTunerFrontend->close();
+        return ClientHelper::getServiceSpecificErrorCode(s);
     }
 
     if (mFrontend != NULL) {