manifest: allow the icon downloader to choose non-square icons

chrome/browser/manifest/manifest_icon_downloader.cc

Index: chrome/browser/manifest/manifest_icon_downloader.cc
diff --git a/chrome/browser/manifest/manifest_icon_downloader.cc b/chrome/browser/manifest/manifest_icon_downloader.cc
index 34ff954d39e64fe15fe901ff85c973b2b19ba0da..04173358c475ce2980ee309d7e77ea6ffd19bbde 100644
--- a/chrome/browser/manifest/manifest_icon_downloader.cc
+++ b/chrome/browser/manifest/manifest_icon_downloader.cc
@@ -126,5 +126,28 @@ int ManifestIconDownloader::FindClosestBitmapIndex(
       best_delta = delta;
     }
   }
+
+  if (best_index != -1)
+    return best_index;
+
+  // There was no square icon of a correct size found. Try to find the most
+  // square possible icon which has both dimensions greater than the minimum
+  // size.
+  float best_ratio_difference = std::numeric_limits<float>::infinity();
+  for (size_t i = 0; i < bitmaps.size(); ++i) {
+    if (bitmaps[i].height() < minimum_icon_size_in_px ||
+        bitmaps[i].width() < minimum_icon_size_in_px) {
+      continue;
+    }
+
+    float ratio = static_cast<float>(bitmaps[i].height()) /

[gone, 2015/09/21 09:51:03]

Can you use some temporary variables to make it easier to parse?

[Lalit Maganti, 2015/09/21 09:52:34]

Done.

+        static_cast<float>(bitmaps[i].width());
+    float ratio_difference = abs(ratio - 1);
+    if (ratio_difference < best_ratio_difference) {
+      best_index = i;
+      best_ratio_difference = ratio_difference;
+    }
+  }
+
   return best_index;
 }