Bug 28844

Summary: V8 binding of WebSocket
Product: WebKit Reporter: Fumitoshi Ukai <ukai>
Component: WebCore JavaScriptAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, dglazkov, eric
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
V8 Bindings for WebSocket API none

Description Fumitoshi Ukai 2009-08-31 03:34:07 PDT
V8 binding of WebSocket API.
Comment 1 Fumitoshi Ukai 2009-09-01 02:25:55 PDT
Created attachment 38851 [details]
V8 Bindings for WebSocket API


---
 10 files changed, 318 insertions(+), 1 deletions(-)
Comment 2 Adam Barth 2009-09-01 08:38:38 PDT
Comment on attachment 38851 [details]
V8 Bindings for WebSocket API

This looks good.  I presume we already have LayoutTests from the JSC implementation?

nit:  In WebKit-land, we use FIXME instead of TODO.

Also, is there a bug on file for Worker support for WebSockets?
Comment 3 Eric Seidel (no email) 2009-09-01 08:54:25 PDT
Comment on attachment 38851 [details]
V8 Bindings for WebSocket API

Rejecting patch 38851 from commit-queue.  This patch will require manual commit.

['WebKitTools/Scripts/run-webkit-tests', '--no-launch-safari', '--quiet', '--exit-after-n-failures=1'] failed with exit code 1
Comment 4 Eric Seidel (no email) 2009-09-01 09:07:52 PDT
Comment on attachment 38851 [details]
V8 Bindings for WebSocket API

Rejecting patch 38851 from commit-queue.  This patch will require manual commit.

['WebKitTools/Scripts/run-webkit-tests', '--no-launch-safari', '--quiet', '--exit-after-n-failures=1'] failed with exit code 1
Comment 5 Adam Barth 2009-09-01 09:39:24 PDT
Comment on attachment 38851 [details]
V8 Bindings for WebSocket API

Clearing flags on attachment: 38851

Committed r47941: <http://trac.webkit.org/changeset/47941>
Comment 6 Adam Barth 2009-09-01 09:39:31 PDT
All reviewed patches have been landed.  Closing bug.
Comment 7 Eric Seidel (no email) 2009-09-01 16:16:30 PDT
media/video-source-add-src.html -> crashed

The commit-queue hit bug 28845.
Comment 8 Eric Seidel (no email) 2009-09-01 16:18:26 PDT
Sorry.  commit-queue failed because a layout test got left around from a previously failed patch:
svg/dom/smil-methods.svg -> failed

This type of error is tracked by bug 28603.