[webkit-reviews] review granted: [Bug 82442] Consider removal of removes jsNull()/v8::Null() from JSInternalsCustom.cpp/V8InternalsCustom.cpp : [Attachment 134250] proposed patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Mar 28 15:08:31 PDT 2012


Kentaro Hara <haraken at chromium.org> has granted Vineet Chaudhary (vineetc)
<rgf748 at motorola.com>'s request for review:
Bug 82442: Consider removal of removes jsNull()/v8::Null() from
JSInternalsCustom.cpp/V8InternalsCustom.cpp
https://bugs.webkit.org/show_bug.cgi?id=82442

Attachment 134250: proposed patch
https://bugs.webkit.org/attachment.cgi?id=134250&action=review

------- Additional Comments from Kentaro Hara <haraken at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=134250&action=review


> Source/WebCore/ChangeLog:9
> +	   No new tests. LayoutTests/fast/harness/user-preferred-language.html
should pass
> +	   even after these changes.

OK, it would be hard to add a new test case for this change.


More information about the webkit-reviews mailing list