Bug 76608

Summary: [Chromium] [WebSocket] Remove binary communication using WebData in WebKit API
Product: WebKit Reporter: Takashi Toyoshima <toyoshim>
Component: New BugsAssignee: Takashi Toyoshima <toyoshim>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov, fishd, webkit.review.bot, yutak
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch for landing none

Takashi Toyoshima
Reported 2012-01-18 23:41:32 PST
WebData is used experimentally for binary communication between WebKit and Chromium. Now, WebArrayBuffer is the better way to implement it, and Chromium use it now. I'd like to remove this obsoleted WebData support from WebKit API because it's not used now.
Attachments
Patch (6.79 KB, patch)
2012-01-18 23:50 PST, Takashi Toyoshima 		no flags
DetailsFormatted DiffDiff
Patch for landing (6.79 KB, patch)
2012-01-23 05:04 PST, Takashi Toyoshima 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Takashi Toyoshima
Comment 1 2012-01-18 23:50:10 PST
Created attachment 123083 [details] Patch
WebKit Review Bot
Comment 2 2012-01-18 23:54:49 PST
Please wait for approval from fishd@chromium.org before submitting because this patch contains changes to the Chromium public API.
WebKit Review Bot
Comment 3 2012-01-19 00:16:56 PST
Comment on attachment 123083 [details] Patch Attachment 123083 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/11247035
Takashi Toyoshima
Comment 4 2012-01-19 00:30:21 PST
Ah, Cr > r118258 is required to pass. I'll try it again later.
Takashi Toyoshima
Comment 5 2012-01-23 05:04:43 PST
Created attachment 123542 [details] Patch for landing
WebKit Review Bot
Comment 6 2012-01-23 18:19:14 PST
Comment on attachment 123542 [details] Patch for landing Clearing flags on attachment: 123542 Committed r105675: <http://trac.webkit.org/changeset/105675>
WebKit Review Bot
Comment 7 2012-01-23 18:19:19 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.