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

content/public/browser/content_browser_client.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_
 #define CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_
 
 #include <map>
 #include <string>
 #include <utility>
@@ skipping 327 matching lines @@
       const GURL& url,
       const base::string16& name,
       const base::string16& display_name,
       unsigned long estimated_size,
       ResourceContext* context,
       const std::vector<std::pair<int, int> >& render_frames);
 
   // Allow the embedder to control if access to file system by a shared worker
   // is allowed.
   // This is called on the IO thread.
-  virtual bool AllowWorkerFileSystem(
+  virtual void RequestWorkerFileSystemAccessSync(

[jam, 2014/06/19 16:47:27]

I'm confused, this has a Sync suffix but takes a callback. should it be Async?

[Fady Samuel, 2014/06/19 16:50:11]

This is for the sync filesystem API. For webview's case, we're simply blocking
the guest until the embedder makes a permission decision, hence the callback.
This is a delayed response.

[Xi Han, 2014/06/19 17:22:27]

The name comes from the webview guest, whether the guest calls a sync api or
async filesystem 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 IPC message (WorkerProcessHostMsg_RequestFileSystemAccessSync) to
send request.

       const GURL& url,
       ResourceContext* context,
-      const std::vector<std::pair<int, int> >& render_frames);
+      const std::vector<std::pair<int, int> >& render_frames,
+      base::Callback<void(bool)> callback);
 
   // Allow the embedder to control if access to IndexedDB by a shared worker
   // is allowed.
   // This is called on the IO thread.
   virtual bool AllowWorkerIndexedDB(
       const GURL& url,
       const base::string16& name,
       ResourceContext* context,
       const std::vector<std::pair<int, int> >& render_frames);
 
@@ skipping 311 matching lines @@
   // Allows an embedder to provide its own ExternalVideoSurfaceContainer
   // implementation.  Return NULL to disable external surface video.
   virtual ExternalVideoSurfaceContainer*
   OverrideCreateExternalVideoSurfaceContainer(WebContents* web_contents);
 #endif
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_