Makes the pinned tab title change indicator hide correctly

chrome/browser/ui/views/tabs/tab_strip_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 "chrome/browser/ui/views/tabs/tab_strip.h"
 
 #include "base/macros.h"
 #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"
 #include "chrome/test/base/testing_profile.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/base/material_design/material_design_controller.h"
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/geometry/rect_conversions.h"
 #include "ui/gfx/path.h"
 #include "ui/gfx/skia_util.h"
@@ skipping 99 matching lines @@
     widget_->Init(init_params);
     widget_->SetContentsView(&parent_);
   }
 
   void TearDown() override {
     widget_.reset();
     views::ViewsTestBase::TearDown();
   }
 
  protected:
+  bool IsShowingPinnedTabTitleChangedIndicator(int model_index) {
+    return tab_strip_->tab_at(model_index)
+        ->showing_pinned_tab_title_changed_indicator_;
+  }
+
   // Returns the rectangular hit test region of |tab| in |tab|'s local
   // coordinate space.
   gfx::Rect GetTabHitTestMask(Tab* tab) {
     views::ViewTargeter* targeter = tab->targeter();
     DCHECK(targeter);
     views::MaskedTargeterDelegate* delegate =
         static_cast<views::MaskedTargeterDelegate*>(tab);
 
     gfx::Path mask;
     bool valid_mask = delegate->GetHitTestMask(&mask);
@@ skipping 488 matching lines @@
   const int kTabStripWidth = 500;
   tab_strip_->SetBounds(0, 0, kTabStripWidth, 20);
 
   for (int i = 0; i < 100; ++i)
     controller_->AddTab(i, (i == 0));
 
   DoLayout();
 
   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(IsShowingPinnedTabTitleChangedIndicator(0));
+  EXPECT_FALSE(IsShowingPinnedTabTitleChangedIndicator(1));
+
+  // Change the title of the second tab (first tab is selected).
+  tab_strip_->TabTitleChangedNotLoading(1);
+  // Indicator should be shown.
+  EXPECT_TRUE(IsShowingPinnedTabTitleChangedIndicator(1));
+  // Select the second tab.
+  controller_->SelectTab(1);
+  // Indicator should hide.
+  EXPECT_FALSE(IsShowingPinnedTabTitleChangedIndicator(1));
+}