Execution line in selected call frame should keep showing after toggling format source.
Created attachment 112485 [details] Patch
Comment on attachment 112485 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=112485&action=review > LayoutTests/inspector/debugger/selected-call-frame-after-formatting-source.html:50 > +<p>Tests execution line in selected call frame keeps showing after formatting source..</p> I think it actually tests that selected call frame does not change when pretty-print is toggled.
Comment on attachment 112485 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=112485&action=review > Source/WebCore/inspector/front-end/DebuggerPresentationModel.js:-315 > - if (WebInspector.debuggerModel.callFrames) So we are not rebuilding call frames now. Why?
Comment on attachment 112485 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=112485&action=review >> Source/WebCore/inspector/front-end/DebuggerPresentationModel.js:-315 >> - if (WebInspector.debuggerModel.callFrames) > > So we are not rebuilding call frames now. Why? They are already correctly updated based on events from rawSourceCodes. Meanwhile calling debuggerPaused is bad because it resets selectedCallFrame to the top of the stack.
Comment on attachment 112485 [details] Patch Please update test description as Pavel suggests before landing.
Committed r98549: <http://trac.webkit.org/changeset/98549>