Browser Plugin: New Implementation (Browser Side)

content/browser/browser_plugin/browser_plugin_embedder.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.
+
+// A BrowserPluginEmbedder has a list of guests it manages.
+// In the beginning when a renderer sees one or more guests (BrowserPlugin
+// instance(s)) and there is a request to navigate to them, the WebContents for
+// that renderer creates a BrowserPluginEmbedder for itself. The
+// BrowserPluginEmbedder, in turn, manages a set of BrowserPluginGuests -- one
+// BrowserPluginGuest for each guest in the embedding WebContents. Note that
+// each of these BrowserPluginGuest objects has its own WebContents.
+// BrowserPluginEmbedder routes any messages directed to a guest from the
+// renderer (BrowserPlugin) to the appropriate guest (identified by the guest's
+// |instance_id|).
+//
+// BrowserPluginEmbedder is responsible for cleaning up the guests when the
+// embedder frame navigates away to a different page or deletes the guests from
+// the existing page.
+
+#ifndef CONTENT_BROWSER_BROWSER_PLUGIN_BROWSER_PLUGIN_EMBEDDER_H_
+#define CONTENT_BROWSER_BROWSER_PLUGIN_BROWSER_PLUGIN_EMBEDDER_H_
+
+#include <map>
+
+#include "base/compiler_specific.h"
+#include "content/public/browser/notification_observer.h"
+#include "content/public/browser/notification_registrar.h"
+#include "content/public/browser/web_contents_observer.h"
+
+class TransportDIB;
+class WebContentsImpl;
+
+namespace WebKit {
+class WebInputEvent;
+}
+
+namespace gfx {
+class Rect;
+class Size;
+}
+
+namespace content {
+
+class BrowserPluginGuest;
+class BrowserPluginHostFactory;
+
+// A browser plugin embedder provides functionality for WebContents to operate
+// in the 'embedder' role. It manages list of guests inside the embedder.
+//
+// The embedder's WebContents manages the lifetime of the embedder. They are
+// created when a renderer asks WebContents to navigate (for the first time) to
+// some guest. It gets destroyed when either the WebContents goes away or there
+// is a RenderViewHost swap in WebContents.
+class CONTENT_EXPORT BrowserPluginEmbedder : public WebContentsObserver,
+                                             public NotificationObserver {
+ public:
+  typedef std::map<int, WebContents*> ContainerInstanceMap;
+
+  virtual ~BrowserPluginEmbedder();
+
+  static BrowserPluginEmbedder* Create(WebContentsImpl* web_contents,
+                                       RenderViewHost* render_view_host);
+
+  // Navigates in a guest (new or existing).
+  void NavigateGuest(RenderViewHost* render_view_host,
+                     int instance_id,
+                     int64 frame_id,
+                     const std::string& src,
+                     const gfx::Size& size);
+
+  // WebContentsObserver implementation.
+  virtual void RenderViewDeleted(RenderViewHost* render_view_host) OVERRIDE;
+  virtual void RenderViewGone(base::TerminationStatus status) OVERRIDE;
+
+  // NotificationObserver method override.
+  virtual void Observe(int type,
+                       const NotificationSource& source,
+                       const NotificationDetails& details) OVERRIDE;
+
+  // Overrides factory for testing. Default (NULL) value indicates regular
+  // (non-test) environment.
+  static void set_factory_for_testing(BrowserPluginHostFactory* factory) {
+    factory_ = factory;
+  }
+
+ private:
+  friend class BrowserPluginEmbedderHelper;

[Charlie Reis, 2012/09/19 21:51:02]

Oh, I missed this.  We generally don't like friend classes, since they make it
harder to tell where private members are used (and thus what invariants hold). 
Can we avoid this?

[lazyboy, 2012/09/19 22:11:14]

Also, I'm assuming it's OK to keep TestFoo classes friends?

I will fix this tomorrow (I think I already have try success for 4:
linux_rel/mac/mac_rel/win_rel in the CQ so far).

[lazyboy, 2012/09/20 00:55:17]

Removed friend classes.

+  friend class TestBrowserPluginEmbedder;
+
+  BrowserPluginEmbedder(WebContentsImpl* web_contents,
+                        RenderViewHost* render_view_host);
+
+  // Returns a guest browser plugin delegate by its container ID specified
+  // in BrowserPlugin.
+  BrowserPluginGuest* GetGuestByInstanceID(int instance_id) const;
+  // Adds a new guest web_contents to the embedder (overridable in test).
+  virtual void AddGuest(int instance_id,
+                        WebContents* guest_web_contents,
+                        int64 frame_id);
+  void DestroyGuestByInstanceID(int instance_id);
+  void DestroyGuests();
+
+  // Message handlers (direct/indirect via BrowserPluginEmbedderHelper).
+  // Routes update rect ack message to the appropriate guest.
+  void UpdateRectACK(int instance_id, int message_id, const gfx::Size& size);
+  void SetFocus(int instance_id, bool focused);
+  void ResizeGuest(int instance_id,
+                   TransportDIB* damage_buffer,
+#if defined(OS_WIN)
+                   int damage_buffer_size,
+#endif
+                   int width,
+                   int height,
+                   bool resize_pending,
+                   float scale_factor);
+  // Handles input events sent from the BrowserPlugin (embedder's renderer
+  // process) by passing them to appropriate guest's input handler.
+  void HandleInputEvent(int instance_id,
+                        RenderViewHost* render_view_host,
+                        const gfx::Rect& guest_rect,
+                        const WebKit::WebInputEvent& event,
+                        IPC::Message* reply_message);
+  void PluginDestroyed(int instance_id);
+
+  // Called when visiblity of web_contents changes, so the embedder will
+  // show/hide its guest.
+  void WebContentsVisibilityChanged(bool visible);
+
+  // Static factory instance (always NULL for non-test).
+  static BrowserPluginHostFactory* factory_;
+
+  // A scoped container for notification registries.
+  NotificationRegistrar registrar_;
+
+  // Contains guests' WebContents, mapping from their instance ids.
+  ContainerInstanceMap guest_web_contents_by_instance_id_;
+  RenderViewHost* render_view_host_;
+
+  DISALLOW_COPY_AND_ASSIGN(BrowserPluginEmbedder);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_BROWSER_PLUGIN_BROWSER_PLUGIN_EMBEDDER_H_