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

Side by Side Diff: chrome/browser/ui/ash/launcher/chrome_launcher_controller_mus.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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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_controller_mus.h" 5 #include "chrome/browser/ui/ash/launcher/chrome_launcher_controller_mus.h"
6 6
7 #include "base/strings/string_util.h" 7 #include "base/strings/string_util.h"
8 #include "base/strings/utf_string_conversions.h" 8 #include "base/strings/utf_string_conversions.h"
9 #include "chrome/browser/profiles/profile.h" 9 #include "chrome/browser/profiles/profile.h"
10 #include "chrome/browser/profiles/profile_manager.h" 10 #include "chrome/browser/profiles/profile_manager.h"
(...skipping 163 matching lines...) Expand 10 before | Expand all | Expand 10 after
174 void ChromeLauncherControllerMus::ActiveUserChanged( 174 void ChromeLauncherControllerMus::ActiveUserChanged(
175 const std::string& user_email) { 175 const std::string& user_email) {
176 NOTIMPLEMENTED(); 176 NOTIMPLEMENTED();
177 } 177 }
178 178
179 void ChromeLauncherControllerMus::AdditionalUserAddedToSession( 179 void ChromeLauncherControllerMus::AdditionalUserAddedToSession(
180 Profile* profile) { 180 Profile* profile) {
181 NOTIMPLEMENTED(); 181 NOTIMPLEMENTED();
182 } 182 }
183 183
184 ChromeLauncherAppMenuItems ChromeLauncherControllerMus::GetApplicationList( 184 ash::ShelfAppMenuItemList ChromeLauncherControllerMus::GetAppMenuItems(
185 const ash::ShelfItem& item, 185 const ash::ShelfItem& item,
186 int event_flags) { 186 int event_flags) {
187 NOTIMPLEMENTED(); 187 NOTIMPLEMENTED();
188 return ChromeLauncherAppMenuItems(); 188 return ash::ShelfAppMenuItemList();
189 } 189 }
190 190
191 std::vector<content::WebContents*> 191 std::vector<content::WebContents*>
192 ChromeLauncherControllerMus::GetV1ApplicationsFromAppId( 192 ChromeLauncherControllerMus::GetV1ApplicationsFromAppId(
193 const std::string& app_id) { 193 const std::string& app_id) {
194 NOTIMPLEMENTED(); 194 NOTIMPLEMENTED();
195 return std::vector<content::WebContents*>(); 195 return std::vector<content::WebContents*>();
196 } 196 }
197 197
198 void ChromeLauncherControllerMus::ActivateShellApp(const std::string& app_id, 198 void ChromeLauncherControllerMus::ActivateShellApp(const std::string& app_id,
(...skipping 98 matching lines...) Expand 10 before | Expand all | Expand 10 after
297 app_id_to_item_delegate_.insert( 297 app_id_to_item_delegate_.insert(
298 std::make_pair(app_id, std::move(delegate))); 298 std::make_pair(app_id, std::move(delegate)));
299 299
300 AppIconLoader* app_icon_loader = GetAppIconLoaderForApp(app_id); 300 AppIconLoader* app_icon_loader = GetAppIconLoaderForApp(app_id);
301 if (app_icon_loader) { 301 if (app_icon_loader) {
302 app_icon_loader->FetchImage(app_id); 302 app_icon_loader->FetchImage(app_id);
303 app_icon_loader->UpdateImage(app_id); 303 app_icon_loader->UpdateImage(app_id);
304 } 304 }
305 } 305 }
306 } 306 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698