views: Make sure the bubble-delegate removes itself from the anchor-widget's observer list.

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 f593a1b23e21fe762c086a2ce229c8f447b0181f..04d77cae751e3a534c51d38b9e759957293ab5e5 100644
--- a/ui/views/bubble/bubble_delegate.cc
+++ b/ui/views/bubble/bubble_delegate.cc
@@ -163,6 +163,13 @@ View* BubbleDelegateView::GetInitiallyFocusedView() {
   return this;
 }
 
+void BubbleDelegateView::WindowClosing() {
+  if (anchor_widget_) {

[msw, 2012/04/24 18:10:41]

nit: use anchor_widget() here and @line168 for r-value access.

[sadrul, 2012/04/24 18:15:42]

I think we actually prefer directly using the variable (at least in the rest of
the views code). (see all the uses of parent_ in view.cc for example)

[msw, 2012/04/24 18:24:58]

Fair enough.

+    anchor_widget_->RemoveObserver(this);
+    anchor_widget_ = NULL;

[msw, 2012/04/24 18:10:41]

Sorry, would you mind also setting anchor_view_ to null here?

[sadrul, 2012/04/24 18:15:42]

Done.

+  }
+}
+
 BubbleDelegateView* BubbleDelegateView::AsBubbleDelegate() {
   return this;
 }