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

Unified Diff: ui/compositor/layer_animation_element_unittest.cc

Issue 11896017: Thread ui opacity animations (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Correctly deal with sequences meant to start together Created 7 years, 11 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: ui/compositor/layer_animation_element_unittest.cc
diff --git a/ui/compositor/layer_animation_element_unittest.cc b/ui/compositor/layer_animation_element_unittest.cc
index 5912023739a61de922c7fb819bcd73d90bb98e1a..6ca1338974abe5f958f2dd75b9dcdecc0af34b12 100644
--- a/ui/compositor/layer_animation_element_unittest.cc
+++ b/ui/compositor/layer_animation_element_unittest.cc
@@ -34,7 +34,7 @@ TEST(LayerAnimationElementTest, TransformElement) {
for (int i = 0; i < 2; ++i) {
start_time += delta;
- element->set_start_time(start_time);
+ element->set_requested_start_time(start_time);
delegate.SetTransformFromAnimation(start_transform);
element->Progress(start_time, &delegate);
CheckApproximatelyEqual(start_transform,
@@ -72,7 +72,7 @@ TEST(LayerAnimationElementTest, BoundsElement) {
for (int i = 0; i < 2; ++i) {
start_time += delta;
- element->set_start_time(start_time);
+ element->set_requested_start_time(start_time);
delegate.SetBoundsFromAnimation(start);
element->Progress(start_time, &delegate);
CheckApproximatelyEqual(start, delegate.GetBoundsForAnimation());
@@ -105,7 +105,7 @@ TEST(LayerAnimationElementTest, OpacityElement) {
for (int i = 0; i < 2; ++i) {
start_time += delta;
- element->set_start_time(start_time);
+ element->set_requested_start_time(start_time);
delegate.SetOpacityFromAnimation(start);
element->Progress(start_time, &delegate);
EXPECT_FLOAT_EQ(start, delegate.GetOpacityForAnimation());
@@ -137,7 +137,7 @@ TEST(LayerAnimationElementTest, VisibilityElement) {
for (int i = 0; i < 2; ++i) {
start_time += delta;
- element->set_start_time(start_time);
+ element->set_requested_start_time(start_time);
delegate.SetVisibilityFromAnimation(start);
element->Progress(start_time, &delegate);
EXPECT_TRUE(delegate.GetVisibilityForAnimation());
@@ -170,7 +170,7 @@ TEST(LayerAnimationElementTest, BrightnessElement) {
for (int i = 0; i < 2; ++i) {
start_time += delta;
- element->set_start_time(start_time);
+ element->set_requested_start_time(start_time);
delegate.SetBrightnessFromAnimation(start);
element->Progress(start_time, &delegate);
EXPECT_FLOAT_EQ(start, delegate.GetBrightnessForAnimation());
@@ -203,7 +203,7 @@ TEST(LayerAnimationElementTest, GrayscaleElement) {
for (int i = 0; i < 2; ++i) {
start_time += delta;
- element->set_start_time(start_time);
+ element->set_requested_start_time(start_time);
delegate.SetGrayscaleFromAnimation(start);
element->Progress(start_time, &delegate);
EXPECT_FLOAT_EQ(start, delegate.GetGrayscaleForAnimation());
@@ -241,7 +241,7 @@ TEST(LayerAnimationElementTest, PauseElement) {
TestLayerAnimationDelegate copy = delegate;
start_time += delta;
- element->set_start_time(start_time);
+ element->set_requested_start_time(start_time);
element->Progress(start_time + delta, &delegate);
// Nothing should have changed.

Powered by Google App Engine
This is Rietveld 408576698