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

chrome/renderer/in_renderer_worker_permission_client_proxy.h

Index: chrome/renderer/in_renderer_worker_permission_client_proxy.h
diff --git a/chrome/renderer/in_renderer_worker_permission_client_proxy.h b/chrome/renderer/in_renderer_worker_permission_client_proxy.h
new file mode 100644
index 0000000000000000000000000000000000000000..c041b4b989612b355ead688115336a6935c35d56
--- /dev/null
+++ b/chrome/renderer/in_renderer_worker_permission_client_proxy.h
@@ -0,0 +1,48 @@
+// Copyright 2013 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.
+
+#ifndef CHROME_RENDERER_IN_RENDERER_WORKER_PERMISSION_CLIENT_PROXY_H_
+#define CHROME_RENDERER_IN_RENDERER_WORKER_PERMISSION_CLIENT_PROXY_H_
+
+#include "base/basictypes.h"
+#include "base/memory/ref_counted.h"
+#include "third_party/WebKit/public/web/WebWorkerPermissionClientProxy.h"
+#include "url/gurl.h"
+
+namespace IPC {
+class SyncMessageFilter;
+}
+
+namespace WebKit {
+class WebFrame;
+}
+
+// This proxy is created on the main renderer thread then passed onto
+// the blink's worker thread.
+class InRendererWorkerPermissionClientProxy

[jam, 2013/10/29 17:19:57]

nit: InRenderer in the class name is redundant since this code is in
chrome\renderer...

[michaeln, 2013/10/29 21:37:00]

I think this is used specifically for dedicated workers, which happens to occur
'in renderer'. If so would it make sense to name this class
DedicatedWorkerPermissionClientProxy and the other
SharedWorkerPermissionClientProxy?

[kinuko, 2013/10/30 00:24:56]

Looks like for dedicated worker stuff we don't always prefix class names, so I
renamed this to WorkerPermissionClientProxy and the other to
SharedWorkerPermissionClientProxy.

+    : public WebKit::WebWorkerPermissionClientProxy {
+ public:
+  InRendererWorkerPermissionClientProxy(WebKit::WebFrame* frame,
+                                        int routing_id);
+  virtual ~InRendererWorkerPermissionClientProxy();
+
+  // WebWorkerPermissionClientProxy overrides.
+  virtual bool allowDatabase(const WebKit::WebString& name,
+                             const WebKit::WebString& display_name,
+                             unsigned long estimated_size);
+  virtual bool allowFileSystem();
+  virtual bool allowIndexedDB(const WebKit::WebString& name);
+
+ private:
+  // Loading document context for this worker.
+  const int routing_id_;
+  bool is_unique_origin_;
+  GURL document_origin_url_;
+  GURL top_frame_origin_url_;
+  scoped_refptr<IPC::SyncMessageFilter> sync_message_filter_;
+
+  DISALLOW_COPY_AND_ASSIGN(InRendererWorkerPermissionClientProxy);
+};
+
+#endif  // CHROME_RENDERER_IN_RENDERER_WORKER_PERMISSION_CLIENT_PROXY_H_