[webkit-reviews] review denied: [Bug 25944] Remove the uses of V8 Locker in worker execution code. : [Attachment 30553] Proposed Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri May 22 07:06:52 PDT 2009
Eric Seidel <eric at webkit.org> has denied Jian Li <jianli at chromium.org>'s
request for review:
Bug 25944: Remove the uses of V8 Locker in worker execution code.
https://bugs.webkit.org/show_bug.cgi?id=25944
Attachment 30553: Proposed Patch
https://bugs.webkit.org/attachment.cgi?id=30553&action=review
------- Additional Comments from Eric Seidel <eric at webkit.org>
Your ChangeLog needs explanation as to why this change is correct and
necessary.
Is this possible to test? If so it needs a test case. If not, then the
ChangeLog needs and explanation of why it's not possible to test, and possibly
a link to a bug requesting additional features in our testing framework. Since
it's not possible to run DumpRenderTree with V8 such a bug would be against
test_shell in the Chromium tree.
Thanks for the patch! Dimitri is probably your best reviewer for actual
content however. Above was mostly just me applying standard patch review
policy. :)
More information about the webkit-reviews
mailing list