Layout panels on top of their launcher icons

ash/wm/panel_layout_manager.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ash/wm/panel_layout_manager.h"
 
 #include <algorithm>
 
 #include "ash/launcher/launcher.h"
+#include "ash/launcher/launcher_model.h"
 #include "ash/shell.h"
 #include "ash/wm/property_util.h"
 #include "base/auto_reset.h"
 #include "ui/aura/client/aura_constants.h"
 #include "ui/aura/root_window.h"
 #include "ui/aura/window.h"
 #include "ui/gfx/rect.h"
 #include "ui/gfx/size.h"
 #include "ui/views/widget/widget.h"
 
 namespace {
 const int kPanelMarginEdge = 4;
 const int kPanelMarginMiddle = 8;
 
 const int kMinimizedHeight = 24;
 
 const float kMaxHeightFactor = .80f;
 const float kMaxWidthFactor = .50f;
 }
 
 namespace ash {
 namespace internal {
 
 ////////////////////////////////////////////////////////////////////////////////
 // PanelLayoutManager public implementation:
 
 PanelLayoutManager::PanelLayoutManager(aura::Window* panel_container)
     : panel_container_(panel_container),
       in_layout_(false),
-      dragged_panel_(NULL) {
+      dragged_panel_(NULL),
+      ALLOW_THIS_IN_INITIALIZER_LIST(launcher_icons_observer_(this)){
   DCHECK(panel_container);
 }
 
 PanelLayoutManager::~PanelLayoutManager() {
+  ash::Launcher* launcher = ash::Shell::GetInstance()->launcher();

[sky, 2012/03/23 21:41:14]

Did you make sure this is deleted before the launcher?

[Dmitry Lomov (no reviews), 2012/03/23 22:34:38]

Hopefully the getter will return null? 

+  if (launcher != NULL)
+    launcher->RemoveIconsObserver(&launcher_icons_observer_);
 }
 
 void PanelLayoutManager::StartDragging(aura::Window* panel) {
   DCHECK(dragged_panel_ == NULL);
   DCHECK(panel->parent() == panel_container_);
   dragged_panel_ = panel;
 }
 
 void PanelLayoutManager::FinishDragging() {
   DCHECK(dragged_panel_ != NULL);
   dragged_panel_ = NULL;
   Relayout();
 }
 
+void PanelLayoutManager::SetLauncher(ash::Launcher* launcher) {
+  launcher->AddIconsObserver(&launcher_icons_observer_);
+}
+
 void PanelLayoutManager::ToggleMinimize(aura::Window* panel) {
   DCHECK(panel->parent() == panel_container_);
   if (panel->GetProperty(aura::client::kShowStateKey) ==
       ui::SHOW_STATE_MINIMIZED) {
     const gfx::Rect& old_bounds = panel->bounds();
     panel->SetProperty(aura::client::kShowStateKey, ui::SHOW_STATE_NORMAL);
 
     gfx::Rect new_bounds(old_bounds);
     const gfx::Rect* restore_bounds = GetRestoreBounds(panel);
     if (restore_bounds != NULL) {
@@ skipping 79 matching lines @@
 
 ////////////////////////////////////////////////////////////////////////////////
 // PanelLayoutManager private implementation:
 
 // This is a rough outline of a simple panel layout manager.
 void PanelLayoutManager::Relayout() {
   if (in_layout_)
     return;
   AutoReset<bool> auto_reset_in_layout(&in_layout_, true);
 
-  // Panels are currently laid out just above the launcher (if it exists),
-  // otherwise at the bottom of the root window.
-  int right, bottom;
   ash::Shell* shell = ash::Shell::GetInstance();
-  if (shell->launcher() && shell->launcher()->widget()->IsVisible()) {
-    const gfx::Rect& bounds =
-        shell->launcher()->widget()->GetWindowScreenBounds();
-    right = bounds.width() - 1 - kPanelMarginEdge;
-    bottom = bounds.y() - 1;
-  } else {
-    const gfx::Rect& bounds = panel_container_->GetRootWindow()->bounds();
-    right = bounds.width() - 1 - kPanelMarginEdge;
-    bottom = bounds.bottom() - 1;
-  }
 
-  // Layout the panel windows right to left.
   for (PanelList::iterator iter = panel_windows_.begin();
        iter != panel_windows_.end(); ++iter) {
     aura::Window* panel_win = *iter;
+    gfx::Rect icon_rect =
+        shell->launcher()->GetScreenBoundsOfItemIconForWindow(panel_win);

[sky, 2012/03/23 21:41:14]

This may be empty.

[Dmitry Lomov (no reviews), 2012/03/23 22:34:38]

True. The layout manager has no idea what to do in this case :) 

+    gfx::Point icon_origin = icon_rect.origin();
+    aura::Window::ConvertPointToWindow(panel_container_->GetRootWindow(),
+                                       panel_container_, &icon_origin);
+
     if (!panel_win->IsVisible())
       continue;
-    int x = right - panel_win->bounds().width();
-    int y = bottom - panel_win->bounds().height();
 
     // Do not relayout dragged panel, but pretend it is in place
     if (panel_win != dragged_panel_) {
-      gfx::Rect bounds(x, y,
-                       panel_win->bounds().width(),
-                       panel_win->bounds().height());
+      gfx::Rect current_bounds = panel_win->bounds();
+
+      gfx::Rect bounds(
+          icon_origin.x() + icon_rect.width()/2 - current_bounds.width()/2,
+          icon_origin.y() - current_bounds.height(),
+          current_bounds.width(),
+          current_bounds.height());
       SetChildBoundsDirect(panel_win, bounds);
     }
-    right = x - kPanelMarginMiddle;
   }
 }
 
 }  // namespace internal
 }  // namespace ash