[PPAPI] Fixed FileSystems from JavaScript not having a context.

content/browser/renderer_host/pepper/pepper_file_system_browser_host.h

Index: content/browser/renderer_host/pepper/pepper_file_system_browser_host.h
diff --git a/content/browser/renderer_host/pepper/pepper_file_system_browser_host.h b/content/browser/renderer_host/pepper/pepper_file_system_browser_host.h
index ebf55f48546d2fab78ecb29a1214112ebf40c519..f3aec5ba14eef2c7f9db5dbb1af5123476170960 100644
--- a/content/browser/renderer_host/pepper/pepper_file_system_browser_host.h
+++ b/content/browser/renderer_host/pepper/pepper_file_system_browser_host.h
@@ -6,6 +6,7 @@
 #define CONTENT_BROWSER_RENDERER_HOST_PEPPER_PEPPER_FILE_SYSTEM_BROWSER_HOST_H_
 
 #include "base/basictypes.h"
+#include "base/callback.h"
 #include "base/memory/weak_ptr.h"
 #include "ppapi/c/pp_file_info.h"
 #include "ppapi/host/host_message_context.h"
@@ -28,17 +29,14 @@ class PepperFileSystemBrowserHost :
                               PP_Instance instance,
                               PP_Resource resource,
                               PP_FileSystemType type);
-  // Creates a new PepperFileSystemBrowserHost with an existing file system at
-  // the given |root_url| and of the given |type|. The file system at |root_url|
-  // must already be opened. Once created, the PepperFileSystemBrowserHost may
-  // be used without being opened.
-  PepperFileSystemBrowserHost(BrowserPpapiHost* host,
-                              PP_Instance instance,
-                              PP_Resource resource,
-                              const GURL& root_url,
-                              PP_FileSystemType type);
   virtual ~PepperFileSystemBrowserHost();
 
+  // Opens the PepperFileSystemBrowserHost to use an existing file system at the
+  // given |root_url|. The file system at |root_url| must already be opened and
+  // have the type given by GetType().
+  // Calls |callback| when complete.
+  void OpenExisting(const GURL& root_url, base::Closure callback);

[yzshen1, 2013/11/01 22:27:39]

const &, please.

[Matt Giuca, 2013/11/01 23:19:47]

Done.

+
   // ppapi::host::ResourceHost override.
   virtual int32_t OnResourceMessageReceived(
       const IPC::Message& msg,
@@ -54,6 +52,9 @@ class PepperFileSystemBrowserHost :
   }
 
  private:
+  void OpenExistingHaveContext(

[dmichael (off chromium), 2013/11/01 17:33:52]

nit: s/HaveContext/WithContext
?

[Matt Giuca, 2013/11/01 23:19:47]

Done.

+      base::Closure callback,

[yzshen1, 2013/11/01 22:27:39]

const &, please.

[Matt Giuca, 2013/11/01 23:19:47]

Done.

+      scoped_refptr<fileapi::FileSystemContext> fs_context);
   void GotFileSystemContext(
       ppapi::host::ReplyMessageContext reply_context,
       fileapi::FileSystemType file_system_type,