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

Unified Diff: cc/layer_tree_host.h

Issue 11491003: Revert 171714 - Use an auxiliary list of animation controllers. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 8 years 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 | « cc/layer_impl.cc ('k') | cc/layer_tree_host.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/layer_tree_host.h
diff --git a/cc/layer_tree_host.h b/cc/layer_tree_host.h
index 3f6b9bae737cb26c289e6b574ef6aa920ad4f438..7e600bd67e8f62c7b75b00ae4d822185c1b9c960 100644
--- a/cc/layer_tree_host.h
+++ b/cc/layer_tree_host.h
@@ -14,7 +14,6 @@
#include "base/memory/scoped_ptr.h"
#include "base/time.h"
#include "cc/animation_events.h"
-#include "cc/animation_registrar.h"
#include "cc/cc_export.h"
#include "cc/layer_tree_host_client.h"
#include "cc/layer_tree_host_common.h"
@@ -37,15 +36,6 @@ struct hash<WebKit::WebGraphicsContext3D*> {
}
};
} // namespace BASE_HASH_NAMESPACE
-
-namespace BASE_HASH_NAMESPACE {
-template<>
-struct hash<cc::LayerAnimationController*> {
- size_t operator()(cc::LayerAnimationController* ptr) const {
- return hash<size_t>()(reinterpret_cast<size_t>(ptr));
- }
-};
-} // namespace BASE_HASH_NAMESPACE
#endif // COMPILER
namespace cc {
@@ -123,8 +113,7 @@ struct CC_EXPORT RendererCapabilities {
int maxTextureSize;
};
-class CC_EXPORT LayerTreeHost : public RateLimiterClient,
- public AnimationRegistrar {
+class CC_EXPORT LayerTreeHost : public RateLimiterClient {
public:
static scoped_ptr<LayerTreeHost> create(LayerTreeHostClient*, const LayerTreeSettings&, scoped_ptr<Thread> implThread);
virtual ~LayerTreeHost();
@@ -201,6 +190,7 @@ public:
bool commitRequested() const;
void setAnimationEvents(scoped_ptr<AnimationEventsVector>, base::Time wallClockTime);
+ virtual void didAddAnimation();
Layer* rootLayer() { return m_rootLayer.get(); }
const Layer* rootLayer() const { return m_rootLayer.get(); }
@@ -258,15 +248,8 @@ protected:
bool initialize(scoped_ptr<Thread> implThread);
bool initializeForTesting(scoped_ptr<Proxy> proxyForTesting);
- // AnimationRegistar implementation.
- virtual void DidActivateAnimationController(LayerAnimationController*) OVERRIDE;
- virtual void DidDeactivateAnimationController(LayerAnimationController*) OVERRIDE;
- virtual void RegisterAnimationController(LayerAnimationController*) OVERRIDE;
- virtual void UnregisterAnimationController(LayerAnimationController*) OVERRIDE;
-
private:
typedef std::vector<scoped_refptr<Layer> > LayerList;
- typedef base::hash_set<LayerAnimationController*> AnimationControllerSet;
bool initializeProxy(scoped_ptr<Proxy> proxy);
void initializeRenderer();
@@ -288,6 +271,7 @@ private:
void setAnimationEventsRecursive(const AnimationEventsVector&, Layer*, base::Time wallClockTime);
bool m_animating;
+ bool m_needsAnimateLayers;
bool m_needsFullTreeSync;
base::CancelableClosure m_prepaintCallback;
@@ -334,12 +318,6 @@ private:
static bool s_needsFilterContext;
- AnimationControllerSet m_activeAnimationControllers;
-
-#if !defined(NDEBUG)
- AnimationControllerSet m_allAnimationControllers;
-#endif
-
DISALLOW_COPY_AND_ASSIGN(LayerTreeHost);
};
« no previous file with comments | « cc/layer_impl.cc ('k') | cc/layer_tree_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698