Attachment 356273 Details for Bug 192248 – [PATCH] Proposed Fix

[patch] [PATCH] Proposed Fix

timeline-1.patch (text/plain), 2.69 KB, created by Joseph Pecoraro on 2018-11-30 17:46:29 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Joseph Pecoraro

Created: 2018-11-30 17:46:29 PST

Size: 2.69 KB

patch

obsolete

>diff --git a/Source/WebInspectorUI/ChangeLog b/Source/WebInspectorUI/ChangeLog
>index 55e1def1224..7d821964925 100644
>--- a/Source/WebInspectorUI/ChangeLog
>+++ b/Source/WebInspectorUI/ChangeLog
>@@ -1,3 +1,22 @@
>+2018-11-30  Joseph Pecoraro  <pecoraro@apple.com>
>+
>+        Web Inspector: REGRESSION(r238330): Timeline auto-capture does not work after page transition
>+        https://bugs.webkit.org/show_bug.cgi?id=192248
>+        <rdar://problem/46390199>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * UserInterface/Base/Main.js:
>+        (WI.transitionPageTarget):
>+        Let the TimelineManager perform work on page transitions.
>+
>+        * UserInterface/Controllers/TimelineManager.js:
>+        (WI.TimelineManager.prototype.initializeTarget):
>+        Initialize the autocapture state of the target.
>+
>+        (WI.TimelineManager.prototype.transitionPageTarget):
>+        When transitioning pages perform a legacy (frontend based) timeline capture.
>+
> 2018-11-30  Nikita Vasilyev  <nvasilyev@apple.com>
> 
>         Web Inspector: Jumping from Computed to Styles should select property
>diff --git a/Source/WebInspectorUI/UserInterface/Base/Main.js b/Source/WebInspectorUI/UserInterface/Base/Main.js
>index 39dd620afd1..0b388d759a9 100644
>--- a/Source/WebInspectorUI/UserInterface/Base/Main.js
>+++ b/Source/WebInspectorUI/UserInterface/Base/Main.js
>@@ -213,6 +213,7 @@ WI.transitionPageTarget = function(target)
>     // Actions to transition the page target.
>     this.notifications.dispatchEventToListeners(WI.Notification.TransitionPageTarget);
>     WI.domManager.transitionPageTarget();
>+    WI.timelineManager.transitionPageTarget();
>     WI.networkManager.transitionPageTarget();
> };
> 
>diff --git a/Source/WebInspectorUI/UserInterface/Controllers/TimelineManager.js b/Source/WebInspectorUI/UserInterface/Controllers/TimelineManager.js
>index 4414e731d2e..7c58e3f5a74 100644
>--- a/Source/WebInspectorUI/UserInterface/Controllers/TimelineManager.js
>+++ b/Source/WebInspectorUI/UserInterface/Controllers/TimelineManager.js
>@@ -65,7 +65,19 @@ WI.TimelineManager = class TimelineManager extends WI.Object
> 
>     initializeTarget(target)
>     {
>-        this._updateAutoCaptureInstruments([target]);
>+        if (target.TimelineAgent) {
>+            this._updateAutoCaptureInstruments([target]);
>+
>+            // COMPATIBILITY (iOS 9): Timeline.setAutoCaptureEnabled did not exist.
>+            if (target.TimelineAgent.setAutoCaptureEnabled)
>+                target.TimelineAgent.setAutoCaptureEnabled(this._autoCaptureOnPageLoad);
>+        }
>+    }
>+
>+    transitionPageTarget()
>+    {
>+        if (this._autoCaptureOnPageLoad)
>+            this._legacyAttemptStartAutoCapturingForFrame(WI.networkManager.mainFrame);
>     }
> 
>     // Static

Flags:

hi: review-

Actions: View | Formatted Diff | Diff

Attachments on bug 192248: 356273 | 356434 | 356488