Part 1 of dragging extensions on the shelf. This part was just about getting...

chrome/browser/extensions/extension_view.cc

 // Copyright (c) 2006-2009 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.
 
 #include "chrome/browser/extensions/extension_view.h"
 
 #include "chrome/browser/extensions/extension_host.h"
 #include "chrome/browser/renderer_host/render_view_host.h"
 #include "chrome/browser/renderer_host/render_widget_host_view.h"
 #if defined(OS_WIN)
@@ skipping 24 matching lines @@
       if (is_visible)
         render_view_host()->view()->Show();
       else
         render_view_host()->view()->Hide();
     }
   }
 }
 
 void ExtensionView::DidChangeBounds(const gfx::Rect& previous,
                                     const gfx::Rect& current) {
+  View::DidChangeBounds(previous, current);
   // Propagate the new size to RenderWidgetHostView.
   // We can't send size zero because RenderWidget DCHECKs that.
   if (render_view_host()->view() && !current.IsEmpty())
     render_view_host()->view()->SetSize(gfx::Size(width(), height()));
-  // Layout is where the HWND is properly positioned.
-  // TODO(erikkay) - perhaps this should be in NativeViewHost
-  Layout();
 }
 
 void ExtensionView::ShowIfCompletelyLoaded() {
   // We wait to show the ExtensionView until it has loaded and our parent has
   // given us a background. These can happen in different orders.
   if (!IsVisible() && host_->did_stop_loading() && render_view_host()->view() &&
       !render_view_host()->view()->background().empty()) {
     SetVisible(true);
     DidContentsPreferredWidthChange(pending_preferred_width_);
   }
@@ skipping 17 matching lines @@
     SetPreferredSize(gfx::Size(pref_width, height()));
   }
 }
 
 void ExtensionView::ViewHierarchyChanged(bool is_add,
                                          views::View *parent,
                                          views::View *child) {
   NativeViewHost::ViewHierarchyChanged(is_add, parent, child);
   if (is_add && GetWidget() && !initialized_) {
     initialized_ = true;
-
-    RenderWidgetHostView* view = RenderWidgetHostView::CreateViewForWidget(
-        render_view_host());
+    RenderWidgetHostView* view =
+        RenderWidgetHostView::CreateViewForWidget(render_view_host());
 
     // TODO(mpcomplete): RWHV needs a cross-platform Init function.
 #if defined(OS_WIN)
     // Create the HWND. Note:
     // RenderWidgetHostHWND supports windowed plugins, but if we ever also
     // wanted to support constrained windows with this, we would need an
     // additional HWND to parent off of because windowed plugin HWNDs cannot
     // exist in the same z-order as constrained windows.
     RenderWidgetHostViewWin* view_win =
         static_cast<RenderWidgetHostViewWin*>(view);
@@ skipping 16 matching lines @@
 
 void ExtensionView::HandleMouseEvent() {
   if (container_)
     container_->OnExtensionMouseEvent(this);
 }
 
 void ExtensionView::HandleMouseLeave() {
   if (container_)
     container_->OnExtensionMouseLeave(this);
 }