Title: [198563] trunk/Source/WebInspectorUI
Revision
198563
Author
[email protected]
Date
2016-03-22 16:47:12 -0700 (Tue, 22 Mar 2016)

Log Message

Web Inspector: REGRESSION: Switching Timelines via Path Component does not work
https://bugs.webkit.org/show_bug.cgi?id=155767
<rdar://problem/25298945>

Patch by Joseph Pecoraro <[email protected]> on 2016-03-22
Reviewed by Timothy Hatcher.

* UserInterface/Views/TimelineRecordingContentView.js:
(WebInspector.TimelineRecordingContentView.prototype._timelinePathComponentSelected):
(WebInspector.TimelineRecordingContentView.prototype._instrumentAdded):

Modified Paths

Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (198562 => 198563)


--- trunk/Source/WebInspectorUI/ChangeLog	2016-03-22 23:33:30 UTC (rev 198562)
+++ trunk/Source/WebInspectorUI/ChangeLog	2016-03-22 23:47:12 UTC (rev 198563)
@@ -1,3 +1,15 @@
+2016-03-22  Joseph Pecoraro  <[email protected]>
+
+        Web Inspector: REGRESSION: Switching Timelines via Path Component does not work
+        https://bugs.webkit.org/show_bug.cgi?id=155767
+        <rdar://problem/25298945>
+
+        Reviewed by Timothy Hatcher.
+
+        * UserInterface/Views/TimelineRecordingContentView.js:
+        (WebInspector.TimelineRecordingContentView.prototype._timelinePathComponentSelected):
+        (WebInspector.TimelineRecordingContentView.prototype._instrumentAdded):
+
 2016-03-22  Timothy Hatcher  <[email protected]>
 
         Web Inspector: Consider Automation script names as internal and hide them

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordingContentView.js (198562 => 198563)


--- trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordingContentView.js	2016-03-22 23:33:30 UTC (rev 198562)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordingContentView.js	2016-03-22 23:47:12 UTC (rev 198563)
@@ -371,8 +371,10 @@
         this.dispatchEventToListeners(WebInspector.ContentView.Event.NavigationItemsDidChange);
     }
 
-    _pathComponentSelected(event)
+    _timelinePathComponentSelected(event)
     {
+        let selectedTimeline = event.data.pathComponent.representedObject;
+        this.showTimelineViewForTimeline(selectedTimeline);
     }
 
     _timeRangePathComponentSelected(event)
@@ -601,7 +603,7 @@
         let displayName = WebInspector.TimelineTabContentView.displayNameForTimelineType(timeline.type);
         let iconClassName = WebInspector.TimelineTabContentView.iconClassNameForTimelineType(timeline.type);
         let pathComponent = new WebInspector.HierarchicalPathComponent(displayName, iconClassName, timeline);
-        pathComponent.addEventListener(WebInspector.HierarchicalPathComponent.Event.SiblingWasSelected, this._pathComponentSelected, this);
+        pathComponent.addEventListener(WebInspector.HierarchicalPathComponent.Event.SiblingWasSelected, this._timelinePathComponentSelected, this);
         this._pathComponentMap.set(timeline, pathComponent);
 
         this._timelineCountChanged();
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to