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

Side by Side Diff: chrome/browser/ui/app_list/search/app_result.cc

Issue 72463002: Move InstallObserver::OnBeginExtensionInstall parameters to a struct (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 1 month 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/app_list/search/app_result.h" 5 #include "chrome/browser/ui/app_list/search/app_result.h"
6 6
7 #include "chrome/browser/extensions/extension_service.h" 7 #include "chrome/browser/extensions/extension_service.h"
8 #include "chrome/browser/extensions/extension_system_factory.h" 8 #include "chrome/browser/extensions/extension_system_factory.h"
9 #include "chrome/browser/extensions/install_tracker.h" 9 #include "chrome/browser/extensions/install_tracker.h"
10 #include "chrome/browser/extensions/install_tracker_factory.h" 10 #include "chrome/browser/extensions/install_tracker_factory.h"
(...skipping 164 matching lines...) Expand 10 before | Expand all | Expand 10 after
175 175
176 // Automatically open app after enabling. 176 // Automatically open app after enabling.
177 Open(ui::EF_NONE); 177 Open(ui::EF_NONE);
178 } 178 }
179 179
180 void AppResult::ExtensionEnableFlowAborted(bool user_initiated) { 180 void AppResult::ExtensionEnableFlowAborted(bool user_initiated) {
181 extension_enable_flow_.reset(); 181 extension_enable_flow_.reset();
182 controller_->OnCloseExtensionPrompt(); 182 controller_->OnCloseExtensionPrompt();
183 } 183 }
184 184
185 void AppResult::OnBeginExtensionInstall(const std::string& extension_id, 185 void AppResult::OnBeginExtensionInstall(
186 const std::string& extension_name, 186 const ExtensionInstallParams& params) {}
187 const gfx::ImageSkia& installing_icon,
188 bool is_app,
189 bool is_platform_app) {}
190 187
191 void AppResult::OnDownloadProgress(const std::string& extension_id, 188 void AppResult::OnDownloadProgress(const std::string& extension_id,
192 int percent_downloaded) {} 189 int percent_downloaded) {}
193 190
194 void AppResult::OnInstallFailure(const std::string& extension_id) {} 191 void AppResult::OnInstallFailure(const std::string& extension_id) {}
195 192
196 void AppResult::OnExtensionInstalled(const extensions::Extension* extension) {} 193 void AppResult::OnExtensionInstalled(const extensions::Extension* extension) {}
197 194
198 void AppResult::OnExtensionLoaded(const extensions::Extension* extension) { 195 void AppResult::OnExtensionLoaded(const extensions::Extension* extension) {
199 UpdateIcon(); 196 UpdateIcon();
200 } 197 }
201 198
202 void AppResult::OnExtensionUnloaded(const extensions::Extension* extension) {} 199 void AppResult::OnExtensionUnloaded(const extensions::Extension* extension) {}
203 200
204 void AppResult::OnExtensionUninstalled(const extensions::Extension* extension) { 201 void AppResult::OnExtensionUninstalled(const extensions::Extension* extension) {
205 if (extension->id() != app_id_) 202 if (extension->id() != app_id_)
206 return; 203 return;
207 204
208 NotifyItemUninstalled(); 205 NotifyItemUninstalled();
209 } 206 }
210 207
211 void AppResult::OnAppsReordered() {} 208 void AppResult::OnAppsReordered() {}
212 209
213 void AppResult::OnAppInstalledToAppList(const std::string& extension_id) {} 210 void AppResult::OnAppInstalledToAppList(const std::string& extension_id) {}
214 211
215 void AppResult::OnShutdown() { StopObservingInstall(); } 212 void AppResult::OnShutdown() { StopObservingInstall(); }
216 213
217 } // namespace app_list 214 } // namespace app_list
OLDNEW
« no previous file with comments | « chrome/browser/ui/app_list/search/app_result.h ('k') | chrome/browser/ui/app_list/search/webstore/webstore_result.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698