Refactoring of new IPC-only pepper implementation

content/browser/renderer_host/pepper/content_browser_pepper_host_factory.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_BROWSER_PEPPER_CONTENT_BROWSER_PEPPER_HOST_FACTORY_H_
 #define CONTENT_BROWSER_PEPPER_CONTENT_BROWSER_PEPPER_HOST_FACTORY_H_
 
 #include "base/compiler_specific.h"
 #include "ppapi/host/host_factory.h"
 
-class PepperMessageFilter;
+namespace content {
 
-namespace content {
+class BrowserPpapiHostImpl;
 
 class ContentBrowserPepperHostFactory : public ppapi::host::HostFactory {

[raymes, 2012/07/25 16:53:22]

I realize this is not directly related to this CL but I think it would be nicer
if HostFactory and friends were named ResourceHostFactory (since it's creating
ResourceHosts and not PpapiHosts).

Even clearer (though longer) might be:
we have PluginResource
PluginResourceHost
PluginResourceHostFactory

[brettw, 2012/07/30 05:46:37]

Sounds good, I'll make a note.
I'll consider this too when I do that.

  public:
   // Non-owning pointer to the filter must outlive this class.
-  explicit ContentBrowserPepperHostFactory(PepperMessageFilter* filter);
+  explicit ContentBrowserPepperHostFactory(BrowserPpapiHostImpl* host);
   virtual ~ContentBrowserPepperHostFactory();
 
   virtual scoped_ptr<ppapi::host::ResourceHost> CreateResourceHost(
       ppapi::host::PpapiHost* host,
       const ppapi::proxy::ResourceMessageCallParams& params,
       PP_Instance instance,
       const IPC::Message& message) OVERRIDE;
 
  private:
   // Non-owning pointer.
-  PepperMessageFilter* filter_;
+  BrowserPpapiHostImpl* host_;
 
   DISALLOW_COPY_AND_ASSIGN(ContentBrowserPepperHostFactory);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_PEPPER_CONTENT_BROWSER_PEPPER_HOST_FACTORY_H_