[webkit-reviews] review denied: [Bug 89147] Web Inspector: Support 'Restart frame' in protocol and backend : [Attachment 147708] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Jun 15 09:38:15 PDT 2012
Pavel Feldman <pfeldman at chromium.org> has denied 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 147708: Patch
https://bugs.webkit.org/attachment.cgi?id=147708&action=review
------- Additional Comments from Pavel Feldman <pfeldman at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=147708&action=review
> Source/WebCore/bindings/v8/custom/V8JavaScriptCallFrameCustom.cpp:49
> +v8::Handle<v8::Value> V8JavaScriptCallFrame::restartCallback(const
v8::Arguments& args)
You don't need this.
> Source/WebCore/inspector/Inspector.json:2439
> + "name": "restartFrame",
"hidden": true
> Source/WebCore/inspector/Inspector.json:2444
> + { "name": "callFrames", "type": "array", "optional":
true, "items": { "$ref": "CallFrame"}, "description": "New stack trace." },
Should not be optional.
> Source/WebCore/inspector/Inspector.json:2445
> + { "name": "result", "type": "object", "optional": true,
"description": "VM-specific description of the changes applied.", "hidden":
true }
Should not be optional. Nuke the "changes applied part".
More information about the webkit-reviews
mailing list