Oilpan: Completely move core/animations/ to oilpan's heap

Source/core/animation/ActiveAnimations.h

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 46 matching lines @@
 
     // Animations that are currently active for this element, their effects will be applied
     // during a style recalc. CSS Transitions are included in this stack.
     AnimationStack& defaultStack() { return m_defaultStack; }
     // Tracks the state of active CSS Animations and Transitions. The individual animations
     // will also be part of the default stack, but the mapping betwen animation name and
     // player is kept here.
     CSSAnimations& cssAnimations() { return m_cssAnimations; }
     const CSSAnimations& cssAnimations() const { return m_cssAnimations; }
 
-    typedef HashMap<AnimationPlayer*, int> AnimationPlayerCountedSet;
+    typedef WillBeHeapHashMap<RawPtrWillBeMember<AnimationPlayer>, int> AnimationPlayerCountedSet;

[Mads Ager (chromium), 2014/04/29 10:29:41]

Maybe rename the type to AnimationPlayerSet since it is no longer counted.

[Timothy Loh, 2014/05/02 05:46:00]

This should probably be a weak member.

[haraken, 2014/05/04 15:31:47]

Agreed, this should be a weak member.

     // AnimationPlayers which have animations targeting this element.
     const AnimationPlayerCountedSet& players() const { return m_players; }
     void addPlayer(AnimationPlayer*);
     void removePlayer(AnimationPlayer*);
 
     bool isEmpty() const { return m_defaultStack.isEmpty() && m_cssAnimations.isEmpty() && m_animations.isEmpty(); }
 
     void cancelAnimationOnCompositor();
 
     void updateAnimationFlags(RenderStyle&);
@@ skipping 16 matching lines @@
     // won't be needed once Element and Animation are moved to Oilpan.
     Vector<Animation*> m_animations;
 
     // CSSAnimations checks if a style change is due to animation.
     friend class CSSAnimations;
 };
 
 } // namespace WebCore
 
 #endif