Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(162)

Issue 2623933002: Use full viewport height for layout if URL bar is locked shown or hidden. (Closed)

Created:
3 years, 11 months ago by bokan
Modified:
3 years, 11 months ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/2924
Project:
chromium
Visibility:
Public.

Description

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} (cherry picked from commit 18940be9f2a3912efbdbed71863a7749ed243e9a) Review-Url: https://codereview.chromium.org/2623933002 . Cr-Commit-Position: refs/branch-heads/2924@{#720} Cr-Branched-From: 3a87aecc31cd1ffe751dd72c04e5a96a1fc8108a-refs/heads/master@{#433059} Committed: https://chromium.googlesource.com/chromium/src/+/6db97b43abbc7b0fcbeae646f219836342eb9545

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+99 lines, -8 lines) Patch
M third_party/WebKit/Source/core/frame/BrowserControls.h View 1 chunk +2 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/web/WebViewImpl.h View 1 chunk +1 line, -2 lines 0 comments Download
M third_party/WebKit/Source/web/WebViewImpl.cpp View 4 chunks +16 lines, -6 lines 0 comments Download
M third_party/WebKit/Source/web/tests/BrowserControlsTest.cpp View 1 chunk +80 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
bokan
3 years, 11 months ago (2017-01-10 20:20:03 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
6db97b43abbc7b0fcbeae646f219836342eb9545.

Powered by Google App Engine
This is Rietveld 408576698