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

chrome/browser/ui/task_manager/task_manager_table_model.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_TASK_MANAGER_TASK_MANAGER_TABLE_MODEL_H_
 #define CHROME_BROWSER_UI_TASK_MANAGER_TASK_MANAGER_TABLE_MODEL_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 27 matching lines @@
   virtual ~TableViewDelegate() {}
 
   virtual bool IsColumnVisible(int column_id) const = 0;
 
   virtual void SetColumnVisibility(int column_id, bool new_visibility) = 0;
 
   virtual bool IsTableSorted() const = 0;
 
   virtual TableSortDescriptor GetSortDescriptor() const = 0;
 
-  // As the name of |visible_column_index| implies, it must by the index of the
-  // column in the visible columns array.
-  virtual void ToggleSortOrder(int visible_column_index) = 0;
+  virtual void SetSortDescriptor(TableSortDescriptor sort_descriptor) = 0;

[sky, 2016/07/18 16:18:58]

const TableSortDescriptor& ?

[ncarter (slow), 2016/07/25 20:04:01]

Done.

 
  private:
   DISALLOW_COPY_AND_ASSIGN(TableViewDelegate);
 };
 
 class TaskManagerTableModel
     : public TaskManagerObserver,
       public ui::TableModel {
  public:
   TaskManagerTableModel(int64_t refresh_flags, TableViewDelegate* delegate);
@@ skipping 75 matching lines @@
 
   // The status of the flag #enable-nacl-debug.
   bool is_nacl_debugging_flag_enabled_;
 
   DISALLOW_COPY_AND_ASSIGN(TaskManagerTableModel);
 };
 
 }  // namespace task_management
 
 #endif  // CHROME_BROWSER_UI_TASK_MANAGER_TASK_MANAGER_TABLE_MODEL_H_