Browser Plugin: New Implementation (Browser Side)

content/browser/browser_plugin/browser_plugin_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_BROWSER_PLUGIN_BROWSER_PLUGIN_HOST_FACTORY_H_
+#define CONTENT_BROWSER_BROWSER_PLUGIN_BROWSER_PLUGIN_HOST_FACTORY_H_
+
+#include "base/base_export.h"
+#include "base/basictypes.h"
+#include "base/compiler_specific.h"
+#include "base/string16.h"
+#include "content/common/content_export.h"
+
+class WebContentsImpl;
+
+namespace content {
+
+class BrowserPluginEmbedder;
+class BrowserPluginGuest;
+class RenderViewHost;
+
+// Factory to create BrowserPlugin embedder and guest.
+class CONTENT_EXPORT BrowserPluginHostFactory {
+ public:
+  virtual BrowserPluginGuest* CreateBrowserPluginGuest(
+      int instance_id,
+      WebContentsImpl* web_contents,
+      RenderViewHost* render_view_host) = 0;
+
+  virtual BrowserPluginEmbedder* CreateBrowserPluginEmbedder(
+      WebContentsImpl* web_contents,
+      RenderViewHost* render_view_host) = 0;

[awong, 2012/09/06 19:55:26]

Nice catch on the virtual destructor :)

nit: add a newline above private:

[lazyboy, 2012/09/07 19:33:19]

Need to make it protected and need providing blank impl. (didn't compile
browsertest on last iter)
Done.

+ private:
+  virtual ~BrowserPluginHostFactory();
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_BROWSER_PLUGIN_BROWSER_PLUGIN_HOST_FACTORY_H_