Merge "Always let standard autofill try on new partitions even if session was marked as augmented only." into sc-dev am: 2b5ab22b9f

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/14970180

Change-Id: I6b8dc5123d9a59abf741426afe5fdc786d862eaf
This commit is contained in:
TreeHugger Robot
2021-06-17 21:53:14 +00:00
committed by Automerger Merge Worker

View File

@@ -2648,6 +2648,10 @@ final class Session implements RemoteFillService.FillServiceCallbacks, ViewState
Slog.d(TAG, "Starting partition or augmented request for view id " + id + ": "
+ viewState.getStateAsString());
}
// Fix to always let standard autofill start.
// Sometimes activity contain IMPORTANT_FOR_AUTOFILL_NO fields which marks session as
// augmentedOnly, but other fields are still fillable by standard autofill.
mSessionFlags.mAugmentedAutofillOnly = false;
requestNewFillResponseLocked(viewState, ViewState.STATE_STARTED_PARTITION, flags);
return true;
}
@@ -2847,12 +2851,18 @@ final class Session implements RemoteFillService.FillServiceCallbacks, ViewState
if (sDebug) Slog.d(TAG, "trigger augmented autofill.");
triggerAugmentedAutofillLocked(flags);
} else {
if (sDebug) Slog.d(TAG, "skip augmented autofill for same view.");
if (sDebug) {
Slog.d(TAG, "skip augmented autofill for same view: "
+ "same view entered");
}
}
return;
} else if (mSessionFlags.mAugmentedAutofillOnly && isSameViewEntered) {
// Regular autofill is disabled.
if (sDebug) Slog.d(TAG, "skip augmented autofill for same view.");
if (sDebug) {
Slog.d(TAG, "skip augmented autofill for same view: "
+ "standard autofill disabled.");
}
return;
}
}