Convert PepperBrowserConnection to be a RenderFrameObserver.

content/renderer/pepper/pepper_browser_connection.h

Index: content/renderer/pepper/pepper_browser_connection.h
===================================================================
--- content/renderer/pepper/pepper_browser_connection.h	(revision 238258)
+++ content/renderer/pepper/pepper_browser_connection.h	(working copy)
@@ -11,12 +11,14 @@
 
 #include "base/callback.h"
 #include "base/files/file_path.h"
-#include "content/public/renderer/render_view_observer.h"
-#include "content/public/renderer/render_view_observer_tracker.h"
+#include "content/public/renderer/render_frame_observer.h"
+#include "content/public/renderer/render_frame_observer_tracker.h"
 #include "ppapi/c/pp_file_info.h"
 #include "ppapi/c/pp_instance.h"
 #include "ppapi/c/pp_resource.h"
 
+class GURL;

[nasko, 2013/12/03 18:27:52]

Why is this new declaration needed?

[jam, 2013/12/03 20:32:09]

because it was forward declared in render_view_observer.h but not
render_frame_observer.h

+
 namespace content {
 
 // This class represents a connection from the renderer to the browser for
@@ -24,12 +26,12 @@
 // and renderer communicate about ResourceHosts, they should pass the plugin
 // process ID to identify which plugin they are talking about.
 class PepperBrowserConnection
-    : public RenderViewObserver,
-      public RenderViewObserverTracker<PepperBrowserConnection> {
+    : public RenderFrameObserver,
+      public RenderFrameObserverTracker<PepperBrowserConnection> {
  public:
   typedef base::Callback<void(const std::vector<int>&)>
       PendingResourceIDCallback;
-  explicit PepperBrowserConnection(RenderView* render_view);
+  explicit PepperBrowserConnection(RenderFrame* render_frame);
   virtual ~PepperBrowserConnection();
 
   virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;