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

Commit 58fe1e02 authored by Jay Aliomer's avatar Jay Aliomer
Browse files

Log HUN view changes and actual removal 

Bug: 198359689
Test: manual logging
Change-Id: I3fa137a4e34b74a1f9edf99a231f14435176b4c3
parent 5cc27dd6

packages/SystemUI/src/com/android/systemui/statusbar/AlertingNotificationManager.java

+1 −1
Original line number Diff line number Diff line
@@ -320,7 +320,7 @@ public abstract class AlertingNotificationManager implements NotificationLifetim 
         * @param updatePostTime whether or not to refresh the post time

         */

        public void updateEntry(boolean updatePostTime) {

            mLogger.logUpdateEntry(updatePostTime);

            mLogger.logUpdateEntry(mEntry.getKey(), updatePostTime);



            long currentTime = mClock.currentTimeMillis();

            mEarliestRemovaltime = currentTime + mMinimumDisplayTime;

packages/SystemUI/src/com/android/systemui/statusbar/notification/row/ActivatableNotificationView.java

+2 −1
Original line number Diff line number Diff line
@@ -424,7 +424,8 @@ public abstract class ActivatableNotificationView extends ExpandableOutlineView 
    }



    @Override

    public void performAddAnimation(long delay, long duration, boolean isHeadsUpAppear) {

    public void performAddAnimation(long delay, long duration, boolean isHeadsUpAppear,

            Runnable onFinishRunnable) {

        enableAppearDrawing(true);

        mIsHeadsUpAnimation = isHeadsUpAppear;

        if (mDrawingAppearAnimation) {

packages/SystemUI/src/com/android/systemui/statusbar/notification/row/ExpandableView.java

+6 −1
Original line number Diff line number Diff line
@@ -358,7 +358,12 @@ public abstract class ExpandableView extends FrameLayout implements Dumpable { 
            Runnable onFinishedRunnable,

            AnimatorListenerAdapter animationListener);



    public abstract void performAddAnimation(long delay, long duration, boolean isHeadsUpAppear);

    public void performAddAnimation(long delay, long duration, boolean isHeadsUpAppear) {

        performAddAnimation(delay, duration, isHeadsUpAppear, null);

    }



    public abstract void performAddAnimation(long delay, long duration, boolean isHeadsUpAppear,

            Runnable onEndRunnable);



    /**

     * Set the notification appearance to be below the speed bump.

packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotifBindPipelineLogger.kt

+1 −2
Original line number Diff line number Diff line
@@ -18,7 +18,6 @@ package com.android.systemui.statusbar.notification.row 


import com.android.systemui.log.LogBuffer

import com.android.systemui.log.LogLevel.INFO

import com.android.systemui.log.LogLevel.WARNING

import com.android.systemui.log.dagger.NotificationLog

import javax.inject.Inject
@@ -50,7 +49,7 @@ class NotifBindPipelineLogger @Inject constructor( 
    }



    fun logRequestPipelineRowNotSet(notifKey: String) {

        buffer.log(TAG, WARNING, {

        buffer.log(TAG, INFO, {

            str1 = notifKey

        }, {

            "Row is not set so pipeline will not run. notif = $str1"

packages/SystemUI/src/com/android/systemui/statusbar/notification/row/StackScrollerDecorView.java

+7 −0
Original line number Diff line number Diff line
@@ -238,6 +238,13 @@ public abstract class StackScrollerDecorView extends ExpandableView { 
        setContentVisible(true);

    }



    @Override

    public void performAddAnimation(long delay, long duration, boolean isHeadsUpAppear,

            Runnable endRunnable) {

        // TODO: use delay and duration

        setContentVisible(true);

    }



    @Override

    public boolean needsClippingToShelf() {

        return false;