Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(415)

Side by Side Diff: chrome/browser/ui/views/new_task_manager_view.cc

Issue 2133013002: AcceleratorProvider: Make GetAcceleratorForCommandId const. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix overrides on Mac and Chrome OS. Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/views/new_task_manager_view.h" 5 #include "chrome/browser/ui/views/new_task_manager_view.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include "build/build_config.h" 9 #include "build/build_config.h"
10 #include "chrome/browser/browser_process.h" 10 #include "chrome/browser/browser_process.h"
(...skipping 262 matching lines...) Expand 10 before | Expand all | Expand 10 after
273 bool NewTaskManagerView::IsCommandIdChecked(int id) const { 273 bool NewTaskManagerView::IsCommandIdChecked(int id) const {
274 return tab_table_->IsColumnVisible(id); 274 return tab_table_->IsColumnVisible(id);
275 } 275 }
276 276
277 bool NewTaskManagerView::IsCommandIdEnabled(int id) const { 277 bool NewTaskManagerView::IsCommandIdEnabled(int id) const {
278 return true; 278 return true;
279 } 279 }
280 280
281 bool NewTaskManagerView::GetAcceleratorForCommandId( 281 bool NewTaskManagerView::GetAcceleratorForCommandId(
282 int command_id, 282 int command_id,
283 ui::Accelerator* accelerator) { 283 ui::Accelerator* accelerator) const {
284 return false; 284 return false;
285 } 285 }
286 286
287 void NewTaskManagerView::ExecuteCommand(int id, int event_flags) { 287 void NewTaskManagerView::ExecuteCommand(int id, int event_flags) {
288 table_model_->ToggleColumnVisibility(id); 288 table_model_->ToggleColumnVisibility(id);
289 } 289 }
290 290
291 NewTaskManagerView::NewTaskManagerView() 291 NewTaskManagerView::NewTaskManagerView()
292 : tab_table_(nullptr), 292 : tab_table_(nullptr),
293 tab_table_parent_(nullptr), 293 tab_table_parent_(nullptr),
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
354 if (!g_browser_process->local_state()) 354 if (!g_browser_process->local_state())
355 return; 355 return;
356 356
357 const base::DictionaryValue* dictionary = 357 const base::DictionaryValue* dictionary =
358 g_browser_process->local_state()->GetDictionary(GetWindowName()); 358 g_browser_process->local_state()->GetDictionary(GetWindowName());
359 if (dictionary) 359 if (dictionary)
360 dictionary->GetBoolean("always_on_top", &is_always_on_top_); 360 dictionary->GetBoolean("always_on_top", &is_always_on_top_);
361 } 361 }
362 362
363 } // namespace task_management 363 } // namespace task_management
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698