Changeset 202353 in webkit

Timestamp:

Jun 22, 2016, 3:19:22 PM (9 years ago)

Author:

BJ Burg

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

Location:

trunk/Source/WebInspectorUI

Files:

• ChangeLog (modified) (1 diff)
• UserInterface/Views/TimelineTabContentView.js (modified) (1 diff)

trunk/Source/WebInspectorUI/ChangeLog

@@ +1 @@
+2016-06-22  Brian Burg  <bburg@apple.com>
+
+        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  <bburg@apple.com>
 

trunk/Source/WebInspectorUI/UserInterface/Views/TimelineTabContentView.js

@@ -296 +296 @@
     closed()
     {
+        super.closed();
+
         if (WebInspector.FPSInstrument.supported())
             this.contentBrowser.navigationBar.removeEventListener(null, null, this);