[webkit-reviews] review denied: [Bug 32442] REGRESSION (r47686-r48034): Inspector console outputs the global value for a variable instead of the local value : [Attachment 51845] Proposed fix v1.0
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Sat Mar 27 16:07:51 PDT 2010
Pavel Feldman <pfeldman at chromium.org> has denied Dmitry Gorbik
<socket.h at gmail.com>'s request for review:
Bug 32442: REGRESSION (r47686-r48034): Inspector console outputs the global
value for a variable instead of the local value
https://bugs.webkit.org/show_bug.cgi?id=32442
Attachment 51845: Proposed fix v1.0
https://bugs.webkit.org/attachment.cgi?id=51845&action=review
------- Additional Comments from Pavel Feldman <pfeldman at chromium.org>
Looks great! One nit and we can land this!
> + if (!dontUseCommandLineAPI) {
> + expression = "with (window.console._inspectorCommandLineAPI) { with
(window) {\n" + expression + "\n} }";
> + }
Single-line conditions should not have {}.
More information about the webkit-reviews
mailing list