CompositorViewHolder is source of truth for viewports

CompositorViewHolder is source of truth for viewports

This change makes the CompositorViewHolder responsible for knowing
what the window and visible viewports are. Any class that needs to
know this information can pull it from the CompositorViewHolder
(a LayoutManagerHost) instead of having it pushed or caching it.
This removes a fair amount of state and complication in
LayoutManager.

BUG=662427
Committed: https://crrev.com/fd6719290c9dce87292ea653ac370542014030c7
Committed: https://crrev.com/7b2ace3be27bcaa12f371215c435d4cd59c5ff5d
Cr-Original-Commit-Position: refs/heads/master@{#433916}
Cr-Commit-Position: refs/heads/master@{#433965}

[mdjones, 2016-11-18 16:54:45]

mdjones@chromium.org changed reviewers:
+ aelias@chromium.org, dtrainor@chromium.org

[mdjones, 2016-11-18 16:54:46]

ptal

[aelias_OOO_until_Jul13, 2016-11-18 22:10:38]

lgtm

[David Trainor- moved to gerrit, 2016-11-21 20:17:38]

lgtm

[mdjones, 2016-11-22 02:41:41]

Description was changed from

==========
CompositorViewHolder is source of truth for viewports

This change makes the CompositorViewHolder responsible for knowing
what the window and visible viewports are. Any class that needs to
know this information can pull it from the CompositorViewHolder
(a LayoutManagerHost) instead of having it pushed.

BUG=662427
==========

to

==========
CompositorViewHolder is source of truth for viewports

This change makes the CompositorViewHolder responsible for knowing
what the window and visible viewports are. Any class that needs to
know this information can pull it from the CompositorViewHolder
(a LayoutManagerHost) instead of having it pushed or caching it.
This removes a fair amount of state and complication in
LayoutManager.

BUG=662427
==========

[mdjones, 2016-11-22 17:33:49]

The CQ bit was checked by mdjones@chromium.org

[commit-bot: I haz the power, 2016-11-22 17:34:27]

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-11-22 18:16:29]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1479836029177420,
"parent_rev": "ca3c4eda69e16c08bad7cb463929ea25175e3f86", "commit_rev":
"1e58075c2a11ea1f4818f04966d9556f2944d7a4"}

[commit-bot: I haz the power, 2016-11-22 18:16:52]

Description was changed from

==========
CompositorViewHolder is source of truth for viewports

This change makes the CompositorViewHolder responsible for knowing
what the window and visible viewports are. Any class that needs to
know this information can pull it from the CompositorViewHolder
(a LayoutManagerHost) instead of having it pushed or caching it.
This removes a fair amount of state and complication in
LayoutManager.

BUG=662427
==========

to

==========
CompositorViewHolder is source of truth for viewports

This change makes the CompositorViewHolder responsible for knowing
what the window and visible viewports are. Any class that needs to
know this information can pull it from the CompositorViewHolder
(a LayoutManagerHost) instead of having it pushed or caching it.
This removes a fair amount of state and complication in
LayoutManager.

BUG=662427
==========

[commit-bot: I haz the power, 2016-11-22 18:16:53]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-11-22 18:19:32]

Description was changed from

==========
CompositorViewHolder is source of truth for viewports

This change makes the CompositorViewHolder responsible for knowing
what the window and visible viewports are. Any class that needs to
know this information can pull it from the CompositorViewHolder
(a LayoutManagerHost) instead of having it pushed or caching it.
This removes a fair amount of state and complication in
LayoutManager.

BUG=662427
==========

to

==========
CompositorViewHolder is source of truth for viewports

This change makes the CompositorViewHolder responsible for knowing
what the window and visible viewports are. Any class that needs to
know this information can pull it from the CompositorViewHolder
(a LayoutManagerHost) instead of having it pushed or caching it.
This removes a fair amount of state and complication in
LayoutManager.

BUG=662427

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

[commit-bot: I haz the power, 2016-11-22 18:19:33]

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

[Maria, 2016-11-22 19:10:54]

