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

Side by Side Diff: chrome/browser/ui/ash/launcher/chrome_launcher_app_menu_item_v2app.cc

Issue 2671923002: mash: Cleanup ash shelf application menu code. (Closed)
Patch Set: Add comments Created 3 years, 10 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/ash/launcher/chrome_launcher_app_menu_item_v2app.h" 5 #include "chrome/browser/ui/ash/launcher/chrome_launcher_app_menu_item_v2app.h"
6 6
7 #include "chrome/browser/ui/ash/launcher/chrome_launcher_controller.h" 7 #include "chrome/browser/ui/ash/launcher/chrome_launcher_controller.h"
8 8
9 ChromeLauncherAppMenuItemV2App::ChromeLauncherAppMenuItemV2App( 9 ChromeLauncherAppMenuItemV2App::ChromeLauncherAppMenuItemV2App(
10 const base::string16 title, 10 const base::string16 title,
11 const gfx::Image* icon, 11 const gfx::Image* icon,
12 const std::string& app_id, 12 const std::string& app_id,
13 ChromeLauncherController* launcher_controller, 13 ChromeLauncherController* launcher_controller,
14 int app_index, 14 int app_index)
15 bool has_leading_separator) 15 : ash::ShelfApplicationMenuItem(title, icon),
16 : ChromeLauncherAppMenuItem(title, icon, has_leading_separator),
17 launcher_controller_(launcher_controller), 16 launcher_controller_(launcher_controller),
18 app_id_(app_id), 17 app_id_(app_id),
19 app_index_(app_index) { 18 app_index_(app_index) {}
20 }
21 19
22 ChromeLauncherAppMenuItemV2App::~ChromeLauncherAppMenuItemV2App() { 20 ChromeLauncherAppMenuItemV2App::~ChromeLauncherAppMenuItemV2App() {}
23 }
24
25 bool ChromeLauncherAppMenuItemV2App::IsEnabled() const {
26 return true;
27 }
28 21
29 void ChromeLauncherAppMenuItemV2App::Execute(int event_flags) { 22 void ChromeLauncherAppMenuItemV2App::Execute(int event_flags) {
30 // Note: At this time there is only a single app running at any point. as 23 // Note: At this time there is only a single app running at any point. as
31 // such we will never come here with usable |event_flags|. If that ever 24 // such we will never come here with usable |event_flags|. If that ever
32 // changes we should add some special close code here. 25 // changes we should add some special close code here.
33 // Note: If the application item did go away since the menu was created, 26 // Note: If the application item did go away since the menu was created,
34 // The controller will take care of it. 27 // The controller will take care of it.
35 launcher_controller_->ActivateShellApp(app_id_, app_index_); 28 launcher_controller_->ActivateShellApp(app_id_, app_index_);
36 } 29 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698