Title: [202353] trunk/Source/WebInspectorUI
Revision
202353
Author
[email protected]
Date
2016-06-22 15:19:22 -0700 (Wed, 22 Jun 2016)

Log Message

Web Inspector: don't start auto capturing if the Inspector window is not visible
https://bugs.webkit.org/show_bug.cgi?id=159014

Reviewed by Joseph Pecoraro.
<rdar://problem/26931269>

Followup, add a missing super.closed() call mentioned in review comments.

* UserInterface/Views/TimelineTabContentView.js:
(WebInspector.TimelineTabContentView.prototype.closed):

Modified Paths

Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (202352 => 202353)


--- trunk/Source/WebInspectorUI/ChangeLog	2016-06-22 22:05:40 UTC (rev 202352)
+++ trunk/Source/WebInspectorUI/ChangeLog	2016-06-22 22:19:22 UTC (rev 202353)
@@ -2,8 +2,21 @@
 
         Web Inspector: don't start auto capturing if the Inspector window is not visible
         https://bugs.webkit.org/show_bug.cgi?id=159014
+
+        Reviewed by Joseph Pecoraro.
         <rdar://problem/26931269>
 
+        Followup, add a missing super.closed() call mentioned in review comments.
+
+        * UserInterface/Views/TimelineTabContentView.js:
+        (WebInspector.TimelineTabContentView.prototype.closed):
+
+2016-06-22  Brian Burg  <[email protected]>
+
+        Web Inspector: don't start auto capturing if the Inspector window is not visible
+        https://bugs.webkit.org/show_bug.cgi?id=159014
+        <rdar://problem/26931269>
+
         Reviewed by Joseph Pecoraro.
 
         TimelineTabContentView should not tell the timeline manager to enable

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/TimelineTabContentView.js (202352 => 202353)


--- trunk/Source/WebInspectorUI/UserInterface/Views/TimelineTabContentView.js	2016-06-22 22:05:40 UTC (rev 202352)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/TimelineTabContentView.js	2016-06-22 22:19:22 UTC (rev 202353)
@@ -295,6 +295,8 @@
 
     closed()
     {
+        super.closed();
+
         if (WebInspector.FPSInstrument.supported())
             this.contentBrowser.navigationBar.removeEventListener(null, null, this);
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to