Bug 137038

Summary: Web Inspector: Automatic Inspection should continue once all breakpoints are loaded
Product: WebKit Reporter: Joseph Pecoraro <joepeck>
Component: Web InspectorAssignee: Joseph Pecoraro <joepeck>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, graouts, joepeck, timothy, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
[PATCH] Proposed Fix
timothy: review+, joepeck: commit-queue-
[PATCH] Proposed Fix none

Description Joseph Pecoraro 2014-09-23 12:44:45 PDT 
Currently Automatic Inspection pauses immediately. This gives the frontend time to send to the backend all the breakpoints it knows about, so by the time the backend continues it has all the breakpoints loaded.

Instead of enforcing this automatic pause on auto-inspect, we should just pause the JSContext until all the frontend's state is loaded, including all the breakpoints.
Comment 1 Radar WebKit Bug Importer 2014-09-23 12:45:36 PDT 
<rdar://problem/18429875>
Comment 2 Joseph Pecoraro 2014-09-23 12:54:13 PDT 
Created attachment 238559 [details]
[PATCH] Proposed Fix
Comment 3 Timothy Hatcher 2014-09-23 13:33:07 PDT 
Comment on attachment 238559 [details]
[PATCH] Proposed Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=238559&action=review

> Source/JavaScriptCore/inspector/remote/RemoteInspectorDebuggable.cpp:94
> +void RemoteInspectorDebuggable::unpauseInspectorInitialized()

The wording here is kind of confusing. Lets go with something like: unpauseForInitializedInspector.
Comment 4 Joseph Pecoraro 2014-09-23 14:05:09 PDT 
Created attachment 238570 [details]
[PATCH] Proposed Fix

See how this looks on the bots.
Comment 5 WebKit Commit Bot 2014-09-26 16:25:22 PDT 
Comment on attachment 238570 [details]
[PATCH] Proposed Fix

Clearing flags on attachment: 238570

Committed r174028: <http://trac.webkit.org/changeset/174028>
Comment 6 WebKit Commit Bot 2014-09-26 16:25:25 PDT 
All reviewed patches have been landed.  Closing bug.