Web Inspector: stop using RawSourceCode in BreakpointManager. See bug 79130 for why it's needed.
Created attachment 130135 [details] Patch
Comment on attachment 130135 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=130135&action=review > Source/WebCore/inspector/front-end/BreakpointManager.js:37 > + * @param {WebInspector.DebuggerPresentationModel} sourceMapping Maybe scriptMapping? Could you please add an interface (ScriptMapping?) and implement it in DebuggerPresentationModel? > Source/WebCore/inspector/front-end/DebuggerPresentationModel.js:45 > + this._breakpointManager = new WebInspector.BreakpointManager(WebInspector.settings.breakpoints, I think one long line would be better.
Created attachment 130153 [details] Patch
Comment on attachment 130153 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=130153&action=review > Source/WebCore/inspector/front-end/DebuggerPresentationModel.js:45 > + this._breakpointManager = new WebInspector.BreakpointManager(WebInspector.settings.breakpoints, One line? > Source/WebCore/inspector/front-end/ScriptMapping.js:48 > + rawLocationToUILocation: function(rawLocation) rawLocationToUILocation: function(rawLocation) { }, > Source/WebCore/inspector/front-end/ScriptMapping.js:58 > + uiLocationToRawLocation: function(uiSourceCode, lineNumber, columnNumber) ditto > Source/WebCore/inspector/front-end/ScriptMapping.js:68 > + { ditto > Source/WebCore/inspector/front-end/ScriptMapping.js:75 > + { ditto
Committed r109872: <http://trac.webkit.org/changeset/109872>