Implementation of shared worker code path for WebView file system permission.

android_webview/browser/aw_content_browser_client.h

Index: android_webview/browser/aw_content_browser_client.h
diff --git a/android_webview/browser/aw_content_browser_client.h b/android_webview/browser/aw_content_browser_client.h
index 355f74327e981b1d43af847d5f07226acd6e1388..c890c6e8d6ed88869a036bf94fd5955783efa47d 100644
--- a/android_webview/browser/aw_content_browser_client.h
+++ b/android_webview/browser/aw_content_browser_client.h
@@ -81,10 +81,11 @@ class AwContentBrowserClient : public content::ContentBrowserClient {
       unsigned long estimated_size,
       content::ResourceContext* context,
       const std::vector<std::pair<int, int> >& render_frames) OVERRIDE;
-  virtual bool AllowWorkerFileSystem(
+  virtual void RequestFileSystemAccessSync(

[boliu, 2014/06/19 15:57:35]

Weird name? It appears the callback allows this to be async, yet the name says
sync?

[Xi Han, 2014/06/19 16:37:46]

The name comes from the webview guest, when a webview guest requests file system
access, whether it calls a sync api or async api. Now we have both apis:
1) sync api for a request from shared worker & worker (the request will block
workers);
2) async api for a request from a webview guest main thread (request won't block
main thread).

We use a sync message (WorkerProcessHostMsg_RequestFileSystemAccessSync) to
request this api.

Sorry for the confusion.

       const GURL& url,
       content::ResourceContext* context,
-      const std::vector<std::pair<int, int> >& render_frames) OVERRIDE;
+      const std::vector<std::pair<int, int> >& render_frames,
+      base::Callback<void(bool)> callback) OVERRIDE;
   virtual bool AllowWorkerIndexedDB(
       const GURL& url,
       const base::string16& name,