extensions: Add ExtensionView interface.

chrome/browser/ui/views/extensions/extension_view_views.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 CHROME_BROWSER_UI_VIEWS_EXTENSIONS_EXTENSION_VIEW_VIEWS_H_
 #define CHROME_BROWSER_UI_VIEWS_EXTENSIONS_EXTENSION_VIEW_VIEWS_H_
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
+#include "chrome/browser/extensions/extension_view.h"
 #include "third_party/skia/include/core/SkBitmap.h"
 #include "ui/views/controls/native/native_view_host.h"
 
-class Browser;
-
 namespace content {
 class RenderViewHost;
 }
 
 namespace extensions {
 class Extension;
 class ExtensionHost;
 }
 
 // This handles the display portion of an ExtensionHost.
-class ExtensionViewViews : public views::NativeViewHost {
+class ExtensionViewViews : public ExtensionView,

[Ben Goodger (Google), 2012/09/17 16:07:17]

Because ExtensionViews are owned by the ExtensionHost, I think you'll need to
set owned by client on this one to prevent a double-free.

[tfarina, 2012/09/19 01:21:39]

Done.

+                           public views::NativeViewHost {
  public:
   ExtensionViewViews(extensions::ExtensionHost* host, Browser* browser);
   virtual ~ExtensionViewViews();
 
-  // A class that represents the container that this view is in.
-  // (bottom shelf, side bar, etc.)
-  class Container {
-   public:
-    virtual ~Container() {}
-    virtual void OnExtensionSizeChanged(ExtensionViewViews* view) {}
-    virtual void OnViewWasResized() {}
-  };
-
-  extensions::ExtensionHost* host() const { return host_; }
-  Browser* browser() const { return browser_; }
-  const extensions::Extension* extension() const;
-  content::RenderViewHost* render_view_host() const;
-  void DidStopLoading();
-  void SetIsClipped(bool is_clipped);
-
-  // Notification from ExtensionHost.
-  void ResizeDueToAutoResize(const gfx::Size& new_size);
-
-  // Method for the ExtensionHost to notify us when the RenderViewHost has a
-  // connection.
-  void RenderViewCreated();
-
   // Set a custom background for the view. The background will be tiled.
   void SetBackground(const SkBitmap& background);
 
-  // Sets the container for this view.
-  void SetContainer(Container* container) { container_ = container; }
+  // Overridden from views::View:
+  virtual void SetVisible(bool is_visible) OVERRIDE;
 
+ protected:
   // Overridden from views::NativeViewHost:
   virtual gfx::NativeCursor GetCursor(const ui::MouseEvent& event) OVERRIDE;
-  virtual void SetVisible(bool is_visible) OVERRIDE;
   virtual void ViewHierarchyChanged(bool is_add,
                                     views::View* parent,
                                     views::View* child) OVERRIDE;
 
- protected:
   // Overridden from views::View.
   virtual void PreferredSizeChanged() OVERRIDE;
   virtual bool SkipDefaultKeyEventProcessing(const ui::KeyEvent& e) OVERRIDE;
   virtual void OnBoundsChanged(const gfx::Rect& previous_bounds) OVERRIDE;
 
  private:
   friend class extensions::ExtensionHost;
 
+  // Overridden from ExtensionView:
+  virtual Browser* GetBrowser() OVERRIDE;
+  virtual const Browser* GetBrowser() const OVERRIDE;
+  virtual gfx::NativeView GetNativeView() OVERRIDE;
+  virtual content::RenderViewHost* GetRenderViewHost() const OVERRIDE;
+  virtual void SetContainer(ExtensionViewContainer* container) OVERRIDE;
+  virtual void ResizeDueToAutoResize(const gfx::Size& new_size) OVERRIDE;
+  virtual void RenderViewCreated() OVERRIDE;
+  virtual void DidStopLoading() OVERRIDE;
+  virtual void WindowFrameChanged() OVERRIDE;
+
   // Initializes the RenderWidgetHostView for this object.
   void CreateWidgetHostView();
 
   // We wait to show the ExtensionViewViews until several things have loaded.
   void ShowIfCompletelyLoaded();
 
   // Restore object to initial state. Called on shutdown or after a renderer
   // crash.
   void CleanUp();
 
@@ skipping 10 matching lines @@
   // The background the view should have once it is initialized. This is set
   // when the view has a custom background, but hasn't been initialized yet.
   SkBitmap pending_background_;
 
   // What we should set the preferred width to once the ExtensionViewViews has
   // loaded.
   gfx::Size pending_preferred_size_;
 
   // The container this view is in (not necessarily its direct superview).
   // Note: the view does not own its container.
-  Container* container_;
-
-  // Whether this extension view is clipped.
-  bool is_clipped_;
+  ExtensionViewContainer* container_;
 
   DISALLOW_COPY_AND_ASSIGN(ExtensionViewViews);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_EXTENSIONS_EXTENSION_VIEW_VIEWS_H_