Add FrameView::MarkViewportConstrainedObjectsForLayout

Add FrameView::MarkViewportConstrainedObjectsForLayout

A minor refactoring; the method will get a couple of other callers
in subsequent patches (which I'm breaking up for digestibility).

The new condition is something that currently will never be false;
subsequent patches will cause it be false sometimes.

BUG=701575
Review-Url: https://codereview.chromium.org/2846263002
Cr-Commit-Position: refs/heads/master@{#468066}
Committed: https://chromium.googlesource.com/chromium/src/+/90f1043d53ecd259599da1af4a17ca1a1b0a3164

[szager1, 2017-04-28 15:56:11]

The CQ bit was checked by szager@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-28 15:56:35]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[szager1, 2017-04-28 15:57:07]

Description was changed from

==========
Add FrameView::MarkViewportConstrainedObjectsForLayout

A minor refactoring; the method will get a couple of other callers
in subsequent patches (which I'm breaking up for digestibility).

The new condition is something that currently will never be false;
subsequent patches will cause it be false sometimes.

BUG=701575
==========

to

==========
Add FrameView::MarkViewportConstrainedObjectsForLayout

A minor refactoring; the method will get a couple of other callers
in subsequent patches (which I'm breaking up for digestibility).

The new condition is something that currently will never be false;
subsequent patches will cause it be false sometimes.

BUG=701575
==========

[szager1, 2017-04-28 15:57:07]

szager@chromium.org changed reviewers:
+ skobes@chromium.org

[skobes, 2017-04-28 15:59:04]

lgtm

[commit-bot: I haz the power, 2017-04-28 17:25:57]

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

[commit-bot: I haz the power, 2017-04-28 17:25:58]

Dry run: This issue passed the CQ dry run.

[szager1, 2017-04-28 18:18:55]

The CQ bit was checked by szager@chromium.org

[commit-bot: I haz the power, 2017-04-28 18:19: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, 2017-04-28 18:28:04]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493403535829400, "parent_rev":
"6c5572401ee0e399869a389c68c6f5b33aee4ed6", "commit_rev":
"90f1043d53ecd259599da1af4a17ca1a1b0a3164"}

[commit-bot: I haz the power, 2017-04-28 18:28:25]

Description was changed from

==========
Add FrameView::MarkViewportConstrainedObjectsForLayout

A minor refactoring; the method will get a couple of other callers
in subsequent patches (which I'm breaking up for digestibility).

The new condition is something that currently will never be false;
subsequent patches will cause it be false sometimes.

BUG=701575
==========

to

==========
Add FrameView::MarkViewportConstrainedObjectsForLayout

A minor refactoring; the method will get a couple of other callers
in subsequent patches (which I'm breaking up for digestibility).

The new condition is something that currently will never be false;
subsequent patches will cause it be false sometimes.

BUG=701575

Review-Url: https://codereview.chromium.org/2846263002
Cr-Commit-Position: refs/heads/master@{#468066}
Committed:
https://chromium.googlesource.com/chromium/src/+/90f1043d53ecd259599da1af4a17...
==========

[commit-bot: I haz the power, 2017-04-28 18:28:26]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/90f1043d53ecd259599da1af4a17...