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

Commit d1d82c71 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Add traces to the main thread inflation of NotificationHeaderView and... 

Merge "Add traces to the main thread inflation of NotificationHeaderView and HybridNotificationView" into udc-dev am: 8386f847

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/23482442



Change-Id: Ic9b8e48e2701577a739ab58655e42ec8b85f23a0
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 22f3b82b 8386f847

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

+10 −0
Original line number Diff line number Diff line
@@ -556,6 +556,11 @@ public class ExpandableNotificationRow extends ActivatableNotificationView 
    }



    public void onNotificationUpdated() {

        if (mIsSummaryWithChildren) {

            Trace.beginSection("ExpNotRow#onNotifUpdated (summary)");

        } else {

            Trace.beginSection("ExpNotRow#onNotifUpdated (leaf)");

        }

        for (NotificationContentView l : mLayouts) {

            l.onNotificationUpdated(mEntry);

        }
@@ -591,6 +596,7 @@ public class ExpandableNotificationRow extends ActivatableNotificationView 
            mUpdateSelfBackgroundOnUpdate = false;

            updateBackgroundColorsOfSelf();

        }

        Trace.endSection();

    }



    private void updateBackgroundColorsOfSelf() {
@@ -2588,6 +2594,7 @@ public class ExpandableNotificationRow extends ActivatableNotificationView 
        mIsSummaryWithChildren = mChildrenContainer != null

                && mChildrenContainer.getNotificationChildCount() > 0;

        if (mIsSummaryWithChildren) {

            Trace.beginSection("ExpNotRow#onChildCountChanged (summary)");

            NotificationViewWrapper wrapper = mChildrenContainer.getNotificationViewWrapper();

            if (wrapper == null || wrapper.getNotificationHeader() == null) {

                mChildrenContainer.recreateNotificationHeader(mExpandClickListener,
@@ -2599,6 +2606,9 @@ public class ExpandableNotificationRow extends ActivatableNotificationView 
        updateChildrenAppearance();

        updateChildrenVisibility();

        applyChildrenRoundness();

        if (mIsSummaryWithChildren) {

            Trace.endSection();

        }

    }



    protected void expandNotification() {

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

+9 −0
Original line number Diff line number Diff line
@@ -22,6 +22,7 @@ import android.annotation.Nullable; 
import android.app.Notification;

import android.content.Context;

import android.content.res.Resources;

import android.os.Trace;

import android.service.notification.StatusBarNotification;

import android.util.TypedValue;

import android.view.LayoutInflater;
@@ -57,6 +58,7 @@ public class HybridGroupManager { 
    }



    private HybridNotificationView inflateHybridView(View contentView, ViewGroup parent) {

        Trace.beginSection("HybridGroupManager#inflateHybridView");

        LayoutInflater inflater = LayoutInflater.from(mContext);

        int layout = contentView instanceof ConversationLayout

                ? R.layout.hybrid_conversation_notification
@@ -64,6 +66,7 @@ public class HybridGroupManager { 
        HybridNotificationView hybrid = (HybridNotificationView)

                inflater.inflate(layout, parent, false);

        parent.addView(hybrid);

        Trace.endSection();

        return hybrid;

    }
@@ -90,12 +93,18 @@ public class HybridGroupManager { 
    public HybridNotificationView bindFromNotification(HybridNotificationView reusableView,

            View contentView, StatusBarNotification notification,

            ViewGroup parent) {

        boolean isNewView = false;

        if (reusableView == null) {

            Trace.beginSection("HybridGroupManager#bindFromNotification");

            reusableView = inflateHybridView(contentView, parent);

            isNewView = true;

        }

        CharSequence titleText = resolveTitle(notification.getNotification());

        CharSequence contentText = resolveText(notification.getNotification());

        reusableView.bind(titleText, contentText, contentView);

        if (isNewView) {

            Trace.endSection();

        }

        return reusableView;

    }

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

+3 −0
Original line number Diff line number Diff line
@@ -26,6 +26,7 @@ import android.graphics.Rect; 
import android.graphics.drawable.Drawable;

import android.os.Build;

import android.os.RemoteException;

import android.os.Trace;

import android.service.notification.StatusBarNotification;

import android.util.ArrayMap;

import android.util.AttributeSet;
@@ -1193,6 +1194,7 @@ public class NotificationContentView extends FrameLayout implements Notification 


    private void updateSingleLineView() {

        if (mIsChildInGroup) {

            Trace.beginSection("NotifContentView#updateSingleLineView");

            boolean isNewView = mSingleLineView == null;

            mSingleLineView = mHybridGroupManager.bindFromNotification(

                    mSingleLineView, mContractedChild, mNotificationEntry.getSbn(), this);
@@ -1200,6 +1202,7 @@ public class NotificationContentView extends FrameLayout implements Notification 
                updateViewVisibility(mVisibleType, VISIBLE_TYPE_SINGLELINE,

                        mSingleLineView, mSingleLineView);

            }

            Trace.endSection();

        } else if (mSingleLineView != null) {

            removeView(mSingleLineView);

            mSingleLineView = null;

packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/NotificationChildrenContainer.java

+2 −0
Original line number Diff line number Diff line
@@ -379,6 +379,7 @@ public class NotificationChildrenContainer extends ViewGroup 
    }



    public void recreateNotificationHeader(OnClickListener listener, boolean isConversation) {

        Trace.beginSection("NotifChildCont#recreateHeader");

        mHeaderClickListener = listener;

        mIsConversation = isConversation;

        StatusBarNotification notification = mContainingNotification.getEntry().getSbn();
@@ -406,6 +407,7 @@ public class NotificationChildrenContainer extends ViewGroup 
        recreateLowPriorityHeader(builder, isConversation);

        updateHeaderVisibility(false /* animate */);

        updateChildrenAppearance();

        Trace.endSection();

    }



    /**