Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(379)

Unified Diff: Source/core/animation/ActiveAnimations.h

Issue 225073004: Oilpan: Completely move core/animations/ to oilpan's heap (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | Source/core/animation/ActiveAnimations.cpp » ('j') | Source/core/animation/Animation.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/animation/ActiveAnimations.h
diff --git a/Source/core/animation/ActiveAnimations.h b/Source/core/animation/ActiveAnimations.h
index 1a4c0a0bb119a4b2f93e11c8cf203ba90549652f..8f41b050fcb87b6856b98d098b0238de901d9f80 100644
--- a/Source/core/animation/ActiveAnimations.h
+++ b/Source/core/animation/ActiveAnimations.h
@@ -64,7 +64,7 @@ public:
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
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*);
« no previous file with comments | « no previous file | Source/core/animation/ActiveAnimations.cpp » ('j') | Source/core/animation/Animation.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698