Send Allow{Database,FileSystem,IndexedDB} sync IPCs directly from worker thread

content/worker/websharedworkerclient_proxy.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/worker/websharedworkerclient_proxy.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/message_loop/message_loop.h"
 #include "content/child/webmessageportchannel_impl.h"
 #include "content/common/worker_messages.h"
 #include "content/public/common/content_switches.h"
 #include "content/worker/shared_worker_devtools_agent.h"
 #include "content/worker/websharedworker_stub.h"
+#include "content/worker/worker_permission_client_proxy.h"
 #include "content/worker/worker_thread.h"
 #include "content/worker/worker_webapplicationcachehost_impl.h"
 #include "ipc/ipc_logging.h"
 #include "third_party/WebKit/public/platform/WebString.h"
 #include "third_party/WebKit/public/platform/WebURL.h"
 #include "third_party/WebKit/public/web/WebDocument.h"
 #include "third_party/WebKit/public/web/WebFrame.h"
 #include "third_party/WebKit/public/web/WebSecurityOrigin.h"
 
 using WebKit::WebApplicationCacheHost;
@@ skipping 45 matching lines @@
     WebKit::WebApplicationCacheHostClient* client) {
   WorkerWebApplicationCacheHostImpl* host =
       new WorkerWebApplicationCacheHostImpl(stub_->appcache_init_info(),
                                             client);
   // Remember the id of the instance we create so we have access to that
   // value when creating nested dedicated workers in createWorker.
   appcache_host_id_ = host->host_id();
   return host;
 }
 
-// TODO(abarth): Security checks should use WebDocument or WebSecurityOrigin,
-// not WebFrame as the context object because WebFrames can contain different
-// WebDocuments at different times.
+WebKit::WebWorkerPermissionClientProxy*
+WebSharedWorkerClientProxy::createWorkerPermissionClientProxy(
+    const WebKit::WebSecurityOrigin& origin) {
+  if (origin.isUnique())
+    return NULL;
+  return new WorkerPermissionClientProxy(
+      GURL(origin.toString()), route_id_,
+      ChildThread::current()->thread_safe_sender());
+}
+
+// TODO(kinuko): Deprecate these methods.
 bool WebSharedWorkerClientProxy::allowDatabase(WebFrame* frame,
                                          const WebString& name,
                                          const WebString& display_name,
                                          unsigned long estimated_size) {
-  WebSecurityOrigin origin = frame->document().securityOrigin();
-  if (origin.isUnique())
-    return false;
-
-  bool result = false;
-  Send(new WorkerProcessHostMsg_AllowDatabase(
-      route_id_, GURL(origin.toString().utf8()), name, display_name,
-      estimated_size, &result));
-  return result;
+  return false;
 }
 
 bool WebSharedWorkerClientProxy::allowFileSystem() {
-  bool result = false;
-  Send(new WorkerProcessHostMsg_AllowFileSystem(
-      route_id_, stub_->url().GetOrigin(), &result));
-  return result;
+  return false;
 }
 
 bool WebSharedWorkerClientProxy::allowIndexedDB(const WebKit::WebString& name) {
-  bool result = false;
-  Send(new WorkerProcessHostMsg_AllowIndexedDB(
-      route_id_, stub_->url().GetOrigin(), name, &result));
-  return result;
+  return false;
 }
 
 void WebSharedWorkerClientProxy::dispatchDevToolsMessage(
     const WebString& message) {
   if (devtools_agent_)
     devtools_agent_->SendDevToolsMessage(message);
 }
 
 void WebSharedWorkerClientProxy::saveDevToolsAgentState(
     const WebKit::WebString& state) {
@@ skipping 11 matching lines @@
   // page. It's ok to post several of theese, because the first executed task
   // will exit the message loop and subsequent ones won't be executed.
   base::MessageLoop::current()->PostDelayedTask(
       FROM_HERE,
       base::Bind(&WebSharedWorkerClientProxy::workerContextDestroyed,
                  weak_factory_.GetWeakPtr()),
       base::TimeDelta::FromSeconds(kMaxTimeForRunawayWorkerSeconds));
 }
 
 }  // namespace content