arc: Support non-standard display scale factors.

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

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 06a52ec6a5051a8a2843f9e0177627256c130754..d0cf8ca3b1d797bdc6badb75c931ba3a45253168 100644
--- a/chrome/browser/ui/app_list/arc/arc_app_icon.cc
+++ b/chrome/browser/ui/app_list/arc/arc_app_icon.cc
@@ -213,11 +213,7 @@ void ArcAppIcon::DecodeRequest::OnImageDecoded(const SkBitmap& bitmap) {
     return;
   }
 
-  gfx::ImageSkia image_skia;
-  image_skia.AddRepresentation(gfx::ImageSkiaRep(
-      bitmap,
-      ui::GetScaleForScaleFactor(scale_factor_)));
-  host_->Update(&image_skia);
+  host_->Update(scale_factor_, bitmap);
   host_->DiscardDecodeRequest(this);
 }
 
@@ -354,18 +350,32 @@ void ArcAppIcon::OnIconRead(
   }
 }
 
-void ArcAppIcon::Update(const gfx::ImageSkia* image) {
+void ArcAppIcon::Update(ui::ScaleFactor scale_factor, const SkBitmap& bitmap) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
-  CHECK(image && !image->isNull());
 
-  std::vector<gfx::ImageSkiaRep> reps = image->image_reps();
-  for (const auto& image_rep : reps) {
-    if (ui::IsSupportedScale(image_rep.scale())) {
-      image_skia_.RemoveRepresentation(image_rep.scale());
-      image_skia_.AddRepresentation(image_rep);
+  gfx::ImageSkiaRep image_rep(bitmap, ui::GetScaleForScaleFactor(scale_factor));
+  DCHECK(ui::IsSupportedScale(image_rep.scale()));
+
+  image_skia_.RemoveRepresentation(image_rep.scale());
+
+  // Regenerate all non-supported image representations if they exist. Handle
+  // scales that are mapped to updated scale.

[xiyuan, 2017/05/24 16:37:09]

nit: Emphasize on removing dependent reps of the new bitmap,

e.g.
  // Regenerate existing non-supported image representations that
  // depend on the rep of |scale_factor|.

[khmel, 2017/05/24 20:20:58]

Nice...

+  std::vector<float> scales_to_regenerate;
+  for (const gfx::ImageSkiaRep& image_rep_to_test : image_skia_.image_reps()) {
+    if (gfx::ImageSkia::MapToSupportedScale(image_rep_to_test.scale()) ==
+        image_rep.scale()) {
+      DCHECK(!ui::IsSupportedScale(image_rep_to_test.scale()));
+      scales_to_regenerate.push_back(image_rep_to_test.scale());
     }
   }
 
+  image_skia_.AddRepresentation(image_rep);
+
+  for (float scale_to_regenerate : scales_to_regenerate) {
+    image_skia_.RemoveRepresentation(scale_to_regenerate);
+    image_skia_.GetRepresentation(scale_to_regenerate);
+  }
+
   observer_->OnIconUpdated(this);
 }