Use full viewport height for layout if URL bar is locked shown or hidden.

Use full viewport height for layout if URL bar is locked shown or hidden.

With inert-top-controls turned on by default, we don't change the layout height
when the top controls are normally hidden. That is, if a user hides the URL bar
by scrolling, the page won't resize to use the newly available space. It'll lay
out as though the URL bar was still showing.

This patch makes an exception for cases where Chrome locks the URL bar in the
hidden state. e.g. Fullscreen or WebApp mode. In those cases, since the URL bar
can't be shown at all, there's no reason not to use the full height.

BUG=678649
Review-Url: https://codereview.chromium.org/2614993002
Cr-Commit-Position: refs/heads/master@{#441995}
Committed: https://chromium.googlesource.com/chromium/src/+/18940be9f2a3912efbdbed71863a7749ed243e9a

[bokan, 2017-01-05 15:16:33]

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

[commit-bot: I haz the power, 2017-01-05 15:16:43]

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

[bokan, 2017-01-05 16:38:55]

bokan@chromium.org changed reviewers:
+ aelias@chromium.org

[bokan, 2017-01-05 16:38:56]

ptal

[commit-bot: I haz the power, 2017-01-05 17:03:34]

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

[commit-bot: I haz the power, 2017-01-05 17:03:36]

Dry run: This issue passed the CQ dry run.

[bokan, 2017-01-05 17:16:08]

Description was changed from

==========
Use full viewport height for layout if URL bar is locked shown or hidden.

With inert-top-controls turned on by default, we don't change the layout height
when the top controls are normally hidden. That is, if a user hides the URL bar
by scrolling, the page won't resize to use the newly available space. It'll lay
out as though the URL bar was still showing.

This patch makes an exception for cases where Chrome locks the URL bar in the
hidden state. e.g. Fullscreen or WebApp mode. In those cases, since the URL bar
can't be shown at all, there's no reason not to use the full height.

BUG=676634
==========

to

==========
Use full viewport height for layout if URL bar is locked shown or hidden.

With inert-top-controls turned on by default, we don't change the layout height
when the top controls are normally hidden. That is, if a user hides the URL bar
by scrolling, the page won't resize to use the newly available space. It'll lay
out as though the URL bar was still showing.

This patch makes an exception for cases where Chrome locks the URL bar in the
hidden state. e.g. Fullscreen or WebApp mode. In those cases, since the URL bar
can't be shown at all, there's no reason not to use the full height.

BUG=678649
==========

[aelias_OOO_until_Jul13, 2017-01-06 02:00:05]

lgtm

[bokan, 2017-01-06 15:40:38]

The CQ bit was checked by bokan@chromium.org

[commit-bot: I haz the power, 2017-01-06 15:40:53]

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-01-06 18:53:25]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483717238483810,
"parent_rev": "a8ac477c3ad2abf257520f22bda13163a763f15d", "commit_rev":
"18940be9f2a3912efbdbed71863a7749ed243e9a"}

[commit-bot: I haz the power, 2017-01-06 18:53:50]

Description was changed from

==========
Use full viewport height for layout if URL bar is locked shown or hidden.

With inert-top-controls turned on by default, we don't change the layout height
when the top controls are normally hidden. That is, if a user hides the URL bar
by scrolling, the page won't resize to use the newly available space. It'll lay
out as though the URL bar was still showing.

This patch makes an exception for cases where Chrome locks the URL bar in the
hidden state. e.g. Fullscreen or WebApp mode. In those cases, since the URL bar
can't be shown at all, there's no reason not to use the full height.

BUG=678649
==========

to

==========
Use full viewport height for layout if URL bar is locked shown or hidden.

With inert-top-controls turned on by default, we don't change the layout height
when the top controls are normally hidden. That is, if a user hides the URL bar
by scrolling, the page won't resize to use the newly available space. It'll lay
out as though the URL bar was still showing.

This patch makes an exception for cases where Chrome locks the URL bar in the
hidden state. e.g. Fullscreen or WebApp mode. In those cases, since the URL bar
can't be shown at all, there's no reason not to use the full height.

BUG=678649

Review-Url: https://codereview.chromium.org/2614993002
Cr-Commit-Position: refs/heads/master@{#441995}
Committed:
https://chromium.googlesource.com/chromium/src/+/18940be9f2a3912efbdbed71863a...
==========

[commit-bot: I haz the power, 2017-01-06 18:53:50]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/18940be9f2a3912efbdbed71863a...

[bokan, 2017-01-27 15:28:01]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2657773005/ by bokan@chromium.org.

The reason for reverting is: Temporarily reverting to see if it fixes the memory
regression in the linked bug.
BUG=679546.