[webkit-reviews] review granted: [Bug 89147] Web Inspector: Support 'Restart frame' in protocol and backend : [Attachment 148040] Patch

[bugzilla-daemon at webkit.org]

Yury Semikhatsky <yurys at chromium.org> has granted Peter Rybin
<prybin at chromium.org>'s request for review:
Bug 89147: Web Inspector: Support 'Restart frame' in protocol and backend
https://bugs.webkit.org/show_bug.cgi?id=89147

Attachment 148040: Patch
https://bugs.webkit.org/attachment.cgi?id=148040&action=review

------- Additional Comments from Yury Semikhatsky <yurys at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=148040&action=review


> Source/WebCore/bindings/v8/ScriptDebugServer.cpp:260
> +    // Call stack may have changed after if the edited function was on the
stack.

after -> after script editing, I think making currentCallFrame() public would
also be fine.