TableView: 3-phase ToggleSortOrder (sorted/reversed/unsorted)

chrome/browser/ui/views/new_task_manager_view.cc

 // Copyright 2015 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/ui/views/new_task_manager_view.h"
 
 #include <stddef.h>
 
 #include "build/build_config.h"
 #include "chrome/browser/browser_process.h"
@@ skipping 110 matching lines @@
 
 void NewTaskManagerView::SetColumnVisibility(int column_id,
                                              bool new_visibility) {
   tab_table_->SetColumnVisibility(column_id, new_visibility);
 }
 
 bool NewTaskManagerView::IsTableSorted() const {
   return tab_table_->is_sorted();
 }
 
-TableSortDescriptor
-NewTaskManagerView::GetSortDescriptor() const {
+TableSortDescriptor NewTaskManagerView::GetSortDescriptor() const {
   if (!IsTableSorted())
     return TableSortDescriptor();
 
   const auto& descriptor = tab_table_->sort_descriptors().front();
   return TableSortDescriptor(descriptor.column_id, descriptor.ascending);
 }
 
-void NewTaskManagerView::ToggleSortOrder(int visible_column_index) {
-  tab_table_->ToggleSortOrder(visible_column_index);
+void NewTaskManagerView::SetSortDescriptor(TableSortDescriptor descriptor) {
+  views::TableView::SortDescriptors descriptor_list;
+
+  // If |sorted_column_id| is the default value, it means to clear the sort.
+  if (descriptor.sorted_column_id != TableSortDescriptor().sorted_column_id) {
+    descriptor_list.emplace_back(descriptor.sorted_column_id,
+                                 descriptor.is_ascending);
+  }
+
+  tab_table_->SetSortDescriptors(std::move(descriptor_list));
 }
 
 gfx::Size NewTaskManagerView::GetPreferredSize() const {
   return gfx::Size(460, 270);
 }
 
 bool NewTaskManagerView::AcceleratorPressed(
     const ui::Accelerator& accelerator) {
   DCHECK_EQ(ui::VKEY_W, accelerator.key_code());
   DCHECK_EQ(ui::EF_CONTROL_DOWN, accelerator.modifiers());
@@ skipping 202 matching lines @@
   if (!g_browser_process->local_state())
     return;
 
   const base::DictionaryValue* dictionary =
     g_browser_process->local_state()->GetDictionary(GetWindowName());
   if (dictionary)
     dictionary->GetBoolean("always_on_top", &is_always_on_top_);
 }
 
 }  // namespace task_management