[css-align] Overflow alignment value 'true' renamed to 'unsafe'

[css-align] Overflow alignment value 'true' renamed to 'unsafe'

The CSS Box Alignment spec changed the syntax of overflow alignment
so that the value 'true' has been renamed to 'unsafe'.

There is no change in the behavior of the old keyword, just a syntax
change, which affects just parsing and computed style logic.

diff: https://hg.csswg.org/drafts/rev/f34a2a8b8177

BUG=565883
Committed: https://crrev.com/ead445bd13a3bbb35e1518856fbd23665b6a0716
Cr-Commit-Position: refs/heads/master@{#364691}

[jfernandez, 2015-12-04 23:02:01]

Description was changed from

==========
[css-align] Overflow alignment value 'true' is renamed to 'unsafe'

The CSS Box Alignment spec changed the syntax of overflow alignment
so that the value 'true' has been renamed to 'unsafe'.

There is no change in the behavior of the old keyword, just a syntax
change, which affects just parsing and computed style logic.

diff: https://hg.csswg.org/drafts/rev/f34a2a8b8177

BUG=565883
==========

to

==========
[css-align] Overflow alignment value 'true' renamed to 'unsafe'

The CSS Box Alignment spec changed the syntax of overflow alignment
so that the value 'true' has been renamed to 'unsafe'.

There is no change in the behavior of the old keyword, just a syntax
change, which affects just parsing and computed style logic.

diff: https://hg.csswg.org/drafts/rev/f34a2a8b8177

BUG=565883
==========

[jfernandez, 2015-12-04 23:02:01]

jfernandez@igalia.com changed reviewers:
+ cbiesinger@chromium.org, rego@igalia.com, svillar@igalia.com,
timloh@chromium.org

[cbiesinger, 2015-12-04 23:04:51]

lgtm for layout

[Timothy Loh, 2015-12-07 05:56:13]

lgtm

https://codereview.chromium.org/1497403002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/css/CSSValueKeywords.in (right):

https://codereview.chromium.org/1497403002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/css/CSSValueKeywords.in:490: unsafe
Probably shouldn't list values multiple times in this file.

https://codereview.chromium.org/1497403002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/css/CSSValueKeywords.in:510: // unsafe
BTW I wouldn't bother adding these commented out lines, I don't know if anyone
actually finds them useful

[jfernandez, 2015-12-08 11:16:16]

The CQ bit was checked by jfernandez@igalia.com

[jfernandez, 2015-12-08 11:16:16]

The patchset sent to the CQ was uploaded after l-g-t-m from
cbiesinger@chromium.org, timloh@chromium.org
Link to the patchset: https://codereview.chromium.org/1497403002/#ps20001
(title: "Applied suggested changes.")

[commit-bot: I haz the power, 2015-12-08 11:16:26]

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

[commit-bot: I haz the power, 2015-12-08 11:45:54]

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

[commit-bot: I haz the power, 2015-12-08 11:45:54]

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

[jfernandez, 2015-12-09 09:43:25]

The CQ bit was checked by jfernandez@igalia.com

[commit-bot: I haz the power, 2015-12-09 09:43:38]

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

[commit-bot: I haz the power, 2015-12-09 11:06:09]

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

[commit-bot: I haz the power, 2015-12-09 11:06:11]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[jfernandez, 2015-12-11 10:23:14]

The CQ bit was checked by jfernandez@igalia.com

[jfernandez, 2015-12-11 10:23:15]

The patchset sent to the CQ was uploaded after l-g-t-m from
cbiesinger@chromium.org, timloh@chromium.org
Link to the patchset: https://codereview.chromium.org/1497403002/#ps40001
(title: "Fixed repaint tests issues.")

[commit-bot: I haz the power, 2015-12-11 10:24:58]

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

[commit-bot: I haz the power, 2015-12-11 10:47:01]

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

[commit-bot: I haz the power, 2015-12-11 10:47:01]

Try jobs failed on following builders:
  win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)

[jfernandez, 2015-12-11 12:46:17]

The CQ bit was checked by jfernandez@igalia.com

[commit-bot: I haz the power, 2015-12-11 12:46:52]

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

[commit-bot: I haz the power, 2015-12-11 13:46:11]

Description was changed from

==========
[css-align] Overflow alignment value 'true' renamed to 'unsafe'

The CSS Box Alignment spec changed the syntax of overflow alignment
so that the value 'true' has been renamed to 'unsafe'.

There is no change in the behavior of the old keyword, just a syntax
change, which affects just parsing and computed style logic.

diff: https://hg.csswg.org/drafts/rev/f34a2a8b8177

BUG=565883
==========

to

==========
[css-align] Overflow alignment value 'true' renamed to 'unsafe'

The CSS Box Alignment spec changed the syntax of overflow alignment
so that the value 'true' has been renamed to 'unsafe'.

There is no change in the behavior of the old keyword, just a syntax
change, which affects just parsing and computed style logic.

diff: https://hg.csswg.org/drafts/rev/f34a2a8b8177

BUG=565883
==========

[commit-bot: I haz the power, 2015-12-11 13:46:12]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-12-11 13:47:05]

Description was changed from

==========
[css-align] Overflow alignment value 'true' renamed to 'unsafe'

The CSS Box Alignment spec changed the syntax of overflow alignment
so that the value 'true' has been renamed to 'unsafe'.

There is no change in the behavior of the old keyword, just a syntax
change, which affects just parsing and computed style logic.

diff: https://hg.csswg.org/drafts/rev/f34a2a8b8177

BUG=565883
==========

to

==========
[css-align] Overflow alignment value 'true' renamed to 'unsafe'

The CSS Box Alignment spec changed the syntax of overflow alignment
so that the value 'true' has been renamed to 'unsafe'.

There is no change in the behavior of the old keyword, just a syntax
change, which affects just parsing and computed style logic.

diff: https://hg.csswg.org/drafts/rev/f34a2a8b8177

BUG=565883

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

[commit-bot: I haz the power, 2015-12-11 13:47:06]

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