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

Unified Diff: chrome/browser/ui/app_list/arc/arc_app_icon.cc

Issue 2604523002: arc: Update PlayStore icons to round style (Closed)
Patch Set: updater Created 4 years 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/resources/chromeos/arc_support/icon/96.png ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/app_list/arc/arc_app_icon.cc
diff --git a/chrome/browser/ui/app_list/arc/arc_app_icon.cc b/chrome/browser/ui/app_list/arc/arc_app_icon.cc
index c5df2c4092a3a9acafc4f5f6daa260f256982ec2..6f1bbc73356684ecad55bca2a93522a7dbf2ffcd 100644
--- a/chrome/browser/ui/app_list/arc/arc_app_icon.cc
+++ b/chrome/browser/ui/app_list/arc/arc_app_icon.cc
@@ -127,15 +127,17 @@ ArcAppIcon::Source::~Source() {
gfx::ImageSkiaRep ArcAppIcon::Source::GetImageForScale(float scale) {
DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
- if (host_)
- host_->LoadForScaleFactor(ui::GetSupportedScaleFactor(scale));
// Host loads icon asynchronously, so use default icon so far.
int resource_id;
if (host_ && host_->app_id() == arc::kPlayStoreAppId) {
+ // Don't request icon from Android side. Use overloaded Chrome icon for Play
+ // Store that is adopted according Chrome style.
resource_id = scale >= 1.5f ?
IDR_ARC_SUPPORT_ICON_96 : IDR_ARC_SUPPORT_ICON_48;
} else {
+ if (host_)
+ host_->LoadForScaleFactor(ui::GetSupportedScaleFactor(scale));
resource_id = IDR_APP_DEFAULT_ICON;
}
« no previous file with comments | « chrome/browser/resources/chromeos/arc_support/icon/96.png ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698