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

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>.