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

Unified Diff: cc/trees/layer_tree_host_common.cc

Issue 23572033: cc: Add test for AA quads due to precision loss (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Don't bother converting a literal Created 7 years, 3 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: cc/trees/layer_tree_host_common.cc
diff --git a/cc/trees/layer_tree_host_common.cc b/cc/trees/layer_tree_host_common.cc
index 785542108fa5fa5d6f39d24aadf8628263ea487e..b1b1f8f9f895eceacb093f5c5984f6a4d64b31fc 100644
--- a/cc/trees/layer_tree_host_common.cc
+++ b/cc/trees/layer_tree_host_common.cc
@@ -1409,8 +1409,9 @@ static void CalculateDrawPropertiesInternal(
// case, the render_surface re-parents the transforms.
layer_draw_properties.target_space_transform = combined_transform;
// M[draw] = M[parent] * LT * S[layer2content]
- layer_draw_properties.target_space_transform.Scale
- (1.f / layer->contents_scale_x(), 1.f / layer->contents_scale_y());
+ SkMScalar one(1.0);
danakj 2013/09/07 00:54:59 nit: use SK_MScalar1? or maybe not?
enne (OOO) 2013/09/07 01:25:18 Sure, done.
+ layer_draw_properties.target_space_transform.Scale(
+ one / layer->contents_scale_x(), one / layer->contents_scale_y());
// The layer's screen_space_transform represents the transform between root
// layer's "screen space" and local content space.

Powered by Google App Engine
This is Rietveld 408576698