Fixes incorrect values seen in clip property

Fixes incorrect values seen in clip property

Previously getComputedStyle(el).clip would be 'box: 0 0 0 0' when the
element's clip was 'rect(auto, auto, auto, auto)'. This change corrects the
behaviour so that each side of the box is considered correctly (i.e. as
rect(auto auto auto auto)), even the box as a whole is not 'auto'.

Firefox displays rect(auto, auto, auto, auto) in the case where
rect(auto, auto, auto, auto) is used, and rect(auto, X, auto, Y) where rect(auto, X, auto, Y) is used.

Our behavior now matches that.

BUG=412921
Committed: https://crrev.com/19c08f5cdff2046ee2fd143c3582c2e5994fb7aa
Cr-Commit-Position: refs/heads/master@{#363747}

[Zachary Forman, 2015-12-08 02:16:36]

zforman@google.com changed reviewers:
+ dstockwell@chromium.org

[Zachary Forman, 2015-12-08 02:16:36]

Hi, could you please review this cl?

Thanks,
Zac

[dstockwell, 2015-12-08 02:43:38]

On 2015/12/08 at 02:16:36, zforman wrote:
> Hi, could you please review this cl?
> 
> Thanks,
> Zac

I think you've got the wrong bug link. Could you also update the description to
mention what happens in other browsers?

[Zachary Forman, 2015-12-08 02:47:47]

Description was changed from

==========
Fixes incorrect values seen in clip property

Previously getComputedStyle(el).clip would be 'box: 0 0 0 0' when the
element's clip was 'box: auto auto auto auto'. This change corrects the
behaviour so that each side of the box is considered correctly, even
when the box as a whole is not 'auto'.

BUG=41291
==========

to

==========
Fixes incorrect values seen in clip property

Previously getComputedStyle(el).clip would be 'box: 0 0 0 0' when the
element's clip was 'box: auto auto auto auto'. This change corrects the
behaviour so that each side of the box is considered correctly, even
when the box as a whole is not 'auto'.

BUG=412921
==========

[Zachary Forman, 2015-12-08 02:55:03]

Description was changed from

==========
Fixes incorrect values seen in clip property

Previously getComputedStyle(el).clip would be 'box: 0 0 0 0' when the
element's clip was 'box: auto auto auto auto'. This change corrects the
behaviour so that each side of the box is considered correctly, even
when the box as a whole is not 'auto'.

BUG=412921
==========

to

==========
Fixes incorrect values seen in clip property

Previously getComputedStyle(el).clip would be 'box: 0 0 0 0' when the
element's clip was 'box: auto auto auto auto'. This change corrects the
behaviour so that each side of the box is considered correctly (i.e. as
rect(auto, auto, auto, auto)), even the box as a whole is not 'auto'.

Firefox displays rect(auto, auto, auto, auto).

BUG=412921
==========

[Zachary Forman, 2015-12-08 03:01:40]

Description was changed from

==========
Fixes incorrect values seen in clip property

Previously getComputedStyle(el).clip would be 'box: 0 0 0 0' when the
element's clip was 'box: auto auto auto auto'. This change corrects the
behaviour so that each side of the box is considered correctly (i.e. as
rect(auto, auto, auto, auto)), even the box as a whole is not 'auto'.

Firefox displays rect(auto, auto, auto, auto).

BUG=412921
==========

to

==========
Fixes incorrect values seen in clip property

Previously getComputedStyle(el).clip would be 'box: 0 0 0 0' when the
element's clip was 'rect(auto, auto, auto, auto)'. This change corrects the
behaviour so that each side of the box is considered correctly (i.e. as
rect(auto auto auto auto)), even the box as a whole is not 'auto'.

Firefox displays rect(auto, auto, auto, auto) in the case where
rect(auto, auto, auto, auto) is used, and rect(auto, X, auto, Y) where
rect(auto, X, auto, Y) is used.

Our behavior now matches that.

BUG=412921
==========

[Zachary Forman, 2015-12-08 04:01:25]

Have done.

[dstockwell, 2015-12-08 04:09:59]

lgtm

[dstockwell, 2015-12-08 04:10:04]

The CQ bit was checked by dstockwell@chromium.org

[dstockwell, 2015-12-08 04:10:04]

The CQ bit was checked by dstockwell@chromium.org

[commit-bot: I haz the power, 2015-12-08 04:10:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1506883004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1506883004/1

[commit-bot: I haz the power, 2015-12-08 05:30:01]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-08 05:30:33]

Description was changed from

==========
Fixes incorrect values seen in clip property

Previously getComputedStyle(el).clip would be 'box: 0 0 0 0' when the
element's clip was 'rect(auto, auto, auto, auto)'. This change corrects the
behaviour so that each side of the box is considered correctly (i.e. as
rect(auto auto auto auto)), even the box as a whole is not 'auto'.

Firefox displays rect(auto, auto, auto, auto) in the case where
rect(auto, auto, auto, auto) is used, and rect(auto, X, auto, Y) where
rect(auto, X, auto, Y) is used.

Our behavior now matches that.

BUG=412921
==========

to

==========
Fixes incorrect values seen in clip property

Previously getComputedStyle(el).clip would be 'box: 0 0 0 0' when the
element's clip was 'rect(auto, auto, auto, auto)'. This change corrects the
behaviour so that each side of the box is considered correctly (i.e. as
rect(auto auto auto auto)), even the box as a whole is not 'auto'.

Firefox displays rect(auto, auto, auto, auto) in the case where
rect(auto, auto, auto, auto) is used, and rect(auto, X, auto, Y) where
rect(auto, X, auto, Y) is used.

Our behavior now matches that.

BUG=412921

Committed: https://crrev.com/19c08f5cdff2046ee2fd143c3582c2e5994fb7aa
Cr-Commit-Position: refs/heads/master@{#363747}
==========

[commit-bot: I haz the power, 2015-12-08 05:30:34]

Patchset 1 (id:??) landed as
https://crrev.com/19c08f5cdff2046ee2fd143c3582c2e5994fb7aa
Cr-Commit-Position: refs/heads/master@{#363747}