Make Task Manager sort more meaningful

chrome/browser/ui/task_manager/task_manager_table_model.cc

Index: chrome/browser/ui/task_manager/task_manager_table_model.cc
diff --git a/chrome/browser/ui/task_manager/task_manager_table_model.cc b/chrome/browser/ui/task_manager/task_manager_table_model.cc
index 075e2e23eee318cb6b38105d38447a5b90cd0bfb..319591422e18d81ea62b45143304a59eb179ad2d 100644
--- a/chrome/browser/ui/task_manager/task_manager_table_model.cc
+++ b/chrome/browser/ui/task_manager/task_manager_table_model.cc
@@ -421,6 +421,8 @@ int TaskManagerTableModel::CompareValues(int row1,
                                          int column_id) {
   switch (column_id) {
     case IDS_TASK_MANAGER_TASK_COLUMN:
+      // Sorting by IDS_TASK_MANAGER_TASK_COLUMN gives the model order.
+      return ValueCompare(row1, row2);

[afakhry, 2016/06/03 17:53:42]

Be prepared for a storm of regression bugs saying "sorting by tasks doesn't sort
alphabetically". Have you talked to UX about this?

[ncarter (slow), 2016/06/20 17:40:00]

I'll start a thread with UI folks. We definitely have a few options for ways to
expose the logical sorting.

In the mean time, I've removed this part of the diff (I think the tests will
still pass). We can expose this to the UI as a separate CL.

[afakhry, 2016/06/22 13:49:46]

Acknowledged.

     case IDS_TASK_MANAGER_PROFILE_NAME_COLUMN:
       return ui::TableModel::CompareValues(row1, row2, column_id);