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

Commit 68266197 authored by Ahan Wu's avatar Ahan Wu
Browse files

Catch unexpected detached view while creating FrameTracker 

Bug: 245083316
Test: atest FrameTrackerTest InteractionJankMonitorTest
Test: LatencyTrackerTest
Change-Id: I55941c78b7200bd70a4b30a29defc3c8306eead8
parent 64d61c11

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

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

    @VisibleForTesting

    public FrameTracker createFrameTracker(Configuration config, Session session) {

    public FrameTracker createFrameTracker(Configuration config, Session session) {

        final View view = config.mView;

        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 =

        final ThreadedRendererWrapper threadedRenderer =

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

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

        final ViewRootWrapper viewRoot =

        final ViewRootWrapper viewRoot =
@@ -541,6 +557,7 @@ public class InteractionJankMonitor { 




        // begin a new trace session.

        // begin a new trace session.

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

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

        if (tracker == null) return false;

        putTracker(cujType, tracker);

        putTracker(cujType, tracker);

        tracker.begin();

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

                    msg.append("Must pass in a valid surface control if only instrument surface; ");

                }

                }

            } else {

            } else {

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

                if (!hasValidView()) {

                    shouldThrow = true;

                    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) {

            if (shouldThrow) {
@@ -1068,6 +1095,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

         * @return true if only instrumenting surface, false otherwise

         */

         */