[Webkit-unassigned] [Bug 159014] New: Web Inspector: don't start auto capturing if the Inspector window is not visible

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=159014

            Bug ID: 159014
           Summary: Web Inspector: don't start auto capturing if the
                    Inspector window is not visible
    Classification: Unclassified
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: All
                OS: All
            Status: NEW
          Keywords: InRadar
          Severity: Normal
          Priority: P2
         Component: Web Inspector
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: bburg at apple.com
                CC: bburg at apple.com, joepeck at webkit.org,
                    mattbaker at apple.com, nvasilyev at apple.com,
                    timothy at apple.com, webkit-bug-importer at group.apple.com

After https://webkit.org/b/159004, we still sometimes can't hit a `debugger` statement from Automation.evaluateJavaScriptFunction. This happens if the Timelines tab was last visible in the previous Web Inspector session.

1. When Automation.inspectBrowsingContext runs, it opens up the Inspector, which restores the last shown tab.
2. TimelineTabContentView.shown() is called, and it set WebInspector.timelineManager.autoCaptureOnPageLoad = true
3. TimelineManager tells the backend to enable auto capturing
4. Shortly thereafter, the backend disables breakpoints and tells the frontend that auto capturing started.

I think it is easiest to alter TimelineTabContentView to not set autoCaptureOnPageLoad if WebInspector.visible is not true. Also it should listen to the VisibilityStateChanged notification and re-enable it when the page becomes visible.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160622/1fc3a92f/attachment.html>