Fix several painting glitches for toolstrips.

chrome/browser/views/hwnd_html_view.cc

 // Copyright (c) 2006-2008 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/views/hwnd_html_view.h"
 
 #include "chrome/browser/renderer_host/render_view_host.h"
 #include "chrome/browser/renderer_host/render_widget_host_view_win.h"
 #include "chrome/browser/renderer_host/render_view_host_delegate.h"
 #include "chrome/browser/tab_contents/site_instance.h"
@@ skipping 14 matching lines @@
 }
 
 HWNDHtmlView::~HWNDHtmlView() {
   if (render_view_host_) {
     Detach();
     render_view_host_->Shutdown();
     render_view_host_ = NULL;
   }
 }
 
+void HWNDHtmlView::SetVisible(bool is_visible) {
+  HWNDView::SetVisible(is_visible);
+
+  // Also tell RenderWidgetHostView the new visibility. Despite its name, it is
+  // not part of the View heirarchy and does not know about the change unless we
+  // tell it.
+  if (render_view_host() && render_view_host()->view()) {
+    if (is_visible)
+      render_view_host()->view()->Show();
+    else
+      render_view_host()->view()->Hide();
+  }
+}
+
+void HWNDHtmlView::DidChangeBounds(const gfx::Rect& previous,
+                                   const gfx::Rect& current) {
+  // Propagate the new size to RenderWidgetHostView.
+  if (render_view_host() && render_view_host()->view() && !current.IsEmpty())

[Erik does not do reviews, 2009/04/27 19:47:30]

I still think this isn't the correct final fix (but is good for this CL). You
should reference the two bugs that I had originally, so that we remove this code
when the right fixes arrive.

See: http://codereview.chromium.org/96008/diff2/3001:4011/3010 for the original
comments on the left.

+    render_view_host()->view()->SetSize(gfx::Size(width(), height()));
+}
+
 void HWNDHtmlView::SetBackground(const SkBitmap& background) {
   if (initialized_) {
     DCHECK(render_view_host_);
     render_view_host_->view()->SetBackground(background);
   } else {
     pending_background_ = background;
   }
 }
 
 void HWNDHtmlView::InitHidden() {
@@ skipping 32 matching lines @@
   }
   rvh->NavigateToURL(content_url_);
   initialized_ = true;
 }
 
 void HWNDHtmlView::ViewHierarchyChanged(bool is_add, View* parent,
                                         View* child) {
   if (is_add && GetWidget() && !initialized_)
     Init(GetWidget()->GetNativeView());
 }