Mostly fixes black flashing that happens during popup resize.

chrome/browser/views/browser_bubble_win.cc

Index: chrome/browser/views/browser_bubble_win.cc
diff --git a/chrome/browser/views/browser_bubble_win.cc b/chrome/browser/views/browser_bubble_win.cc
index d25fea387d586e65526ccb0fbf2042fdc54d2ab6..3b0b9b8d69fcb92a6ea40a6091816510dddeadf6 100644
--- a/chrome/browser/views/browser_bubble_win.cc
+++ b/chrome/browser/views/browser_bubble_win.cc
@@ -14,6 +14,8 @@ class BubbleWidget : public views::WidgetWin {
  public:
   explicit BubbleWidget(BrowserBubble* bubble)
       : bubble_(bubble), closed_(false) {
+    set_window_style(WS_POPUP | WS_CLIPCHILDREN);

[amit, 2009/11/16 18:06:27]

This will work but it sets the window style per instance. Ideally for all
instances of BubbleWidget, the style should be specified by the class style.
Even better would be if all the WidgetWin window class could have this style.
This could be done by adding WS_CLIPCHILDREN | WS_CLIPSIBLINGS to the
kWindowDefaultStyle in window_impl.cc  Adding it here would make sure that the
flicker issue would not have to be debugged and fixed elsewhere in future :)

+    set_window_ex_style(WS_EX_TOOLWINDOW);
   }
 
   void Show(bool activate) {
@@ -70,7 +72,6 @@ void BrowserBubble::InitPopup() {
   // popup_ is a Widget, but we need to do some WidgetWin stuff first, then
   // we'll assign it into popup_.
   views::WidgetWin* pop = new BubbleWidget(this);
-  pop->set_window_style(WS_POPUP);
   pop->Init(frame_native_view_, bounds_);
   pop->SetContentsView(view_);
 
@@ -81,7 +82,7 @@ void BrowserBubble::InitPopup() {
 
 void BrowserBubble::MovePopup(int x, int y, int w, int h) {
   views::WidgetWin* pop = static_cast<views::WidgetWin*>(popup_);
-  pop->MoveWindow(x, y, w, h);
+  pop->SetBounds(gfx::Rect(x, y, w, h));
 }
 
 void BrowserBubble::Show(bool activate) {