Introduce WebWorkerPermissionClientProxy to deprecate WorkerAllowMainThreadBridge

Source/web/WorkerPermissionClient.h

Index: Source/web/WorkerPermissionClient.h
diff --git a/Source/modules/filesystem/WorkerLocalFileSystem.h b/Source/web/WorkerPermissionClient.h
similarity index 62%
copy from Source/modules/filesystem/WorkerLocalFileSystem.h
copy to Source/web/WorkerPermissionClient.h
index 72242b5f06cc068bfd5e2f734e1a90338fe641f3..f132f3be515d4e860f5f4554c29fd38f247311bd 100644
--- a/Source/modules/filesystem/WorkerLocalFileSystem.h
+++ b/Source/web/WorkerPermissionClient.h
@@ -28,26 +28,45 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef WorkerLocalFileSystem_h
-#define WorkerLocalFileSystem_h
+#ifndef WorkerPermissionClient_h
+#define WorkerPermissionClient_h
 
 #include "core/workers/WorkerClients.h"
-#include "modules/filesystem/LocalFileSystem.h"
 #include "wtf/Forward.h"
 
 namespace WebCore {
+class ExecutionContext;
+}
 
-class WorkerLocalFileSystem : public LocalFileSystemBase, public Supplement<WorkerClients> {
+namespace WebKit {
+
+class WebFrame;
+class WebString;
+class WebWorkerPermissionClientProxy;
+
+class WorkerPermissionClient : public WebCore::Supplement<WebCore::WorkerClients> {
 public:
-    static PassOwnPtr<WorkerLocalFileSystem> create(PassOwnPtr<FileSystemClient>);
+    static PassOwnPtr<WorkerPermissionClient> create(PassOwnPtr<WebWorkerPermissionClientProxy>);
+
+    virtual ~WorkerPermissionClient();
+
+    bool allowDatabase(const WebString& name, const WebString& displayName, unsigned long estimatedSize);
+    bool allowFileSystem();
+    bool allowIndexedDB(const WebString& name);
+
+    WebWorkerPermissionClientProxy* proxy() { return m_proxy.get(); }

[michaeln, 2013/10/30 19:47:04]

does this need to be public?

[kinuko, 2013/10/31 01:48:09]

This is used just to see if the embedder has proxy implementation until we
completely switch the code (i.e. if proxy is null we use old code for now), and
to be removed in the next blink patch. I'll add a FIXME comment.

+
     static const char* supplementName();
-    static WorkerLocalFileSystem* from(ExecutionContext*);
-    virtual ~WorkerLocalFileSystem();
+    static WorkerPermissionClient* from(WebCore::ExecutionContext*);
 
 private:
-    explicit WorkerLocalFileSystem(PassOwnPtr<FileSystemClient>);
+    explicit WorkerPermissionClient(PassOwnPtr<WebWorkerPermissionClientProxy>);
+
+    OwnPtr<WebWorkerPermissionClientProxy> m_proxy;
 };
 
-} // namespace WebCore
+void providePermissionClientToWorker(WebCore::WorkerClients*, PassOwnPtr<WebWorkerPermissionClientProxy>);
+
+} // namespace WebKit
 
-#endif // WorkerLocalFileSystem_h
+#endif // WorkerPermissionClient_h