[Chromium] Move BlobRegistryProxy into WebCore

[Mark Pilgrim (Google)]

[Chromium] Move BlobRegistryProxy into WebCore

[Mark Pilgrim (Google)]

Created attachment 182631 [details]
Patch

[Adam Barth]

Comment on attachment 182631 [details]
Patch

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

I think this is fine for a first patch.  There's more cleanup we can do to better integrate BlobRegistryProxy with BlobRegistryImpl

> Source/WebCore/platform/network/chromium/BlobRegistryProxy.cpp:48
> +// We are part of the WebKit implementation.
> +using namespace WebKit;

As in the other patch, it would be better to remove this using directive and be explicit about when we're using things in the WebKit namespace.

> Source/WebCore/platform/network/chromium/BlobRegistryProxy.cpp:52
> +BlobRegistry& blobRegistry()

It looks like this function is declared in Source/WebCore/platform/network/BlobRegistry.h.  I wonder if we should put this function in Source/WebCore/platform/network/BlobRegistryImpl.cpp?

[WebKit Review Bot]

Comment on attachment 182631 [details]
Patch

Clearing flags on attachment: 182631

Committed r139698: <http://trac.webkit.org/changeset/139698>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.

[Mark Pilgrim (Google)]

Filed Bug 106859 for WebKit prefix.