commit 5f9753d770549db9c65e4e13c1e297a7f5fd6791
author Derek Sollenberger <djsollen@google.com> Wed Apr 01 15:59:02 2020 -0400
committer Derek Sollenberger <djsollen@google.com> Tue Apr 07 11:26:52 2020 -0400
tree 2b7393a7f3867fde68342d5d098f79c2b59dd0be
parent 2728fa3c52526c64622a912fdebb9d1229e8969d

Don't reset the capture mode after each frame when using the CallbackAPI

When multiframe skp support was introduced it set the capture mode to
none for everything but the multiframe use case. This was incorrect in
the case of the CallbackAPI where we want to continue to capture frames.

Bug: 152084866
Test: hwui_unit_tests
Change-Id: I7e3caf41a3725d03252df1ab7bedfe1b69a238c2

libs/hwui/pipeline/skia/SkiaPipeline.cpp

diff --git a/libs/hwui/pipeline/skia/SkiaPipeline.cpp b/libs/hwui/pipeline/skia/SkiaPipeline.cpp
index 41aa1ff..5088494 100644
--- a/libs/hwui/pipeline/skia/SkiaPipeline.cpp
+++ b/libs/hwui/pipeline/skia/SkiaPipeline.cpp
@@ -418,9 +418,9 @@
                 auto data = picture->serialize();
                 savePictureAsync(data, mCapturedFile);
                 mCaptureSequence = 0;
+                mCaptureMode = CaptureMode::None;
             }
         }
-        mCaptureMode = CaptureMode::None;
         mRecorder.reset();
     }
 }