Fix several painting glitches for toolstrips.

chrome/browser/views/hwnd_html_view.cc

Index: chrome/browser/views/hwnd_html_view.cc
diff --git a/chrome/browser/views/hwnd_html_view.cc b/chrome/browser/views/hwnd_html_view.cc
index a0174d6f04e619049909189d584f31cdcfb1c52d..d51c259b8d12d5673c76b69bfed392dccb95aa4b 100644
--- a/chrome/browser/views/hwnd_html_view.cc
+++ b/chrome/browser/views/hwnd_html_view.cc
@@ -32,6 +32,27 @@ HWNDHtmlView::~HWNDHtmlView() {
   }
 }
 
+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_);