Bug 133920 - Copy SharedBuffer data into IPC message directly
Summary: Copy SharedBuffer data into IPC message directly
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Pratik Solanki
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-06-14 15:51 PDT by Pratik Solanki
Modified: 2014-06-19 11:33 PDT (History)
8 users (show)

See Also:


Attachments
Patch (6.90 KB, patch)
2014-06-14 15:57 PDT, Pratik Solanki
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from webkit-ews-09 for mac-mountainlion-wk2 (521.85 KB, application/zip)
2014-06-14 19:11 PDT, Build Bot
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Pratik Solanki 2014-06-14 15:51:16 PDT
When Data array callbacks are enabled, we currently merge all the CFDataRefs in SharedBuffer into one contiguous memory buffer before copying into the IPC message. We can avoid the merge into a single buffer here.
Comment 1 Pratik Solanki 2014-06-14 15:57:59 PDT
Created attachment 233130 [details]
Patch
Comment 2 Sam Weinig 2014-06-14 16:46:12 PDT
I don't think there is any reason to add a SharedBufferDataReference type. Instead, we should just add a coder for SharedBuffer.
Comment 3 Build Bot 2014-06-14 19:11:00 PDT
Comment on attachment 233130 [details]
Patch

Attachment 233130 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/4560879307718656

New failing tests:
media/W3C/video/networkState/networkState_during_loadstart.html
Comment 4 Build Bot 2014-06-14 19:11:03 PDT
Created attachment 233133 [details]
Archive of layout-test-results from webkit-ews-09 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-09  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5
Comment 5 Pratik Solanki 2014-06-16 11:30:31 PDT
(In reply to comment #3)
> (From update of attachment 233130 [details])
> Attachment 233130 [details] did not pass mac-wk2-ews (mac-wk2):
> Output: http://webkit-queues.appspot.com/results/4560879307718656
> 
> New failing tests:
> media/W3C/video/networkState/networkState_during_loadstart.html

Looks like a flaky test. I see these failures in other bugs as well so this is not related to my change.
Comment 6 Pratik Solanki 2014-06-19 11:33:44 PDT
Comment on attachment 233130 [details]
Patch

Clearing flags on attachment: 233130

Committed r170154: <http://trac.webkit.org/changeset/170154>
Comment 7 Pratik Solanki 2014-06-19 11:33:48 PDT
All reviewed patches have been landed.  Closing bug.