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

Unified Diff: cc/trees/layer_tree_host_unittest_animation.cc

Issue 2105743006: cc : Use EXPECT_TRANSFORMATION_MATRIX_EQ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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 | tools/valgrind/gtest_exclude/cc_unittests.gtest-drmemory_win32.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/trees/layer_tree_host_unittest_animation.cc
diff --git a/cc/trees/layer_tree_host_unittest_animation.cc b/cc/trees/layer_tree_host_unittest_animation.cc
index a75f6c4b32482be48e49e58df36c0681b0d14851..9d7ba1fc71095231760f9b95f54b87fbfbb166e5 100644
--- a/cc/trees/layer_tree_host_unittest_animation.cc
+++ b/cc/trees/layer_tree_host_unittest_animation.cc
@@ -1202,7 +1202,7 @@ class LayerTreeHostAnimationTestPendingTreeAnimatesFirstCommit
// starting state which is 6,7.
gfx::Transform expected_transform;
expected_transform.Translate(6.0, 7.0);
- EXPECT_EQ(expected_transform, child->DrawTransform());
+ EXPECT_TRANSFORMATION_MATRIX_EQ(expected_transform, child->DrawTransform());
// And the sync tree layer should know it is animating.
EXPECT_TRUE(child->screen_space_transform_is_animating());
@@ -1429,7 +1429,8 @@ class LayerTreeHostAnimationTestRemoveAnimation
// applied.
gfx::Transform expected_transform;
expected_transform.Translate(10.f, 10.f);
- EXPECT_EQ(expected_transform, child->DrawTransform());
+ EXPECT_TRANSFORMATION_MATRIX_EQ(expected_transform,
+ child->DrawTransform());
EXPECT_FALSE(child->screen_space_transform_is_animating());
EndTest();
break;
@@ -1572,7 +1573,8 @@ class LayerTreeHostAnimationTestAnimationFinishesDuringCommit
gfx::Transform expected_transform;
expected_transform.Translate(5.f, 5.f);
LayerImpl* layer_impl = host_impl->sync_tree()->LayerById(layer_->id());
- EXPECT_EQ(expected_transform, layer_impl->DrawTransform());
+ EXPECT_TRANSFORMATION_MATRIX_EQ(expected_transform,
+ layer_impl->DrawTransform());
EndTest();
break;
}
@@ -1684,7 +1686,7 @@ class LayerTreeHostAnimationTestChangeAnimationPlayer
translate.Translate(5, 5);
switch (host_impl->sync_tree()->source_frame_number()) {
case 2:
- EXPECT_EQ(node->data.local, translate);
+ EXPECT_TRANSFORMATION_MATRIX_EQ(node->data.local, translate);
EndTest();
break;
default:
« no previous file with comments | « no previous file | tools/valgrind/gtest_exclude/cc_unittests.gtest-drmemory_win32.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698