arc: Use high-resolution icon for Play Store default item for scale >= 1.5f.

chrome/browser/ui/app_list/arc/arc_app_icon.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/app_list/arc/arc_app_icon.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/files/file_path.h"
@@ skipping 70 matching lines @@
 }
 
 gfx::ImageSkiaRep ArcAppIcon::Source::GetImageForScale(float scale) {
   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) {
     resource_id = scale >= 1.5f ?
-        IDR_ARC_SUPPORT_ICON_48 : IDR_ARC_SUPPORT_ICON_96;
+        IDR_ARC_SUPPORT_ICON_96 : IDR_ARC_SUPPORT_ICON_48;

[khmel, 2016/09/23 01:44:45]

could you please retest that it works as expected. I pained one icon one color
and another icon different color and made sure that app list contains right one.

I probably formatted the code and made that typo but not sure 100%

Thanks

[hshi1, 2016/09/23 01:47:50]

Yes I did just retest this (I'm using a paint tool to modify one of the PNGs). 

Note that you must test this BEFORE the Play Store opt-in flow. Because after
that the icon is not the one shipped with Chrome but the actual play store icon.

[khmel, 2016/09/23 01:48:54]

Thanks, lgtm

   } else {
     resource_id = IDR_APP_DEFAULT_ICON;
   }
   const gfx::ImageSkia* default_image = ResourceBundle::GetSharedInstance().
       GetImageSkiaNamed(resource_id);
   CHECK(default_image);
   return gfx::ImageSkiaOperations::CreateResizedImage(
       *default_image,
       skia::ImageOperations::RESIZE_BEST,
       gfx::Size(resource_size_in_dip_, resource_size_in_dip_)).
@@ skipping 204 matching lines @@
 
 void ArcAppIcon::DiscardDecodeRequest(DecodeRequest* request) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
 
   ScopedVector<DecodeRequest>::iterator it = std::find(decode_requests_.begin(),
                                                        decode_requests_.end(),
                                                        request);
   CHECK(it != decode_requests_.end());
   decode_requests_.erase(it);
 }