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

Unified Diff: chrome/browser/manifest/manifest_icon_downloader.h

Issue 1261143004: Implement manifest icon downloader (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix review comments Created 5 years, 4 months 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
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..594337f3a6e1cc5425f738e1e9e3e894d1cf9fab
--- /dev/null
+++ b/chrome/browser/manifest/manifest_icon_downloader.h
@@ -0,0 +1,69 @@
+// 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 <vector>
+
+#include "base/basictypes.h"
+#include "base/callback_forward.h"
+#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
mlamouri (slow - plz ping) 2015/08/21 13:34:18 nit: say it's a helper.
Lalit Maganti 2015/08/21 13:45:51 Done.
+// 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 final {
+ public:
+ using IconFetchCallback = base::Callback<void(const SkBitmap&)>;
+
+ ManifestIconDownloader() = delete;
+ ~ManifestIconDownloader() = delete;
+
+ // Returns whether the download has started.
+ // It will return false if the current context or information do not allow to
+ // download the image.
+ static 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);
+
+ 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_

Powered by Google App Engine
This is Rietveld 408576698