Move BookmarkBubble with anchor

ui/views/bubble/bubble_delegate.cc

Index: ui/views/bubble/bubble_delegate.cc
diff --git a/ui/views/bubble/bubble_delegate.cc b/ui/views/bubble/bubble_delegate.cc
index 53ff9e69b7e56f075dd900172d24f536cc9b95bc..972743dc964fc8bebdda8bd67a539749020fd36c 100644
--- a/ui/views/bubble/bubble_delegate.cc
+++ b/ui/views/bubble/bubble_delegate.cc
@@ -176,10 +176,16 @@ void BubbleDelegateView::OnWidgetActivationChanged(Widget* widget,
 void BubbleDelegateView::OnWidgetBoundsChanged(Widget* widget,
                                                const gfx::Rect& new_bounds) {
   if (anchor_widget() == widget) {
-    if (move_with_anchor())
+    if (move_with_anchor()) {
       SizeToContents();
-    else
-      GetWidget()->Close();
+    } else {
+      // Do not close if the widget gets shrinked vertically without move.

[msw, 2014/02/13 00:32:48]

This seems odd, and I don't think this is the right fix. The widget bounds may
have the same origin and height, but the anchor bounds within that widget still
may have changed (the anchor view may have moved within the bounds of the
widget, due to layout changes). I think we could probably just close if the
widget origin changes, or the anchor bounds have changed; that should solve your
bug. LMK if you need help.

[oshima, 2014/02/13 01:28:29]

You're right that anchor position may change even if only the height changes,
but for the same reason, we need to close when the width changes because the
location of star icon does change when width changes.

I actually wonder why bookmark bubble isn't set to move with anchor. Do you know
why?

[msw, 2014/02/13 01:38:07]

I don't see a reason offhand why the bookmark bubble doesn't move with its
anchor (just tried this and it doesn't seem too broken). If that fixes your
issue and doesn't cause other issues, let's go with that.

+      gfx::Rect bounds = widget->GetWindowBoundsInScreen();
+      if (bounds.origin() != new_bounds.origin() ||
+          bounds.width() != new_bounds.width()) {
+        GetWidget()->Close();
+      }
+    }
   }
 }