********** WebCore blob hacking

WebCore blob hacking

Added BlobDataHandle. Renderer/worker side scoper object for a reference to a blob.

Change blob processing/handling classes to work with BlobDataHandles insead of URLs.
    -- WebCore::Blob
    -- WebCore::BlobBuilder
    -- postMessage() plumbing
    -- WebSocket.send() plumbing
    -- fileWriter.write() plumbing
    -- SerializedScriptValue
    -- FormData

Switched to using string uuids instead of urls in structs and params in the WebKit API.
      -- WebBlobData::Item
      -- WebHTTPBody::Element
      -- WebFileWriter::write()
      -- WebBlobRegistry

Defined/implemented WebCore::BlobRegistry to be callable from any renderer/worker thread.
   -- BlobRegistryImpl for 'native' webcore
   -- BlobRegistryProxy for chrome

Removed ThreadableBlobRegistry.

Added didCreateSnapshotFile plumbing that carries a WebCore::BlobDataHandle with it thru the bridging/callback forest (done in a previous patch).

Got more explicit about naming: FileSystemURL vs BlobUUID vs PublicBlobURL.

BUG=174200

[kinuko, 2012-10-31 14:19:46]

http://codereview.chromium.org/11192017/diff/52007/WebCore/platform/network/B...
File WebCore/platform/network/BlobRegistry.h (right):

http://codereview.chromium.org/11192017/diff/52007/WebCore/platform/network/B...
WebCore/platform/network/BlobRegistry.h:63: // For use on the main therad only.
nit: typo (therad -> thread)

[kinuko, 2013-01-07 09:38:06]

(cosmetic comments only)

https://codereview.chromium.org/11192017/diff/119001/WebCore/WebCore.gypi
File WebCore/WebCore.gypi (left):

https://codereview.chromium.org/11192017/diff/119001/WebCore/WebCore.gypi#old...
WebCore/WebCore.gypi:2779: 'fileapi/ThreadableBlobRegistry.h',
yay

https://codereview.chromium.org/11192017/diff/119001/WebCore/platform/network...
File WebCore/platform/network/BlobRegistryImpl.cpp (right):

https://codereview.chromium.org/11192017/diff/119001/WebCore/platform/network...
WebCore/platform/network/BlobRegistryImpl.cpp:212: {
nit: can we have ASSERT(isMainThread()) here? (though we seem to call it in
registerBlobResourceHandleConstructor())

https://codereview.chromium.org/11192017/diff/119001/WebCore/platform/network...
WebCore/platform/network/BlobRegistryImpl.cpp:219:
registry->m_publicURLs.remove(url);
nit: can we have ASSERT(isMainThread()) here?

[michaeln, 2013-01-25 21:19:26]

https://codereview.chromium.org/11192017/diff/119001/WebCore/platform/network...
File WebCore/platform/network/BlobRegistryImpl.cpp (right):

https://codereview.chromium.org/11192017/diff/119001/WebCore/platform/network...
WebCore/platform/network/BlobRegistryImpl.cpp:212: {
On 2013/01/07 09:38:06, kinuko wrote:
> nit: can we have ASSERT(isMainThread()) here? (though we seem to call it in
> registerBlobResourceHandleConstructor())

Done.

https://codereview.chromium.org/11192017/diff/119001/WebCore/platform/network...
WebCore/platform/network/BlobRegistryImpl.cpp:219:
registry->m_publicURLs.remove(url);
On 2013/01/07 09:38:06, kinuko wrote:
> nit: can we have ASSERT(isMainThread()) here?

Done.