Bug 132478 - Don't abuse Blob deserialization constructor in WebSocket
Summary: Don't abuse Blob deserialization constructor in WebSocket
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Alexey Proskuryakov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-05-02 12:23 PDT by Alexey Proskuryakov
Modified: 2014-05-02 12:44 PDT (History)
0 users

See Also:

Attachments
proposed patch (13.56 KB, patch)
2014-05-02 12:27 PDT, Alexey Proskuryakov 		sam: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Proskuryakov 2014-05-02 12:23:23 PDT 
WebSocket makes a copy of Blob for no reason.
Comment 1 Alexey Proskuryakov 2014-05-02 12:27:46 PDT 
Created attachment 230683 [details]
proposed patch
Comment 2 Sam Weinig 2014-05-02 12:36:03 PDT 
Comment on attachment 230683 [details]
proposed patch

I would keep this a reference (you can take off the const).  Otherwise, r=me
Comment 3 Alexey Proskuryakov 2014-05-02 12:44:38 PDT 
Committed <http://trac.webkit.org/r168183>.