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

ash/tooltips/tooltip_controller_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 "ash/display/display_controller.h"
 #include "ash/shell.h"
 #include "ash/test/ash_test_base.h"
 #include "ash/tooltips/tooltip_controller.h"
 #include "ash/wm/cursor_manager.h"
 #include "base/utf_string_conversions.h"
 #include "ui/aura/client/tooltip_client.h"
 #include "ui/aura/env.h"
 #include "ui/aura/root_window.h"
 #include "ui/aura/test/event_generator.h"
 #include "ui/aura/window.h"
@@ skipping 21 matching lines @@
     *tooltip = tooltip_text_;
     return true;
   }
 
  private:
   string16 tooltip_text_;
 
   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;
     widget->SetContentsView(contents_view);
   }
 
   views::View* contents_view = widget->GetContentsView();
   contents_view->AddChildView(view);
   view->SetBounds(contents_view->width(), 0, 100, 100);
   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() {
   return static_cast<ash::internal::TooltipController*>(
       aura::client::GetTooltipClient(Shell::GetPrimaryRootWindow()));
 }
 
 gfx::Font GetDefaultFont() {
   return ui::ResourceBundle::GetSharedInstance().GetFont(
       ui::ResourceBundle::BaseFont);
@@ skipping 382 matching lines @@
   EXPECT_TRUE(IsTooltipTimerRunning());
   FireTooltipTimer();
   EXPECT_TRUE(IsTooltipVisible());
   EXPECT_TRUE(IsTooltipShownTimerRunning());
   string16 expected_tooltip = ASCIIToUTF16("Tooltip Text for view 2");
   EXPECT_EQ(expected_tooltip, aura::client::GetTooltipText(window));
   EXPECT_EQ(expected_tooltip, GetTooltipText());
   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