diff options
| author | pkotwicz@chromium.org <pkotwicz@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-05-06 22:54:12 +0000 |
|---|---|---|
| committer | pkotwicz@chromium.org <pkotwicz@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-05-06 22:54:12 +0000 |
| commit | 76ff1e3601252b13d0ad4b6521fbe64259fcf7fc (patch) | |
| tree | fdfa22600a391b9159ae407978c188a286eaa0b3 /media/base/media_stub.cc | |
| parent | a21c13963d75ea8d85237fd768db7c08d4417d45 (diff) | |
| download | chromium_src-76ff1e3601252b13d0ad4b6521fbe64259fcf7fc.zip chromium_src-76ff1e3601252b13d0ad4b6521fbe64259fcf7fc.tar.gz chromium_src-76ff1e3601252b13d0ad4b6521fbe64259fcf7fc.tar.bz2 | |
Fix the size of the tabs when exiting immersive fullscreen with the tab indicators hidden.
The following occurs when exiting immersive fullscreen, in order:
1) The aura::Window's kShowStateKey is updated. This causes a layout of the BrowserView's non client view. (Note that BrowserView::ProcessFullscreen() does not disable laying out the non client view.)
2) Immersive fullscreen is disabled
3) BrowserView::Layout()
This CL fixes a bug where the y position of the browser view in its parent is incorrect due to BrowserView::IsFullscreen() returning false and ImmersiveModeController::IsEnabled() returning true during step 1.
BUG=233271
Test=Manual, see repro steps below
1) Run chrome with --ash-immersive-hide-tab-indicators --ash-enable-immersive-fullscreen
2) Use F4 to enter immersive fullscreen
3) With the top-of-window views hidden, hit F4 to exit immersive fullscreen
4) The tab strip should be properly visible
Review URL: https://chromiumcodereview.appspot.com/14634003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@198558 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'media/base/media_stub.cc')
0 files changed, 0 insertions, 0 deletions