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

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

Merge "Fix AugmentedAutofillService onFillRequest() wrong focused AutofillId... 

Merge "Fix AugmentedAutofillService onFillRequest() wrong focused AutofillId for WebView" into rvc-dev am: cc4f13e0 am: b1fe173f

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

Change-Id: Id7f585c4dbef4335e9f2a1e63a68888ecafd434c
parents 02f0a638 b1fe173f

core/java/android/view/autofill/AutofillId.java

+4 −1
Original line number Diff line number Diff line
@@ -75,7 +75,10 @@ public final class AutofillId implements Parcelable { 
    /** @hide */

    public static AutofillId withoutSession(@NonNull AutofillId id) {

        final int flags = id.mFlags & ~FLAG_HAS_SESSION;

        return new AutofillId(flags, id.mViewId, id.mVirtualLongId, NO_SESSION);

        final long virtualChildId =

                ((id.mFlags & FLAG_IS_VIRTUAL_LONG) != 0) ? id.mVirtualLongId

                        : id.mVirtualIntId;

        return new AutofillId(flags, id.mViewId, virtualChildId, NO_SESSION);

    }



    /** @hide */

core/tests/coretests/src/android/view/autofill/AutofillIdTest.java

+26 −0
Original line number Diff line number Diff line
@@ -125,6 +125,32 @@ public class AutofillIdTest { 
        assertNonVirtual(idWithoutSession, 42, NO_SESSION);

    }



    @Test

    public void testVirtual_Long_withoutSession() {

        final AutofillId id = new AutofillId(new AutofillId(42), 108L, 666);

        final AutofillId idWithoutSession = AutofillId.withoutSession(id);

        assertThat(idWithoutSession.getViewId()).isEqualTo(42);

        assertThat(idWithoutSession.isVirtualLong()).isTrue();

        assertThat(idWithoutSession.isVirtualInt()).isFalse();

        assertThat(idWithoutSession.isNonVirtual()).isFalse();

        assertThat(idWithoutSession.getVirtualChildLongId()).isEqualTo(108L);

        assertThat(idWithoutSession.getVirtualChildIntId()).isEqualTo(View.NO_ID);

        assertThat(idWithoutSession.getSessionId()).isEqualTo(NO_SESSION);

    }



    @Test

    public void testVirtual_Int_withoutSession() {

        final AutofillId id = new AutofillId(42, 108);

        final AutofillId idWithoutSession = AutofillId.withoutSession(id);

        assertThat(idWithoutSession.getViewId()).isEqualTo(42);

        assertThat(idWithoutSession.isVirtualLong()).isFalse();

        assertThat(idWithoutSession.isVirtualInt()).isTrue();

        assertThat(idWithoutSession.isNonVirtual()).isFalse();

        assertThat(idWithoutSession.getVirtualChildIntId()).isEqualTo(108);

        assertThat(idWithoutSession.getVirtualChildLongId()).isEqualTo(View.NO_ID);

        assertThat(idWithoutSession.getSessionId()).isEqualTo(NO_SESSION);

    }



    @Test

    public void testSetResetSession() {

        final AutofillId id = new AutofillId(42);