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

Unified Diff: ui/compositor/layer.h

Issue 1749573002: CC Animation: Erase old animation system in UI Compositor. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@eraseblink
Patch Set: Exclude histograms.xml Created 4 years, 10 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 | « ui/compositor/compositor_switches.cc ('k') | ui/compositor/layer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/compositor/layer.h
diff --git a/ui/compositor/layer.h b/ui/compositor/layer.h
index 75567ffe48b79755c8dca1e0db81f6ca6951d48b..2dcb9c765be1bdd098bcb1d43b0098988428ac9f 100644
--- a/ui/compositor/layer.h
+++ b/ui/compositor/layer.h
@@ -27,7 +27,6 @@
#include "ui/compositor/compositor.h"
#include "ui/compositor/layer_animation_delegate.h"
#include "ui/compositor/layer_delegate.h"
-#include "ui/compositor/layer_threaded_animation_delegate.h"
#include "ui/compositor/layer_type.h"
#include "ui/gfx/geometry/rect.h"
#include "ui/gfx/image/image_skia.h"
@@ -53,6 +52,7 @@ namespace ui {
class Compositor;
class LayerAnimator;
class LayerOwner;
+class LayerThreadedAnimationDelegate;
// Layer manages a texture, transform and a set of child Layers. Any View that
// has enabled layers ends up creating a Layer to manage the texture.
@@ -66,7 +66,6 @@ class LayerOwner;
// NULL, but the children are not deleted.
class COMPOSITOR_EXPORT Layer
: public LayerAnimationDelegate,
- public LayerThreadedAnimationDelegate,
NON_EXPORTED_BASE(public cc::ContentLayerClient),
NON_EXPORTED_BASE(public cc::TextureLayerClient),
NON_EXPORTED_BASE(public cc::LayerClient) {
@@ -75,9 +74,6 @@ class COMPOSITOR_EXPORT Layer
explicit Layer(LayerType type);
~Layer() override;
- static const cc::LayerSettings& UILayerSettings();
- static void InitializeUILayerSettings();
-
// Retrieves the Layer's compositor. The Layer will walk up its parent chain
// to locate it. Returns NULL if the Layer is not attached to a compositor.
Compositor* GetCompositor() {
@@ -406,10 +402,6 @@ class COMPOSITOR_EXPORT Layer
LayerThreadedAnimationDelegate* GetThreadedAnimationDelegate() override;
LayerAnimatorCollection* GetLayerAnimatorCollection() override;
- // Implementation of LayerThreadedAnimationDelegate.
- void AddThreadedAnimation(scoped_ptr<cc::Animation> animation) override;
- void RemoveThreadedAnimation(int animation_id) override;
-
// Creates a corresponding composited layer for |type_|.
void CreateCcLayer();
@@ -426,12 +418,6 @@ class COMPOSITOR_EXPORT Layer
// Cleanup |cc_layer_| and replaces it with |new_layer|.
void SwitchToLayer(scoped_refptr<cc::Layer> new_layer);
- // We cannot send animations to our cc_layer_ until we have been added to a
- // layer tree. Instead, we hold on to these animations in
- // pending_threaded_animations_, and expect SendPendingThreadedAnimations to
- // be called once we have been added to a tree.
- void SendPendingThreadedAnimations();
-
void SetCompositorForAnimatorsInTree(Compositor* compositor);
void ResetCompositorForAnimatorsInTree(Compositor* compositor);
@@ -493,10 +479,6 @@ class COMPOSITOR_EXPORT Layer
scoped_refptr<LayerAnimator> animator_;
- // Animations that are passed to AddThreadedAnimation before this layer is
- // added to a tree.
- std::vector<scoped_ptr<cc::Animation>> pending_threaded_animations_;
-
// Ownership of the layer is held through one of the strongly typed layer
// pointers, depending on which sort of layer this is.
scoped_refptr<cc::Layer> content_layer_;
« no previous file with comments | « ui/compositor/compositor_switches.cc ('k') | ui/compositor/layer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698