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

Unified Diff: cc/layer_sorter_unittest.cc

Issue 11361067: Avoid using WebTransformationMatrix::translateRight* (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 1 month 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 | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/layer_sorter_unittest.cc
diff --git a/cc/layer_sorter_unittest.cc b/cc/layer_sorter_unittest.cc
index 6c54fc550974b2ad6bdb1aec116983d580b584f4..eb2e6bcdb4f2f680c12511077dbeee72ea9675e4 100644
--- a/cc/layer_sorter_unittest.cc
+++ b/cc/layer_sorter_unittest.cc
@@ -67,8 +67,8 @@ TEST(LayerSorterTest, RightAngleOverlap)
// Two layers forming a right angle with a perspective viewing transform.
WebTransformationMatrix leftFaceMatrix;
+ leftFaceMatrix.translate3d(-1, 0, -5);
leftFaceMatrix.rotate3d(0, 1, 0, -90);
- leftFaceMatrix.translateRight3d(-1, 0, -5);
leftFaceMatrix.translate(-1, -1);
LayerShape leftFace(2, 2, perspectiveMatrix * leftFaceMatrix);
WebTransformationMatrix frontFaceMatrix;
@@ -97,8 +97,8 @@ TEST(LayerSorterTest, IntersectingLayerOverlap)
LayerShape frontFace(2, 2, perspectiveMatrix * frontFaceMatrix);
WebTransformationMatrix throughMatrix;
+ throughMatrix.translate3d(0, 0, -4);
throughMatrix.rotate3d(0, 1, 0, 45);
- throughMatrix.translateRight3d(0, 0, -4);
throughMatrix.translate(-1, -1);
LayerShape rotatedFace(2, 2, perspectiveMatrix * throughMatrix);
overlapResult = LayerSorter::checkOverlap(&frontFace, &rotatedFace, zThreshold, weight);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698