[Webkit-unassigned] [Bug 89678] Web Inspector: Support 'Restart frame' in inspector frontend

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=89678


Vsevolod Vlasov <vsevik at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #148840|review-                     |review?
               Flag|                            |




--- Comment #3 from Vsevolod Vlasov <vsevik at chromium.org>  2012-06-21 11:24:03 PST ---
(From update of attachment 148840)
View in context: https://bugs.webkit.org/attachment.cgi?id=148840&action=review

> Source/WebCore/inspector/front-end/CallStackSidebarPane.js:192
> +        this._pane.contributeToContextMenu(contextMenu);

I would keep context menu population in one method of CallStackSidebarPane.
E.g. in eclipse "restart frame" is in the middle of context menu which would be hard to implement with current approach.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.