diff options
| author | nainar <nainar@chromium.org> | 2015-09-02 18:04:10 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-09-03 01:04:59 +0000 |
| commit | 8ca8ee6d5165acb571a5f9d298ca3d80310e5f15 (patch) | |
| tree | 9fe0309b79367e3af3c12c4dc6b0e05706e40ab6 /mojo/converters | |
| parent | 81937dc4a4fae2028c3c70f9c1c3f3414448f6de (diff) | |
| download | chromium_src-8ca8ee6d5165acb571a5f9d298ca3d80310e5f15.zip chromium_src-8ca8ee6d5165acb571a5f9d298ca3d80310e5f15.tar.gz chromium_src-8ca8ee6d5165acb571a5f9d298ca3d80310e5f15.tar.bz2 | |
Apply skew on both axes together rather than sequentially on Compositor
thread.
Currently skew is not implemented correctly on the Compositor thread.
Instead of calling skew(angle_x, angle_y), the implementation calls
skewX(angle_x) followed by skewY(angle_y). This leads to a different
result. This patch applied skew on both axes together rather than
calling skewX(angle_x) and skewX(angle_Y) sequentially. FF and IE
correctly call skew(angle_x, angle_y).
BUG=268468
Review URL: https://codereview.chromium.org/1325803002
Cr-Commit-Position: refs/heads/master@{#347089}
Diffstat (limited to 'mojo/converters')
| -rw-r--r-- | mojo/converters/surfaces/tests/surface_unittest.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mojo/converters/surfaces/tests/surface_unittest.cc b/mojo/converters/surfaces/tests/surface_unittest.cc index a23f6cb..0dcfeea 100644 --- a/mojo/converters/surfaces/tests/surface_unittest.cc +++ b/mojo/converters/surfaces/tests/surface_unittest.cc @@ -208,7 +208,7 @@ TEST(SurfaceLibTest, RenderPass) { gfx::Rect output_rect(4, 9, 13, 71); gfx::Rect damage_rect(9, 17, 41, 45); gfx::Transform transform_to_root_target; - transform_to_root_target.SkewY(43.0); + transform_to_root_target.Skew(0.0, 43.0); bool has_transparent_background = false; pass->SetAll(pass_id, output_rect, |