commit 6c8512978af64592305e967c838d1b22d348e297
author Chong Zhang <chz@google.com> Wed Mar 14 18:56:56 2018 -0700
committer Chong Zhang <chz@google.com> Thu Mar 15 15:05:29 2018 -0700
tree ee63c71f05e71ebb22cf8ea9834e11da1ed91f3a
parent 4985fdef8bee4598df89f014a0886be24cd17fdb

heif: address api review comments

- renaming of MediaFormat keys related to grid config

- add methods to MediaMetadataRetriever to get image/frame
  with default bitmap config

- fix java doc

bug: 74831433

Change-Id: Iec607e615d34bea0620070592e4adcfc04cbccae

media/jni/android_media_MediaMetadataRetriever.cpp

diff --git a/media/jni/android_media_MediaMetadataRetriever.cpp b/media/jni/android_media_MediaMetadataRetriever.cpp
index 3a67142..4f6763e 100644
--- a/media/jni/android_media_MediaMetadataRetriever.cpp
+++ b/media/jni/android_media_MediaMetadataRetriever.cpp
@@ -449,13 +449,14 @@
     std::vector<sp<IMemory> > frames;
     status_t err = retriever->getFrameAtIndex(&frames, frameIndex, numFrames, colorFormat);
     if (err != OK || frames.size() == 0) {
-        ALOGE("failed to get frames from retriever, err=%d, size=%zu",
-                err, frames.size());
+        jniThrowException(env,
+                "java/lang/IllegalStateException", "No frames from retriever");
         return NULL;
     }
     jobject arrayList = env->NewObject(fields.arrayListClazz, fields.arrayListInit);
     if (arrayList == NULL) {
-        ALOGE("can't create bitmap array list object");
+        jniThrowException(env,
+                "java/lang/IllegalStateException", "Can't create bitmap array");
         return NULL;
     }