Merge "Add isInline info to augmented autofill request log" into rvc-dev am:... (f267e95b) · Commits · e / os / android_frameworks_base

services/autofill/java/com/android/server/autofill/Session.java

+41 −27

Original line number	Diff line number	Diff line
		@@ -117,6 +117,7 @@ import java.util.Optional;
		import java.util.concurrent.CountDownLatch;
		import java.util.concurrent.atomic.AtomicInteger;
		import java.util.function.Consumer;
		import java.util.function.Function;

		/**
		* A session for a given activity.
		@@ -3079,19 +3080,13 @@ final class Session implements RemoteFillService.FillServiceCallbacks, ViewState
		final boolean isWhitelisted = mService
		.isWhitelistedForAugmentedAutofillLocked(mComponentName);

		final String historyItem =
		"aug:id=" + id + " u=" + uid + " m=" + mode
		+ " a=" + ComponentName.flattenToShortString(mComponentName)
		+ " f=" + mCurrentViewId
		+ " s=" + remoteService.getComponentName()
		+ " w=" + isWhitelisted;
		mService.getMaster().logRequestLocked(historyItem);

		if (!isWhitelisted) {
		if (sVerbose) {
		Slog.v(TAG, "triggerAugmentedAutofillLocked(): "
		+ ComponentName.flattenToShortString(mComponentName) + " not whitelisted ");
		}
		logAugmentedAutofillRequestLocked(mode, remoteService.getComponentName(),
		mCurrentViewId, isWhitelisted, /isInline/null);
		return null;
		}

		@@ -3116,24 +3111,27 @@ final class Session implements RemoteFillService.FillServiceCallbacks, ViewState

		final AutofillId focusedId = mCurrentViewId;

		final Consumer<InlineSuggestionsRequest> requestAugmentedAutofill =
		(inlineSuggestionsRequest) -> {
		remoteService.onRequestAutofillLocked(id, mClient, taskId, mComponentName,
		AutofillId.withoutSession(focusedId),
		currentValue, inlineSuggestionsRequest,
		/inlineSuggestionsCallback=/
		final Function<InlineSuggestionsResponse, Boolean> inlineSuggestionsResponseCallback =
		response -> {
		synchronized (mLock) {
		return mInlineSessionController
		.onInlineSuggestionsResponseLocked(
		return mInlineSessionController.onInlineSuggestionsResponseLocked(
		focusedId, response);
		}
		},
		};
		final Consumer<InlineSuggestionsRequest> requestAugmentedAutofill =
		(inlineSuggestionsRequest) -> {
		synchronized (mLock) {
		logAugmentedAutofillRequestLocked(mode, remoteService.getComponentName(),
		focusedId, isWhitelisted, inlineSuggestionsRequest != null);
		remoteService.onRequestAutofillLocked(id, mClient, taskId, mComponentName,
		AutofillId.withoutSession(focusedId), currentValue,
		inlineSuggestionsRequest, inlineSuggestionsResponseCallback,
		/onErrorCallback=/ () -> {
		synchronized (mLock) {
		cancelAugmentedAutofillLocked();
		}
		}, mService.getRemoteInlineSuggestionRenderServiceLocked());
		}
		};

		// When the inline suggestion render service is available, there are 2 cases when
		@@ -3150,9 +3148,11 @@ final class Session implements RemoteFillService.FillServiceCallbacks, ViewState
		if (sDebug) Slog.d(TAG, "Create inline request for augmented autofill");
		remoteRenderService.getInlineSuggestionsRendererInfo(new RemoteCallback(
		(extras) -> {
		synchronized (mLock) {
		mInlineSessionController.onCreateInlineSuggestionsRequestLocked(
		focusedId, /requestConsumer=/ requestAugmentedAutofill,
		extras);
		}
		}, mHandler));
		} else {
		requestAugmentedAutofill.accept(
		@@ -3164,6 +3164,20 @@ final class Session implements RemoteFillService.FillServiceCallbacks, ViewState
		return mAugmentedAutofillDestroyer;
		}

		@GuardedBy("mLock")
		private void logAugmentedAutofillRequestLocked(int mode,
		ComponentName augmentedRemoteServiceName, AutofillId focusedId, boolean isWhitelisted,
		Boolean isInline) {
		final String historyItem =
		"aug:id=" + id + " u=" + uid + " m=" + mode
		+ " a=" + ComponentName.flattenToShortString(mComponentName)
		+ " f=" + focusedId
		+ " s=" + augmentedRemoteServiceName
		+ " w=" + isWhitelisted
		+ " i=" + isInline;
		mService.getMaster().logRequestLocked(historyItem);
		}

		@GuardedBy("mLock")
		private void cancelAugmentedAutofillLocked() {
		final RemoteAugmentedAutofillService remoteService = mService