Double clicking on an item in the task manager should bring the relevant tab forward.

Review URL: http://codereview.chromium.org/1822

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@2026 0039d316-1c4b-4281-b951-d872f2087c98

4 files changed, 70 insertions, 0 deletions

diff --git a/chrome/browser/task_manager.cc b/chrome/browser/task_manager.cc
index 7cb75ff..5d9b8f0 100644
--- a/chrome/browser/task_manager.cc
+++ b/chrome/browser/task_manager.cc
@@ -511,12 +511,15 @@ class TaskManagerContents : public ChromeViews::View,
   virtual void ViewHierarchyChanged(bool is_add, ChromeViews::View* parent,
                                     ChromeViews::View* child);
   void GetSelection(std::vector<int>* selection);
+  void GetFocused(std::vector<int>* focused);
 
   // NativeButton::Listener implementation.
   virtual void ButtonPressed(ChromeViews::NativeButton* sender);
 
   // ChromeViews::TableViewObserver implementation.
   virtual void OnSelectionChanged();
+  virtual void OnDoubleClick();
+  virtual void OnKeyDown(unsigned short virtual_keycode);
 
   // ChromeViews::LinkController implementation.
   virtual void LinkActivated(ChromeViews::Link* source, int event_flags);
@@ -721,6 +724,16 @@ void TaskManagerContents::GetSelection(std::vector<int>* selection) {
   }
 }
 
+void TaskManagerContents::GetFocused(std::vector<int>* focused) {
+  DCHECK(focused);
+  int row_count = tab_table_->RowCount();
+  for (int i = 0; i < row_count; ++i) {
+    // The TableView returns the selection starting from the end.
+    if (tab_table_->ItemHasTheFocus(i))
+      focused->insert(focused->begin(), i);
+  }
+}
+
 // NativeButton::Listener implementation.
 void TaskManagerContents::ButtonPressed(ChromeViews::NativeButton* sender) {
   if (sender == kill_button_)
@@ -733,6 +746,15 @@ void TaskManagerContents::OnSelectionChanged() {
                            tab_table_->SelectedRowCount() > 0);
 }
 
+void TaskManagerContents::OnDoubleClick() {
+  task_manager_->ActivateFocusedTab();
+}
+
+void TaskManagerContents::OnKeyDown(unsigned short virtual_keycode) {
+  if (virtual_keycode == VK_RETURN)
+    task_manager_->ActivateFocusedTab();
+}
+
 // ChromeViews::LinkController implementation
 void TaskManagerContents::LinkActivated(ChromeViews::Link* source,
                                         int event_flags) {
@@ -826,6 +848,34 @@ void TaskManager::KillSelectedProcesses() {
   }
 }
 
+void TaskManager::ActivateFocusedTab() {
+  std::vector<int> focused;
+  contents_->GetFocused(&focused);
+  int focused_size = static_cast<int>(focused.size());
+
+  DCHECK(focused_size == 1);
+
+  // Gracefully return if there is not exactly one item in focus.
+  if (focused_size != 1)
+    return;
+
+  // Otherwise, the one focused thing should be one the user intends to bring
+  // forth, so get see if GetTabContents returns non-null.  If it does, activate
+  // those contents.
+  int index = focused[0];
+
+  // GetTabContents returns a pointer to the relevant tab contents for the
+  // resource.  If the index doesn't correspond to a Tab (i.e. refers to the
+  // Browser process or a plugin), GetTabContents will return NULL.
+  TabContents* chosen_tab_contents =
+      table_model_->resources_[index]->GetTabContents();
+
+  if (!chosen_tab_contents)
+    return;
+
+  chosen_tab_contents->Activate();
+}
+
 void TaskManager::AddResourceProvider(ResourceProvider* provider) {
   table_model_->AddResourceProvider(provider);
 }
diff --git a/chrome/browser/task_manager.h b/chrome/browser/task_manager.h
index 9b8687f..d35b1c2 100644
--- a/chrome/browser/task_manager.h
+++ b/chrome/browser/task_manager.h
@@ -5,6 +5,9 @@
 #ifndef CHROME_BROWSER_TASK_MANAGER_H__
 #define CHROME_BROWSER_TASK_MANAGER_H__
 
+#include <map>
+#include <vector>
+
 #include "base/lock.h"
 #include "base/singleton.h"
 #include "base/ref_counted.h"
@@ -12,6 +15,7 @@
 #include "chrome/views/dialog_delegate.h"
 #include "chrome/views/group_table_view.h"
 #include "chrome/browser/cache_manager_host.h"
+#include "chrome/browser/tab_contents.h"
 #include "net/url_request/url_request_job_tracker.h"
 
 class MessageLoop;
@@ -42,10 +46,16 @@ class TaskManager : public ChromeViews::DialogDelegate {
   // Resources from similar processes are grouped together by the task manager.
   class Resource {
    public:
+    virtual ~Resource() {}
+
     virtual std::wstring GetTitle() const = 0;
     virtual SkBitmap GetIcon() const = 0;
     virtual HANDLE GetProcess() const = 0;
 
+    // A helper function for ActivateFocusedTab.  Returns NULL by default
+    // because not all resources have an assoiciated tab.
+    virtual TabContents* GetTabContents() const {return NULL;}
+
     // Whether this resource does report the network usage accurately.
     // This controls whether 0 or N/A is displayed when no bytes have been
     // reported as being read. This is because some plugins do not report the
@@ -99,6 +109,10 @@ class TaskManager : public ChromeViews::DialogDelegate {
   // Terminates the selected tab(s) in the list.
   void KillSelectedProcesses();
 
+  // Activates the browser tab associated with the focused row in the task
+  // manager table.  This happens when the user double clicks or hits return.
+  void ActivateFocusedTab();
+
   void AddResourceProvider(ResourceProvider* provider);
   void RemoveResourceProvider(ResourceProvider* provider);
 
diff --git a/chrome/browser/task_manager_resource_providers.cc b/chrome/browser/task_manager_resource_providers.cc
index ad34101..b3fe407 100644
--- a/chrome/browser/task_manager_resource_providers.cc
+++ b/chrome/browser/task_manager_resource_providers.cc
@@ -62,6 +62,10 @@ HANDLE TaskManagerWebContentsResource::GetProcess() const {
   return process_;
 }
 
+TabContents* TaskManagerWebContentsResource::GetTabContents() const {
+  return dynamic_cast<TabContents*>(web_contents_);
+}
+
 ////////////////////////////////////////////////////////////////////////////////
 // TaskManagerWebContentsResourceProvider class
 ////////////////////////////////////////////////////////////////////////////////
diff --git a/chrome/browser/task_manager_resource_providers.h b/chrome/browser/task_manager_resource_providers.h
index cd300f8..bece876 100644
--- a/chrome/browser/task_manager_resource_providers.h
+++ b/chrome/browser/task_manager_resource_providers.h
@@ -24,6 +24,8 @@ class TaskManagerWebContentsResource : public TaskManager::Resource {
   std::wstring GetTitle() const;
   SkBitmap GetIcon() const;
   HANDLE GetProcess() const;
+  TabContents* GetTabContents() const;
+
   // WebContents always provide the network usage.
   bool SupportNetworkUsage() const { return true; }
   void SetSupportNetworkUsage() { };