[Webkit-unassigned] [Bug 67447] Generate a Worker constructor of V8 using the IDL 'Constructor' extended attribute

[bugzilla-daemon at webkit.org]

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





--- Comment #4 from Kentaro Hara <haraken at google.com>  2011-09-01 16:20:14 PST ---
(In reply to comment #2)
> (From update of attachment 106047 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=106047&action=review
> 
> > Source/WebCore/workers/Worker.idl:37
> > +        ConstructorRaisesException,
> 
> shouldn't this be Constructor(in DOMString scriptUrl) raises (DOMException)?

Dimitri: Thank you very much for the review. 

The spec of the 'Constructor' extended attribute does not allow the "Constructor(...) raises (...)" format (http://www.w3.org/TR/WebIDL/#Constructor). For example, the spec says that the IDL of a Worker constructor should be "Constructor(DOMString scriptURL)" (http://dev.w3.org/html5/workers/#dedicated-workers-and-the-worker-interface). On the other hand, the spec of the Worker constructor says that it can throw SYNTAX_ERR (http://dev.w3.org/html5/workers/#dom-worker).

Therefore, I think that the possible approaches for us are as follows:

Appraoch1: Change the spec of constructors, so that it can allow the "Constructor(...) raises (...)" format.

Approach2: We introduce a special extended attribute for WebKit (as we have done so far), 'ConstructorRaisesException'.

For now, this patch takes Approach2.

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