A revert of this CL (patchset #5 id:80001) has been created in
https://codereview.chromium.org/2524833002/ by mariakhomenko@chromium.org.

The reason for reverting is: Broke the builder:
https://build.chromium.org/p/chromium.android/builders/Android%20arm64%20Buil....

[mdjones, 2016-11-22 19:31:27]

Description was changed from

==========
CompositorViewHolder is source of truth for viewports

This change makes the CompositorViewHolder responsible for knowing
what the window and visible viewports are. Any class that needs to
know this information can pull it from the CompositorViewHolder
(a LayoutManagerHost) instead of having it pushed or caching it.
This removes a fair amount of state and complication in
LayoutManager.

BUG=662427

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

to

==========
CompositorViewHolder is source of truth for viewports

This change makes the CompositorViewHolder responsible for knowing
what the window and visible viewports are. Any class that needs to
know this information can pull it from the CompositorViewHolder
(a LayoutManagerHost) instead of having it pushed or caching it.
This removes a fair amount of state and complication in
LayoutManager.

BUG=662427

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

[mdjones, 2016-11-22 19:54:10]

The CQ bit was checked by mdjones@chromium.org

[mdjones, 2016-11-22 19:54:11]

The patchset sent to the CQ was uploaded after l-g-t-m from aelias@chromium.org,
dtrainor@chromium.org
Link to the patchset: https://codereview.chromium.org/2509873003/#ps100001
(title: "fix test merge conflict")

[commit-bot: I haz the power, 2016-11-22 19:54:58]

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-11-22 20:30:54]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1479844450641550,
"parent_rev": "9448c1f8fcde38c10ae2df02de3b18cc0cd20204", "commit_rev":
"f7a4341655eb8f7386fca58ca843d762a25dee04"}

[commit-bot: I haz the power, 2016-11-22 20:31:19]

Description was changed from

==========
CompositorViewHolder is source of truth for viewports

This change makes the CompositorViewHolder responsible for knowing
what the window and visible viewports are. Any class that needs to
know this information can pull it from the CompositorViewHolder
(a LayoutManagerHost) instead of having it pushed or caching it.
This removes a fair amount of state and complication in
LayoutManager.

BUG=662427

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

to

==========
CompositorViewHolder is source of truth for viewports

This change makes the CompositorViewHolder responsible for knowing
what the window and visible viewports are. Any class that needs to
know this information can pull it from the CompositorViewHolder
(a LayoutManagerHost) instead of having it pushed or caching it.
This removes a fair amount of state and complication in
LayoutManager.

BUG=662427

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

[commit-bot: I haz the power, 2016-11-22 20:31:20]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-11-22 20:35:21]

Description was changed from

==========
CompositorViewHolder is source of truth for viewports

This change makes the CompositorViewHolder responsible for knowing
what the window and visible viewports are. Any class that needs to
know this information can pull it from the CompositorViewHolder
(a LayoutManagerHost) instead of having it pushed or caching it.
This removes a fair amount of state and complication in
LayoutManager.

BUG=662427

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

to

==========
CompositorViewHolder is source of truth for viewports

This change makes the CompositorViewHolder responsible for knowing
what the window and visible viewports are. Any class that needs to
know this information can pull it from the CompositorViewHolder
(a LayoutManagerHost) instead of having it pushed or caching it.
This removes a fair amount of state and complication in
LayoutManager.

BUG=662427

Committed: https://crrev.com/fd6719290c9dce87292ea653ac370542014030c7
Committed: https://crrev.com/7b2ace3be27bcaa12f371215c435d4cd59c5ff5d
Cr-Original-Commit-Position: refs/heads/master@{#433916}
Cr-Commit-Position: refs/heads/master@{#433965}
==========

[commit-bot: I haz the power, 2016-11-22 20:35:22]

Patchset 6 (id:??) landed as
https://crrev.com/7b2ace3be27bcaa12f371215c435d4cd59c5ff5d
Cr-Commit-Position: refs/heads/master@{#433965}