Summary: | Web Inspector: breakpoints in named evals are not restored after a reload | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Patrick Mueller <pmuellr> | ||||||
Component: | Web Inspector (Deprecated) | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bweinstein, commit-queue, joepeck, keishi, pfeldman, pmuellr, rik, timothy | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Patrick Mueller
2009-11-11 14:13:52 PST
We could try to reinstall them by matching up the @sourceURL. Created attachment 43002 [details]
preliminary patch
The patches are actually simpler than that - the code was pretty much set up to handle this, except the calculated sourceURL wasn't used in ScriptsPanel.addScript(), and in the same method the _sourceIDMap[] map entry is simply set too late for the big if block to take advantage of it.
I'm reusing an existing manual test case to test this, but haven't changed the manual description; I'll do that tomorrow, but thought folks might want to see the current patch. Looks reasonable.
Also note, I was planning on reorging the breakpoint stuff with this change, but now I'm thinking that I should just do the re-org in a separate patch. I suspect it will be messy.
This patch looks good. Make a ChangeLog and I will r+ it. Created attachment 43063 [details]
proposed patch 2009/11/12 - a
per previous comment:
- in ScriptsPanel::addScript(), changed to use the calculated sourceURL rather than the original sourceURL. The calculated one will have an actual name if the script was an eval that used the //@sourceURL= annotation comment
- in ScriptsPanel::addScript(), moved the cache-filling of _sourceIDmap[] to ABOVE the place it's used, later, in a called function, to paint the breakpoint marker in the source panel; without this fix, everything works, but the breakpoint markers are not repainted for these breakpoints
- added a new manual test case
Comment on attachment 43063 [details] proposed patch 2009/11/12 - a Clearing flags on attachment: 43063 Committed r50880: <http://trac.webkit.org/changeset/50880> All reviewed patches have been landed. Closing bug. |