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

Unified Diff: views/view.cc

Issue 8362006: Reland r107720 - Enable the new layer animation framework. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge with master. Created 9 years, 2 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
Index: views/view.cc
diff --git a/views/view.cc b/views/view.cc
index 1464067e85cf588d167126474daf9dc34be9256d..5b9292fb4eb75156a7d4883d7bb36aff7df6386e 100644
--- a/views/view.cc
+++ b/views/view.cc
@@ -18,6 +18,7 @@
#include "ui/gfx/canvas_skia.h"
#include "ui/gfx/compositor/compositor.h"
#include "ui/gfx/compositor/layer.h"
+#include "ui/gfx/compositor/layer_animator.h"
#include "ui/gfx/interpolated_transform.h"
#include "ui/gfx/path.h"
#include "ui/gfx/point3.h"
@@ -25,7 +26,6 @@
#include "views/background.h"
#include "views/context_menu_controller.h"
#include "views/drag_controller.h"
-#include "views/layer_property_setter.h"
#include "views/layout/layout_manager.h"
#include "views/views_delegate.h"
#include "views/widget/native_widget_private.h"
@@ -424,7 +424,7 @@ const ui::Transform& View::GetTransform() const {
void View::SetTransform(const ui::Transform& transform) {
if (!transform.HasChange()) {
if (layer()) {
- layer_property_setter_->SetTransform(layer(), transform);
+ layer()->SetTransform(transform);
if (!paint_to_layer_)
DestroyLayer();
} else {
@@ -433,7 +433,7 @@ void View::SetTransform(const ui::Transform& transform) {
} else {
if (!layer())
CreateLayer();
- layer_property_setter_->SetTransform(layer(), transform);
+ layer()->SetTransform(transform);
layer()->ScheduleDraw();
}
}
@@ -447,20 +447,6 @@ void View::SetPaintToLayer(bool paint_to_layer) {
}
}
-void View::SetLayerPropertySetter(LayerPropertySetter* setter) {
- DCHECK(layer());
- LayerPropertySetter* old_setter = layer_property_setter_.get();
- if (!layer() || (old_setter && old_setter == setter))
- return;
- if (!setter)
- setter = LayerPropertySetter::CreateDefaultSetter();
-
- if (old_setter)
- old_setter->Uninstalled(layer());
- layer_property_setter_.reset(setter);
- layer_property_setter_->Installed(layer());
-}
-
// RTL positioning -------------------------------------------------------------
gfx::Rect View::GetMirroredBounds() const {
@@ -1155,8 +1141,7 @@ void View::UpdateChildLayerVisibility(bool ancestor_visible) {
void View::UpdateChildLayerBounds(const gfx::Point& offset) {
if (layer()) {
- layer_property_setter_->SetBounds(layer(), gfx::Rect(offset.x(), offset.y(),
- width(), height()));
+ layer()->SetBounds(gfx::Rect(offset.x(), offset.y(), width(), height()));
} else {
for (int i = 0, count = child_count(); i < count; ++i) {
gfx::Point new_offset(offset.x() + child_at(i)->x(),
@@ -1171,7 +1156,7 @@ void View::OnPaintLayer(gfx::Canvas* canvas) {
PaintCommon(canvas);
}
-void View::OnLayerAnimationEnded(const ui::Animation* animation) {
+void View::OnLayerAnimationEnded(const ui::LayerAnimationSequence* animation) {
}
void View::ReorderLayers() {
@@ -1596,9 +1581,9 @@ void View::BoundsChanged(const gfx::Rect& previous_bounds) {
gfx::Point offset;
parent_->CalculateOffsetToAncestorWithLayer(&offset, NULL);
offset.Offset(x(), y());
- layer_property_setter_->SetBounds(layer(), gfx::Rect(offset, size()));
+ layer()->SetBounds(gfx::Rect(offset, size()));
} else {
- layer_property_setter_->SetBounds(layer(), bounds_);
+ layer()->SetBounds(bounds_);
}
// TODO(beng): this seems redundant with the SchedulePaint at the top of
// this function. explore collapsing.
@@ -1782,10 +1767,6 @@ void View::CreateLayer() {
layer_.reset(new ui::Layer(NULL));
layer_->set_delegate(this);
- if (layer_property_setter_.get())
- layer_property_setter_->Installed(layer());
- else
- SetLayerPropertySetter(NULL);
UpdateParentLayers();
UpdateLayerVisibility();
@@ -1856,9 +1837,6 @@ void View::DestroyLayer() {
new_parent->Add(children[i]);
}
- if (layer_property_setter_.get())
- layer_property_setter_->Uninstalled(layer());
-
layer_.reset();
if (new_parent)
« ui/gfx/compositor/layer_animator.cc ('K') | « views/view.h ('k') | views/view_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698