Fix placeholder bug.

Bug: 18947927
Change-Id: I08542d614b15f31398150ea52caa6055ec63ccfb

diff --git a/src/com/android/camera/processing/imagebackend/TaskCompressImageToJpeg.java b/src/com/android/camera/processing/imagebackend/TaskCompressImageToJpeg.java
index 596332e..3878bcc 100644 (file)
--- a/src/com/android/camera/processing/imagebackend/TaskCompressImageToJpeg.java
+++ b/src/com/android/camera/processing/imagebackend/TaskCompressImageToJpeg.java
@@ -72,7 +72,6 @@ public class TaskCompressImageToJpeg extends TaskJpegEncode {
                 ImageFormat.JPEG);
 
         onStart(mId, inputImage, resultImage);
-        mSession.startEmpty();
         logWrapper("TIMER_END Full-size YUV buffer available, w=" + img.proxy.getWidth() + " h="
                 + img.proxy.getHeight() + " of format " + img.proxy.getFormat()
                 + " (35==YUV_420_888)");
@@ -96,6 +95,8 @@ public class TaskCompressImageToJpeg extends TaskJpegEncode {
 
         onJpegEncodeDone(mId, inputImage, resultImage, writeOut);
 
+        // TODO: the app actually crashes here on a race condition: TaskCompressImageToJpeg might
+        // complete before TaskConvertImageToRGBPreview.
         mSession.saveAndFinish(writeOut, resultImage.width, resultImage.height,
                 resultImage.orientation.getDegrees(), createExif(resultImage),
                 new MediaSaver.OnMediaSavedListener() {