********** Chromium Blob hacking

Chromium Blob hacking

Added BlobStorageContext as a replacement for BlobStorageController. The new class manages blob data by uuid and seperately maintains a mapping from of public blob urls to uuids.

Added BlobStorageConsumer class, a container for blob uuid and public url usage associated with a renderer/worker process. Cleans up on exit.

Added BlobDataHandle. In-browser-process scoper object for a reference to a blob.

Changed blob processing classes to take BlobDataHandles as input instead of GURLs. Not long after IPC deserizliation, get a BlobDataHandle and pass that around.
- FileSystemOperation.Write (and famlity) for FileWriter support
- ResourceDispatcherHost (and famility) for blob url requests
- (PostMessage handling and WebIntent plumbing ultimately needs to be treated in this way too)

Added OnDidCreateSnapshotFile callback path, no longer overloading OnDidReadMetadata for that purpose.

Removed dependency on blobs from the browser-side of SnapshotFile creation.

Switched to using string uuids instead of urls in IPC messages (and structs carried in ipc messages)
- FileSystemHostMsg_Write
- webkit_base::DataElement (and consumers: BlobData, ResourceRequestBody)
- blob registry/building ipc messages

Implemented the WebKit::WebBlobRegistryImpl such that it can be invoked on any renderer/worker thread.

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

BUG=174200

[kinuko, 2013-01-07 08:52:00]

https://codereview.chromium.org/11410019/diff/56001/content/browser/fileapi/f...
File content/browser/fileapi/fileapi_message_filter.cc (right):

https://codereview.chromium.org/11410019/diff/56001/content/browser/fileapi/f...
content/browser/fileapi/fileapi_message_filter.cc:246: FileSystemOperation*
operation = GetNewOperation(src_url, request_id);
note: in the current code this needs to take dest_url

https://codereview.chromium.org/11410019/diff/56001/content/browser/fileapi/f...
content/browser/fileapi/fileapi_message_filter.cc:266: FileSystemOperation*
operation = GetNewOperation(src_url, request_id);
ditto

https://codereview.chromium.org/11410019/diff/56001/content/browser/fileapi/f...
content/browser/fileapi/fileapi_message_filter.cc:555:
operation->SyncGetPlatformPath(url, platform_path);
nit: indent is off

https://codereview.chromium.org/11410019/diff/56001/content/browser/fileapi/f...
content/browser/fileapi/fileapi_message_filter.cc:806: // and takes  a ref of
its own.
nit: extra space before 'a ref'

https://codereview.chromium.org/11410019/diff/56001/content/common/child_thre...
File content/common/child_thread.cc (right):

https://codereview.chromium.org/11410019/diff/56001/content/common/child_thre...
content/common/child_thread.cc:200: }
This change can probably be landed independently?

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

https://codereview.chromium.org/11410019/diff/56001/content/browser/fileapi/f...
File content/browser/fileapi/fileapi_message_filter.cc (right):

https://codereview.chromium.org/11410019/diff/56001/content/browser/fileapi/f...
content/browser/fileapi/fileapi_message_filter.cc:246: FileSystemOperation*
operation = GetNewOperation(src_url, request_id);
On 2013/01/07 08:52:01, kinuko wrote:
> note: in the current code this needs to take dest_url

Done.

https://codereview.chromium.org/11410019/diff/56001/content/browser/fileapi/f...
content/browser/fileapi/fileapi_message_filter.cc:266: FileSystemOperation*
operation = GetNewOperation(src_url, request_id);
On 2013/01/07 08:52:01, kinuko wrote:
> ditto

Done.

https://codereview.chromium.org/11410019/diff/56001/content/browser/fileapi/f...
content/browser/fileapi/fileapi_message_filter.cc:555:
operation->SyncGetPlatformPath(url, platform_path);
On 2013/01/07 08:52:01, kinuko wrote:
> nit: indent is off

Done.

https://codereview.chromium.org/11410019/diff/56001/content/browser/fileapi/f...
content/browser/fileapi/fileapi_message_filter.cc:806: // and takes  a ref of
its own.
On 2013/01/07 08:52:01, kinuko wrote:
> nit: extra space before 'a ref'

Done.