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

Unified Diff: ash/wm/base_layout_manager.cc

Issue 9515003: Fix the full screen switching browser window dipping below launcher bar issue. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix the issue for managed window mode, add tests. Created 8 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ash/wm/base_layout_manager.cc
diff --git a/ash/wm/base_layout_manager.cc b/ash/wm/base_layout_manager.cc
index e8a5014feaa51e42e9cf13a3552d83d27d20f4d1..11b24b89bf36ab27e93a86cf12a752cb670ae27d 100644
--- a/ash/wm/base_layout_manager.cc
+++ b/ash/wm/base_layout_manager.cc
@@ -67,10 +67,19 @@ void BaseLayoutManager::SetChildBounds(aura::Window* child,
// BaseLayoutManager, RootWindowObserver overrides:
void BaseLayoutManager::OnRootWindowResized(const gfx::Size& new_size) {
+ AdjustWindowSizesForScreenChange();
+}
+
+void BaseLayoutManager::OnScreenWorkAreaInsetsChanged() {
+ AdjustWindowSizesForScreenChange();
+}
+
+void BaseLayoutManager::AdjustWindowSizesForScreenChange() {
sky 2012/03/01 01:43:19 Order should match that of header.
jennyz 2012/03/01 18:27:45 Done.
// If a user plugs an external monitor into a laptop running Aura the
// monitor size will change. Maximized windows need to resize to match.
// We also do this when developers running Aura on a desktop manually resize
// the host window.
+ // We also need to do this when the work area insets changes.
for (WindowSet::const_iterator it = windows_.begin();
it != windows_.end();
++it) {

Powered by Google App Engine
This is Rietveld 408576698