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

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

Merge "Catch unexpected detached view while creating FrameTracker" into tm-qpr-dev am: 05914b98 

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



Change-Id: I70f58c963ee1e25a276e88a290d3f6bda3f94c8b
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 06eab40b 05914b98

core/java/com/android/internal/jank/InteractionJankMonitor.java

+35 −2
Original line number Diff line number Diff line
@@ -439,6 +439,22 @@ public class InteractionJankMonitor { 
    @VisibleForTesting

    public FrameTracker createFrameTracker(Configuration config, Session session) {

        final View view = config.mView;



        if (!config.hasValidView()) {

            boolean attached = false;

            boolean hasViewRoot = false;

            boolean hasRenderer = false;

            if (view != null) {

                attached = view.isAttachedToWindow();

                hasViewRoot = view.getViewRootImpl() != null;

                hasRenderer = view.getThreadedRenderer() != null;

            }

            Log.d(TAG, "create FrameTracker fails: view=" + view

                    + ", attached=" + attached + ", hasViewRoot=" + hasViewRoot

                    + ", hasRenderer=" + hasRenderer, new Throwable());

            return null;

        }



        final ThreadedRendererWrapper threadedRenderer =

                view == null ? null : new ThreadedRendererWrapper(view.getThreadedRenderer());

        final ViewRootWrapper viewRoot =
@@ -545,6 +561,7 @@ public class InteractionJankMonitor { 


        // begin a new trace session.

        tracker = createFrameTracker(conf, new Session(cujType, conf.mTag));

        if (tracker == null) return false;

        putTracker(cujType, tracker);

        tracker.begin();
@@ -1066,9 +1083,19 @@ public class InteractionJankMonitor { 
                    msg.append("Must pass in a valid surface control if only instrument surface; ");

                }

            } else {

                if (mView == null || !mView.isAttachedToWindow()) {

                if (!hasValidView()) {

                    shouldThrow = true;

                    msg.append("Null view or unattached view while instrumenting view; ");

                    boolean attached = false;

                    boolean hasViewRoot = false;

                    boolean hasRenderer = false;

                    if (mView != null) {

                        attached = mView.isAttachedToWindow();

                        hasViewRoot = mView.getViewRootImpl() != null;

                        hasRenderer = mView.getThreadedRenderer() != null;

                    }

                    String err = "invalid view: view=" + mView + ", attached=" + attached

                            + ", hasViewRoot=" + hasViewRoot + ", hasRenderer=" + hasRenderer;

                    msg.append(err);

                }

            }

            if (shouldThrow) {
@@ -1076,6 +1103,12 @@ public class InteractionJankMonitor { 
            }

        }



        boolean hasValidView() {

            return mSurfaceOnly

                    || (mView != null && mView.isAttachedToWindow()

                    && mView.getViewRootImpl() != null && mView.getThreadedRenderer() != null);

        }



        /**

         * @return true if only instrumenting surface, false otherwise

         */