LayoutState doesn't need to store both layout and pagination offset.

LayoutState doesn't need to store both layout and pagination offset.

We only ever used those two in combination to figure out how far away we were
from the start of the pagination context. So, let's just store that directly instead.
This allows us to clean up quite a bit. Also changed LayoutState() to do more
early returns, when we have no more work left to do.

Also consolidated two sections that disabled pagination for unsupported content
(one for SVG and one for other unbreakable content).

Committed: https://crrev.com/fb9e4782a189a3122fd7d313a1d3f8d6cae37076
Cr-Commit-Position: refs/heads/master@{#427945}

[mstensho (USE GERRIT), 2016-10-26 22:34:36]

The CQ bit was checked by mstensho@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-10-26 22:35:07]

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

[mstensho (USE GERRIT), 2016-10-26 22:35:10]

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

[eae, 2016-10-26 22:36:37]

LGTM

[commit-bot: I haz the power, 2016-10-27 01:07:12]

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

[commit-bot: I haz the power, 2016-10-27 01:07:13]

Dry run: This issue passed the CQ dry run.

[mstensho (USE GERRIT), 2016-10-27 05:12:45]

Description was changed from

==========
LayoutState doesn't need to store both layout and pagination offset.

We only ever used those two in combination to figure out how far away we were
from the start of the pagination offset. So, let's just store that directly
instead.
This allows us to clean up quite a bit. Also changed LayoutState() to do more
early returns, when we have no more work left to do.

Also consolidated two sections that disabled pagination for unsupported content
(one for SVG and one for other unbreakable content).
==========

to

==========
LayoutState doesn't need to store both layout and pagination offset.

We only ever used those two in combination to figure out how far away we were
from the start of the pagination context. So, let's just store that directly
instead.
This allows us to clean up quite a bit. Also changed LayoutState() to do more
early returns, when we have no more work left to do.

Also consolidated two sections that disabled pagination for unsupported content
(one for SVG and one for other unbreakable content).
==========

[mstensho (USE GERRIT), 2016-10-27 05:13:20]

On 2016/10/26 22:36:37, eae wrote:
> LGTM

Thanks. Fixed one typo in the commit message:

- start of the pagination offset
+ start of the pagination context

[mstensho (USE GERRIT), 2016-10-27 05:13:26]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2016-10-27 05:13:44]

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 05:17:51]

Description was changed from

==========
LayoutState doesn't need to store both layout and pagination offset.

We only ever used those two in combination to figure out how far away we were
from the start of the pagination context. So, let's just store that directly
instead.
This allows us to clean up quite a bit. Also changed LayoutState() to do more
early returns, when we have no more work left to do.

Also consolidated two sections that disabled pagination for unsupported content
(one for SVG and one for other unbreakable content).
==========

to

==========
LayoutState doesn't need to store both layout and pagination offset.

We only ever used those two in combination to figure out how far away we were
from the start of the pagination context. So, let's just store that directly
instead.
This allows us to clean up quite a bit. Also changed LayoutState() to do more
early returns, when we have no more work left to do.

Also consolidated two sections that disabled pagination for unsupported content
(one for SVG and one for other unbreakable content).
==========

[commit-bot: I haz the power, 2016-10-27 05:17:52]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-27 05:20:17]

Description was changed from

==========
LayoutState doesn't need to store both layout and pagination offset.

We only ever used those two in combination to figure out how far away we were
from the start of the pagination context. So, let's just store that directly
instead.
This allows us to clean up quite a bit. Also changed LayoutState() to do more
early returns, when we have no more work left to do.

Also consolidated two sections that disabled pagination for unsupported content
(one for SVG and one for other unbreakable content).
==========

to

==========
LayoutState doesn't need to store both layout and pagination offset.

We only ever used those two in combination to figure out how far away we were
from the start of the pagination context. So, let's just store that directly
instead.
This allows us to clean up quite a bit. Also changed LayoutState() to do more
early returns, when we have no more work left to do.

Also consolidated two sections that disabled pagination for unsupported content
(one for SVG and one for other unbreakable content).

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

[commit-bot: I haz the power, 2016-10-27 05:20:18]

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

[mstensho (USE GERRIT), 2016-10-28 08:36:14]

This fixed https://bugs.chromium.org/p/chromium/issues/detail?id=583713