extensions: Add ExtensionView interface.

chrome/browser/ui/views/extensions/extension_view_views.cc

Index: chrome/browser/ui/views/extensions/extension_view_views.cc
diff --git a/chrome/browser/ui/views/extensions/extension_view_views.cc b/chrome/browser/ui/views/extensions/extension_view_views.cc
index 9f083cd5123578b476fa955880c37562c7e4b8fe..512813b184200ad24c03c7ea06bc2b43f074cf06 100644
--- a/chrome/browser/ui/views/extensions/extension_view_views.cc
+++ b/chrome/browser/ui/views/extensions/extension_view_views.cc
@@ -20,9 +20,11 @@ ExtensionViewViews::ExtensionViewViews(extensions::ExtensionHost* host,
     : host_(host),
       browser_(browser),
       initialized_(false),
-      container_(NULL),
-      is_clipped_(false) {
-  host_->set_view(this);
+      container_(NULL) {
+  // We are owned by ExtensionHost, so views hierarchy shouldn't delete us!
+  set_owned_by_client();
+
+  host_->SetExtensionView(this);
 
   // This view needs to be focusable so it can act as the focused view for the
   // focus manager. This is required to have SkipDefaultKeyEventProcessing
@@ -36,28 +38,14 @@ ExtensionViewViews::~ExtensionViewViews() {
   CleanUp();
 }
 
-const extensions::Extension* ExtensionViewViews::extension() const {
-  return host_->extension();
-}
-
-content::RenderViewHost* ExtensionViewViews::render_view_host() const {
-  return host_->render_view_host();
-}
-
-void ExtensionViewViews::DidStopLoading() {
-  ShowIfCompletelyLoaded();
-}
-
-void ExtensionViewViews::SetIsClipped(bool is_clipped) {
-  if (is_clipped_ != is_clipped) {
-    is_clipped_ = is_clipped;
-    if (visible())
-      ShowIfCompletelyLoaded();
+void ExtensionViewViews::SetBackground(const SkBitmap& background) {
+  if (GetRenderViewHost()->IsRenderViewLive() &&
+      GetRenderViewHost()->GetView()) {
+    GetRenderViewHost()->GetView()->SetBackground(background);
+  } else {
+    pending_background_ = background;
   }
-}
-
-gfx::NativeCursor ExtensionViewViews::GetCursor(const ui::MouseEvent& event) {
-  return gfx::kNullCursor;
+  ShowIfCompletelyLoaded();
 }
 
 void ExtensionViewViews::SetVisible(bool is_visible) {
@@ -67,15 +55,84 @@ void ExtensionViewViews::SetVisible(bool is_visible) {
     // Also tell RenderWidgetHostView the new visibility. Despite its name, it
     // is not part of the View hierarchy and does not know about the change
     // unless we tell it.
-    if (render_view_host()->GetView()) {
+    if (GetRenderViewHost()->GetView()) {
       if (is_visible)
-        render_view_host()->GetView()->Show();
+        GetRenderViewHost()->GetView()->Show();
       else
-        render_view_host()->GetView()->Hide();
+        GetRenderViewHost()->GetView()->Hide();
     }
   }
 }
 
+gfx::NativeCursor ExtensionViewViews::GetCursor(const ui::MouseEvent& event) {
+  return gfx::kNullCursor;
+}
+
+void ExtensionViewViews::ViewHierarchyChanged(bool is_add,
+                                              views::View* parent,
+                                              views::View* child) {
+  NativeViewHost::ViewHierarchyChanged(is_add, parent, child);
+  if (is_add && GetWidget() && !initialized_)
+    CreateWidgetHostView();
+}
+
+Browser* ExtensionViewViews::GetBrowser() {
+  return browser_;
+}
+
+const Browser* ExtensionViewViews::GetBrowser() const {
+  return browser_;
+}
+
+gfx::NativeView ExtensionViewViews::GetNativeView() {
+  return host_->host_contents()->GetView()->GetNativeView();
+}
+
+content::RenderViewHost* ExtensionViewViews::GetRenderViewHost() const {
+  return host_->render_view_host();
+}
+
+void ExtensionViewViews::SetContainer(ExtensionViewContainer* container) {
+  container_ = container;
+}
+
+void ExtensionViewViews::ResizeDueToAutoResize(const gfx::Size& new_size) {
+  // Don't actually do anything with this information until we have been shown.
+  // Size changes will not be honored by lower layers while we are hidden.
+  if (!visible()) {
+    pending_preferred_size_ = new_size;
+    return;
+  }
+
+  gfx::Size preferred_size = GetPreferredSize();
+  if (new_size != preferred_size)
+    SetPreferredSize(new_size);
+}
+
+void ExtensionViewViews::RenderViewCreated() {
+  if (!pending_background_.empty() && GetRenderViewHost()->GetView()) {
+    GetRenderViewHost()->GetView()->SetBackground(pending_background_);
+    pending_background_.reset();
+  }
+
+  chrome::ViewType host_type = host_->extension_host_type();
+  if (host_type == chrome::VIEW_TYPE_EXTENSION_POPUP) {
+    gfx::Size min_size(ExtensionPopup::kMinWidth,
+                       ExtensionPopup::kMinHeight);
+    gfx::Size max_size(ExtensionPopup::kMaxWidth,
+                       ExtensionPopup::kMaxHeight);
+    GetRenderViewHost()->EnableAutoResize(min_size, max_size);
+  }
+}
+
+void ExtensionViewViews::DidStopLoading() {
+  ShowIfCompletelyLoaded();
+}
+
+void ExtensionViewViews::WindowFrameChanged() {
+  NOTIMPLEMENTED();
+}
+
 void ExtensionViewViews::CreateWidgetHostView() {
   DCHECK(!initialized_);
   initialized_ = true;
@@ -85,7 +142,7 @@ void ExtensionViewViews::CreateWidgetHostView() {
 }
 
 void ExtensionViewViews::ShowIfCompletelyLoaded() {
-  if (visible() || is_clipped_)
+  if (visible())
     return;
 
   // We wait to show the ExtensionViewViews until it has loaded, and the view
@@ -104,40 +161,10 @@ void ExtensionViewViews::CleanUp() {
   initialized_ = false;
 }
 
-void ExtensionViewViews::SetBackground(const SkBitmap& background) {
-  if (render_view_host()->IsRenderViewLive() && render_view_host()->GetView()) {
-    render_view_host()->GetView()->SetBackground(background);
-  } else {
-    pending_background_ = background;
-  }
-  ShowIfCompletelyLoaded();
-}
-
-void ExtensionViewViews::ResizeDueToAutoResize(const gfx::Size& new_size) {
-  // Don't actually do anything with this information until we have been shown.
-  // Size changes will not be honored by lower layers while we are hidden.
-  if (!visible()) {
-    pending_preferred_size_ = new_size;
-    return;
-  }
-
-  gfx::Size preferred_size = GetPreferredSize();
-  if (new_size != preferred_size)
-    SetPreferredSize(new_size);
-}
-
-void ExtensionViewViews::ViewHierarchyChanged(bool is_add,
-                                              views::View* parent,
-                                              views::View* child) {
-  NativeViewHost::ViewHierarchyChanged(is_add, parent, child);
-  if (is_add && GetWidget() && !initialized_)
-    CreateWidgetHostView();
-}
-
 void ExtensionViewViews::PreferredSizeChanged() {
   View::PreferredSizeChanged();
   if (container_)
-    container_->OnExtensionSizeChanged(this);
+    container_->OnExtensionSizeChanged(this, gfx::Size());
 }
 
 bool ExtensionViewViews::SkipDefaultKeyEventProcessing(const ui::KeyEvent& e) {
@@ -152,31 +179,8 @@ bool ExtensionViewViews::SkipDefaultKeyEventProcessing(const ui::KeyEvent& e) {
 void ExtensionViewViews::OnBoundsChanged(const gfx::Rect& previous_bounds) {
   // Propagate the new size to RenderWidgetHostView.
   // We can't send size zero because RenderWidget DCHECKs that.
-  if (render_view_host()->GetView() && !bounds().IsEmpty()) {
-    render_view_host()->GetView()->SetSize(size());
-
-    if (container_)
-      container_->OnViewWasResized();
-  }
-}
-
-void ExtensionViewViews::RenderViewCreated() {
-  if (!pending_background_.empty() && render_view_host()->GetView()) {
-    render_view_host()->GetView()->SetBackground(pending_background_);
-    pending_background_.reset();
-  }
-
-  chrome::ViewType host_type = host_->extension_host_type();
-  if (host_type == chrome::VIEW_TYPE_EXTENSION_POPUP) {
-    gfx::Size min_size(ExtensionPopup::kMinWidth,
-                       ExtensionPopup::kMinHeight);
-    gfx::Size max_size(ExtensionPopup::kMaxWidth,
-                       ExtensionPopup::kMaxHeight);
-    render_view_host()->EnableAutoResize(min_size, max_size);
-  }
-
-  if (container_)
-    container_->OnViewWasResized();
+  if (GetRenderViewHost()->GetView() && !bounds().IsEmpty())
+    GetRenderViewHost()->GetView()->SetSize(size());
 }
 
 void ExtensionViewViews::HandleKeyboardEvent(
@@ -184,3 +188,9 @@ void ExtensionViewViews::HandleKeyboardEvent(
   unhandled_keyboard_event_handler_.HandleKeyboardEvent(event,
                                                         GetFocusManager());
 }
+
+// static
+ExtensionView* ExtensionView::Create(extensions::ExtensionHost* host,
+                                     Browser* browser) {
+  return new ExtensionViewViews(host, browser);
+}