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

Unified Diff: ui/aura_shell/status_area_layout_manager.h

Issue 8743014: [cros, Aura] Refresh status area widget bounds on StatusAreaView layout. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: nit fixes, merge, prevent double deletion of ShelfLayoutManager Created 9 years 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
« no previous file with comments | « ui/aura_shell/shell.cc ('k') | ui/aura_shell/status_area_layout_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/aura_shell/status_area_layout_manager.h
diff --git a/ui/aura_shell/desktop_layout_manager.h b/ui/aura_shell/status_area_layout_manager.h
similarity index 50%
copy from ui/aura_shell/desktop_layout_manager.h
copy to ui/aura_shell/status_area_layout_manager.h
index 7af3039e4bf303cf539c81533bf7ca969d3d4b05..a5984b0beed42fc437d183705e6a873b156b06e8 100644
--- a/ui/aura_shell/desktop_layout_manager.h
+++ b/ui/aura_shell/status_area_layout_manager.h
@@ -2,41 +2,26 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef UI_AURA_SHELL_DESKTOP_LAYOUT_MANAGER_H_
-#define UI_AURA_SHELL_DESKTOP_LAYOUT_MANAGER_H_
+#ifndef UI_AURA_SHELL_STATUS_AREA_LAYOUT_MANAGER_H_
+#define UI_AURA_SHELL_STATUS_AREA_LAYOUT_MANAGER_H_
#pragma once
#include "base/basictypes.h"
#include "base/compiler_specific.h"
#include "ui/aura/layout_manager.h"
-namespace aura {
-class Window;
-}
-namespace gfx {
-class Rect;
-}
-namespace views {
-class Widget;
-}
-
namespace aura_shell {
namespace internal {
-class ShelfLayoutController;
+class ShelfLayoutManager;
-// A layout manager for the root window.
-// Resizes all of its immediate children to fill the bounds of the root window.
-class DesktopLayoutManager : public aura::LayoutManager {
+// StatusAreaLayoutManager is a layout manager responsible for the status area.
+// In any case when status area needs relayout it redirects this call to
+// ShelfLayoutManager.
+class StatusAreaLayoutManager : public aura::LayoutManager {
public:
- explicit DesktopLayoutManager(aura::Window* owner);
- virtual ~DesktopLayoutManager();
-
- void set_shelf(ShelfLayoutController* shelf) { shelf_ = shelf; }
-
- void set_background_widget(views::Widget* background_widget) {
- background_widget_ = background_widget;
- }
+ explicit StatusAreaLayoutManager(ShelfLayoutManager* shelf);
+ virtual ~StatusAreaLayoutManager();
// Overridden from aura::LayoutManager:
virtual void OnWindowResized() OVERRIDE;
@@ -48,16 +33,20 @@ class DesktopLayoutManager : public aura::LayoutManager {
const gfx::Rect& requested_bounds) OVERRIDE;
private:
- aura::Window* owner_;
+ // Updates layout of the status area. Effectively calls ShelfLayoutManager
+ // to update layout of the shelf.
+ void LayoutStatusArea();
- views::Widget* background_widget_;
+ // True when inside LayoutStatusArea method.
+ // Used to prevent calling itself again from SetChildBounds().
+ bool in_layout_;
- ShelfLayoutController* shelf_;
+ ShelfLayoutManager* shelf_;
- DISALLOW_COPY_AND_ASSIGN(DesktopLayoutManager);
+ DISALLOW_COPY_AND_ASSIGN(StatusAreaLayoutManager);
};
} // namespace internal
} // namespace aura_shell
-#endif // UI_AURA_SHELL_DESKTOP_LAYOUT_MANAGER_H_
+#endif // UI_AURA_SHELL_STATUS_AREA_LAYOUT_MANAGER_H_
« no previous file with comments | « ui/aura_shell/shell.cc ('k') | ui/aura_shell/status_area_layout_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698