Merge 157903 - Turn off animations for page actions and browser actions.

chrome/common/extensions/extension_action_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 "base/message_loop.h"
 #include "chrome/common/extensions/extension_action.h"
 #include "googleurl/src/gurl.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
@@ skipping 15 matching lines @@
   action.SetTitle(100, "bar");
   ASSERT_EQ("foo", action.GetTitle(1));
   ASSERT_EQ("bar", action.GetTitle(100));
   action.SetTitle(ExtensionAction::kDefaultTabId, "baz");
   ASSERT_EQ("baz", action.GetTitle(1));
   action.ClearAllValuesForTab(100);
   ASSERT_EQ("baz", action.GetTitle(100));
 }
 
 TEST_F(ExtensionActionTest, Visibility) {
-  // Supports the icon animation.
-  MessageLoop message_loop;
-
   ASSERT_FALSE(action.GetIsVisible(1));
-  EXPECT_FALSE(action.GetIconAnimation(ExtensionAction::kDefaultTabId));
   action.SetAppearance(ExtensionAction::kDefaultTabId, ExtensionAction::ACTIVE);
   ASSERT_TRUE(action.GetIsVisible(1));
   ASSERT_TRUE(action.GetIsVisible(100));
-  EXPECT_FALSE(action.GetIconAnimation(ExtensionAction::kDefaultTabId));
 
   action.SetAppearance(ExtensionAction::kDefaultTabId,
                        ExtensionAction::INVISIBLE);
   ASSERT_FALSE(action.GetIsVisible(1));
   ASSERT_FALSE(action.GetIsVisible(100));
-  EXPECT_FALSE(action.GetIconAnimation(100));
   action.SetAppearance(100, ExtensionAction::ACTIVE);
   ASSERT_FALSE(action.GetIsVisible(1));
   ASSERT_TRUE(action.GetIsVisible(100));
-  EXPECT_TRUE(action.GetIconAnimation(100));
+  EXPECT_FALSE(action.GetIconAnimation(100))
+      << "Page actions should not animate.";
 
   action.ClearAllValuesForTab(100);
   ASSERT_FALSE(action.GetIsVisible(1));
   ASSERT_FALSE(action.GetIsVisible(100));
-  EXPECT_FALSE(action.GetIconAnimation(100));
+}
+
+TEST_F(ExtensionActionTest, ScriptBadgeAnimation) {
+  // Supports the icon animation.
+  MessageLoop message_loop;
+
+  ExtensionAction script_badge("", ExtensionAction::TYPE_SCRIPT_BADGE);
+  EXPECT_FALSE(script_badge.GetIconAnimation(ExtensionAction::kDefaultTabId));
+  script_badge.SetAppearance(ExtensionAction::kDefaultTabId,
+                             ExtensionAction::ACTIVE);
+  EXPECT_FALSE(script_badge.GetIconAnimation(ExtensionAction::kDefaultTabId))
+      << "Showing the default tab should not animate script badges.";
+
+  script_badge.SetAppearance(ExtensionAction::kDefaultTabId,
+                             ExtensionAction::INVISIBLE);
+  EXPECT_FALSE(script_badge.GetIconAnimation(1))
+      << "Making a script badge invisible should not show its animation.";
+  script_badge.SetAppearance(1, ExtensionAction::ACTIVE);
+  EXPECT_TRUE(script_badge.GetIconAnimation(1))
+      << "Making a script badge visible should show its animation.";
+
+  script_badge.ClearAllValuesForTab(1);
+  EXPECT_FALSE(script_badge.GetIconAnimation(100));
 }
 
 TEST_F(ExtensionActionTest, GetAttention) {
   // Supports the icon animation.
   scoped_ptr<MessageLoop> message_loop(new MessageLoop);
 
-  EXPECT_FALSE(action.GetIsVisible(1));
-  EXPECT_FALSE(action.GetIconAnimation(1));
-  action.SetAppearance(1, ExtensionAction::WANTS_ATTENTION);
-  EXPECT_TRUE(action.GetIsVisible(1));
-  EXPECT_TRUE(action.GetIconAnimation(1));
+  ExtensionAction script_badge("", ExtensionAction::TYPE_SCRIPT_BADGE);
+  EXPECT_FALSE(script_badge.GetIsVisible(1));
+  EXPECT_FALSE(script_badge.GetIconAnimation(1));
+  script_badge.SetAppearance(1, ExtensionAction::WANTS_ATTENTION);
+  EXPECT_TRUE(script_badge.GetIsVisible(1));
+  EXPECT_TRUE(script_badge.GetIconAnimation(1));
 
   // Simulate waiting long enough for the animation to end.
   message_loop.reset();  // Can't have 2 MessageLoops alive at once.
   message_loop.reset(new MessageLoop);
-  EXPECT_FALSE(action.GetIconAnimation(1));  // Sanity check.
+  EXPECT_FALSE(script_badge.GetIconAnimation(1));  // Sanity check.
 
-  action.SetAppearance(1, ExtensionAction::ACTIVE);
-  EXPECT_FALSE(action.GetIconAnimation(1))
+  script_badge.SetAppearance(1, ExtensionAction::ACTIVE);
+  EXPECT_FALSE(script_badge.GetIconAnimation(1))
       << "The animation should not play again if the icon was already visible.";
 }
 
 TEST_F(ExtensionActionTest, Badge) {
   ASSERT_EQ("", action.GetBadgeText(1));
   action.SetBadgeText(ExtensionAction::kDefaultTabId, "foo");
   ASSERT_EQ("foo", action.GetBadgeText(1));
   ASSERT_EQ("foo", action.GetBadgeText(100));
   action.SetBadgeText(100, "bar");
   ASSERT_EQ("foo", action.GetBadgeText(1));
@@ skipping 56 matching lines @@
   action.SetPopupUrl(ExtensionAction::kDefaultTabId, url_baz);
   ASSERT_EQ(url_baz, action.GetPopupUrl(1));
   ASSERT_EQ(url_bar, action.GetPopupUrl(100));
 
   action.ClearAllValuesForTab(100);
   ASSERT_EQ(url_baz, action.GetPopupUrl(1));
   ASSERT_EQ(url_baz, action.GetPopupUrl(100));
 }
 
 }  // namespace