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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestMetricsTest.java

Issue 2363113003: Lint suppressions needed for Android SDK roll. (Closed)
Patch Set: Suppress alert box presubmit. Created 4 years, 2 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/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestMetricsTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestMetricsTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestMetricsTest.java
index 0a71bbc3b7aabd252aa1a92eefb8cae091fe0ea6..214506a842446b68a5e0002a2ff023f9897d8879 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestMetricsTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestMetricsTest.java
@@ -4,6 +4,7 @@
package org.chromium.chrome.browser.payments;
+import android.annotation.SuppressLint;
import android.content.DialogInterface;
import android.test.suitebuilder.annotation.MediumTest;
@@ -255,6 +256,8 @@ public class PaymentRequestMetricsTest extends PaymentRequestTestBase {
*
* @param abortReason The only bucket in the abort histogram that should have a record.
*/
+ // TODO(crbug.com/635567): Fix this properly.
+ @SuppressLint("DefaultLocale")
private void assertOnlySpecificAbortMetricLogged(int abortReason) {
for (int i = 0; i < PaymentRequestMetrics.ABORT_REASON_MAX; ++i) {
assertEquals(String.format("Found %d instead of %d", i, abortReason),

Powered by Google App Engine
This is Rietveld 408576698