Pepper: Move FileIO host from renderer to browser.

content/browser/renderer_host/pepper/pepper_file_ref_host.h

 // 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 CONTENT_BROWSER_RENDERER_HOST_PEPPER_PEPPER_FILE_REF_HOST_H_
 #define CONTENT_BROWSER_RENDERER_HOST_PEPPER_PEPPER_FILE_REF_HOST_H_
 
 #include <string>
 
 #include "base/memory/weak_ptr.h"
 #include "content/public/browser/browser_ppapi_host.h"
 #include "ppapi/c/pp_file_info.h"
 #include "ppapi/c/pp_instance.h"
 #include "ppapi/c/pp_resource.h"
 #include "ppapi/c/pp_time.h"
 #include "ppapi/host/host_message_context.h"
 #include "ppapi/host/resource_host.h"
 #include "webkit/browser/fileapi/file_system_url.h"
 
 namespace content {
 
 class PepperFileRefHost;
+class PepperFileSystemBrowserHost;
 
 // Internal and external filesystems have very different codepaths for
 // performing FileRef operations. The logic is split into separate classes
 // to make it easier to read.
 class PepperFileRefBackend {
  public:
   virtual ~PepperFileRefBackend();
 
   virtual int32_t MakeDirectory(ppapi::host::ReplyMessageContext context,
                                 bool make_ancestors) = 0;
@@ skipping 62 matching lines @@
   int32_t OnDelete(ppapi::host::HostMessageContext* context);
   int32_t OnRename(ppapi::host::HostMessageContext* context,
                    PP_Resource new_file_ref);
   int32_t OnQuery(ppapi::host::HostMessageContext* context);
   int32_t OnReadDirectoryEntries(ppapi::host::HostMessageContext* context);
   int32_t OnGetAbsolutePath(ppapi::host::HostMessageContext* context);
 
   BrowserPpapiHost* host_;
   scoped_ptr<PepperFileRefBackend> backend_;
   PP_FileSystemType fs_type_;
+  base::WeakPtr<PepperFileSystemBrowserHost> file_system_host_;

[bbudge, 2013/10/22 01:59:39]

It looks like this is only used in a single method, which passes it to the
backend. Is there a reason to hold onto the weak ptr here?

[teravest, 2013/10/22 15:45:05]

Good catch, removed.

 
   DISALLOW_COPY_AND_ASSIGN(PepperFileRefHost);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_PEPPER_PEPPER_FILE_REF_HOST_H_