Correct the initial value of CSS independent transform properties

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
Committed: https://crrev.com/a022d98375b213ad9f497f220d0b64d6ed77cbc9
Cr-Commit-Position: refs/heads/master@{#427941}

[trchen, 2016-10-26 01:19:06]

trchen@chromium.org changed reviewers:
+ alancutter@chromium.org

[trchen, 2016-10-26 01:19:11]

The CQ bit was checked by trchen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-26 01:19:40]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[alancutter (OOO until 2018), 2016-10-26 01:39:39]

lgtm after changes.

Please add a link to the spec in the description and a bug number.

https://codereview.chromium.org/2454543002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/transforms/independent-transform-initial-value.html
(right):

https://codereview.chromium.org/2454543002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/transforms/independent-transform-initial-value.html:3:
<div style="translate:initial; margin-top:-100px; width:100px; height:100px;
background:red;"></div>
Include rotate and scale as well.

[trchen, 2016-10-26 01:51:42]

Description was changed from

==========
Correct the initial value of CSS independent transform properties

According to the spec, 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 intialize these properties to
initial value.
==========

to

==========
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
==========

[trchen, 2016-10-26 01:54:24]

On 2016/10/26 01:39:39, alancutter wrote:
> lgtm after changes.
> 
> Please add a link to the spec in the description and a bug number.

Done.

>
https://codereview.chromium.org/2454543002/diff/20001/third_party/WebKit/Layo...
> File
>
third_party/WebKit/LayoutTests/transforms/independent-transform-initial-value.html
> (right):
> 
>
https://codereview.chromium.org/2454543002/diff/20001/third_party/WebKit/Layo...
>
third_party/WebKit/LayoutTests/transforms/independent-transform-initial-value.html:3:
> <div style="translate:initial; margin-top:-100px; width:100px; height:100px;
> background:red;"></div>
> Include rotate and scale as well.

Done.

[trchen, 2016-10-26 01:54:35]

The CQ bit was checked by trchen@chromium.org

[trchen, 2016-10-26 01:54:36]

The patchset sent to the CQ was uploaded after l-g-t-m from
alancutter@chromium.org
Link to the patchset: https://codereview.chromium.org/2454543002/#ps40001
(title: "revised")

[commit-bot: I haz the power, 2016-10-26 01:55:22]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-26 02:03:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-26 02:03:35]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[trchen, 2016-10-26 02:49:49]

trchen@chromium.org changed reviewers:
+ dstockwell@chromium.org

[trchen, 2016-10-26 02:49:49]

Sorry Alan I thought you are an owner of Source/core. orz

+cc dstockwell

[trchen, 2016-10-26 02:49:57]

The CQ bit was checked by trchen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-26 02:50:08]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-26 04:00:53]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-26 04:00:53]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[trchen, 2016-10-26 22:12:04]

The CQ bit was checked by trchen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-26 22:12:49]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-26 23:34:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-26 23:34:26]

Dry run: This issue passed the CQ dry run.

[dstockwell, 2016-10-27 04:31:43]

lgtm

[trchen, 2016-10-27 04:35:28]

The CQ bit was checked by trchen@chromium.org

[trchen, 2016-10-27 04:35:28]

The patchset sent to the CQ was uploaded after l-g-t-m from
alancutter@chromium.org
Link to the patchset: https://codereview.chromium.org/2454543002/#ps60001
(title: "git cl format")

[commit-bot: I haz the power, 2016-10-27 04:35:54]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-27 04:39:56]

Description was changed from

==========
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
==========

to

==========
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
==========

[commit-bot: I haz the power, 2016-10-27 04:39:57]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-10-27 04:41:50]

Description was changed from

==========
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
==========

to

==========
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

Committed: https://crrev.com/a022d98375b213ad9f497f220d0b64d6ed77cbc9
Cr-Commit-Position: refs/heads/master@{#427941}
==========

[commit-bot: I haz the power, 2016-10-27 04:41:51]

Patchset 4 (id:??) landed as
https://crrev.com/a022d98375b213ad9f497f220d0b64d6ed77cbc9
Cr-Commit-Position: refs/heads/master@{#427941}