Bug 66196
Summary: | Web Inspector: [protocol] LiveEdit does not report about a new script | ||
---|---|---|---|
Product: | WebKit | Reporter: | Peter Rybin <peter.rybin> |
Component: | Web Inspector (Deprecated) | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED INVALID | ||
Severity: | Normal | CC: | apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, yurys |
Priority: | P2 | ||
Version: | 528+ (Nightly build) | ||
Hardware: | All | ||
OS: | All |
Peter Rybin
-- create a script with a function;
-- in live edit add a function parameter; save;
expected:
protocol reports that new script appears
actual:
protocol doesn't mention a new script
-- reconnect debugger; see that the new script is there (named "...(old)")
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Yury Semikhatsky
The problem here is that when the script is edited we enter debugger context before calling Debug.LiveEdit.SetScriptSource and the context is used to compile the new script which is wrong. Instead live-edit code should compute the context where the original script has been compiled and use it for compiling the new script.
Brian Burg
Closing as invalid, as this bug pertains to the old inspector UI and/or its tests.
Please file a new bug (https://www.webkit.org/new-inspector-bug) if the bug/feature/issue is still relevant to WebKit trunk.