Merge "Move CentralSurface into initialize()." into tm-qpr-dev

This commit is contained in:
Julia Tuttle
2022-10-21 18:04:04 +00:00
committed by Android (Google) Code Review
4 changed files with 8 additions and 3 deletions

View File

@@ -23,6 +23,7 @@ import com.android.systemui.statusbar.notification.NotificationActivityStarter
import com.android.systemui.statusbar.notification.collection.inflation.NotificationRowBinderImpl import com.android.systemui.statusbar.notification.collection.inflation.NotificationRowBinderImpl
import com.android.systemui.statusbar.notification.collection.render.NotifStackController import com.android.systemui.statusbar.notification.collection.render.NotifStackController
import com.android.systemui.statusbar.notification.stack.NotificationListContainer import com.android.systemui.statusbar.notification.stack.NotificationListContainer
import com.android.systemui.statusbar.phone.CentralSurfaces
/** /**
* The master controller for all notifications-related work * The master controller for all notifications-related work
@@ -32,6 +33,7 @@ import com.android.systemui.statusbar.notification.stack.NotificationListContain
*/ */
interface NotificationsController { interface NotificationsController {
fun initialize( fun initialize(
centralSurfaces: CentralSurfaces,
presenter: NotificationPresenter, presenter: NotificationPresenter,
listContainer: NotificationListContainer, listContainer: NotificationListContainer,
stackController: NotifStackController, stackController: NotifStackController,

View File

@@ -56,7 +56,6 @@ import javax.inject.Inject
*/ */
@SysUISingleton @SysUISingleton
class NotificationsControllerImpl @Inject constructor( class NotificationsControllerImpl @Inject constructor(
private val centralSurfaces: Lazy<CentralSurfaces>,
private val notificationListener: NotificationListener, private val notificationListener: NotificationListener,
private val commonNotifCollection: Lazy<CommonNotifCollection>, private val commonNotifCollection: Lazy<CommonNotifCollection>,
private val notifPipeline: Lazy<NotifPipeline>, private val notifPipeline: Lazy<NotifPipeline>,
@@ -76,6 +75,7 @@ class NotificationsControllerImpl @Inject constructor(
) : NotificationsController { ) : NotificationsController {
override fun initialize( override fun initialize(
centralSurfaces: CentralSurfaces,
presenter: NotificationPresenter, presenter: NotificationPresenter,
listContainer: NotificationListContainer, listContainer: NotificationListContainer,
stackController: NotifStackController, stackController: NotifStackController,
@@ -92,8 +92,8 @@ class NotificationsControllerImpl @Inject constructor(
notificationRowBinder.setNotificationClicker( notificationRowBinder.setNotificationClicker(
clickerBuilder.build( clickerBuilder.build(
Optional.of( Optional.ofNullable(centralSurfaces), bubblesOptional,
centralSurfaces.get()), bubblesOptional, notificationActivityStarter)) notificationActivityStarter))
notificationRowBinder.setUpWithPresenter( notificationRowBinder.setUpWithPresenter(
presenter, presenter,
listContainer, listContainer,

View File

@@ -24,6 +24,7 @@ import com.android.systemui.statusbar.notification.NotificationActivityStarter
import com.android.systemui.statusbar.notification.collection.inflation.NotificationRowBinderImpl import com.android.systemui.statusbar.notification.collection.inflation.NotificationRowBinderImpl
import com.android.systemui.statusbar.notification.collection.render.NotifStackController import com.android.systemui.statusbar.notification.collection.render.NotifStackController
import com.android.systemui.statusbar.notification.stack.NotificationListContainer import com.android.systemui.statusbar.notification.stack.NotificationListContainer
import com.android.systemui.statusbar.phone.CentralSurfaces
import javax.inject.Inject import javax.inject.Inject
/** /**
@@ -34,6 +35,7 @@ class NotificationsControllerStub @Inject constructor(
) : NotificationsController { ) : NotificationsController {
override fun initialize( override fun initialize(
centralSurfaces: CentralSurfaces,
presenter: NotificationPresenter, presenter: NotificationPresenter,
listContainer: NotificationListContainer, listContainer: NotificationListContainer,
stackController: NotifStackController, stackController: NotifStackController,

View File

@@ -1426,6 +1426,7 @@ public class CentralSurfacesImpl implements CoreStartable, CentralSurfaces {
mStackScrollerController.setNotificationActivityStarter(mNotificationActivityStarter); mStackScrollerController.setNotificationActivityStarter(mNotificationActivityStarter);
mGutsManager.setNotificationActivityStarter(mNotificationActivityStarter); mGutsManager.setNotificationActivityStarter(mNotificationActivityStarter);
mNotificationsController.initialize( mNotificationsController.initialize(
this,
mPresenter, mPresenter,
mNotifListContainer, mNotifListContainer,
mStackScrollerController.getNotifStackController(), mStackScrollerController.getNotifStackController(),