[Webkit-unassigned] [Bug 66536] Implement Web IDL Constructor extended attribute in IDLParser.pm and CodeGeneratorV8.pm

[bugzilla-daemon at webkit.org]

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





--- Comment #9 from Dominic Cooney <dominicc at chromium.org>  2011-08-30 15:31:03 PST ---
> >> Source/WebCore/ChangeLog:22
> >> +        [3] Added 'V8ConstructorSetsActiveDOMWrapper' extended attribute.
> > 
> > Looks like you need to update this.
> 
> Update what? I guess that the description is already up-to-date.

My bad!

> >> Source/WebCore/ChangeLog:26
> >> +        [4] Added 'ConstructorRaisesException' extended attribute.
> > 
> > When is this used? Because the Web IDL spec does not have 'raises' for constructors. Maybe we should email the spec author and see if it can be added, although the Web IDL spec is already in last call.
> 
> 'ConstructorRaisesException' does not mean that the constructor can raise exception (as 'raises' means),  but just mean that XXX::create() requires a placeholder for ExceptionCode, like XXX::create(..., ec). Thus, I guess that this may be renamed to "ConstructorWith=RaisesException" or something.

It seems weird that something in the WebCore layer (as opposed to bindings layer) is generating an exception, but that is not described in WebIDL with raises(…).

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