This change allows setting all saved breakpoints in debugger model by url before the site with corresponding scripts was loaded (allows breaking in onload handler on first load). This change also simplifies implementation of snippets debugging.
Created attachment 147294 [details] Patch
Comment on attachment 147294 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=147294&action=review > Source/WebCore/inspector/front-end/DebuggerModel.js:59 > +WebInspector.DebuggerModel.Location = function(scriptId, sourceURL, lineNumber, columnNumber) Shouldn't sourceURL be optional for debugger location? For example it does not exist for eval scripts. > Source/WebCore/inspector/front-end/DebuggerModel.js:136 > setBreakpointByScriptLocation: function(rawLocation, condition, callback) Should we be more specific here and introduce another method instead?
Created attachment 147598 [details] Patch
Committed r120335: <http://trac.webkit.org/changeset/120335>