Task Manager Should remember the most recently enabled columns.

chrome/browser/ui/views/new_task_manager_view.h

Index: chrome/browser/ui/views/new_task_manager_view.h
diff --git a/chrome/browser/ui/views/new_task_manager_view.h b/chrome/browser/ui/views/new_task_manager_view.h
index 2d4d03884030377fc9ba756eb30c3e3956af58c8..65d2616ec2e0b844c431c7ca0796ddee75307f39 100644
--- a/chrome/browser/ui/views/new_task_manager_view.h
+++ b/chrome/browser/ui/views/new_task_manager_view.h
@@ -101,10 +101,25 @@ class NewTaskManagerView
   // Restores saved "always on top" state from a previous session.
   void RetriveSavedAlwaysOnTopState();
 
+  // Restores the saved column visibility from a previous session.
+  void RetrieveSavedColumnVisibility();
+
+  // Updates the visibility of the columns in the table view based on the values
+  // stored in |column_visibility_settings_|.
+  void UpdateColumnVisibility();
+
+  // Stores the current values of columns visibility in
+  // |column_visibility_settings_| to the user prefs.
+  void StoreColumnVisibility();
+
   scoped_ptr<NewTaskManagerView::TableModel> table_model_;
 
   scoped_ptr<views::MenuRunner> menu_runner_;
 
+  // Contains either the retrieved from user preferences columns visibility
+  // settings if any, otherwise it will contain the default column visibility.
+  scoped_ptr<base::DictionaryValue> column_visibility_settings_;

[Lei Zhang, 2015/08/27 01:10:44]

Does this need to be a scoped_ptr? It's always going to a 'non-NULL' scoped_ptr.

[afakhry, 2015/08/27 19:56:52]

It has to be a scoped_ptr<>. DictionaryValue is owned by us and it's non
copiable non assignable. We need to reset it to an new DictionaryValue if
there's one to retrieve from the local state. We can only get it as a pointer or
a scoped_ptr<>.

+
   // We need to own the text of the menu, the Windows API does not copy it.
   base::string16 always_on_top_menu_text_;