Merge changes If0d82a5b,I0616d6f4 into udc-dev (afd58389) · Commits · e / os / android_frameworks_base

packages/SystemUI/src/com/android/systemui/log/dagger/LogModule.java

+1 −1

Original line number	Diff line number	Diff line
		@@ -209,7 +209,7 @@ public class LogModule {
		@SysUISingleton
		@CollapsedSbFragmentLog
		public static LogBuffer provideCollapsedSbFragmentLogBuffer(LogBufferFactory factory) {
		return factory.create("CollapsedSbFragmentLog", 20);
		return factory.create("CollapsedSbFragmentLog", 40);
		}

		/**

packages/SystemUI/src/com/android/systemui/statusbar/phone/fragment/CollapsedStatusBarFragment.java

+55 −61

Original line number	Diff line number	Diff line
		@@ -14,11 +14,7 @@

		package com.android.systemui.statusbar.phone.fragment;

		import static android.app.StatusBarManager.DISABLE2_SYSTEM_ICONS;
		import static android.app.StatusBarManager.DISABLE_CLOCK;
		import static android.app.StatusBarManager.DISABLE_NOTIFICATION_ICONS;
		import static android.app.StatusBarManager.DISABLE_ONGOING_CALL_CHIP;
		import static android.app.StatusBarManager.DISABLE_SYSTEM_INFO;


		import static com.android.systemui.statusbar.events.SystemStatusAnimationSchedulerKt.IDLE;
		import static com.android.systemui.statusbar.events.SystemStatusAnimationSchedulerKt.SHOWING_PERSISTENT_DOT;
		@@ -112,8 +108,13 @@ public class CollapsedStatusBarFragment extends Fragment implements CommandQueue
		private View mClockView;
		private View mOngoingCallChip;
		private View mNotificationIconAreaInner;
		private int mDisabled1;
		private int mDisabled2;
		// Visibilities come in from external system callers via disable flags, but we also sometimes
		// modify the visibilities internally. We need to store both so that we don't accidentally
		// propagate our internally modified flags for too long.
		private StatusBarVisibilityModel mLastSystemVisibility =
		StatusBarVisibilityModel.createDefaultModel();
		private StatusBarVisibilityModel mLastModifiedVisibility =
		StatusBarVisibilityModel.createDefaultModel();
		private DarkIconManager mDarkIconManager;
		private final StatusBarFragmentComponent.Factory mStatusBarFragmentComponentFactory;
		private final CommandQueue mCommandQueue;
		@@ -141,7 +142,7 @@ public class CollapsedStatusBarFragment extends Fragment implements CommandQueue
		private final OngoingCallListener mOngoingCallListener = new OngoingCallListener() {
		@Override
		public void onOngoingCallStateChanged(boolean animate) {
		disable(getContext().getDisplayId(), mDisabled1, mDisabled2, animate);
		updateStatusBarVisibilities(animate);
		}
		};
		private OperatorNameViewController mOperatorNameViewController;
		@@ -388,8 +389,7 @@ public class CollapsedStatusBarFragment extends Fragment implements CommandQueue
		}
		notificationIconArea.addView(mNotificationIconAreaInner);

		// #disable should have already been called, so use the disable values to set visibility.
		updateNotificationIconAreaAndCallChip(mDisabled1, false);
		updateNotificationIconAreaAndCallChip(/* animate= */ false);
		}

		/**
		@@ -408,49 +408,50 @@ public class CollapsedStatusBarFragment extends Fragment implements CommandQueue
		if (displayId != getContext().getDisplayId()) {
		return;
		}
		mCollapsedStatusBarFragmentLogger
		.logDisableFlagChange(new DisableState(state1, state2));
		mLastSystemVisibility =
		StatusBarVisibilityModel.createModelFromFlags(state1, state2);
		updateStatusBarVisibilities(animate);
		}

		int state1BeforeAdjustment = state1;
		state1 = adjustDisableFlags(state1);

		mCollapsedStatusBarFragmentLogger.logDisableFlagChange(
		/* new= */ new DisableState(state1BeforeAdjustment, state2),
		/* newAfterLocalModification= */ new DisableState(state1, state2));
		private void updateStatusBarVisibilities(boolean animate) {
		StatusBarVisibilityModel previousModel = mLastModifiedVisibility;
		StatusBarVisibilityModel newModel = calculateInternalModel(mLastSystemVisibility);
		mCollapsedStatusBarFragmentLogger.logVisibilityModel(newModel);
		mLastModifiedVisibility = newModel;

		final int old1 = mDisabled1;
		final int diff1 = state1 ^ old1;
		final int old2 = mDisabled2;
		final int diff2 = state2 ^ old2;
		mDisabled1 = state1;
		mDisabled2 = state2;
		if ((diff1 & DISABLE_SYSTEM_INFO) != 0 \|\| ((diff2 & DISABLE2_SYSTEM_ICONS) != 0)) {
		if ((state1 & DISABLE_SYSTEM_INFO) != 0 \|\| ((state2 & DISABLE2_SYSTEM_ICONS) != 0)) {
		hideEndSideContent(animate);
		hideOperatorName(animate);
		} else {
		if (newModel.getShowSystemInfo() != previousModel.getShowSystemInfo()) {
		if (newModel.getShowSystemInfo()) {
		showEndSideContent(animate);
		showOperatorName(animate);
		} else {
		hideEndSideContent(animate);
		hideOperatorName(animate);
		}
		}

		// The ongoing call chip and notification icon visibilities are intertwined, so update both
		// if either change.
		if (((diff1 & DISABLE_ONGOING_CALL_CHIP) != 0)
		\|\| ((diff1 & DISABLE_NOTIFICATION_ICONS) != 0)) {
		updateNotificationIconAreaAndCallChip(state1, animate);
		if (newModel.getShowNotificationIcons() != previousModel.getShowNotificationIcons()
		\|\| newModel.getShowOngoingCallChip() != previousModel.getShowOngoingCallChip()) {
		updateNotificationIconAreaAndCallChip(animate);
		}

		// The clock may have already been hidden, but we might want to shift its
		// visibility to GONE from INVISIBLE or vice versa
		if ((diff1 & DISABLE_CLOCK) != 0 \|\| mClockView.getVisibility() != clockHiddenMode()) {
		if ((state1 & DISABLE_CLOCK) != 0) {
		hideClock(animate);
		} else {
		if (newModel.getShowClock() != previousModel.getShowClock()
		\|\| mClockView.getVisibility() != clockHiddenMode()) {
		if (newModel.getShowClock()) {
		showClock(animate);
		} else {
		hideClock(animate);
		}
		}
		}

		protected int adjustDisableFlags(int state) {
		private StatusBarVisibilityModel calculateInternalModel(
		StatusBarVisibilityModel externalModel) {
		boolean headsUpVisible =
		mStatusBarFragmentComponent.getHeadsUpAppearanceController().shouldBeVisible();

		@@ -459,34 +460,31 @@ public class CollapsedStatusBarFragment extends Fragment implements CommandQueue
		&& shouldHideNotificationIcons()
		&& !(mStatusBarStateController.getState() == StatusBarState.KEYGUARD
		&& headsUpVisible)) {
		state \|= DISABLE_NOTIFICATION_ICONS;
		state \|= DISABLE_SYSTEM_INFO;
		state \|= DISABLE_CLOCK;
		}

		if (mOngoingCallController.hasOngoingCall()) {
		state &= ~DISABLE_ONGOING_CALL_CHIP;
		} else {
		state \|= DISABLE_ONGOING_CALL_CHIP;
		}

		if (headsUpVisible) {
		// Disable everything on the left side of the status bar, since the app name for the
		// heads up notification appears there instead.
		state \|= DISABLE_CLOCK;
		state \|= DISABLE_ONGOING_CALL_CHIP;
		// Hide everything
		return new StatusBarVisibilityModel(
		/* showClock= */ false,
		/* showNotificationIcons= */ false,
		/* showOngoingCallChip= */ false,
		/* showSystemInfo= */ false);
		}

		return state;
		boolean showClock = externalModel.getShowClock() && !headsUpVisible;
		boolean showOngoingCallChip = mOngoingCallController.hasOngoingCall() && !headsUpVisible;
		return new StatusBarVisibilityModel(
		showClock,
		externalModel.getShowNotificationIcons(),
		showOngoingCallChip,
		externalModel.getShowSystemInfo());
		}

		/**
		* Updates the visibility of the notification icon area and ongoing call chip based on disabled1
		* state.
		*/
		private void updateNotificationIconAreaAndCallChip(int state1, boolean animate) {
		boolean disableNotifications = (state1 & DISABLE_NOTIFICATION_ICONS) != 0;
		boolean hasOngoingCall = (state1 & DISABLE_ONGOING_CALL_CHIP) == 0;
		private void updateNotificationIconAreaAndCallChip(boolean animate) {
		StatusBarVisibilityModel visibilityModel = mLastModifiedVisibility;
		boolean disableNotifications = !visibilityModel.getShowNotificationIcons();
		boolean hasOngoingCall = visibilityModel.getShowOngoingCallChip();

		// Hide notifications if the disable flag is set or we have an ongoing call.
		if (disableNotifications \|\| hasOngoingCall) {
		@@ -683,7 +681,7 @@ public class CollapsedStatusBarFragment extends Fragment implements CommandQueue

		@Override
		public void onDozingChanged(boolean isDozing) {
		disable(getContext().getDisplayId(), mDisabled1, mDisabled2, false /* animate */);
		updateStatusBarVisibilities(/* animate= */ false);
		}

		@Nullable
		@@ -698,10 +696,6 @@ public class CollapsedStatusBarFragment extends Fragment implements CommandQueue
		return mSystemEventAnimator.onSystemEventAnimationFinish(hasPersistentDot);
		}

		private boolean isSystemIconAreaDisabled() {
		return (mDisabled1 & DISABLE_SYSTEM_INFO) != 0 \|\| (mDisabled2 & DISABLE2_SYSTEM_ICONS) != 0;
		}

		private void updateStatusBarLocation(int left, int right) {
		int leftMargin = left - mStatusBar.getLeft();
		int rightMargin = mStatusBar.getRight() - right;

packages/SystemUI/src/com/android/systemui/statusbar/phone/fragment/CollapsedStatusBarFragmentLogger.kt

+19 −5

Original line number	Diff line number	Diff line
		@@ -37,7 +37,6 @@ class CollapsedStatusBarFragmentLogger @Inject constructor(
		*/
		fun logDisableFlagChange(
		new: DisableFlagsLogger.DisableState,
		newAfterLocalModification: DisableFlagsLogger.DisableState
		) {
		buffer.log(
		TAG,
		@@ -45,19 +44,34 @@ class CollapsedStatusBarFragmentLogger @Inject constructor(
		{
		int1 = new.disable1
		int2 = new.disable2
		long1 = newAfterLocalModification.disable1.toLong()
		long2 = newAfterLocalModification.disable2.toLong()
		},
		{
		disableFlagsLogger.getDisableFlagsString(
		old = null,
		new = DisableFlagsLogger.DisableState(int1, int2),
		newAfterLocalModification =
		DisableFlagsLogger.DisableState(long1.toInt(), long2.toInt())
		)
		}
		)
		}

		fun logVisibilityModel(model: StatusBarVisibilityModel) {
		buffer.log(
		TAG,
		LogLevel.INFO,
		{
		bool1 = model.showClock
		bool2 = model.showNotificationIcons
		bool3 = model.showOngoingCallChip
		bool4 = model.showSystemInfo
		},
		{ "New visibilities calculated internally. " +
		"showClock=$bool1 " +
		"showNotificationIcons=$bool2 " +
		"showOngoingCallChip=$bool3 " +
		"showSystemInfo=$bool4"
		}
		)
		}
		}

		private const val TAG = "CollapsedSbFragment"

packages/SystemUI/src/com/android/systemui/statusbar/phone/fragment/StatusBarVisibilityModel.kt

0 → 100644

+58 −0

Original line number	Diff line number	Diff line
		/*
		* Copyright (C) 2023 The Android Open Source Project
		*
		* Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file
		* except in compliance with the License. You may obtain a copy of the License at
		*
		* http://www.apache.org/licenses/LICENSE-2.0
		*
		* Unless required by applicable law or agreed to in writing, software distributed under the
		* License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
		* KIND, either express or implied. See the License for the specific language governing
		* permissions and limitations under the License.
		*/

		package com.android.systemui.statusbar.phone.fragment

		import android.app.StatusBarManager.DISABLE2_NONE
		import android.app.StatusBarManager.DISABLE2_SYSTEM_ICONS
		import android.app.StatusBarManager.DISABLE_CLOCK
		import android.app.StatusBarManager.DISABLE_NONE
		import android.app.StatusBarManager.DISABLE_NOTIFICATION_ICONS
		import android.app.StatusBarManager.DISABLE_ONGOING_CALL_CHIP
		import android.app.StatusBarManager.DISABLE_SYSTEM_INFO

		/** A model for which parts of the status bar should be visible or not visible. */
		data class StatusBarVisibilityModel(
		val showClock: Boolean,
		val showNotificationIcons: Boolean,
		val showOngoingCallChip: Boolean,
		val showSystemInfo: Boolean,
		) {
		companion object {
		/** Creates the default model. */
		@JvmStatic
		fun createDefaultModel(): StatusBarVisibilityModel {
		return createModelFromFlags(DISABLE_NONE, DISABLE2_NONE)
		}

		/**
		* Given a set of disabled flags, converts them into the correct visibility statuses.
		*
		* See [CommandQueue.Callbacks.disable].
		*/
		@JvmStatic
		fun createModelFromFlags(disabled1: Int, disabled2: Int): StatusBarVisibilityModel {
		return StatusBarVisibilityModel(
		showClock = (disabled1 and DISABLE_CLOCK) == 0,
		showNotificationIcons = (disabled1 and DISABLE_NOTIFICATION_ICONS) == 0,
		// TODO(b/279899176): [CollapsedStatusBarFragment] always overwrites this with the
		// value of [OngoingCallController]. Do we need to process the flag here?
		showOngoingCallChip = (disabled1 and DISABLE_ONGOING_CALL_CHIP) == 0,
		showSystemInfo =
		(disabled1 and DISABLE_SYSTEM_INFO) == 0 &&
		(disabled2 and DISABLE2_SYSTEM_ICONS) == 0
		)
		}
		}
		}

packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/fragment/CollapsedStatusBarFragmentLoggerTest.kt

+28 −4

Original line number	Diff line number	Diff line
		@@ -43,15 +43,39 @@ class CollapsedStatusBarFragmentLoggerTest : SysuiTestCase() {
		fun logDisableFlagChange_bufferHasStates() {
		val state = DisableFlagsLogger.DisableState(0, 1)

		logger.logDisableFlagChange(state, state)
		logger.logDisableFlagChange(state)

		val stringWriter = StringWriter()
		buffer.dump(PrintWriter(stringWriter), tailLength = 0)
		val actualString = stringWriter.toString()
		val expectedLogString = disableFlagsLogger.getDisableFlagsString(
		old = null, new = state, newAfterLocalModification = state
		val expectedLogString =
		disableFlagsLogger.getDisableFlagsString(
		old = null,
		new = state,
		newAfterLocalModification = null,
		)

		assertThat(actualString).contains(expectedLogString)
		}

		@Test
		fun logVisibilityModel_bufferCorrect() {
		logger.logVisibilityModel(
		StatusBarVisibilityModel(
		showClock = false,
		showNotificationIcons = true,
		showOngoingCallChip = false,
		showSystemInfo = true,
		)
		)

		val stringWriter = StringWriter()
		buffer.dump(PrintWriter(stringWriter), tailLength = 0)
		val actualString = stringWriter.toString()

		assertThat(actualString).contains("showClock=false")
		assertThat(actualString).contains("showNotificationIcons=true")
		assertThat(actualString).contains("showOngoingCallChip=false")
		assertThat(actualString).contains("showSystemInfo=true")
		}
		}