Add viewport with assumption that browser controls always show

Add viewport with assumption that browser controls always show

This change adds a viewport and a new sizing flag to the layouts. Some
layouts require the size of the screen to assume the browser controls
are showing when they would otherwise not be. To use this viewport, a
new sizing flag has been added to clearly identify its use:
USE_PREVIOUS_TOOLBAR_STATE.

As a result, this fixes a sizing bug in SimpleAnimationLayout.

BUG=668643, 662427
Committed: https://crrev.com/fdf5564b291c19901f507ad181ad53cbfdac55d2
Cr-Commit-Position: refs/heads/master@{#435433}

[mdjones, 2016-11-29 01:06:44]

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

[mdjones, 2016-11-29 01:06:45]

The sizing flags need to be more clear in general, but I can do that in another
patch, ptal.

[David Trainor- moved to gerrit, 2016-11-30 18:09:38]

lgtm

[mdjones, 2016-11-30 19:39:36]

The CQ bit was checked by mdjones@chromium.org

[mdjones, 2016-11-30 19:39:37]

The patchset sent to the CQ was uploaded after l-g-t-m from
dtrainor@chromium.org
Link to the patchset: https://codereview.chromium.org/2536553005/#ps40001
(title: "fix bug on tablet")

[commit-bot: I haz the power, 2016-11-30 19:40:00]

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-30 20:06:57]

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

[commit-bot: I haz the power, 2016-11-30 20:06:58]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)

[mdjones, 2016-11-30 21:21:49]

The CQ bit was checked by mdjones@chromium.org

[mdjones, 2016-11-30 21:21:49]

The patchset sent to the CQ was uploaded after l-g-t-m from
dtrainor@chromium.org
Link to the patchset: https://codereview.chromium.org/2536553005/#ps60001
(title: "fix mock test class")

[commit-bot: I haz the power, 2016-11-30 21:22:32]

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-30 21:56:17]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1480540909408340,
"parent_rev": "9ea152e3cbadd1b3976f5c0cc8d14b0e570f02ea", "commit_rev":
"b33ee92f57beb22beedb128d5565256ab626d281"}

[commit-bot: I haz the power, 2016-11-30 21:56:47]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-11-30 21:59:11]

Description was changed from

==========
Add viewport with assumption that browser controls always show

This change adds a viewport and a new sizing flag to the layouts. Some
layouts require the size of the screen to assume the browser controls
are showing when they would otherwise not be. To use this viewport, a
new sizing flag has been added to clearly identify its use:
USE_PREVIOUS_TOOLBAR_STATE.

As a result, this fixes a sizing bug in SimpleAnimationLayout.

BUG=668643, 662427
==========

to

==========
Add viewport with assumption that browser controls always show

This change adds a viewport and a new sizing flag to the layouts. Some
layouts require the size of the screen to assume the browser controls
are showing when they would otherwise not be. To use this viewport, a
new sizing flag has been added to clearly identify its use:
USE_PREVIOUS_TOOLBAR_STATE.

As a result, this fixes a sizing bug in SimpleAnimationLayout.

BUG=668643, 662427

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

[commit-bot: I haz the power, 2016-11-30 21:59:13]

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