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

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

Issue 2589503002: Use exact pixel sizes instead of dip in webapp/WebAPK installability code (Closed)
Patch Set: Created 4 years 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.cc
diff --git a/chrome/browser/manifest/manifest_icon_downloader.cc b/chrome/browser/manifest/manifest_icon_downloader.cc
index 4b208cc54287eed44614c3dd740dd2c1caa4bbc0..03ed14d6398f5eae9735f032a853914beb89a848 100644
--- a/chrome/browser/manifest/manifest_icon_downloader.cc
+++ b/chrome/browser/manifest/manifest_icon_downloader.cc
@@ -49,20 +49,13 @@ void ManifestIconDownloader::DevToolsConsoleHelper::AddMessage(
bool ManifestIconDownloader::Download(
content::WebContents* web_contents,
const GURL& icon_url,
- int ideal_icon_size_in_dp,
- int minimum_icon_size_in_dp,
+ int ideal_icon_size_in_px,
+ int minimum_icon_size_in_px,
const ManifestIconDownloader::IconFetchCallback& callback) {
- DCHECK(minimum_icon_size_in_dp <= ideal_icon_size_in_dp);
+ DCHECK(minimum_icon_size_in_px <= ideal_icon_size_in_px);
if (!web_contents || !icon_url.is_valid())
return false;
- const float device_scale_factor =
- display::Screen::GetScreen()->GetPrimaryDisplay().device_scale_factor();
- const int ideal_icon_size_in_px =
- static_cast<int>(round(ideal_icon_size_in_dp * device_scale_factor));
- const int minimum_icon_size_in_px =
- static_cast<int>(round(minimum_icon_size_in_dp * device_scale_factor));
-
web_contents->DownloadImage(
icon_url,
false, // is_favicon

Powered by Google App Engine
This is Rietveld 408576698