Snap for 8892748 from 2d71a01c12a948478c61c1ef841bce4fcd5343d5 to tm-qpr1-release (58889624) · Commits · e / os / android_frameworks_base

core/java/android/app/Activity.java

+1 −1

Original line number	Diff line number	Diff line
		@@ -308,7 +308,7 @@ import java.util.function.Consumer;
		* <li>The <b>foreground lifetime</b> of an activity happens between a call to
		* {@link android.app.Activity#onResume} until a corresponding call to
		* {@link android.app.Activity#onPause}. During this time the activity is
		* in visible, active and interacting with the user. An activity
		* visible, active and interacting with the user. An activity
		* can frequently go between the resumed and paused states -- for example when
		* the device goes to sleep, when an activity result is delivered, when a new
		* intent is delivered -- so the code in these methods should be fairly

core/java/android/hardware/camera2/impl/CameraExtensionSessionImpl.java

+4 −2

Original line number	Diff line number	Diff line
		@@ -1693,8 +1693,10 @@ public final class CameraExtensionSessionImpl extends CameraExtensionSession {
		((i != idx) \|\| notifyCurrentIndex)) {
		TotalCaptureResult result = previewMap.valueAt(i).second;
		Long timestamp = result.get(CaptureResult.SENSOR_TIMESTAMP);
		if (mCaptureResultHandler != null) {
		mCaptureResultHandler.onCaptureCompleted(timestamp,
		initializeFilteredResults(result));
		}

		Log.w(TAG, "Preview frame drop with timestamp: " + previewMap.keyAt(i));
		final long ident = Binder.clearCallingIdentity();

core/java/android/view/contentcapture/MainContentCaptureSession.java

+6 −1

Original line number	Diff line number	Diff line
		@@ -64,6 +64,7 @@ import java.lang.ref.WeakReference;
		import java.util.ArrayList;
		import java.util.Collections;
		import java.util.List;
		import java.util.NoSuchElementException;
		import java.util.concurrent.atomic.AtomicBoolean;

		/**
		@@ -630,7 +631,11 @@ public final class MainContentCaptureSession extends ContentCaptureSession {
		mComponentName = null;
		mEvents = null;
		if (mDirectServiceInterface != null) {
		try {
		mDirectServiceInterface.asBinder().unlinkToDeath(mDirectServiceVulture, 0);
		} catch (NoSuchElementException e) {
		Log.w(TAG, "IContentCaptureDirectManager does not exist");
		}
		}
		mDirectServiceInterface = null;
		mHandler.removeMessages(MSG_FLUSH);

core/java/com/android/internal/util/ScreenshotHelper.java

+80 −120

Original line number	Diff line number	Diff line
		package com.android.internal.util;

		import static android.content.Intent.ACTION_USER_SWITCHED;
		import static android.view.WindowManager.ScreenshotSource.SCREENSHOT_OTHER;

		import android.annotation.NonNull;
		import android.annotation.Nullable;
		@@ -29,6 +28,10 @@ import android.os.RemoteException;
		import android.os.UserHandle;
		import android.util.Log;
		import android.view.WindowManager;
		import android.view.WindowManager.ScreenshotSource;
		import android.view.WindowManager.ScreenshotType;

		import com.android.internal.annotations.VisibleForTesting;

		import java.util.Objects;
		import java.util.function.Consumer;
		@@ -42,24 +45,28 @@ public class ScreenshotHelper {
		* Describes a screenshot request (to make it easier to pass data through to the handler).
		*/
		public static class ScreenshotRequest implements Parcelable {
		private int mSource;
		private boolean mHasStatusBar;
		private boolean mHasNavBar;
		private Bundle mBitmapBundle;
		private Rect mBoundsInScreen;
		private Insets mInsets;
		private int mTaskId;
		private int mUserId;
		private ComponentName mTopComponent;

		ScreenshotRequest(int source, boolean hasStatus, boolean hasNav) {
		private final int mSource;
		private final Bundle mBitmapBundle;
		private final Rect mBoundsInScreen;
		private final Insets mInsets;
		private final int mTaskId;
		private final int mUserId;
		private final ComponentName mTopComponent;

		@VisibleForTesting
		public ScreenshotRequest(int source) {
		mSource = source;
		mHasStatusBar = hasStatus;
		mHasNavBar = hasNav;
		mBitmapBundle = null;
		mBoundsInScreen = null;
		mInsets = null;
		mTaskId = -1;
		mUserId = -1;
		mTopComponent = null;
		}

		ScreenshotRequest(int source, Bundle bitmapBundle, Rect boundsInScreen, Insets insets,
		int taskId, int userId, ComponentName topComponent) {
		@VisibleForTesting
		public ScreenshotRequest(int source, Bundle bitmapBundle, Rect boundsInScreen,
		Insets insets, int taskId, int userId, ComponentName topComponent) {
		mSource = source;
		mBitmapBundle = bitmapBundle;
		mBoundsInScreen = boundsInScreen;
		@@ -71,16 +78,21 @@ public class ScreenshotHelper {

		ScreenshotRequest(Parcel in) {
		mSource = in.readInt();
		mHasStatusBar = in.readBoolean();
		mHasNavBar = in.readBoolean();

		if (in.readInt() == 1) {
		mBitmapBundle = in.readBundle(getClass().getClassLoader());
		mBoundsInScreen = in.readParcelable(Rect.class.getClassLoader(), android.graphics.Rect.class);
		mInsets = in.readParcelable(Insets.class.getClassLoader(), android.graphics.Insets.class);
		mBoundsInScreen = in.readParcelable(Rect.class.getClassLoader(), Rect.class);
		mInsets = in.readParcelable(Insets.class.getClassLoader(), Insets.class);
		mTaskId = in.readInt();
		mUserId = in.readInt();
		mTopComponent = in.readParcelable(ComponentName.class.getClassLoader(), android.content.ComponentName.class);
		mTopComponent = in.readParcelable(ComponentName.class.getClassLoader(),
		ComponentName.class);
		} else {
		mBitmapBundle = null;
		mBoundsInScreen = null;
		mInsets = null;
		mTaskId = -1;
		mUserId = -1;
		mTopComponent = null;
		}
		}

		@@ -88,14 +100,6 @@ public class ScreenshotHelper {
		return mSource;
		}

		public boolean getHasStatusBar() {
		return mHasStatusBar;
		}

		public boolean getHasNavBar() {
		return mHasNavBar;
		}

		public Bundle getBitmapBundle() {
		return mBitmapBundle;
		}
		@@ -112,7 +116,6 @@ public class ScreenshotHelper {
		return mTaskId;
		}


		public int getUserId() {
		return mUserId;
		}
		@@ -129,8 +132,6 @@ public class ScreenshotHelper {
		@Override
		public void writeToParcel(Parcel dest, int flags) {
		dest.writeInt(mSource);
		dest.writeBoolean(mHasStatusBar);
		dest.writeBoolean(mHasNavBar);
		if (mBitmapBundle == null) {
		dest.writeInt(0);
		} else {
		@@ -144,7 +145,8 @@ public class ScreenshotHelper {
		}
		}

		public static final @NonNull Parcelable.Creator<ScreenshotRequest> CREATOR =
		@NonNull
		public static final Parcelable.Creator<ScreenshotRequest> CREATOR =
		new Parcelable.Creator<ScreenshotRequest>() {

		@Override
		@@ -254,113 +256,71 @@ public class ScreenshotHelper {

		/**
		* Request a screenshot be taken.
		*
		* <p>
		* Added to support reducing unit test duration; the method variant without a timeout argument
		* is recommended for general use.
		*
		* @param screenshotType The type of screenshot, for example either
		* {@link android.view.WindowManager#TAKE_SCREENSHOT_FULLSCREEN}
		* or
		* {@link android.view.WindowManager#TAKE_SCREENSHOT_SELECTED_REGION}
		* @param hasStatus {@code true} if the status bar is currently showing. {@code false}
		* if not.
		* @param hasNav {@code true} if the navigation bar is currently showing. {@code
		* false} if not.
		* @param source The source of the screenshot request. One of
		* {SCREENSHOT_GLOBAL_ACTIONS, SCREENSHOT_KEY_CHORD,
		* SCREENSHOT_OVERVIEW, SCREENSHOT_OTHER}
		* @param handler A handler used in case the screenshot times out
		* @param completionConsumer Consumes `false` if a screenshot was not taken, and `true` if the
		* screenshot was taken.
		*/
		public void takeScreenshot(final int screenshotType, final boolean hasStatus,
		final boolean hasNav, int source, @NonNull Handler handler,
		@Nullable Consumer<Uri> completionConsumer) {
		ScreenshotRequest screenshotRequest = new ScreenshotRequest(source, hasStatus, hasNav);
		takeScreenshot(screenshotType, SCREENSHOT_TIMEOUT_MS, handler, screenshotRequest,
		completionConsumer);
		}

		/**
		* Request a screenshot be taken, with provided reason.
		*
		* @param screenshotType The type of screenshot, for example either
		* {@link android.view.WindowManager#TAKE_SCREENSHOT_FULLSCREEN}
		* or
		* {@link android.view.WindowManager#TAKE_SCREENSHOT_SELECTED_REGION}
		* @param hasStatus {@code true} if the status bar is currently showing. {@code false}
		* if
		* not.
		* @param hasNav {@code true} if the navigation bar is currently showing. {@code
		* false}
		* if not.
		* @param handler A handler used in case the screenshot times out
		* @param completionConsumer Consumes `false` if a screenshot was not taken, and `true` if the
		* screenshot was taken.
		* @param screenshotType The type of screenshot, defined by {@link ScreenshotType}
		* @param source The source of the screenshot request, defined by {@link ScreenshotSource}
		* @param handler used to process messages received from the screenshot service
		* @param completionConsumer receives the URI of the captured screenshot, once saved or
		* null if no screenshot was saved
		*/
		public void takeScreenshot(final int screenshotType, final boolean hasStatus,
		final boolean hasNav, @NonNull Handler handler,
		@Nullable Consumer<Uri> completionConsumer) {
		takeScreenshot(screenshotType, hasStatus, hasNav, SCREENSHOT_TIMEOUT_MS, handler,
		public void takeScreenshot(@ScreenshotType int screenshotType, @ScreenshotSource int source,
		@NonNull Handler handler, @Nullable Consumer<Uri> completionConsumer) {
		ScreenshotRequest screenshotRequest = new ScreenshotRequest(source);
		takeScreenshot(screenshotType, handler, screenshotRequest, SCREENSHOT_TIMEOUT_MS,
		completionConsumer);
		}

		/**
		* Request a screenshot be taken with a specific timeout.
		*
		* Request a screenshot be taken.
		* <p>
		* Added to support reducing unit test duration; the method variant without a timeout argument
		* is recommended for general use.
		*
		* @param screenshotType The type of screenshot, for example either
		* {@link android.view.WindowManager#TAKE_SCREENSHOT_FULLSCREEN}
		* or
		* {@link android.view.WindowManager#TAKE_SCREENSHOT_SELECTED_REGION}
		* @param hasStatus {@code true} if the status bar is currently showing. {@code false}
		* if
		* not.
		* @param hasNav {@code true} if the navigation bar is currently showing. {@code
		* false}
		* if not.
		* @param timeoutMs If the screenshot hasn't been completed within this time period,
		* the screenshot attempt will be cancelled and `completionConsumer`
		* will be run.
		* @param handler A handler used in case the screenshot times out
		* @param completionConsumer Consumes `false` if a screenshot was not taken, and `true` if the
		* screenshot was taken.
		* @param screenshotType The type of screenshot, defined by {@link ScreenshotType}
		* @param source The source of the screenshot request, defined by {@link ScreenshotSource}
		* @param handler used to process messages received from the screenshot service
		* @param timeoutMs time limit for processing, intended only for testing
		* @param completionConsumer receives the URI of the captured screenshot, once saved or
		* null if no screenshot was saved
		*/
		public void takeScreenshot(final int screenshotType, final boolean hasStatus,
		final boolean hasNav, long timeoutMs, @NonNull Handler handler,
		@Nullable Consumer<Uri> completionConsumer) {
		ScreenshotRequest screenshotRequest = new ScreenshotRequest(SCREENSHOT_OTHER, hasStatus,
		hasNav);
		takeScreenshot(screenshotType, timeoutMs, handler, screenshotRequest, completionConsumer);
		@VisibleForTesting
		public void takeScreenshot(@ScreenshotType int screenshotType, @ScreenshotSource int source,
		@NonNull Handler handler, long timeoutMs, @Nullable Consumer<Uri> completionConsumer) {
		ScreenshotRequest screenshotRequest = new ScreenshotRequest(source);
		takeScreenshot(screenshotType, handler, screenshotRequest, timeoutMs, completionConsumer);
		}

		/**
		* Request that provided image be handled as if it was a screenshot.
		*
		* @param screenshotBundle Bundle containing the buffer and color space of the screenshot.
		* @param boundsInScreen The bounds in screen coordinates that the bitmap orginated from.
		* @param boundsInScreen The bounds in screen coordinates that the bitmap originated from.
		* @param insets The insets that the image was shown with, inside the screen bounds.
		* @param taskId The taskId of the task that the screen shot was taken of.
		* @param userId The userId of user running the task provided in taskId.
		* @param topComponent The component name of the top component running in the task.
		* @param source The source of the screenshot request, defined by {@link ScreenshotSource}
		* @param handler A handler used in case the screenshot times out
		* @param completionConsumer Consumes `false` if a screenshot was not taken, and `true` if the
		* screenshot was taken.
		* @param completionConsumer receives the URI of the captured screenshot, once saved or
		* null if no screenshot was saved
		*/
		public void provideScreenshot(@NonNull Bundle screenshotBundle, @NonNull Rect boundsInScreen,
		@NonNull Insets insets, int taskId, int userId, ComponentName topComponent, int source,
		@NonNull Handler handler, @Nullable Consumer<Uri> completionConsumer) {
		ScreenshotRequest screenshotRequest =
		new ScreenshotRequest(source, screenshotBundle, boundsInScreen, insets, taskId,
		userId, topComponent);
		takeScreenshot(WindowManager.TAKE_SCREENSHOT_PROVIDED_IMAGE, SCREENSHOT_TIMEOUT_MS,
		handler, screenshotRequest, completionConsumer);
		@NonNull Insets insets, int taskId, int userId, ComponentName topComponent,
		@ScreenshotSource int source, @NonNull Handler handler,
		@Nullable Consumer<Uri> completionConsumer) {
		ScreenshotRequest screenshotRequest = new ScreenshotRequest(source, screenshotBundle,
		boundsInScreen, insets, taskId, userId, topComponent);
		takeScreenshot(WindowManager.TAKE_SCREENSHOT_PROVIDED_IMAGE, handler, screenshotRequest,
		SCREENSHOT_TIMEOUT_MS,
		completionConsumer);
		}

		private void takeScreenshot(final int screenshotType, long timeoutMs, @NonNull Handler handler,
		ScreenshotRequest screenshotRequest, @Nullable Consumer<Uri> completionConsumer) {
		private void takeScreenshot(@ScreenshotType int screenshotType, @NonNull Handler handler,
		ScreenshotRequest screenshotRequest, long timeoutMs,
		@Nullable Consumer<Uri> completionConsumer) {
		synchronized (mScreenshotLock) {

		final Runnable mScreenshotTimeout = () -> {

core/tests/screenshothelpertests/src/com/android/internal/util/ScreenshotHelperTest.java

+7 −4

Original line number	Diff line number	Diff line
		@@ -80,13 +80,14 @@ public final class ScreenshotHelperTest {

		@Test
		public void testFullscreenScreenshot() {
		mScreenshotHelper.takeScreenshot(TAKE_SCREENSHOT_FULLSCREEN, false, false, mHandler, null);
		mScreenshotHelper.takeScreenshot(TAKE_SCREENSHOT_FULLSCREEN,
		WindowManager.ScreenshotSource.SCREENSHOT_OTHER, mHandler, null);
		}

		@Test
		public void testSelectedRegionScreenshot() {
		mScreenshotHelper.takeScreenshot(TAKE_SCREENSHOT_SELECTED_REGION, false, false, mHandler,
		null);
		mScreenshotHelper.takeScreenshot(TAKE_SCREENSHOT_SELECTED_REGION,
		WindowManager.ScreenshotSource.SCREENSHOT_OTHER, mHandler, null);
		}

		@Test
		@@ -101,8 +102,10 @@ public final class ScreenshotHelperTest {
		long timeoutMs = 10;

		CountDownLatch lock = new CountDownLatch(1);
		mScreenshotHelper.takeScreenshot(TAKE_SCREENSHOT_FULLSCREEN, false, false, timeoutMs,
		mScreenshotHelper.takeScreenshot(TAKE_SCREENSHOT_FULLSCREEN,
		WindowManager.ScreenshotSource.SCREENSHOT_OTHER,
		mHandler,
		timeoutMs,
		uri -> {
		assertNull(uri);
		lock.countDown();