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

Unified Diff: content/browser/payments/payment_app_info_fetcher.cc

Issue 2969713002: [Payments] Present web payment app instead of instruments to user (Closed)
Patch Set: address comments Created 3 years, 5 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: content/browser/payments/payment_app_info_fetcher.cc
diff --git a/content/browser/payments/payment_app_info_fetcher.cc b/content/browser/payments/payment_app_info_fetcher.cc
index f4d68e87d696e6587138409d3daf4b4ec1f5939c..cc232764fd1ad97aaf2191128b1d6ccb80bc732e 100644
--- a/content/browser/payments/payment_app_info_fetcher.cc
+++ b/content/browser/payments/payment_app_info_fetcher.cc
@@ -20,7 +20,9 @@ namespace {
// TODO(gogerald): Choose appropriate icon size dynamically on different
// platforms.
-const int kPaymentAppIdealIconSize = 64;
+// Here we choose a large ideal icon size to be big enough for all platforms.
+// Note that we only scale down for this icon size but not scale up.
+const int kPaymentAppIdealIconSize = 0xFFFF;
const int kPaymentAppMinimumIconSize = 0;
} // namespace
« no previous file with comments | « content/browser/payments/payment_app_database.cc ('k') | content/browser/payments/payment_app_provider_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698