Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c81be428 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Always let standard autofill try on new partitions even if session was... 

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

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

Change-Id: I14bd39b5b70cc69cb609ef11c0b94cd2c04a8a9d
parents bda25fdb 2b5ab22b

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

+12 −2
Original line number Diff line number Diff line
@@ -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;

                    }

                }