New issue
Advanced search Search tips

Issue 722696 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

CSS Transforms2: Web Platform tests for Individual Transform Properties

Project Member Reported by ericwilligers@chromium.org, May 16 2017

Issue description

The translate, scale, and rotate properties should have web platform tests for parsing, animation and rendering.

 
Labels: Hotlist-Interop Update-Quarterly
Project Member

Comment 2 by bugdroid1@chromium.org, May 23 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c2dd1a0ce5ba82339906ec3f43bb0c34f1b1743d

commit c2dd1a0ce5ba82339906ec3f43bb0c34f1b1743d
Author: ericwilligers <ericwilligers@chromium.org>
Date: Tue May 23 01:28:09 2017

CSS Transforms 2: Web Platform Tests for translate,rotate,scale parsing

These are the first Web Platform Tests for the Individual Transform
Properties.

Spec:
https://drafts.csswg.org/css-transforms-2/#individual-transforms

BUG= 722696 

Review-Url: https://codereview.chromium.org/2882153004
Cr-Commit-Position: refs/heads/master@{#473775}

[add] https://crrev.com/c2dd1a0ce5ba82339906ec3f43bb0c34f1b1743d/third_party/WebKit/LayoutTests/external/wpt/css/css-transforms-2/parsing/resources/parsing-testcommon.js
[add] https://crrev.com/c2dd1a0ce5ba82339906ec3f43bb0c34f1b1743d/third_party/WebKit/LayoutTests/external/wpt/css/css-transforms-2/parsing/rotate-parsing-invalid.html
[add] https://crrev.com/c2dd1a0ce5ba82339906ec3f43bb0c34f1b1743d/third_party/WebKit/LayoutTests/external/wpt/css/css-transforms-2/parsing/rotate-parsing-valid.html
[add] https://crrev.com/c2dd1a0ce5ba82339906ec3f43bb0c34f1b1743d/third_party/WebKit/LayoutTests/external/wpt/css/css-transforms-2/parsing/scale-parsing-invalid.html
[add] https://crrev.com/c2dd1a0ce5ba82339906ec3f43bb0c34f1b1743d/third_party/WebKit/LayoutTests/external/wpt/css/css-transforms-2/parsing/scale-parsing-valid.html
[add] https://crrev.com/c2dd1a0ce5ba82339906ec3f43bb0c34f1b1743d/third_party/WebKit/LayoutTests/external/wpt/css/css-transforms-2/parsing/translate-parsing-invalid.html
[add] https://crrev.com/c2dd1a0ce5ba82339906ec3f43bb0c34f1b1743d/third_party/WebKit/LayoutTests/external/wpt/css/css-transforms-2/parsing/translate-parsing-valid.html

Project Member

Comment 3 by bugdroid1@chromium.org, Aug 11 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e3edea50288973d4b4a0de0a3b862ea931bc3ae5

commit e3edea50288973d4b4a0de0a3b862ea931bc3ae5
Author: ericwilligers <ericwilligers@chromium.org>
Date: Fri Aug 11 03:55:15 2017

WPT for Individual Transform Properties animation

The individual transform properties are specified in
https://drafts.csswg.org/css-transforms-2/#individual-transforms

translate animates as <length> or <percentage> list.
rotate animats as SLERP (spherical linear interpolation).
scale animates as <number> list.

When animating between 'none' and another value, 'none' is
replaced by the identity value for that transform property.

BUG= 722696 

Change-Id: I3e4bb0f6273260b53a49a845d66922916c86df10
Reviewed-on: https://chromium-review.googlesource.com/607988
Reviewed-by: nainar <nainar@chromium.org>
Commit-Queue: Eric Willigers <ericwilligers@chromium.org>
Cr-Commit-Position: refs/heads/master@{#493660}
[add] https://crrev.com/e3edea50288973d4b4a0de0a3b862ea931bc3ae5/third_party/WebKit/LayoutTests/external/wpt/css/css-transforms-2/animation/resources/interpolation-testcommon.js
[add] https://crrev.com/e3edea50288973d4b4a0de0a3b862ea931bc3ae5/third_party/WebKit/LayoutTests/external/wpt/css/css-transforms-2/animation/rotate-interpolation.html
[add] https://crrev.com/e3edea50288973d4b4a0de0a3b862ea931bc3ae5/third_party/WebKit/LayoutTests/external/wpt/css/css-transforms-2/animation/scale-interpolation.html
[add] https://crrev.com/e3edea50288973d4b4a0de0a3b862ea931bc3ae5/third_party/WebKit/LayoutTests/external/wpt/css/css-transforms-2/animation/translate-interpolation.html

Project Member

Comment 4 by bugdroid1@chromium.org, Aug 30 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/724d2f3086730179aad5e8ff89b389637c1067d7

commit 724d2f3086730179aad5e8ff89b389637c1067d7
Author: Eric Willigers <ericwilligers@chromium.org>
Date: Wed Aug 30 00:12:19 2017

CSS Motion Path: test calc support

https://drafts.csswg.org/css-values-3/#calc-notation
The calc() function ... can be used wherever <length>,
<frequency>, <angle>, <time>, <percentage>, <number>,
or <integer> values are allowed.

We test calc support for CSS Motion Path and
transforms properties.

BUG= 722757 , 722696 

Change-Id: If0f3791522f35b5fbd5f83b8fd6beda214600d96
Reviewed-on: https://chromium-review.googlesource.com/634086
Reviewed-by: Darren Shen <shend@chromium.org>
Commit-Queue: Eric Willigers <ericwilligers@chromium.org>
Cr-Commit-Position: refs/heads/master@{#498296}
[add] https://crrev.com/724d2f3086730179aad5e8ff89b389637c1067d7/third_party/WebKit/LayoutTests/external/wpt/css/css-transforms-2/transforms-support-calc.html
[add] https://crrev.com/724d2f3086730179aad5e8ff89b389637c1067d7/third_party/WebKit/LayoutTests/external/wpt/css/motion-1/offset-supports-calc.html

Status: Fixed (was: Started)

Comment 6 by nainar@chromium.org, Nov 29 2017

Labels: -Hotlist-Interop Test-Layout

Sign in to add a comment