Task Manager Should remember the most recently enabled columns.

chrome/browser/ui/views/new_task_manager_view.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_VIEWS_NEW_TASK_MANAGER_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_NEW_TASK_MANAGER_VIEW_H_
 
 #include <vector>
 
 #include "chrome/browser/ui/host_desktop.h"
@@ skipping 83 matching lines @@
 
   // Initializes the state of the always-on-top setting as the window is shown.
   void InitAlwaysOnTopState();
 
   // Activates the tab associated with the focused row.
   void ActivateFocusedTab();
 
   // Restores saved "always on top" state from a previous session.
   void RetriveSavedAlwaysOnTopState();
 
+  // Restores the saved columns settings from a previous session.
+  void RetrieveSavedColumnsSettings();
+
+  // Updates the table view based on the values currently  stored in

[Lei Zhang, 2015/08/28 22:47:45]

nit: extra space before "stored"

[afakhry, 2015/08/29 00:30:30]

Done.

+  // |columns_settings_|.
+  void UpdateTableFromColumnsSettings();
+
+  // Stores the current values in |column_settings_| to the user prefs so that
+  // it can be restored later next time the task manager view is opened.
+  void StoreColumnsSettings();
+
   scoped_ptr<NewTaskManagerView::TableModel> table_model_;
 
   scoped_ptr<views::MenuRunner> menu_runner_;
 
+  // Contains either the retrieved from user preferences columns settings if

[Lei Zhang, 2015/08/28 22:47:45]

I have a bit of trouble parsing this sentence. How about:

Contains either the column settings retrieved from user preferences if it
exists, or the default column settings.

[afakhry, 2015/08/29 00:30:30]

Done.

+  // any, or it will contain the default columns settings.
+  // The columns settings are the visible columns and the last sorted column
+  // and the direction of the sort.
+  scoped_ptr<base::DictionaryValue> columns_settings_;
+
   // We need to own the text of the menu, the Windows API does not copy it.
   base::string16 always_on_top_menu_text_;
 
   views::LabelButton* kill_button_;
   views::Link* about_memory_link_;
   views::TableView* tab_table_;
   views::View* tab_table_parent_;
 
   // all possible columns, not necessarily visible
   std::vector<ui::TableColumn> columns_;
 
   // The host desktop type this task manager belongs to.
   const chrome::HostDesktopType desktop_type_;
 
   // True when the Task Manager window should be shown on top of other windows.
   bool is_always_on_top_;
 
   DISALLOW_COPY_AND_ASSIGN(NewTaskManagerView);
 };
 
 }  // namespace task_management
 
 #endif  // CHROME_BROWSER_UI_VIEWS_NEW_TASK_MANAGER_VIEW_H_