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

Side by Side Diff: chrome/browser/ui/app_list/app_list_view_delegate.cc

Issue 12207105: [win] Remove 'installing...' icons from the app list if the install fails. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix mac Created 7 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 | Annotate | Revision Log
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/app_list/app_list_view_delegate.h" 5 #include "chrome/browser/ui/app_list/app_list_view_delegate.h"
6 6
7 #include "chrome/browser/profiles/profile_manager.h" 7 #include "chrome/browser/profiles/profile_manager.h"
8 #include "chrome/browser/ui/app_list/app_list_controller_delegate.h" 8 #include "chrome/browser/ui/app_list/app_list_controller_delegate.h"
9 #include "chrome/browser/ui/app_list/apps_model_builder.h" 9 #include "chrome/browser/ui/app_list/apps_model_builder.h"
10 #include "chrome/browser/ui/app_list/chrome_app_list_item.h" 10 #include "chrome/browser/ui/app_list/chrome_app_list_item.h"
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
62 extension_name, 62 extension_name,
63 installing_icon); 63 installing_icon);
64 } 64 }
65 65
66 void AppListViewDelegate::OnDownloadProgress( 66 void AppListViewDelegate::OnDownloadProgress(
67 const std::string& extension_id, 67 const std::string& extension_id,
68 int percent_downloaded) { 68 int percent_downloaded) {
69 apps_builder_->OnDownloadProgress(extension_id, percent_downloaded); 69 apps_builder_->OnDownloadProgress(extension_id, percent_downloaded);
70 } 70 }
71 71
72 void AppListViewDelegate::OnInstallFailure(const std::string& extension_id) {
73 apps_builder_->OnInstallFailure(extension_id);
74 }
75
72 void AppListViewDelegate::ActivateAppListItem( 76 void AppListViewDelegate::ActivateAppListItem(
73 app_list::AppListItemModel* item, 77 app_list::AppListItemModel* item,
74 int event_flags) { 78 int event_flags) {
75 content::RecordAction(content::UserMetricsAction("AppList_ClickOnApp")); 79 content::RecordAction(content::UserMetricsAction("AppList_ClickOnApp"));
76 static_cast<ChromeAppListItem*>(item)->Activate(event_flags); 80 static_cast<ChromeAppListItem*>(item)->Activate(event_flags);
77 } 81 }
78 82
79 void AppListViewDelegate::StartSearch() { 83 void AppListViewDelegate::StartSearch() {
80 if (search_builder_.get()) 84 if (search_builder_.get())
81 search_builder_->StartSearch(); 85 search_builder_->StartSearch();
(...skipping 27 matching lines...) Expand all
109 controller_->ViewClosing(); 113 controller_->ViewClosing();
110 } 114 }
111 115
112 void AppListViewDelegate::ViewActivationChanged(bool active) { 116 void AppListViewDelegate::ViewActivationChanged(bool active) {
113 controller_->ViewActivationChanged(active); 117 controller_->ViewActivationChanged(active);
114 } 118 }
115 119
116 gfx::ImageSkia AppListViewDelegate::GetWindowIcon() { 120 gfx::ImageSkia AppListViewDelegate::GetWindowIcon() {
117 return controller_->GetWindowIcon(); 121 return controller_->GetWindowIcon();
118 } 122 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/app_list/app_list_view_delegate.h ('k') | chrome/browser/ui/app_list/apps_model_builder.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698