New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 659434 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Initial value of CSS independent transform properties should be 'none'

Project Member Reported by trchen@chromium.org, Oct 26 2016

Issue description

Feature not shipped. Affect layout tests only.

What steps will reproduce the problem?
(1) See layout test included in https://codereview.chromium.org/2454543002/

What is the expected output?
Setting initial value to those properties should make computed value to become 'none'.

What do you see instead?
The computed value is identity matrix.
 
Labels: Test-Layout
Project Member

Comment 2 by bugdroid1@chromium.org, Oct 27 2016

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

commit a022d98375b213ad9f497f220d0b64d6ed77cbc9
Author: trchen <trchen@chromium.org>
Date: Thu Oct 27 04:39:35 2016

Correct the initial value of CSS independent transform properties

According to the spec[1], the initial value of translate, scale, and rotate
should be 'none', which doesn't force stacking context unlike identity matrix.
This CL also make sure the constructor always initialize these properties to
initial value.

[1] https://drafts.csswg.org/css-transforms-2/#individual-transforms

BUG= 659434 

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

[add] https://crrev.com/a022d98375b213ad9f497f220d0b64d6ed77cbc9/third_party/WebKit/LayoutTests/transforms/independent-transform-initial-value-expected.html
[add] https://crrev.com/a022d98375b213ad9f497f220d0b64d6ed77cbc9/third_party/WebKit/LayoutTests/transforms/independent-transform-initial-value.html
[modify] https://crrev.com/a022d98375b213ad9f497f220d0b64d6ed77cbc9/third_party/WebKit/Source/core/style/ComputedStyle.h
[modify] https://crrev.com/a022d98375b213ad9f497f220d0b64d6ed77cbc9/third_party/WebKit/Source/core/style/StyleTransformData.cpp

Comment 3 by trchen@chromium.org, Oct 28 2016

Status: Fixed (was: Started)

Sign in to add a comment