Merge "Revert "Reland "Update Glanceable Hub's AppWidgetHostListener impl for... (8dc22d9e) · Commits · e / os / android_frameworks_base

core/java/android/appwidget/AppWidgetHost.java

+1 −2

Original line number	Diff line number	Diff line
		@@ -50,6 +50,7 @@ import com.android.internal.appwidget.IAppWidgetService;

		import java.lang.ref.WeakReference;
		import java.util.ArrayList;
		import java.util.Arrays;
		import java.util.List;

		/**
		@@ -81,7 +82,6 @@ public class AppWidgetHost {
		private final Callbacks mCallbacks;
		private final SparseArray<AppWidgetHostListener> mListeners = new SparseArray<>();
		private InteractionHandler mInteractionHandler;
		private final Handler mMainHandler;

		static class Callbacks extends IAppWidgetHost.Stub {
		private final WeakReference<Handler> mWeakHandler;
		@@ -214,7 +214,6 @@ public class AppWidgetHost {
		mHandler = new UpdateHandler(looper);
		mCallbacks = new Callbacks(mHandler);
		mDisplayMetrics = context.getResources().getDisplayMetrics();
		mMainHandler = new Handler(Looper.getMainLooper());
		bindService(context);
		}

packages/SystemUI/multivalentTests/src/com/android/systemui/communal/widgets/GlanceableHubWidgetManagerServiceTest.kt

+1 −14

Original line number	Diff line number	Diff line
		@@ -46,10 +46,8 @@ import kotlinx.coroutines.test.runTest
		import org.junit.Before
		import org.junit.Test
		import org.junit.runner.RunWith
		import org.mockito.ArgumentMatchers.any
		import org.mockito.ArgumentMatchers.anyInt
		import org.mockito.kotlin.argumentCaptor
		import org.mockito.kotlin.doAnswer
		import org.mockito.kotlin.eq
		import org.mockito.kotlin.mock
		import org.mockito.kotlin.never
		@@ -147,15 +145,7 @@ class GlanceableHubWidgetManagerServiceTest : SysuiTestCase() {
		val service = IGlanceableHubWidgetManagerService.Stub.asInterface(binder)

		// Set listener
		val listener =
		mock<IGlanceableHubWidgetManagerService.IAppWidgetHostListener> {
		on { collectWidgetEvent(any()) } doAnswer
		{
		(it.arguments[0]
		as IGlanceableHubWidgetManagerService.IAppWidgetEventCallback)
		.onResult(null)
		}
		}
		val listener = mock<IGlanceableHubWidgetManagerService.IAppWidgetHostListener>()
		service.setAppWidgetHostListener(1, listener)

		// Verify a listener is set on the host
		@@ -176,9 +166,6 @@ class GlanceableHubWidgetManagerServiceTest : SysuiTestCase() {

		appWidgetHostListener.onViewDataChanged(1)
		verify(listener).onViewDataChanged(1)

		appWidgetHostListener.collectWidgetEvent()
		verify(listener).collectWidgetEvent(any())
		}

		@Test

packages/SystemUI/src/com/android/systemui/communal/widgets/AppWidgetHostListenerDelegate.kt

+0 −19

Original line number	Diff line number	Diff line
		@@ -16,17 +16,14 @@

		package com.android.systemui.communal.widgets

		import android.appwidget.AppWidgetEvent
		import android.appwidget.AppWidgetHost.AppWidgetHostListener
		import android.appwidget.AppWidgetProviderInfo
		import android.os.Looper
		import android.widget.RemoteViews
		import com.android.app.tracing.coroutines.launchTraced
		import com.android.systemui.dagger.qualifiers.Application
		import dagger.assisted.Assisted
		import dagger.assisted.AssistedFactory
		import dagger.assisted.AssistedInject
		import java.util.concurrent.CompletableFuture
		import kotlinx.coroutines.CoroutineScope

		/**
		@@ -58,20 +55,4 @@ constructor(
		override fun onViewDataChanged(viewId: Int) {
		mainScope.launchTraced("$tag#onViewDataChanged") { listener.onViewDataChanged(viewId) }
		}

		override fun collectWidgetEvent(): AppWidgetEvent? {
		if (!android.appwidget.flags.Flags.engagementMetrics()) {
		return null
		}

		if (Looper.getMainLooper().isCurrentThread()) {
		return listener.collectWidgetEvent()
		}

		val future = CompletableFuture<AppWidgetEvent?>()
		mainScope.launchTraced("$tag#collectWidgetEvent") {
		future.complete(listener.collectWidgetEvent())
		}
		return future.get()
		}
		}

packages/SystemUI/src/com/android/systemui/communal/widgets/GlanceableHubWidgetManager.kt

+0 −5

Original line number	Diff line number	Diff line
		@@ -29,7 +29,6 @@ import com.android.server.servicewatcher.ServiceWatcher
		import com.android.server.servicewatcher.ServiceWatcher.ServiceListener
		import com.android.systemui.communal.shared.model.CommunalWidgetContentModel
		import com.android.systemui.communal.shared.model.GlanceableHubMultiUserHelper
		import com.android.systemui.communal.widgets.IGlanceableHubWidgetManagerService.IAppWidgetEventCallback
		import com.android.systemui.communal.widgets.IGlanceableHubWidgetManagerService.IAppWidgetHostListener
		import com.android.systemui.communal.widgets.IGlanceableHubWidgetManagerService.IConfigureWidgetCallback
		import com.android.systemui.communal.widgets.IGlanceableHubWidgetManagerService.IGlanceableHubWidgetsListener
		@@ -213,10 +212,6 @@ constructor(
		override fun onViewDataChanged(viewId: Int) {
		listener.onViewDataChanged(viewId)
		}

		override fun collectWidgetEvent(callback: IAppWidgetEventCallback) {
		callback.onResult(listener.collectWidgetEvent())
		}
		}
		}

packages/SystemUI/src/com/android/systemui/communal/widgets/GlanceableHubWidgetManagerService.kt

+0 −24

Original line number	Diff line number	Diff line
		@@ -16,7 +16,6 @@

		package com.android.systemui.communal.widgets

		import android.appwidget.AppWidgetEvent
		import android.appwidget.AppWidgetHost.AppWidgetHostListener
		import android.appwidget.AppWidgetProviderInfo
		import android.content.ComponentName
		@@ -31,14 +30,12 @@ import androidx.lifecycle.LifecycleService
		import androidx.lifecycle.lifecycleScope
		import com.android.systemui.communal.data.repository.CommunalWidgetRepository
		import com.android.systemui.communal.shared.model.GlanceableHubMultiUserHelper
		import com.android.systemui.communal.widgets.IGlanceableHubWidgetManagerService.IAppWidgetEventCallback
		import com.android.systemui.communal.widgets.IGlanceableHubWidgetManagerService.IAppWidgetHostListener
		import com.android.systemui.communal.widgets.IGlanceableHubWidgetManagerService.IConfigureWidgetCallback
		import com.android.systemui.communal.widgets.IGlanceableHubWidgetManagerService.IGlanceableHubWidgetsListener
		import com.android.systemui.log.LogBuffer
		import com.android.systemui.log.core.Logger
		import com.android.systemui.log.dagger.CommunalLog
		import java.util.concurrent.CompletableFuture
		import javax.inject.Inject
		import kotlinx.coroutines.CompletableDeferred
		import kotlinx.coroutines.Job
		@@ -263,27 +260,6 @@ constructor(
		}
		}
		}

		override fun collectWidgetEvent(): AppWidgetEvent? {
		if (!android.appwidget.flags.Flags.engagementMetrics()) return null

		val future = CompletableFuture<AppWidgetEvent?>()
		val callback =
		object : IAppWidgetEventCallback.Stub() {
		override fun onResult(event: AppWidgetEvent?) {
		future.complete(event)
		}
		}
		return try {
		listener.collectWidgetEvent(callback)
		future.get()
		} catch (e: RemoteException) {
		logger.e({ "Error collecting widget event: $str1" }) {
		str1 = e.localizedMessage
		}
		null
		}
		}
		}
		}