Web Inspector: status message isn't shown in call stack sidebar pane for native breakpoints
Created attachment 76748 [details] Patch. Update CallStackSidebarPane placards and status message in the same event handler so it doesn't depend on event dispatch order.
Comment on attachment 76748 [details] Patch. View in context: https://bugs.webkit.org/attachment.cgi?id=76748&action=review > WebCore/inspector/front-end/BreakpointManager.js:181 > + this._lastHitBreakpointView = breakpointView; Breakpoint manager shouldn't modify breakpoint views, instead each view should listen to the corresponding breakpoint state changes and update itself. BreakpointManager should operate on the breakpoints. r- for this.
(In reply to comment #2) > (From update of attachment 76748 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=76748&action=review > > > WebCore/inspector/front-end/BreakpointManager.js:181 > > + this._lastHitBreakpointView = breakpointView; > > Breakpoint manager shouldn't modify breakpoint views, instead each view should listen to the corresponding breakpoint state changes and update itself. BreakpointManager should operate on the breakpoints. r- for this. This patch is just a quick fix, relations between BreakpointManager and breakpoint views didn't change. I've just renamed _lastHitBreakpoint.view to _lastHitBreakpointView. Let me decouple breakpoint views from breakpoint manager in a separate patch.
Committed r74191: <http://trac.webkit.org/changeset/74191>
http://trac.webkit.org/changeset/74191 might have broken Leopard Intel Debug (Tests)