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

Commit 9ba78f15 authored by Selim Cinek's avatar Selim Cinek
Browse files

Fixed that notifications weren't updating to the display size anymore

In the big refactoring, the call to onDensityOrFontScaleChanged
was dropped, so notifications weren't actually reinflated.
This is happening again.

Change-Id: Ie866bb947a5803dbb650fa0e7ef3764948862825
Fixes: 118402606
Test: change density and observe notification changing size
parent 70022d9c
Loading
Loading
Loading
Loading
+1 −1
Original line number Diff line number Diff line
@@ -588,7 +588,7 @@ public class NotificationData {
        }

        public boolean areGutsExposed() {
            return row != null && row.getGuts().isExposed();
            return row != null && row.getGuts() != null && row.getGuts().isExposed();
        }

        public boolean isChildInGroup() {
+0 −4
Original line number Diff line number Diff line
@@ -121,8 +121,6 @@ import com.android.internal.logging.MetricsLogger;
import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
import com.android.internal.statusbar.IStatusBarService;
import com.android.internal.statusbar.StatusBarIcon;
import com.android.internal.widget.MessagingGroup;
import com.android.internal.widget.MessagingMessage;
import com.android.keyguard.KeyguardUpdateMonitor;
import com.android.keyguard.KeyguardUpdateMonitorCallback;
import com.android.keyguard.ViewMediatorCallback;
@@ -1116,8 +1114,6 @@ public class StatusBar extends SystemUI implements DemoMode,

    @Override
    public void onDensityOrFontScaleChanged() {
        MessagingMessage.dropCache();
        MessagingGroup.dropCache();
        // TODO: Remove this.
        if (mBrightnessMirrorController != null) {
            mBrightnessMirrorController.onDensityOrFontScaleChanged();
+9 −1
Original line number Diff line number Diff line
@@ -37,6 +37,8 @@ import android.widget.TextView;

import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
import com.android.internal.statusbar.IStatusBarService;
import com.android.internal.widget.MessagingGroup;
import com.android.internal.widget.MessagingMessage;
import com.android.keyguard.KeyguardUpdateMonitor;
import com.android.systemui.Dependency;
import com.android.systemui.InitController;
@@ -62,9 +64,11 @@ import com.android.systemui.statusbar.notification.row.NotificationGutsManager;
import com.android.systemui.statusbar.notification.row.NotificationGutsManager.OnSettingsClickListener;
import com.android.systemui.statusbar.notification.row.NotificationInfo.CheckSaveListener;
import com.android.systemui.statusbar.notification.stack.NotificationListContainer;
import com.android.systemui.statusbar.policy.ConfigurationController;
import com.android.systemui.statusbar.policy.KeyguardMonitor;

public class StatusBarNotificationPresenter implements NotificationPresenter {
public class StatusBarNotificationPresenter implements NotificationPresenter,
        ConfigurationController.ConfigurationListener {

    private final LockscreenGestureLogger mLockscreenGestureLogger =
            Dependency.get(LockscreenGestureLogger.class);
@@ -165,9 +169,13 @@ public class StatusBarNotificationPresenter implements NotificationPresenter {

            onUserSwitched(mLockscreenUserManager.getCurrentUserId());
        });
        Dependency.get(ConfigurationController.class).addCallback(this);
    }

    @Override
    public void onDensityOrFontScaleChanged() {
        MessagingMessage.dropCache();
        MessagingGroup.dropCache();
        if (!KeyguardUpdateMonitor.getInstance(mContext).isSwitchingUser()) {
            mEntryManager.updateNotificationsOnDensityOrFontScaleChanged();
        } else {