Cache API: Pass origin to platform, rather than DatabaseIdentifier

content/renderer/renderer_blink_platform_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/renderer_blink_platform_impl.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/files/file_path.h"
@@ skipping 412 matching lines @@
 
 //------------------------------------------------------------------------------
 
 WebIDBFactory* RendererBlinkPlatformImpl::idbFactory() {
   return web_idb_factory_.get();
 }
 
 //------------------------------------------------------------------------------
 
 blink::WebServiceWorkerCacheStorage* RendererBlinkPlatformImpl::cacheStorage(
-    const WebString& origin_identifier) {
-  const GURL origin =
-      storage::GetOriginFromIdentifier(origin_identifier.utf8());
+    const blink::WebSecurityOrigin& security_origin) {
+  const GURL origin = blink::WebStringToGURL(security_origin.toString());

[jsbell, 2016/03/02 19:52:25]

It looks like this WebSecurityOrigin->WebString->GURL conversion is done
elsewhere, but it also seems like that could be more elegant...?

[Mike West, 2016/03/02 20:58:39]

If you make `origin` a `url::Origin`, you'll get the conversion from
`WebSecurityOrigin` for ~free. That's more work elsewhere, of course...

   return new WebServiceWorkerCacheStorageImpl(thread_safe_sender_.get(),
                                               origin);
 }
 
 //------------------------------------------------------------------------------
 
 WebFileSystem* RendererBlinkPlatformImpl::fileSystem() {
   return WebFileSystemImpl::ThreadSpecificInstance(default_task_runner_);
 }
 
@@ skipping 814 matching lines @@
 }
 
 //------------------------------------------------------------------------------
 
 blink::WebTrialTokenValidator*
 RendererBlinkPlatformImpl::trialTokenValidator() {
   return &trial_token_validator_;
 }
 
 }  // namespace content