Bug 80190 - [Chromium] [WebSocket] Add WebSocket extensions query WebKit API
Summary: [Chromium] [WebSocket] Add WebSocket extensions query WebKit API
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Takashi Toyoshima
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-02 13:41 PST by Takashi Toyoshima
Modified: 2012-03-06 02:17 PST (History)
6 users (show)

See Also:


Attachments
Patch (2.81 KB, patch)
2012-03-02 13:47 PST, Takashi Toyoshima
no flags Details | Formatted Diff | Diff
Patch (2.87 KB, patch)
2012-03-05 14:09 PST, Takashi Toyoshima
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Takashi Toyoshima 2012-03-02 13:41:58 PST
Recently, Ishibashi-san implement extensions property on WebSocket.
This must be exported as WebKit API, too.
Comment 1 Takashi Toyoshima 2012-03-02 13:47:28 PST
Created attachment 129952 [details]
Patch
Comment 2 WebKit Review Bot 2012-03-02 13:50:49 PST
Please wait for approval from fishd@chromium.org before submitting because this patch contains changes to the Chromium public API.
Comment 3 Takashi Toyoshima 2012-03-05 14:09:23 PST
Created attachment 130202 [details]
Patch
Comment 4 Takashi Toyoshima 2012-03-05 14:11:12 PST
I rebased the patch.
Now, WebSocketImpl uses OVERRIDE macro.
Comment 5 WebKit Review Bot 2012-03-06 02:17:27 PST
Comment on attachment 130202 [details]
Patch

Clearing flags on attachment: 130202

Committed r109889: <http://trac.webkit.org/changeset/109889>
Comment 6 WebKit Review Bot 2012-03-06 02:17:40 PST
All reviewed patches have been landed.  Closing bug.