RESOLVED FIXED Bug 97231
Web Inspector: Provide a way to distinguish scripts having sourceURL from standalone scripts. 
https://bugs.webkit.org/show_bug.cgi?id=97231
Summary Web Inspector: Provide a way to distinguish scripts having sourceURL from sta...
Vsevolod Vlasov
Reported 2012-09-20 10:20:06 PDT
Currently sourceURL field could be set for - standalone scripts (<script src=foo.js></script>) - evals - scripts in dynamically created script element To distinguish between first two we should provide hasSourceURL flag for each script.
Attachments
Patch (13.15 KB, patch)
2012-09-20 10:25 PDT, Vsevolod Vlasov 		no flags
DetailsFormatted DiffDiff
Patch (13.11 KB, patch)
2012-09-20 10:28 PDT, Vsevolod Vlasov 		pfeldman: review+
peter+ews: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Vsevolod Vlasov
Comment 1 2012-09-20 10:24:35 PDT
This patch depends on https://codereview.chromium.org/10966011/ landed in v8.
Vsevolod Vlasov
Comment 2 2012-09-20 10:25:20 PDT
Created attachment 164939 [details] Patch
Vsevolod Vlasov
Comment 3 2012-09-20 10:28:58 PDT
Created attachment 164940 [details] Patch
Peter Beverloo (cr-android ews)
Comment 4 2012-09-20 10:47:17 PDT
Comment on attachment 164940 [details] Patch Attachment 164940 [details] did not pass cr-android-ews (chromium-android): Output: http://queues.webkit.org/results/13955258
WebKit Review Bot
Comment 5 2012-09-20 12:55:49 PDT
Comment on attachment 164940 [details] Patch Attachment 164940 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/13935277
Vsevolod Vlasov
Comment 6 2012-10-02 01:27:54 PDT
Committed r130138: <http://trac.webkit.org/changeset/130138>
Ojan Vafai
Comment 8 2012-10-02 12:16:28 PDT
Added to TestExpectations for now: http://trac.webkit.org/changeset/130193
Note You need to log in before you can comment on or make changes to this bug.