Bug 157194

Summary: Web Inspector: Jump to Line in auto formatted JavaScript does not work the first time
Product: WebKit Reporter: Joseph Pecoraro <joepeck>
Component: Web InspectorAssignee: Joseph Pecoraro <joepeck>
Status: RESOLVED FIXED    
Severity: Normal CC: bburg, commit-queue, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
[PATCH] Proposed Fix none

Joseph Pecoraro
Reported 2016-04-29 11:19:24 PDT
* SUMMARY Jump to Line in auto formatted JavaScript does not work the first time. * STEPS TO REPRODUCE 1. Inspect <http://bogojoker.com/shell/> 2. Set a breakpoint on formatted easySlider.min.js:56:21 ("if (autoplay)") 3. Select another resource in sidebar 4. Reload 5. Select the breakpoint => Jump to line fails to reveal the right line 6. Select the breakpoint again => Jump to line succeeds
Attachments
[PATCH] Proposed Fix (4.01 KB, patch)
2016-04-29 12:05 PDT, Joseph Pecoraro 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2016-04-29 11:20:33 PDT
<rdar://problem/26008471>
Joseph Pecoraro
Comment 2 2016-04-29 12:05:18 PDT
Created attachment 277726 [details] [PATCH] Proposed Fix This is a quick solution. Really we should rework the interface between TextEditor and SourceCodeTextEditor quite a bit.
WebKit Commit Bot
Comment 3 2016-04-29 14:08:12 PDT
Comment on attachment 277726 [details] [PATCH] Proposed Fix Clearing flags on attachment: 277726 Committed r200262: <http://trac.webkit.org/changeset/200262>
WebKit Commit Bot
Comment 4 2016-04-29 14:08:15 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.