Spec-compliant shorthand parsing of foo-break-(after,before,inside).

Spec-compliant shorthand parsing of foo-break-(after,before,inside).

The CL that introduced the generic break-after, break-before and break-inside
properties deliberately violated the spec when it came to parsing the
page-break-(after,before,inside) shorthand properties. This was in order to
reduce the risk of a big revert, and instead remain as compatible as we could
with how we used to handle it prior to the introduction of the new generic
properties, i.e. when we parsed and stored separate properties for
page-break-foo and -webkit-column-break-foo, rather than treating them as
shorthands for break-foo.

BUG=223068
Committed: https://crrev.com/1e723e74e19e48596466bb2357197dc6836d9c8a
Cr-Commit-Position: refs/heads/master@{#376896}

[mstensho (USE GERRIT), 2016-02-22 21:11:15]

mstensho@opera.com changed reviewers:
+ timloh@chromium.org

[Timothy Loh, 2016-02-22 23:40:14]

On 2016/02/22 21:11:15, mstensho wrote:

lgtm

[mstensho (USE GERRIT), 2016-02-23 01:39:08]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2016-02-23 01:40:54]

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

[commit-bot: I haz the power, 2016-02-23 01:48:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-23 01:49:30]

Description was changed from

==========
Spec-compliant shorthand parsing of foo-break-(after,before,inside).

The CL that introduced the generic break-after, break-before and break-inside
properties deliberately violated the spec when it came to parsing the
page-break-(after,before,inside) shorthand properties. This was in order to
reduce the risk of a big revert, and instead remain as compatible as we could
with how we used to handle it prior to the introduction of the new generic
properties, i.e. when we parsed and stored separate properties for
page-break-foo and -webkit-column-break-foo, rather than treating them as
shorthands for break-foo.

BUG=223068
==========

to

==========
Spec-compliant shorthand parsing of foo-break-(after,before,inside).

The CL that introduced the generic break-after, break-before and break-inside
properties deliberately violated the spec when it came to parsing the
page-break-(after,before,inside) shorthand properties. This was in order to
reduce the risk of a big revert, and instead remain as compatible as we could
with how we used to handle it prior to the introduction of the new generic
properties, i.e. when we parsed and stored separate properties for
page-break-foo and -webkit-column-break-foo, rather than treating them as
shorthands for break-foo.

BUG=223068

Committed: https://crrev.com/1e723e74e19e48596466bb2357197dc6836d9c8a
Cr-Commit-Position: refs/heads/master@{#376896}
==========

[commit-bot: I haz the power, 2016-02-23 01:49:31]

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