Makes the pinned tab title change indicator hide correctly

chrome/browser/ui/views/tabs/tab_strip_unittest.cc

Index: chrome/browser/ui/views/tabs/tab_strip_unittest.cc
diff --git a/chrome/browser/ui/views/tabs/tab_strip_unittest.cc b/chrome/browser/ui/views/tabs/tab_strip_unittest.cc
index 8ab4a56f9da325ca789e7396b9d8cb1232b5ad46..ae77f065e22051edc969e29431cd681c9383b2b3 100644
--- a/chrome/browser/ui/views/tabs/tab_strip_unittest.cc
+++ b/chrome/browser/ui/views/tabs/tab_strip_unittest.cc
@@ -8,6 +8,7 @@
 #include "base/message_loop/message_loop.h"
 #include "chrome/browser/ui/views/tabs/fake_base_tab_strip_controller.h"
 #include "chrome/browser/ui/views/tabs/tab.h"
+#include "chrome/browser/ui/views/tabs/tab_renderer_data.h"
 #include "chrome/browser/ui/views/tabs/tab_strip.h"
 #include "chrome/browser/ui/views/tabs/tab_strip_controller.h"
 #include "chrome/browser/ui/views/tabs/tab_strip_observer.h"
@@ -635,3 +636,30 @@ TEST_F(TabStripTest, NewTabButtonStaysVisible) {
 
   EXPECT_LE(tab_strip_->GetNewTabButtonBounds().right(), kTabStripWidth);
 }
+
+TEST_F(TabStripTest, PinnedTabTitleChangedIndicatorHidesOnSelect) {
+  for (int i = 0; i < 2; ++i)
+    controller_->AddTab(i, (i == 0));
+
+  // Two tabs, both pinned.
+  TabRendererData pinned_data;
+  pinned_data.pinned = true;
+  tab_strip_->SetTabData(0, pinned_data);
+  tab_strip_->SetTabData(1, pinned_data);
+
+  EXPECT_FALSE(
+      tab_strip_->tab_at(0)->showing_pinned_tab_title_changed_indicator());
+  EXPECT_FALSE(
+      tab_strip_->tab_at(1)->showing_pinned_tab_title_changed_indicator());
+
+  // Change the title of the second tab (first tab is selected).
+  tab_strip_->TabTitleChangedNotLoading(1);
+  // Indicator should be shown.
+  EXPECT_TRUE(
+      tab_strip_->tab_at(1)->showing_pinned_tab_title_changed_indicator());
+  // Select the second tab.
+  controller_->SelectTab(1);
+  // Indicator should hide.
+  EXPECT_FALSE(
+      tab_strip_->tab_at(1)->showing_pinned_tab_title_changed_indicator());
+}