Animate the script badges.

chrome/browser/extensions/script_badge_controller.h

 // 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_SCRIPT_BADGE_CONTROLLER_H_
 #define CHROME_BROWSER_EXTENSIONS_SCRIPT_BADGE_CONTROLLER_H_
 #pragma once
 
 #include <map>
 #include <set>
@@ skipping 38 matching lines @@
 class ScriptBadgeController
     : public base::RefCountedThreadSafe<ScriptBadgeController>,
       public LocationBarController,
       public ScriptExecutor,
       public content::WebContentsObserver,
       public content::NotificationObserver {
  public:
   explicit ScriptBadgeController(TabContents* tab_contents);
 
   // LocationBarController implementation.
-  virtual std::vector<ExtensionAction*> GetCurrentActions() OVERRIDE;
+  virtual std::vector<ExtensionAction*> GetCurrentActions() const OVERRIDE;
+  virtual base::WeakPtr<IconAnimation> GetIconAnimation(
+      const ExtensionAction* action) const OVERRIDE;
   virtual Action OnClicked(const std::string& extension_id,
                            int mouse_button) OVERRIDE;
 
   // ScriptExecutor implementation.
   virtual void ExecuteScript(const std::string& extension_id,
                              ScriptType script_type,
                              const std::string& code,
                              FrameScope frame_scope,
                              UserScript::RunLocation run_at,
                              WorldType world_type,
@@ skipping 48 matching lines @@
 
   // Our parent TabContents.
   TabContents* tab_contents_;
 
   // The current extension actions in the order they appeared.
   std::vector<ExtensionAction*> current_actions_;
 
   // The extensions that have called ExecuteScript on the current frame.
   std::set<std::string> extensions_executing_scripts_;
 
+  // Active animations for each ExtensionAction (keyed by extension ID).

[Yoyo Zhou, 2012/06/22 00:50:29]

Have you considered storing these on the ExtensionAction instead? It seems like
the lifetime of the two should be closely related.

[not at google - send to devlin, 2012/06/26 07:30:47]

Done. Ended up being simpler to put it on ExtensionAction.

+  class IconAnimationWrapper;
+  typedef std::map<std::string, linked_ptr<IconAnimationWrapper> >
+      IconAnimationMap;
+  IconAnimationMap icon_animations_;
+
   // Listen to extension unloaded notifications.
   content::NotificationRegistrar registrar_;
 
   DISALLOW_COPY_AND_ASSIGN(ScriptBadgeController);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_SCRIPT_BADGE_CONTROLLER_H_