Aura tooltips do not move on mouse move in case of many neighboring views with the same label

ui/views/corewm/tooltip_controller_unittest.cc

 // Copyright (c) 2013 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 "ui/views/corewm/tooltip_controller.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "ui/aura/client/cursor_client.h"
 #include "ui/aura/client/screen_position_client.h"
 #include "ui/aura/env.h"
@@ skipping 373 matching lines @@
   gfx::Point v2_point(1, 1);
   View::ConvertPointToWidget(v2, &v2_point);
   generator_->MoveMouseRelativeTo(GetWindow(), v2_point);
   generator_->MoveMouseRelativeTo(GetWindow(), v1_point);
 
   helper_->FireTooltipTimer();
   EXPECT_TRUE(helper_->IsTooltipVisible());
   EXPECT_EQ(v1_tt, helper_->GetTooltipText());
 }
 
+TEST_F(TooltipControllerTest, TooltipPositionChangesOnTwoViewsWithSameLabel) {
+  // Owned by |view_|.
+  TooltipTestView* v1 = new TooltipTestView;
+  TooltipTestView* v2 = new TooltipTestView;
+  view_->AddChildView(v1);
+  view_->AddChildView(v2);
+  const base::string16 reference_string(base::ASCIIToUTF16(
+      "Identical Tooltip Text"));
+  v1->set_tooltip_text(reference_string);
+  v2->set_tooltip_text(reference_string);
+
+  gfx::Rect view_bounds(view_->GetLocalBounds());
+  view_bounds.set_height(view_bounds.height() / 2);
+  v1->SetBoundsRect(view_bounds);
+  view_bounds.set_y(view_bounds.height());
+  v2->SetBoundsRect(view_bounds);
+
+  gfx::Point center = v1->bounds().CenterPoint();
+  generator_->MoveMouseRelativeTo(GetWindow(), center);
+  helper_->FireTooltipTimer();
+  EXPECT_TRUE(helper_->IsTooltipVisible());
+  EXPECT_EQ(reference_string, helper_->GetTooltipText());
+  gfx::Point tooltip_bounds1 = helper_->GetTooltipPosition();
+
+  center = v2->bounds().CenterPoint();
+  generator_->MoveMouseRelativeTo(GetWindow(), center);
+  helper_->FireTooltipTimer();
+  EXPECT_TRUE(helper_->IsTooltipVisible());
+  EXPECT_EQ(reference_string, helper_->GetTooltipText());
+  gfx::Point tooltip_bounds2 = helper_->GetTooltipPosition();
+
+  EXPECT_NE(tooltip_bounds1, gfx::Point());
+  EXPECT_NE(tooltip_bounds2, gfx::Point());
+  EXPECT_NE(tooltip_bounds1, tooltip_bounds2);
+}
+
 namespace {
 
 // Returns the index of |window| in its parent's children.
 int IndexInParent(const aura::Window* window) {
   aura::Window::Windows::const_iterator i =
       std::find(window->parent()->children().begin(),
                 window->parent()->children().end(),
                 window);
   return i == window->parent()->children().end() ? -1 :
       static_cast<int>(i - window->parent()->children().begin());
@@ skipping 268 matching lines @@
   }
   virtual void Show() OVERRIDE {
     is_visible_ = true;
   }
   virtual void Hide() OVERRIDE {
     is_visible_ = false;
   }
   virtual bool IsVisible() OVERRIDE {
     return is_visible_;
   }
+  virtual gfx::Point GetTooltipPosition() OVERRIDE {
+    return gfx::Point();
+  }
 
  private:
   bool is_visible_;
   base::string16 tooltip_text_;
 
   DISALLOW_COPY_AND_ASSIGN(TestTooltip);
 };
 
 }  // namespace
 
@@ skipping 67 matching lines @@
   // the tooltip window is closed.
   ui::CancelModeEvent event;
   helper_->controller()->OnCancelMode(&event);
   EXPECT_FALSE(helper_->IsTooltipVisible());
   EXPECT_TRUE(helper_->GetTooltipWindow() == NULL);
 }
 
 }  // namespace test
 }  // namespace corewm
 }  // namespace views