Fixing crash Report - Magic Signature: views::View::ConvertPointToScreen

ui/views/bubble/bubble_delegate_unittest.cc

 // Copyright (c) 2012 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 "base/run_loop.h"
 #include "ui/base/hit_test.h"
 #include "ui/views/bubble/bubble_delegate.h"
 #include "ui/views/bubble/bubble_frame_view.h"
 #include "ui/views/test/test_widget_observer.h"
 #include "ui/views/test/views_test_base.h"
@@ skipping 11 matching lines @@
 class TestBubbleDelegateView : public BubbleDelegateView {
  public:
   TestBubbleDelegateView(View* anchor_view)
       : BubbleDelegateView(anchor_view, BubbleBorder::TOP_LEFT),
         view_(new View()) {
     view_->set_focusable(true);
     AddChildView(view_);
   }
   virtual ~TestBubbleDelegateView() {}
 
+  void SetAnchorRectForTest(gfx::Rect rect) {
+    set_anchor_rect(rect);
+  }
+
+  void SetAnchorViewForTest(View* view) {
+    SetAnchorView(view);
+  }
+
   // BubbleDelegateView overrides:
   virtual View* GetInitiallyFocusedView() OVERRIDE { return view_; }
   virtual gfx::Size GetPreferredSize() OVERRIDE { return gfx::Size(200, 200); }
   virtual int GetFadeDuration() OVERRIDE { return 1; }
 
  private:
   View* view_;
 
   DISALLOW_COPY_AND_ASSIGN(TestBubbleDelegateView);
 };
@@ skipping 60 matching lines @@
   //            aura::test::TestActivationClient::OnWindowDestroyed().
   scoped_ptr<Widget> smoke_and_mirrors_widget(CreateTestWidget());
   EXPECT_FALSE(bubble_observer.widget_closed());
 #endif
 
   // Ensure that closing the anchor widget also closes the bubble itself.
   anchor_widget->CloseNow();
   EXPECT_TRUE(bubble_observer.widget_closed());
 }
 
+TEST_F(BubbleDelegateTest, CloseAnchorViewTest) {
+  // Create an anchor widget and add a view to be used as anchor view.

[msw, 2013/09/27 21:21:53]

nit: "an anchor view."

[Mr4D (OOO till 08-26), 2013/09/27 22:52:21]

Done.

+  scoped_ptr<Widget> anchor_widget(CreateTestWidget());
+  scoped_ptr<View> anchor_view(new View());
+  anchor_widget->GetContentsView()->AddChildView(anchor_view.get());
+  TestBubbleDelegateView* bubble_delegate = new TestBubbleDelegateView(
+      anchor_view.get());
+  Widget* bubble_widget = BubbleDelegateView::CreateBubble(bubble_delegate);
+
+  // Check that the anchor view is correct and set up an anchor view rect.
+  // Make sure that this rect will get ignored (as long as the anchor view is
+  // attached).
+  EXPECT_EQ(anchor_view, bubble_delegate->GetAnchorView());
+  EXPECT_TRUE(bubble_delegate->has_anchor_view());
+  const gfx::Rect set_anchor_rect = gfx::Rect(10, 10, 100, 100);
+  bubble_delegate->SetAnchorRectForTest(set_anchor_rect);
+  const gfx::Rect view_rect = bubble_delegate->GetAnchorRect();
+  EXPECT_NE(view_rect.ToString(), set_anchor_rect.ToString());
+
+  // Create the bubble.
+  bubble_widget->Show();
+  EXPECT_EQ(anchor_widget, bubble_delegate->anchor_widget());
+
+  // Remove now the anchor view and make sure that the original found rect
+  // is still kept, so that the bubble does not jump when the view gets deleted.
+  anchor_widget->GetContentsView()->RemoveChildView(anchor_view.get());
+  anchor_view.reset();
+  EXPECT_TRUE(bubble_delegate->has_anchor_view());
+  EXPECT_EQ(NULL, bubble_delegate->GetAnchorView());
+  EXPECT_EQ(view_rect.ToString(), bubble_delegate->GetAnchorRect().ToString());
+}
+
+TEST_F(BubbleDelegateTest, TestAnchorRectMovesWithViewTest) {
+  // Create an anchor widget and add a view to be used as anchor view.
+  scoped_ptr<Widget> anchor_widget(CreateTestWidget());
+  TestBubbleDelegateView* bubble_delegate = new TestBubbleDelegateView(
+      anchor_widget->GetContentsView());
+  BubbleDelegateView::CreateBubble(bubble_delegate);
+
+  anchor_widget->GetContentsView()->SetBounds(10, 10, 100, 100);
+  const gfx::Rect view_rect = bubble_delegate->GetAnchorRect();
+
+  anchor_widget->GetContentsView()->SetBounds(20, 10, 100, 100);
+  const gfx::Rect view_rect_2 = bubble_delegate->GetAnchorRect();
+  EXPECT_NE(view_rect.ToString(), view_rect_2.ToString());
+

[msw, 2013/09/27 21:21:53]

nit: remove this blank line.

[Mr4D (OOO till 08-26), 2013/09/27 22:52:21]

Done.

+}
+
 TEST_F(BubbleDelegateTest, ResetAnchorWidget) {
   scoped_ptr<Widget> anchor_widget(CreateTestWidget());
   BubbleDelegateView* bubble_delegate = new BubbleDelegateView(
       anchor_widget->GetContentsView(), BubbleBorder::NONE);
 
   // Make sure the bubble widget is parented to a widget other than the anchor
   // widget so that closing the anchor widget does not close the bubble widget.
   scoped_ptr<Widget> parent_widget(CreateTestWidget());
   bubble_delegate->set_parent_window(parent_widget->GetNativeView());
   // Preventing close on deactivate should not prevent closing with the parent.
@@ skipping 117 matching lines @@
             bubble_widget->GetFocusManager()->GetFocusedView());
 
   Observe(bubble_widget);
 
   bubble_delegate->StartFade(false);
   RunNestedLoop();
   EXPECT_TRUE(bubble_destroyed());
 }
 
 }  // namespace views