Summary: | Web Inspector: Simplify script mappings by introducing DefaultScriptMapping. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Vsevolod Vlasov <vsevik> | ||||||
Component: | Web Inspector (Deprecated) | Assignee: | Vsevolod Vlasov <vsevik> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | apavlov, keishi, loislo, pfeldman, pmuellr, vsevik, web-inspector-bugs, yurys | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Vsevolod Vlasov
2012-11-22 01:47:07 PST
Created attachment 175623 [details]
Patch
Created attachment 175625 [details]
Patch
Comment on attachment 175625 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=175625&action=review > Source/WebCore/ChangeLog:8 > + Additional information of the change such as approach, rationale. Please add per-function descriptions below (OOPS!). Remove this line. > Source/WebCore/inspector/front-end/DefaultScriptMapping.js:55 > + return new WebInspector.UILocation(uiSourceCode, lineNumber, columnNumber); We should finally make it a factory method on UISourceCode. Committed r138496: <http://trac.webkit.org/changeset/138496> |