[Webkit-unassigned] [Bug 66756] Implement an Event constructor in V8

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Aug 31 11:18:02 PDT 2011


https://bugs.webkit.org/show_bug.cgi?id=66756





--- Comment #40 from Kentaro Hara <haraken at google.com>  2011-08-31 11:18:02 PST ---
Adam: Thank you for the review. 

> > Source/WebCore/bindings/v8/custom/V8EventConstructors.cpp:59
> > +    if (AllowAllocation::current())
> > +        return args.Holder();
> 
> Crazy.  Is this a v8 function?  Why does it not require the v8 namespace?  Are we using namespace v8 somewhere?  (We shouldn't be.)

AllowAllocation::current() is in the WebCore namespace. (http://codesearch.google.com/codesearch#OAMlx_jo-ck/src/third_party/WebKit/Source/WebCore/bindings/v8/V8Binding.h)

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list