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

Unified Diff: chrome/browser/task_manager.cc

Issue 1822: Task Manager Double Click (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 12 years, 3 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
« no previous file with comments | « chrome/browser/task_manager.h ('k') | chrome/browser/task_manager_resource_providers.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/task_manager.cc
===================================================================
--- chrome/browser/task_manager.cc (revision 1649)
+++ chrome/browser/task_manager.cc (working copy)
@@ -538,12 +538,15 @@
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);
@@ -748,6 +751,16 @@
}
}
+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_)
@@ -760,6 +773,15 @@
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) {
@@ -853,6 +875,34 @@
}
}
+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);
}
« no previous file with comments | « chrome/browser/task_manager.h ('k') | chrome/browser/task_manager_resource_providers.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698