Show the Cast toolbar icon ephemerally when Cast is in use

chrome/browser/ui/toolbar/toolbar_actions_model_unittest.cc

Index: chrome/browser/ui/toolbar/toolbar_actions_model_unittest.cc
diff --git a/chrome/browser/ui/toolbar/toolbar_actions_model_unittest.cc b/chrome/browser/ui/toolbar/toolbar_actions_model_unittest.cc
index a4bbd98b39ed1fe0cd82031bee7b6399578d1b0e..027e73ae7e52e210e75021d9e18d4408aa77ed06 100644
--- a/chrome/browser/ui/toolbar/toolbar_actions_model_unittest.cc
+++ b/chrome/browser/ui/toolbar/toolbar_actions_model_unittest.cc
@@ -1308,7 +1308,7 @@ TEST_F(ToolbarActionsModelUnitTest, ToolbarModelPrefChange) {
             observer()->inserted_count() - observer()->removed_count());
 }
 
-TEST_F(ToolbarActionsModelUnitTest, ComponentExtensionsAddedToEnd) {
+TEST_F(ToolbarActionsModelUnitTest, ComponentExtensionsAddedToRight) {

[Devlin, 2016/07/26 21:59:03]

If we do remove this functionality, isn't this testing the same thing as
NewExtensionsAreVisible?

[Devlin, 2016/07/29 19:10:22]

ping

[takumif, 2016/08/02 04:58:41]

Removing the test.

   Init();
 
   ASSERT_TRUE(AddBrowserActionExtensions());
@@ -1332,10 +1332,10 @@ TEST_F(ToolbarActionsModelUnitTest, ComponentExtensionsAddedToEnd) {
           .Build();
   service()->AddExtension(component_extension.get());
 
-  EXPECT_EQ(component_extension.get()->id(), GetActionIdAtIndex(0));
-  EXPECT_EQ(browser_action_a()->id(), GetActionIdAtIndex(1));
-  EXPECT_EQ(browser_action_b()->id(), GetActionIdAtIndex(2));
-  EXPECT_EQ(browser_action_c()->id(), GetActionIdAtIndex(3));
+  EXPECT_EQ(browser_action_a()->id(), GetActionIdAtIndex(0));
+  EXPECT_EQ(browser_action_b()->id(), GetActionIdAtIndex(1));
+  EXPECT_EQ(browser_action_c()->id(), GetActionIdAtIndex(2));
+  EXPECT_EQ(component_extension.get()->id(), GetActionIdAtIndex(3));
 }
 
 // Test various different reorderings, removals, and reinsertions of the