Implement manifest icon downloader

chrome/browser/manifest/manifest_icon_downloader.h

Index: chrome/browser/manifest/manifest_icon_downloader.h
diff --git a/chrome/browser/manifest/manifest_icon_downloader.h b/chrome/browser/manifest/manifest_icon_downloader.h
new file mode 100644
index 0000000000000000000000000000000000000000..1547d66c0148ffc7c6c8472e5d64fb7d45ddb8a0
--- /dev/null
+++ b/chrome/browser/manifest/manifest_icon_downloader.h
@@ -0,0 +1,70 @@
+// 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.
+
+#ifndef CHROME_BROWSER_MANIFEST_MANIFEST_ICON_DOWNLOADER_H_
+#define CHROME_BROWSER_MANIFEST_MANIFEST_ICON_DOWNLOADER_H_
+
+#include "base/basictypes.h"
+#include "base/bind.h"

[mlamouri (slow - plz ping), 2015/08/21 09:27:11]

Do you need bind.h or callback_fwd.h?

[Lalit Maganti, 2015/08/21 12:14:19]

Done.

+#include "base/memory/ref_counted.h"

[mlamouri (slow - plz ping), 2015/08/21 09:27:12]

Do you need this include?

[Lalit Maganti, 2015/08/21 12:14:19]

Done.

+#include "content/public/browser/web_contents_observer.h"

[mlamouri (slow - plz ping), 2015/08/21 09:27:12]

I think you can remove this #include.

[Lalit Maganti, 2015/08/21 12:14:19]

Done.

[Lalit Maganti, 2015/08/21 12:14:19]

Done.

+#include "third_party/skia/include/core/SkBitmap.h"
+
+namespace content {
+class WebContents;
+}  // namespace content
+
+namespace gfx {
+class Size;
+}  // namespace gfx
+
+class GURL;
+
+// Downloads the icon located at icon_url. If the file contains multiple icons
+// then it attempts to pick the one closest in size bigger than or
+// equal to ideal_icon_size_in_dp, taking into account the density of the
+// device. If a bigger icon is chosen then, the icon is scaled down to be
+// equal to ideal_icon_size_in_dp.
+class ManifestIconDownloader {

[mlamouri (slow - plz ping), 2015/08/21 09:27:11]

make it final?

[Lalit Maganti, 2015/08/21 12:14:19]

Done.

+ public:
+  using IconFetchCallback = base::Callback<void(const SkBitmap&)>;
+
+  ManifestIconDownloader() = default;
+  ~ManifestIconDownloader() = default;
+
+  // Returns whether the download has started. It will return false if the
+  // current context or information do not allow to download the image.

[mlamouri (slow - plz ping), 2015/08/21 09:27:11]

nit: line break after the first sentence.

[Lalit Maganti, 2015/08/21 12:14:19]

Done.

+  bool Download(content::WebContents* web_contents,
+                const GURL& icon_url,
+                int ideal_icon_size_in_dp,
+                const IconFetchCallback& callback);
+
+ private:
+  // Callback run after the manifest icon downloaded successfully or the
+  // download failed.
+  static void OnIconFetched(int ideal_icon_size_in_px,
+                            int minimum_icon_size_in_px,
+                            const IconFetchCallback& callback,
+                            int id,
+                            int http_status_code,
+                            const GURL& url,
+                            const std::vector<SkBitmap>& bitmaps,
+                            const std::vector<gfx::Size>& sizes);
+
+  static void ScaleIcon(int ideal_icon_size_in_px,
+                        const SkBitmap& bitmap,
+                        const IconFetchCallback& callback);
+
+  // |bitmaps| is assumed to be a vector of bitmaps sorted by size from biggest
+  // to smallest.
+  static int FindClosestBitmapIndex(int ideal_icon_size_in_px,
+                                    int minimum_icon_size_in_px,
+                                    const std::vector<SkBitmap>& bitmaps);
+
+  friend class ManifestIconDownloaderTest;
+
+  DISALLOW_COPY_AND_ASSIGN(ManifestIconDownloader);
+};
+
+#endif  // CHROME_BROWSER_MANIFEST_MANIFEST_ICON_DOWNLOADER_H_