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

Unified Diff: ash/wm/shelf_layout_manager.cc

Issue 10979090: Simplify WorkAreaBounds access logic (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Missed dead code Created 8 years, 2 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/shelf_layout_manager.cc
diff --git a/ash/wm/shelf_layout_manager.cc b/ash/wm/shelf_layout_manager.cc
index 75a53b7d6526de8ffae240049847e211174f6d21..de66906b19b4cb18dd1464a8340b00638c6be486 100644
--- a/ash/wm/shelf_layout_manager.cc
+++ b/ash/wm/shelf_layout_manager.cc
@@ -495,32 +495,6 @@ void ShelfLayoutManager::OnWindowActivated(aura::Window* active,
ShelfLayoutManager::TargetBounds::TargetBounds() : opacity(0.0f) {}
-gfx::Rect ShelfLayoutManager::GetMaximizedWindowBounds(
- aura::Window* window) {
- gfx::Rect bounds(ScreenAsh::GetDisplayBoundsInParent(window));
- if (auto_hide_behavior_ == SHELF_AUTO_HIDE_BEHAVIOR_ALWAYS) {
- AdjustBoundsBasedOnAlignment(kAutoHideSize, &bounds);
- return bounds;
- }
- // SHELF_AUTO_HIDE_BEHAVIOR_NEVER maximized windows don't get any taller.
- return GetUnmaximizedWorkAreaBounds(window);
-}
-
-gfx::Rect ShelfLayoutManager::GetUnmaximizedWorkAreaBounds(
- aura::Window* window) {
- gfx::Rect bounds(ScreenAsh::GetDisplayBoundsInParent(window));
- int size;
- if (auto_hide_behavior_ == SHELF_AUTO_HIDE_BEHAVIOR_ALWAYS) {
- size = kAutoHideSize;
- } else {
- int width, height;
- GetShelfSize(&width, &height);
- size = std::max(width, height);
- }
- AdjustBoundsBasedOnAlignment(size, &bounds);
- return bounds;
-}
-
void ShelfLayoutManager::SetState(VisibilityState visibility_state) {
ShellDelegate* delegate = Shell::GetInstance()->delegate();
State state;

Powered by Google App Engine
This is Rietveld 408576698