Implement new Task Manager and Edit Bookmark style, etc.

chrome/browser/ui/views/task_manager_view.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 "chrome/browser/task_manager/task_manager.h"
 
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/metrics/stats_table.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/memory_purger.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/prefs/scoped_user_pref_update.h"
+#include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/host_desktop.h"
 #include "chrome/browser/ui/views/browser_dialogs.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/pref_names.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
 #include "ui/base/accelerators/accelerator.h"
@@ skipping 12 matching lines @@
 #include "ui/views/controls/table/group_table_view.h"
 #include "ui/views/controls/table/table_view_observer.h"
 #include "ui/views/layout/layout_constants.h"
 #include "ui/views/widget/widget.h"
 #include "ui/views/window/dialog_delegate.h"
 
 #if defined(OS_WIN)
 #include "win8/util/win8_util.h"
 #endif
 
-// The task manager window default size.
-static const int kDefaultWidth = 460;
-static const int kDefaultHeight = 270;
-
 // Yellow highlight used when highlighting background resources.
 static const SkColor kBackgroundResourceHighlight =
     SkColorSetRGB(0xff, 0xf1, 0xcd);
 
 namespace {
 
 ////////////////////////////////////////////////////////////////////////////////
 // TaskManagerTableModel class
 ////////////////////////////////////////////////////////////////////////////////
 
@@ skipping 147 matching lines @@
                         public views::ContextMenuController,
                         public ui::SimpleMenuModel::Delegate {
  public:
   TaskManagerView(bool highlight_background_resources,
                   chrome::HostDesktopType desktop_type);
   virtual ~TaskManagerView();
 
   // Shows the Task manager window, or re-activates an existing one. If
   // |highlight_background_resources| is true, highlights the background
   // resources in the resource display.
-  static void Show(bool highlight_background_resources,
-                   chrome::HostDesktopType desktop_type);
+  static void Show(bool highlight_background_resources, Browser* browser);
 
   // views::View:
   virtual void Layout() OVERRIDE;
   virtual gfx::Size GetPreferredSize() OVERRIDE;
   virtual void ViewHierarchyChanged(bool is_add, views::View* parent,
                                     views::View* child) OVERRIDE;
 
   // views::ButtonListener:
   virtual void ButtonPressed(views::Button* sender,
                              const ui::Event& event) OVERRIDE;
@@ skipping 248 matching lines @@
     } else {
       parent->RemoveChildView(kill_button_);
       if (purge_memory_button_)
         parent->RemoveChildView(purge_memory_button_);
       parent->RemoveChildView(about_memory_link_);
     }
   }
 }
 
 void TaskManagerView::Layout() {
+  bool new_style = views::DialogDelegate::UseNewStyle();
   gfx::Size size = kill_button_->GetPreferredSize();
-  int prefered_width = size.width();
-  int prefered_height = size.height();
-
-  tab_table_->SetBounds(
-      x() + views::kPanelHorizMargin,
-      y() + views::kPanelVertMargin,
-      width() - 2 * views::kPanelHorizMargin,
-      height() - 2 * views::kPanelVertMargin - prefered_height);
-
-  // y-coordinate of button top left.
   gfx::Rect parent_bounds = parent()->GetContentsBounds();
-  int y_buttons =
-      parent_bounds.bottom() - prefered_height - views::kButtonVEdgeMargin;
-
-  kill_button_->SetBounds(
-      x() + width() - prefered_width - views::kPanelHorizMargin,
-      y_buttons,
-      prefered_width,
-      prefered_height);
+  int x = width() - size.width() - (new_style ? 0 : views::kPanelHorizMargin);
+  int y_buttons = new_style ? GetLocalBounds().bottom() - size.height() :
+      parent_bounds.bottom() - size.height() - views::kButtonVEdgeMargin;
+  kill_button_->SetBounds(x, y_buttons, size.width(), size.height());
 
   if (purge_memory_button_) {
     size = purge_memory_button_->GetPreferredSize();
     purge_memory_button_->SetBounds(
         kill_button_->x() - size.width() -
             views::kUnrelatedControlHorizontalSpacing,
         y_buttons, size.width(), size.height());
   }
 
   size = about_memory_link_->GetPreferredSize();
-  int link_prefered_width = size.width();
-  int link_prefered_height = size.height();
-  // center between the two buttons horizontally, and line up with
-  // bottom of buttons vertically.
-  int link_y_offset = std::max(0, prefered_height - link_prefered_height) / 2;
-  about_memory_link_->SetBounds(
-      x() + views::kPanelHorizMargin,
-      y_buttons + prefered_height - link_prefered_height - link_y_offset,
-      link_prefered_width,
-      link_prefered_height);
+  about_memory_link_->SetBounds(new_style ? 0 : views::kPanelHorizMargin,
+      y_buttons + (kill_button_->height() - size.height()) / 2,
+      size.width(), size.height());
+
+  gfx::Rect rect = GetLocalBounds();
+  if (!new_style)
+    rect.Inset(views::kPanelHorizMargin, views::kPanelVertMargin);
+  rect.Inset(0, 0, 0,
+             kill_button_->height() + views::kUnrelatedControlVerticalSpacing);
+  tab_table_->SetBoundsRect(rect);
 }
 
 gfx::Size TaskManagerView::GetPreferredSize() {
-  return gfx::Size(kDefaultWidth, kDefaultHeight);
+  return gfx::Size(460, 270);
 }
 
 // static
 void TaskManagerView::Show(bool highlight_background_resources,
-                           chrome::HostDesktopType desktop_type) {
+                           Browser* browser) {
 #if defined(OS_WIN)
   // In Windows Metro it's not good to open this native window.
   DCHECK(!win8::IsSingleWindowMetroMode());
 #endif
+  const chrome::HostDesktopType desktop_type = browser->host_desktop_type();
 
   if (instance_) {
     if (instance_->highlight_background_resources_ !=
         highlight_background_resources ||
         instance_->desktop_type_ != desktop_type) {
       instance_->GetWidget()->Close();
     } else {
       // If there's a Task manager window open already, just activate it.
       instance_->GetWidget()->Activate();
       return;
     }
   }
   instance_ = new TaskManagerView(highlight_background_resources, desktop_type);
-
-  views::Widget* widget = new views::Widget;
-  views::Widget::InitParams params;
-  params.delegate = instance_;
-  if (DialogDelegate::UseNewStyle()) {
-    // TODO(msw): Use table_view_views or support dialog transparency and native
-    // controls (e.g. table_view_win) via a second dialog Widget (like bubbles).
-    // params.transparent = true;
-    params.remove_standard_frame = true;
-  }
-  params.top_level = true;
-  widget->Init(params);
-
+  DialogDelegateView::CreateDialogWidget(instance_,
+      browser->window()->GetNativeWindow(), NULL);
   instance_->InitAlwaysOnTopState();
   instance_->model_->StartUpdating();
   instance_->GetWidget()->Show();
 
   // Set the initial focus to the list of tasks.
   views::FocusManager* focus_manager = instance_->GetFocusManager();
   if (focus_manager)
     focus_manager->SetFocusedView(instance_->tab_table_);
 }
 
@@ skipping 196 matching lines @@
       g_browser_process->local_state()->GetDictionary(GetWindowName().c_str());
   return dictionary &&
       dictionary->GetBoolean("always_on_top", always_on_top) && always_on_top;
 }
 
 }  // namespace
 
 namespace chrome {
 
 // Declared in browser_dialogs.h so others don't need to depend on our header.
-void ShowTaskManager(chrome::HostDesktopType desktop_type) {
-  TaskManagerView::Show(false, desktop_type);
+void ShowTaskManager(Browser* browser) {
+  TaskManagerView::Show(false, browser);
 }
 
-void ShowBackgroundPages(chrome::HostDesktopType desktop_type) {
-  TaskManagerView::Show(true, desktop_type);
+void ShowBackgroundPages(Browser* browser) {
+  TaskManagerView::Show(true, browser);
 }
 
 }  // namespace chrome