RESOLVED FIXED Bug 97680
Web Inspector: Move UISourceCode creation out from ResourceScriptMapping. 
https://bugs.webkit.org/show_bug.cgi?id=97680
Summary Web Inspector: Move UISourceCode creation out from ResourceScriptMapping.
Vsevolod Vlasov
Reported 2012-09-26 07:07:24 PDT
UISourceCodes for scripts having sourceURL are now created by NetworkUISourceCodeProvider. UISourceCodes for anonymous, dynamic and concatenated scripts are now created on demand only. All UISourceCodes created by ResourceScriptMapping are now "temporary". Temporary UISourceCodes are not stored in workspace and removed when a normal UISourceCode with the same url is available. UISourceCodeReplaced event was replaced with TemporaryUISourceCodeAdded/Removed events.
Attachments
Patch (62.91 KB, patch)
2012-09-26 07:21 PDT, Vsevolod Vlasov 		no flags
DetailsFormatted DiffDiff
Patch (49.47 KB, patch)
2012-09-26 07:52 PDT, Vsevolod Vlasov 		pfeldman: review+
webkit.review.bot: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Vsevolod Vlasov
Comment 1 2012-09-26 07:21:23 PDT
Created attachment 165797 [details] Patch
Pavel Feldman
Comment 2 2012-09-26 07:49:32 PDT
Comment on attachment 165797 [details] Patch Too much ChangeLog!
Vsevolod Vlasov
Comment 3 2012-09-26 07:52:39 PDT
Created attachment 165801 [details] Patch
WebKit Review Bot
Comment 4 2012-09-26 08:51:12 PDT
Comment on attachment 165801 [details] Patch Attachment 165801 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/14019569 New failing tests: inspector/debugger/source-url-comment.html
Build Bot
Comment 5 2012-09-27 02:17:28 PDT
Comment on attachment 165801 [details] Patch Attachment 165801 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/14009055 New failing tests: inspector/debugger/source-url-comment.html
Vsevolod Vlasov
Comment 6 2012-10-02 05:12:53 PDT
Committed r130155: <http://trac.webkit.org/changeset/130155>
Note You need to log in before you can comment on or make changes to this bug.