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

Commit d1724517 authored by Ioana Alexandru's avatar Ioana Alexandru Committed by Automerger Merge Worker
Browse files

Merge "Implement visitUris for RemoteViews ViewGroupActionAdd." into udc-dev am: 8bcbcdd7 

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



Change-Id: Id79907d0592cce03ff86e312adf10cd4615e11a3
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 04be871b 8bcbcdd7

core/java/android/widget/RemoteViews.java

+5 −0
Original line number Diff line number Diff line
@@ -2598,6 +2598,11 @@ public class RemoteViews implements Parcelable, Filter { 
        public int getActionTag() {

            return VIEW_GROUP_ACTION_ADD_TAG;

        }



        @Override

        public final void visitUris(@NonNull Consumer<Uri> visitor) {

            mNestedViews.visitUris(visitor);

        }

    }



    /**

core/tests/coretests/src/android/widget/RemoteViewsTest.java

+24 −0
Original line number Diff line number Diff line
@@ -715,6 +715,30 @@ public class RemoteViewsTest { 
        verify(visitor, times(1)).accept(eq(icon4.getUri()));

    }



    @Test

    public void visitUris_nestedViews() {

        final RemoteViews outer = new RemoteViews(mPackage, R.layout.remote_views_test);



        final RemoteViews inner = new RemoteViews(mPackage, 33);

        final Uri imageUriI = Uri.parse("content://inner/image");

        final Icon icon1 = Icon.createWithContentUri("content://inner/icon1");

        final Icon icon2 = Icon.createWithContentUri("content://inner/icon2");

        final Icon icon3 = Icon.createWithContentUri("content://inner/icon3");

        final Icon icon4 = Icon.createWithContentUri("content://inner/icon4");

        inner.setImageViewUri(R.id.image, imageUriI);

        inner.setTextViewCompoundDrawables(R.id.text, icon1, icon2, icon3, icon4);



        outer.addView(R.id.layout, inner);



        Consumer<Uri> visitor = (Consumer<Uri>) spy(Consumer.class);

        outer.visitUris(visitor);

        verify(visitor, times(1)).accept(eq(imageUriI));

        verify(visitor, times(1)).accept(eq(icon1.getUri()));

        verify(visitor, times(1)).accept(eq(icon2.getUri()));

        verify(visitor, times(1)).accept(eq(icon3.getUri()));

        verify(visitor, times(1)).accept(eq(icon4.getUri()));

    }



    @Test

    public void visitUris_separateOrientation() {

        final RemoteViews landscape = new RemoteViews(mPackage, R.layout.remote_views_test);

services/tests/uiservicestests/src/com/android/server/notification/NotificationVisitUrisTest.java

+0 −1
Original line number Diff line number Diff line
@@ -88,7 +88,6 @@ public class NotificationVisitUrisTest extends UiServiceTestCase { 
    private static final Multimap<Class<?>, String> KNOWN_BAD =

            ImmutableMultimap.<Class<?>, String>builder()

                    .put(Notification.Builder.class, "setPublicVersion") // b/276294099

                    .putAll(RemoteViews.class, "addView", "addStableView") // b/277740082

                    .put(RemoteViews.class, "setIcon") // b/281018094

                    .put(Notification.WearableExtender.class, "addAction") // TODO: b/281044385

                    .put(Person.Builder.class, "setUri") // TODO: b/281044385