BrowserPluginGuest/Embedder: Remove friend helper classes, made necessary calls public

content/browser/browser_plugin/browser_plugin_guest.h

Index: content/browser/browser_plugin/browser_plugin_guest.h
diff --git a/content/browser/browser_plugin/browser_plugin_guest.h b/content/browser/browser_plugin/browser_plugin_guest.h
index e007eb0397f49fedc4d5d2671296b97efaa31862..353163f73e361ef62d3548646fe5658eed2eb09d 100644
--- a/content/browser/browser_plugin/browser_plugin_guest.h
+++ b/content/browser/browser_plugin/browser_plugin_guest.h
@@ -76,6 +76,11 @@ class CONTENT_EXPORT BrowserPluginGuest : public WebContentsDelegate,
     guest_hang_timeout_ = timeout;
   }
 
+  void set_embedder_render_process_host(
+      RenderProcessHost* render_process_host) {
+    embedder_render_process_host_ = render_process_host;
+  }
+
   // WebContentsObserver implementation.
   virtual void DidCommitProvisionalLoadForFrame(
       int64 frame_id,
@@ -88,30 +93,6 @@ class CONTENT_EXPORT BrowserPluginGuest : public WebContentsDelegate,
   // WebContentsDelegate implementation.
   virtual void RendererUnresponsive(WebContents* source) OVERRIDE;
 
- private:
-  friend class BrowserPluginEmbedder;
-  friend class BrowserPluginGuestHelper;
-  friend class TestBrowserPluginGuest;
-
-  BrowserPluginGuest(int instance_id,
-                     WebContentsImpl* web_contents,
-                     RenderViewHost* render_view_host);
-
-  void set_embedder_render_process_host(
-      RenderProcessHost* render_process_host) {
-    embedder_render_process_host_ = render_process_host;
-  }
-  RenderProcessHost* embedder_render_process_host() {
-    return embedder_render_process_host_;
-  }
-  // Returns the identifier that uniquely identifies a browser plugin guest
-  // within an embedder.
-  int instance_id() const { return instance_id_; }
-  TransportDIB* damage_buffer() const { return damage_buffer_.get(); }
-  const gfx::Size& damage_view_size() const { return damage_view_size_; }
-  float damage_buffer_scale_factor() const {
-    return damage_buffer_scale_factor_;
-  }
   void SetDamageBuffer(TransportDIB* damage_buffer,
 #if defined(OS_WIN)
                        int damage_buffer_size,
@@ -138,18 +119,41 @@ class CONTENT_EXPORT BrowserPluginGuest : public WebContentsDelegate,
   // embedder) instead of default view/widget host.
   void HandleInputEventAck(RenderViewHost* render_view_host, bool handled);
 
-  // Helper to send messages to embedder. Overridden in test implementation
-  // since we want to intercept certain messages for testing.
-  virtual void SendMessageToEmbedder(IPC::Message* msg);
-  // Overridden in tests.
-  virtual void SetFocus(bool focused);
+  // Exposes the protected web_contents() from WebContentsObserver.
+  WebContents* GetWebContents();
+
   // Overridden in tests.
   virtual bool ViewTakeFocus(bool reverse);
+  // Overridden in tests.
+  virtual void SetFocus(bool focused);
   // Reload the guest.
   virtual void Reload();
   // Stop loading the guest.
   virtual void Stop();
 
+ private:
+  friend class TestBrowserPluginGuest;
+
+  BrowserPluginGuest(int instance_id,
+                     WebContentsImpl* web_contents,
+                     RenderViewHost* render_view_host);
+
+  RenderProcessHost* embedder_render_process_host() {
+    return embedder_render_process_host_;
+  }
+  // Returns the identifier that uniquely identifies a browser plugin guest
+  // within an embedder.
+  int instance_id() const { return instance_id_; }
+  TransportDIB* damage_buffer() const { return damage_buffer_.get(); }
+  const gfx::Size& damage_view_size() const { return damage_view_size_; }
+  float damage_buffer_scale_factor() const {
+    return damage_buffer_scale_factor_;
+  }
+
+  // Helper to send messages to embedder. Overridden in test implementation
+  // since we want to intercept certain messages for testing.
+  virtual void SendMessageToEmbedder(IPC::Message* msg);
+
   // Static factory instance (always NULL for non-test).
   static content::BrowserPluginHostFactory* factory_;