Web Inspector: set breakpoints by line:column.
Created attachment 79251 [details] Patch.
Created attachment 79254 [details] Patch.
Comment on attachment 79254 [details] Patch. View in context: https://bugs.webkit.org/attachment.cgi?id=79254&action=review Please address the comments before landing. > Source/WebCore/bindings/js/ScriptDebugServer.cpp:153 > + *actualColumnNumber = 1; Please file a bug against JSC and add FIXME comment here. > Source/WebCore/bindings/js/ScriptDebugServer.h:138 > + typedef HashMap<long, ScriptBreakpoint> ScriptBreakpoints; ScriptBreakpoints -> LineToScriptBreakpointMap > Source/WebCore/bindings/js/ScriptDebugServer.h:139 > + typedef HashMap<intptr_t, ScriptBreakpoints> BreakpointsMap; BreakpointsMap -> SourceIdToBreakpointsMap, I'd also recommend renaming corresponding fields as m_breakpoints sounds too general. > Source/WebCore/inspector/InspectorDebuggerAgent.h:100 > + typedef HashMap<Location, ScriptBreakpoint> ScriptBreakpoints; ScriptBreakpoints -> LocationToBreakpointMap
Committed r76509: <http://trac.webkit.org/changeset/76509>
http://trac.webkit.org/changeset/76509 might have broken Leopard Intel Release (Build) and Leopard Intel Debug (Build)