commit ff2a495cc65881caa25bae520f1249e92ceb9ac6
author Yin-Chia Yeh <yinchiayeh@google.com> Sat Apr 02 16:31:57 2016 -0700
committer Yin-Chia Yeh <yinchiayeh@google.com> Thu Apr 07 10:48:06 2016 -0700
tree 29c117fd813491d1226ebebf9259d0073dec18df
parent 59a2d786854012e4c52abac263bd751f286fbdfd

CameraNDK: hide more NDK metadata tags

- Codegen update: some of hidden metadata keys should not be
disclosed in NDK.
- Also added capture result frame number plumbing.

Bug: 27102995
Change-Id: Ia238f95a4da726330205c53fc5d3f1f63d7553a1

camera/ndk/impl/ACameraDevice.cpp

diff --git a/camera/ndk/impl/ACameraDevice.cpp b/camera/ndk/impl/ACameraDevice.cpp
index 6bca692..1a1118c 100644
--- a/camera/ndk/impl/ACameraDevice.cpp
+++ b/camera/ndk/impl/ACameraDevice.cpp
@@ -1185,8 +1185,8 @@
     }
 
     CameraMetadata metadataCopy = metadata;
-    // Copied from java implmentation. Why do we need this?
     metadataCopy.update(ANDROID_LENS_INFO_SHADING_MAP_SIZE, dev->mShadingMapSize, /*data_count*/2);
+    metadataCopy.update(ANDROID_SYNC_FRAME_NUMBER, &frameNumber, /*data_count*/1);
 
     auto it = dev->mSequenceCallbackMap.find(sequenceId);
     if (it != dev->mSequenceCallbackMap.end()) {

camera/ndk/impl/ACameraMetadata.cpp

diff --git a/camera/ndk/impl/ACameraMetadata.cpp b/camera/ndk/impl/ACameraMetadata.cpp
index 8366ade..ab99e38 100644
--- a/camera/ndk/impl/ACameraMetadata.cpp
+++ b/camera/ndk/impl/ACameraMetadata.cpp
@@ -294,7 +294,6 @@
         case ACAMERA_LENS_FOCUS_DISTANCE:
         case ACAMERA_LENS_OPTICAL_STABILIZATION_MODE:
         case ACAMERA_NOISE_REDUCTION_MODE:
-        case ACAMERA_REQUEST_ID:
         case ACAMERA_SCALER_CROP_REGION:
         case ACAMERA_SENSOR_EXPOSURE_TIME:
         case ACAMERA_SENSOR_FRAME_DURATION:
@@ -311,9 +310,7 @@
         case ACAMERA_TONEMAP_MODE:
         case ACAMERA_TONEMAP_GAMMA:
         case ACAMERA_TONEMAP_PRESET_CURVE:
-        case ACAMERA_LED_TRANSMIT:
         case ACAMERA_BLACK_LEVEL_LOCK:
-        case ACAMERA_REPROCESS_EFFECTIVE_EXPOSURE_FACTOR:
             return true;
         default:
             return false;