Bug 132363 – Make Blob RawData immutable

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Alexey Proskuryakov]

RawData is essentially immutable, but its interface exposes a way to modify it.

[Alexey Proskuryakov]

Created attachment 230431 [details]
proposed patch

[Build Bot]

Comment on attachment 230431 [details]
proposed patch

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

New failing tests:
http/tests/local/blob/send-data-blob.html

[Build Bot]

Created attachment 230442 [details]
Archive of layout-test-results from webkit-ews-11 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-11  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5

[Build Bot]

Comment on attachment 230431 [details]
proposed patch

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

New failing tests:
fast/files/read-blob-async.html
http/tests/local/blob/send-data-blob.html
fast/files/workers/worker-read-blob-sync.html
fast/files/workers/worker-read-blob-async.html
http/tests/local/blob/send-hybrid-blob.html

[Build Bot]

Created attachment 230450 [details]
Archive of layout-test-results from webkit-ews-04 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-04  Port: mac-mountainlion  Platform: Mac OS X 10.8.5

[Alexey Proskuryakov]

Created attachment 230458 [details]
proposed patch

[Anders Carlsson]

Comment on attachment 230458 [details]
proposed patch

View in context: https://bugs.webkit.org/attachment.cgi?id=230458&action=review

> Source/WebCore/fileapi/WebKitBlobBuilder.cpp:-51
> -// FIXME: Move this file to BlobBuilder.cpp

I think you should keep this FIXME but change the name to whatever we want to rename it to.

> Source/WebCore/fileapi/WebKitBlobBuilder.h:-44
> -// FIXME: Move this file to BlobBuilder.h

I think you should keep this FIXME but change the name to whatever we want to rename it to.

> Source/WebCore/platform/network/BlobData.h:45
> +    static PassRefPtr<RawData> create(Vector<char>&& data)

No need for this to take an rvalue reference, it can just take a Vector<char> by value.

[Alexey Proskuryakov]

Comment on attachment 230458 [details]
proposed patch

> I think you should keep this FIXME but change the name to whatever we want to rename it to.

But I don't know that yet :)

> No need for this to take an rvalue reference, it can just take a Vector<char> by value.

Talked this through in person. I want to ensure that constructing RawData never copies, so this is intentional.

But also, RawData will be going away in one of the next steps. So, just landing as is to not spend more effort on it.

[WebKit Commit Bot]

Comment on attachment 230458 [details]
proposed patch

Clearing flags on attachment: 230458

Committed r168032: <http://trac.webkit.org/changeset/168032>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.