aura: couple of tooltip UX changes:

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/shell.h"
 #include "ash/test/ash_test_base.h"
 #include "ash/tooltips/tooltip_controller.h"
 #include "base/utf_string_conversions.h"
 #include "ui/aura/client/tooltip_client.h"
 #include "ui/aura/cursor_manager.h"
@@ skipping 82 matching lines @@
   }
 
   aura::Window* GetTooltipWindow() {
     return GetController()->tooltip_window_;
   }
 
   void FireTooltipTimer() {
     GetController()->TooltipTimerFired();
   }
 
+  bool IsTooltipTimerRunning() {
+    return GetController()->tooltip_timer_.IsRunning();
+  }
+
+  void FireTooltipShownTimer() {
+    GetController()->tooltip_shown_timer_.Stop();
+    GetController()->TooltipShownTimerFired();
+  }
+
+  bool IsTooltipShownTimerRunning() {
+    return GetController()->tooltip_shown_timer_.IsRunning();
+  }
+
   bool IsTooltipVisible() {
     return GetController()->IsTooltipVisible();
   }
 
   void TrimTooltipToFit(string16* text,
                         int* max_width,
                         int* line_count,
                         int x,
                         int y) {
     ash::internal::TooltipController::TrimTooltipToFit(text, max_width,
@@ skipping 228 matching lines @@
   // Whitespaces in tooltips are preserved.
   tooltip.clear();
   max_width = line_count = -1;
   tooltip = ASCIIToUTF16("Small  Tool  t\tip");
   TrimTooltipToFit(&tooltip, &max_width, &line_count, 0, 0);
   EXPECT_EQ(font.GetStringWidth(ASCIIToUTF16("Small  Tool  t\tip")), max_width);
   EXPECT_EQ(1, line_count);
   EXPECT_EQ(ASCIIToUTF16("Small  Tool  t\tip"), tooltip);
 }
 
+TEST_F(TooltipControllerTest, TooltipHidesOnKeyPressAndStaysHiddenUntilChange) {
+  scoped_ptr<views::Widget> widget(CreateNewWidget());
+  TooltipTestView* view1 = new TooltipTestView;
+  AddViewToWidgetAndResize(widget.get(), view1);
+  view1->set_tooltip_text(ASCIIToUTF16("Tooltip Text for view 1"));
+  EXPECT_EQ(string16(), GetTooltipText());
+  EXPECT_EQ(NULL, GetTooltipWindow());
+
+  TooltipTestView* view2 = new TooltipTestView;
+  AddViewToWidgetAndResize(widget.get(), view2);
+  view2->set_tooltip_text(ASCIIToUTF16("Tooltip Text for view 2"));
+
+  aura::Window* window = widget->GetNativeView();
+
+  // Fire tooltip timer so tooltip becomes visible.
+  aura::test::EventGenerator generator(Shell::GetPrimaryRootWindow());
+  generator.MoveMouseRelativeTo(window,
+                                view1->bounds().CenterPoint());
+  FireTooltipTimer();
+  EXPECT_TRUE(IsTooltipVisible());
+  EXPECT_TRUE(IsTooltipShownTimerRunning());
+
+  generator.PressKey(ui::VKEY_1, 0);
+  EXPECT_FALSE(IsTooltipVisible());
+  EXPECT_FALSE(IsTooltipTimerRunning());
+  EXPECT_FALSE(IsTooltipShownTimerRunning());
+
+  // Moving the mouse inside |view1| should not change the state of the tooltip
+  // or the timers.
+  for (int i = 0; i < 50; i++) {
+    generator.MoveMouseBy(1, 0);
+    EXPECT_FALSE(IsTooltipVisible());
+    EXPECT_FALSE(IsTooltipTimerRunning());
+    EXPECT_FALSE(IsTooltipShownTimerRunning());
+    EXPECT_EQ(window,
+        Shell::GetPrimaryRootWindow()->GetEventHandlerForPoint(
+            generator.current_location()));
+    string16 expected_tooltip = ASCIIToUTF16("Tooltip Text for view 1");
+    EXPECT_EQ(expected_tooltip, aura::client::GetTooltipText(window));
+    EXPECT_EQ(expected_tooltip, GetTooltipText());
+    EXPECT_EQ(window, GetTooltipWindow());
+  }
+
+  // Now we move the mouse on to |view2|. It should re-start the tooltip timer.
+  generator.MoveMouseBy(1, 0);
+  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, TooltipHidesOnTimeoutAndStaysHiddenUntilChange) {
+  scoped_ptr<views::Widget> widget(CreateNewWidget());
+  TooltipTestView* view1 = new TooltipTestView;
+  AddViewToWidgetAndResize(widget.get(), view1);
+  view1->set_tooltip_text(ASCIIToUTF16("Tooltip Text for view 1"));
+  EXPECT_EQ(string16(), GetTooltipText());
+  EXPECT_EQ(NULL, GetTooltipWindow());
+
+  TooltipTestView* view2 = new TooltipTestView;
+  AddViewToWidgetAndResize(widget.get(), view2);
+  view2->set_tooltip_text(ASCIIToUTF16("Tooltip Text for view 2"));
+
+  aura::Window* window = widget->GetNativeView();
+
+  // Fire tooltip timer so tooltip becomes visible.
+  aura::test::EventGenerator generator(Shell::GetPrimaryRootWindow());
+  generator.MoveMouseRelativeTo(window,
+                                view1->bounds().CenterPoint());
+  FireTooltipTimer();
+  EXPECT_TRUE(IsTooltipVisible());
+  EXPECT_TRUE(IsTooltipShownTimerRunning());
+
+  FireTooltipShownTimer();
+  EXPECT_FALSE(IsTooltipVisible());
+  EXPECT_FALSE(IsTooltipTimerRunning());
+  EXPECT_FALSE(IsTooltipShownTimerRunning());
+
+  // Moving the mouse inside |view1| should not change the state of the tooltip
+  // or the timers.
+  for (int i = 0; i < 50; i++) {
+    generator.MoveMouseBy(1, 0);
+    EXPECT_FALSE(IsTooltipVisible());
+    EXPECT_FALSE(IsTooltipTimerRunning());
+    EXPECT_FALSE(IsTooltipShownTimerRunning());
+    EXPECT_EQ(window,
+        Shell::GetPrimaryRootWindow()->GetEventHandlerForPoint(
+            generator.current_location()));
+    string16 expected_tooltip = ASCIIToUTF16("Tooltip Text for view 1");
+    EXPECT_EQ(expected_tooltip, aura::client::GetTooltipText(window));
+    EXPECT_EQ(expected_tooltip, GetTooltipText());
+    EXPECT_EQ(window, GetTooltipWindow());
+  }
+
+  // Now we move the mouse on to |view2|. It should re-start the tooltip timer.
+  generator.MoveMouseBy(1, 0);
+  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());
+}
+
 }  // namespace test
 }  // namespace ash