Bug 81088 - Web Inspector: extract CompilerScriptMapping from RawSourceCode.
Summary: Web Inspector: extract CompilerScriptMapping from RawSourceCode.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Pavel Podivilov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-14 01:01 PDT by Pavel Podivilov
Modified: 2012-03-15 10:03 PDT (History)
10 users (show)

See Also:

Attachments
Patch (48.63 KB, patch)
2012-03-14 01:09 PDT, Pavel Podivilov 		no flags Details | Formatted Diff | Diff
Patch (42.81 KB, patch)
2012-03-14 09:26 PDT, Pavel Podivilov 		vsevik: review+
Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Pavel Podivilov 2012-03-14 01:01:42 PDT 
Web Inspector: extract CompilerScriptMapping from RawSourceCode.
Comment 1 Pavel Podivilov 2012-03-14 01:09:16 PDT 
Created attachment 131804 [details]
Patch
Comment 2 Pavel Podivilov 2012-03-14 09:26:25 PDT 
Created attachment 131869 [details]
Patch
Comment 3 Vsevolod Vlasov 2012-03-15 05:32:55 PDT 
Comment on attachment 131869 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=131869&action=review

> Source/WebCore/inspector/front-end/CompilerScriptMapping.js:1
> +/*

Could you please add new file to compilation.

> Source/WebCore/inspector/front-end/CompilerScriptMapping.js:137
> +        for (var url in this._uiSourceCodeByURL) {

This loop does not make sense, you probably meant to dispatch the event once.
Comment 4 Pavel Podivilov 2012-03-15 10:03:06 PDT 
Committed r110856: <http://trac.webkit.org/changeset/110856>