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

Unified Diff: cc/layers/layer_impl.cc

Issue 2495373003: Match html canvas which is transferred to OffscreenCanvas to CSS style (Closed)
Patch Set: fix Created 4 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
Index: cc/layers/layer_impl.cc
diff --git a/cc/layers/layer_impl.cc b/cc/layers/layer_impl.cc
index be233f65d512fd6ebe4e958db2c06235e3400dd5..b8bd94a1ee982159e0c2789ace973cbd36f90c1a 100644
--- a/cc/layers/layer_impl.cc
+++ b/cc/layers/layer_impl.cc
@@ -179,12 +179,18 @@ void LayerImpl::PopulateSharedQuadState(SharedQuadState* state) const {
void LayerImpl::PopulateScaledSharedQuadState(SharedQuadState* state,
danakj 2016/11/28 22:26:22 I don't think we need 2 methods really. How about
xlai (Olivia) 2016/12/13 17:29:17 Done.
float scale) const {
+ PopulateScaledSharedQuadState(state, scale, scale);
+}
+
+void LayerImpl::PopulateScaledSharedQuadState(SharedQuadState* state,
+ float scale_x,
danakj 2016/11/28 22:26:22 can you rename these input vars to all be like "la
xlai (Olivia) 2016/12/13 17:29:17 Done.
+ float scale_y) const {
gfx::Transform scaled_draw_transform =
draw_properties_.target_space_transform;
- scaled_draw_transform.Scale(SK_MScalar1 / scale, SK_MScalar1 / scale);
- gfx::Size scaled_bounds = gfx::ScaleToCeiledSize(bounds(), scale);
+ scaled_draw_transform.Scale(SK_MScalar1 / scale_x, SK_MScalar1 / scale_y);
+ gfx::Size scaled_bounds = gfx::ScaleToCeiledSize(bounds(), scale_x, scale_y);
gfx::Rect scaled_visible_layer_rect =
- gfx::ScaleToEnclosingRect(visible_layer_rect(), scale);
+ gfx::ScaleToEnclosingRect(visible_layer_rect(), scale_x, scale_y);
scaled_visible_layer_rect.Intersect(gfx::Rect(scaled_bounds));
state->SetAll(scaled_draw_transform, scaled_bounds, scaled_visible_layer_rect,
« no previous file with comments | « cc/layers/layer_impl.h ('k') | cc/layers/surface_layer.h » ('j') | cc/layers/surface_layer.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698