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

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

Issue 2696073002: Merge ShelfItemDelegate::ItemSelected & LauncherItemDelegate::Activate. (Closed)
Patch Set: nit 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 120 matching lines...) Expand 10 before | Expand all | Expand 10 after
131 NOTIMPLEMENTED(); 131 NOTIMPLEMENTED();
132 return false; 132 return false;
133 } 133 }
134 134
135 bool ChromeLauncherControllerMus::IsPlatformApp(ash::ShelfID id) { 135 bool ChromeLauncherControllerMus::IsPlatformApp(ash::ShelfID id) {
136 NOTIMPLEMENTED(); 136 NOTIMPLEMENTED();
137 return false; 137 return false;
138 } 138 }
139 139
140 void ChromeLauncherControllerMus::ActivateApp(const std::string& app_id, 140 void ChromeLauncherControllerMus::ActivateApp(const std::string& app_id,
141 ash::LaunchSource source, 141 ash::ShelfLaunchSource source,
142 int event_flags) { 142 int event_flags) {
143 NOTIMPLEMENTED(); 143 NOTIMPLEMENTED();
144 } 144 }
145 145
146 void ChromeLauncherControllerMus::SetLauncherItemImage( 146 void ChromeLauncherControllerMus::SetLauncherItemImage(
147 ash::ShelfID shelf_id, 147 ash::ShelfID shelf_id,
148 const gfx::ImageSkia& image) { 148 const gfx::ImageSkia& image) {
149 NOTIMPLEMENTED(); 149 NOTIMPLEMENTED();
150 } 150 }
151 151
152 void ChromeLauncherControllerMus::UpdateAppState(content::WebContents* contents, 152 void ChromeLauncherControllerMus::UpdateAppState(content::WebContents* contents,
153 AppState app_state) { 153 AppState app_state) {
154 NOTIMPLEMENTED(); 154 NOTIMPLEMENTED();
155 } 155 }
156 156
157 ash::ShelfID ChromeLauncherControllerMus::GetShelfIDForWebContents( 157 ash::ShelfID ChromeLauncherControllerMus::GetShelfIDForWebContents(
158 content::WebContents* contents) { 158 content::WebContents* contents) {
159 NOTIMPLEMENTED(); 159 NOTIMPLEMENTED();
160 return ash::TYPE_UNDEFINED; 160 return ash::TYPE_UNDEFINED;
161 } 161 }
162 162
163 void ChromeLauncherControllerMus::SetRefocusURLPatternForTest(ash::ShelfID id, 163 void ChromeLauncherControllerMus::SetRefocusURLPatternForTest(ash::ShelfID id,
164 const GURL& url) { 164 const GURL& url) {
165 NOTIMPLEMENTED(); 165 NOTIMPLEMENTED();
166 } 166 }
167 167
168 ash::ShelfItemDelegate::PerformedAction 168 ash::ShelfAction ChromeLauncherControllerMus::ActivateWindowOrMinimizeIfActive(
169 ChromeLauncherControllerMus::ActivateWindowOrMinimizeIfActive(
170 ui::BaseWindow* window, 169 ui::BaseWindow* window,
171 bool allow_minimize) { 170 bool allow_minimize) {
172 NOTIMPLEMENTED(); 171 NOTIMPLEMENTED();
173 return ash::ShelfItemDelegate::kNoAction; 172 return ash::SHELF_ACTION_NONE;
174 } 173 }
175 174
176 void ChromeLauncherControllerMus::ActiveUserChanged( 175 void ChromeLauncherControllerMus::ActiveUserChanged(
177 const std::string& user_email) { 176 const std::string& user_email) {
178 NOTIMPLEMENTED(); 177 NOTIMPLEMENTED();
179 } 178 }
180 179
181 void ChromeLauncherControllerMus::AdditionalUserAddedToSession( 180 void ChromeLauncherControllerMus::AdditionalUserAddedToSession(
182 Profile* profile) { 181 Profile* profile) {
183 NOTIMPLEMENTED(); 182 NOTIMPLEMENTED();
(...skipping 115 matching lines...) Expand 10 before | Expand all | Expand 10 after
299 app_id_to_item_delegate_.insert( 298 app_id_to_item_delegate_.insert(
300 std::make_pair(app_id, std::move(delegate))); 299 std::make_pair(app_id, std::move(delegate)));
301 300
302 AppIconLoader* app_icon_loader = GetAppIconLoaderForApp(app_id); 301 AppIconLoader* app_icon_loader = GetAppIconLoaderForApp(app_id);
303 if (app_icon_loader) { 302 if (app_icon_loader) {
304 app_icon_loader->FetchImage(app_id); 303 app_icon_loader->FetchImage(app_id);
305 app_icon_loader->UpdateImage(app_id); 304 app_icon_loader->UpdateImage(app_id);
306 } 305 }
307 } 306 }
308 } 307 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698