Merge "Adds additional UiEvent logs for LongScreenshotActivity" into sc-qpr1-dev

This commit is contained in:
Mark Renouf
2021-08-21 02:25:09 +00:00
committed by Android (Google) Code Review
2 changed files with 12 additions and 1 deletions

View File

@@ -154,6 +154,7 @@ public class LongScreenshotActivity extends Activity {
@Override
public void onStart() {
super.onStart();
mUiEventLogger.log(ScreenshotEvent.SCREENSHOT_LONG_SCREENSHOT_ACTIVITY_STARTED);
if (mPreview.getDrawable() != null) {
// We already have an image, so no need to try to load again.
@@ -245,6 +246,8 @@ public class LongScreenshotActivity extends Activity {
}
private void onCachedImageLoaded(ImageLoader.Result imageResult) {
mUiEventLogger.log(ScreenshotEvent.SCREENSHOT_LONG_SCREENSHOT_ACTIVITY_CACHED_IMAGE_LOADED);
BitmapDrawable drawable = new BitmapDrawable(getResources(), imageResult.bitmap);
mPreview.setImageDrawable(drawable);
mPreview.setAlpha(1f);
@@ -282,6 +285,8 @@ public class LongScreenshotActivity extends Activity {
finish();
}
if (isFinishing()) {
mUiEventLogger.log(ScreenshotEvent.SCREENSHOT_LONG_SCREENSHOT_ACTIVITY_FINISHED);
if (mScrollCaptureResponse != null) {
mScrollCaptureResponse.close();
}

View File

@@ -77,7 +77,13 @@ public enum ScreenshotEvent implements UiEventLogger.UiEventEnum {
@UiEvent(doc = "The long screenshot capture failed")
SCREENSHOT_LONG_SCREENSHOT_FAILURE(881),
@UiEvent(doc = "The long screenshot capture completed successfully")
SCREENSHOT_LONG_SCREENSHOT_COMPLETED(882);
SCREENSHOT_LONG_SCREENSHOT_COMPLETED(882),
@UiEvent(doc = "Long screenshot editor activity started")
SCREENSHOT_LONG_SCREENSHOT_ACTIVITY_STARTED(889),
@UiEvent(doc = "Long screenshot editor activity loaded a previously saved screenshot")
SCREENSHOT_LONG_SCREENSHOT_ACTIVITY_CACHED_IMAGE_LOADED(890),
@UiEvent(doc = "Long screenshot editor activity finished")
SCREENSHOT_LONG_SCREENSHOT_ACTIVITY_FINISHED(891);
private final int mId;