New pipeline: remove non-silent section headers
Fixes: 204127880 Bug: 199765418 Test: manual Merged-In: I729cca549154c5762c2bb7082111d19b3ec1f9d9 Change-Id: I729cca549154c5762c2bb7082111d19b3ec1f9d9
This commit is contained in:
@@ -48,7 +48,9 @@ class ConversationCoordinator @Inject constructor(
|
|||||||
val sectioner = object : NotifSectioner("People") {
|
val sectioner = object : NotifSectioner("People") {
|
||||||
override fun isInSection(entry: ListEntry): Boolean =
|
override fun isInSection(entry: ListEntry): Boolean =
|
||||||
isConversation(entry.representativeEntry!!)
|
isConversation(entry.representativeEntry!!)
|
||||||
override fun getHeaderNodeController() = peopleHeaderController
|
override fun getHeaderNodeController() =
|
||||||
|
// TODO: remove SHOW_ALL_SECTIONS, this redundant method, and peopleHeaderController
|
||||||
|
if (RankingCoordinator.SHOW_ALL_SECTIONS) peopleHeaderController else null
|
||||||
}
|
}
|
||||||
|
|
||||||
override fun attach(pipeline: NotifPipeline) {
|
override fun attach(pipeline: NotifPipeline) {
|
||||||
|
|||||||
@@ -205,7 +205,11 @@ public class HeadsUpCoordinator implements Coordinator {
|
|||||||
@Nullable
|
@Nullable
|
||||||
@Override
|
@Override
|
||||||
public NodeController getHeaderNodeController() {
|
public NodeController getHeaderNodeController() {
|
||||||
return mIncomingHeaderController;
|
// TODO: remove SHOW_ALL_SECTIONS, this redundant method, and mIncomingHeaderController
|
||||||
|
if (RankingCoordinator.SHOW_ALL_SECTIONS) {
|
||||||
|
return mIncomingHeaderController;
|
||||||
|
}
|
||||||
|
return null;
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|||||||
@@ -41,6 +41,7 @@ import javax.inject.Inject;
|
|||||||
*/
|
*/
|
||||||
@SysUISingleton
|
@SysUISingleton
|
||||||
public class RankingCoordinator implements Coordinator {
|
public class RankingCoordinator implements Coordinator {
|
||||||
|
public static final boolean SHOW_ALL_SECTIONS = false;
|
||||||
private final StatusBarStateController mStatusBarStateController;
|
private final StatusBarStateController mStatusBarStateController;
|
||||||
private final HighPriorityProvider mHighPriorityProvider;
|
private final HighPriorityProvider mHighPriorityProvider;
|
||||||
private final NodeController mSilentHeaderController;
|
private final NodeController mSilentHeaderController;
|
||||||
@@ -83,7 +84,11 @@ public class RankingCoordinator implements Coordinator {
|
|||||||
@Nullable
|
@Nullable
|
||||||
@Override
|
@Override
|
||||||
public NodeController getHeaderNodeController() {
|
public NodeController getHeaderNodeController() {
|
||||||
return mAlertingHeaderController;
|
// TODO: remove SHOW_ALL_SECTIONS, this redundant method, and mAlertingHeaderController
|
||||||
|
if (SHOW_ALL_SECTIONS) {
|
||||||
|
return mAlertingHeaderController;
|
||||||
|
}
|
||||||
|
return null;
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user