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

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: rebase 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;
please use gerrit instead 2017/07/05 13:14:22 Why jump from 64 to 65535?
gogerald1 2017/07/05 19:49:39 As I mentioned in the above comments. 64 pixels is
const int kPaymentAppMinimumIconSize = 0;
} // namespace

Powered by Google App Engine
This is Rietveld 408576698