aura: Fix crash when the aura window hosting a tooltip is destroyed. We fix

ash/tooltips/tooltip_controller_unittest.cc

Index: ash/tooltips/tooltip_controller_unittest.cc
diff --git a/ash/tooltips/tooltip_controller_unittest.cc b/ash/tooltips/tooltip_controller_unittest.cc
index bd8405fa9b15cc790d579f5122c10151d6b78c4e..fe5376e4f11b4ffd3103446ec904e02cee7ed7b4 100644
--- a/ash/tooltips/tooltip_controller_unittest.cc
+++ b/ash/tooltips/tooltip_controller_unittest.cc
@@ -2,6 +2,7 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#include "ash/display/display_controller.h"
 #include "ash/shell.h"
 #include "ash/test/ash_test_base.h"
 #include "ash/tooltips/tooltip_controller.h"
@@ -43,19 +44,24 @@ class TooltipTestView : public views::View {
   DISALLOW_COPY_AND_ASSIGN(TooltipTestView);
 };
 
-views::Widget* CreateNewWidget() {
+views::Widget* CreateNewWidgetWithBounds(const gfx::Rect& bounds) {
   views::Widget* widget = new views::Widget;
   views::Widget::InitParams params;
   params.type = views::Widget::InitParams::TYPE_WINDOW_FRAMELESS;
   params.accept_events = true;
   params.ownership = views::Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
-  params.parent = Shell::GetPrimaryRootWindow();
+  params.parent_widget = NULL;
   params.child = true;
+  params.bounds = bounds;
   widget->Init(params);
   widget->Show();
   return widget;
 }
 
+views::Widget* CreateNewWidget() {
+  return CreateNewWidgetWithBounds(gfx::Rect());
+}
+
 void AddViewToWidgetAndResize(views::Widget* widget, views::View* view) {
   if (!widget->GetContentsView()) {
     views::View* contents_view = new views::View;
@@ -68,7 +74,8 @@ void AddViewToWidgetAndResize(views::Widget* widget, views::View* view) {
   gfx::Rect contents_view_bounds = contents_view->bounds();
   contents_view_bounds = contents_view_bounds.Union(view->bounds());
   contents_view->SetBoundsRect(contents_view_bounds);
-  widget->SetBounds(contents_view_bounds);
+  widget->SetBounds(gfx::Rect(widget->GetWindowBoundsInScreen().origin(),
+                              contents_view_bounds.size()));
 }
 
 ash::internal::TooltipController* GetController() {
@@ -471,5 +478,45 @@ TEST_F(TooltipControllerTest, TooltipHidesOnTimeoutAndStaysHiddenUntilChange) {
   EXPECT_EQ(window, GetTooltipWindow());
 }
 
+TEST_F(TooltipControllerTest, TooltipsOnMultiDisplayShouldNotCrash) {
+  internal::DisplayController::SetExtendedDesktopEnabled(true);
+  UpdateDisplay("1000x600,600x400");
+  Shell::RootWindowList root_windows = Shell::GetAllRootWindows();
+  scoped_ptr<views::Widget> widget1(CreateNewWidgetWithBounds(
+      gfx::Rect(10, 10, 100, 100)));
+  TooltipTestView* view1 = new TooltipTestView;
+  AddViewToWidgetAndResize(widget1.get(), view1);
+  view1->set_tooltip_text(ASCIIToUTF16("Tooltip Text for view 1"));
+  EXPECT_EQ(widget1->GetNativeView()->GetRootWindow(), root_windows[0]);
+
+  scoped_ptr<views::Widget> widget2(CreateNewWidgetWithBounds(
+      gfx::Rect(1200, 10, 100, 100)));
+  TooltipTestView* view2 = new TooltipTestView;
+  AddViewToWidgetAndResize(widget2.get(), view2);
+  view2->set_tooltip_text(ASCIIToUTF16("Tooltip Text for view 2"));
+  EXPECT_EQ(widget2->GetNativeView()->GetRootWindow(), root_windows[1]);
+
+  // Show tooltip on second display.
+  aura::test::EventGenerator generator(root_windows[1]);
+  generator.MoveMouseRelativeTo(widget2->GetNativeView(),
+                                view2->bounds().CenterPoint());
+  FireTooltipTimer();
+  EXPECT_TRUE(IsTooltipVisible());
+
+  // Get rid of secondary display. This destroy's the tooltip's aura window. If
+  // we have handled this case, we will not crash in the following statement.
+  UpdateDisplay("1000x600");
+  EXPECT_FALSE(IsTooltipVisible());
+  EXPECT_EQ(widget2->GetNativeView()->GetRootWindow(), root_windows[0]);
+
+  // The tooltip should create a new aura window for itself, so we should still
+  // be able to show tooltips on the primary display.
+  aura::test::EventGenerator generator1(root_windows[0]);
+  generator1.MoveMouseRelativeTo(widget1->GetNativeView(),
+                                 view1->bounds().CenterPoint());
+  FireTooltipTimer();
+  EXPECT_TRUE(IsTooltipVisible());
+}
+
 }  // namespace test
 }  // namespace ash