Bug 117877

Summary: Web Inspector: REGRESSION: Breakpoints often missed when reloading page
Product: WebKit Reporter: Timothy Hatcher <timothy>
Component: Web InspectorAssignee: Timothy Hatcher <timothy>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, graouts, joepeck, timothy, webkit-bug-importer
Priority: P2 Keywords: InRadar, Regression
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Patch none

Timothy Hatcher
Reported 2013-06-21 08:20:09 PDT
Setting a breakpoint to trigger on page load, it often doesn't trigger on reloads. I see this primarily in the remote scenario; most likely because of the larger delay and asynchronisity between messages. Steps: 1. On an iOS device/simulator load <http://bogojoker.com/shell/> 2. Open a remote inspector 3. Set a brekapoint on easySlider.min.js:2 or :3 (pretty printed) 4. Reload the page with Cmd+R in the inspector => sometimes the breakpoint is not hit Notes: In the protocol I'm seeing the frontend remove the breakpoint and readd it frequently. Why this extra churn? <rdar://problem/13955160>
Attachments
Patch (2.35 KB, patch)
2013-06-21 08:27 PDT, Timothy Hatcher
no flags
Patch (3.52 KB, patch)
2013-06-21 08:38 PDT, Timothy Hatcher
no flags
Timothy Hatcher
Comment 1 2013-06-21 08:27:07 PDT
Timothy Hatcher
Comment 2 2013-06-21 08:38:18 PDT
I have a better fix.
Timothy Hatcher
Comment 3 2013-06-21 08:38:57 PDT
Joseph Pecoraro
Comment 4 2013-06-21 10:53:37 PDT
Comment on attachment 205188 [details] Patch r=me
WebKit Commit Bot
Comment 5 2013-06-21 11:17:22 PDT
Comment on attachment 205188 [details] Patch Clearing flags on attachment: 205188 Committed r151846: <http://trac.webkit.org/changeset/151846>
WebKit Commit Bot
Comment 6 2013-06-21 11:17:24 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.