Bug 74160 – Use [Supplemental] IDL in WebSocket

• Component: WebCore Misc.
• Status: RESOLVED FIXED

[Kentaro Hara]

By using the [Supplemental] IDL, we can move declarations of WebSocket attributes from DOMWindow.idl to websocket/DOMWindowWebSocket.idl, which helps make WebSocket a self-contained module.

See bug 72138 for more details of our motivation.

[Kentaro Hara]

Created attachment 118526 [details]
Patch

[Adam Barth]

Comment on attachment 118526 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=118526&action=review

> Source/WebCore/websockets/DOMWindowWebSocket.idl:37
> +        attribute CloseEventConstructor CloseEvent;

Is CloseEvent specific to WebSockets?  I guess it is!

[Kentaro Hara]

(In reply to comment #2)
> (From update of attachment 118526 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=118526&action=review
> 
> > Source/WebCore/websockets/DOMWindowWebSocket.idl:37
> > +        attribute CloseEventConstructor CloseEvent;
> 
> Is CloseEvent specific to WebSockets?  I guess it is!

Yes, it is!
http://dev.w3.org/html5/websockets/#closeevent

[Adam Barth]

We should do more build system integration.  I can help.  I'll try to do the Mac build tomorrow.

[Kentaro Hara]

(In reply to comment #4)
> We should do more build system integration.  I can help.  I'll try to do the Mac build tomorrow.

What do you mean by "more build system integration"?

After this patch is landed, I am planning to change build flows of all platforms as follows:

[1] Make a change on DerivedSources.make. (for Mac)
[2] Make a change on GNUmakefile.am and bindings/gobject/GNUmakefile.am. (for GTK)
[3] Make a change on DerivedSources.pri. (for Qt)
[4] Make a change on WebCore.vcproj/MigrateScripts and WebCore.vcproj/WebCore.vcproj. (for Win)
[5] Make a change on UseJSC.cmake, UseV8.cmake and PlatformBlackBerry.cmake. (for Efl, WinCE and BlackBerry)

Since ENABLE_WEBSOCKET is enabled on all platforms, I hope that now we can confirm if our build flow change is correct or not.

[Adam Barth]

Sounds great.

[WebKit Review Bot]

Comment on attachment 118526 [details]
Patch

Attachment 118526 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/10833373

[Kentaro Hara]

Created attachment 118726 [details]
rebased patch for commit

[WebKit Review Bot]

Comment on attachment 118726 [details]
rebased patch for commit

Clearing flags on attachment: 118726

Committed r102558: <http://trac.webkit.org/changeset/102558>

[Kentaro Hara]

Reverted r102558 for reason:

clobber build failure

Committed r102571: <http://trac.webkit.org/changeset/102571>

[WebKit Review Bot]

Comment on attachment 118726 [details]
rebased patch for commit

Clearing flags on attachment: 118726

Committed r102774: <http://trac.webkit.org/changeset/102774>