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

Unified Diff: base/android/javatests/src/org/chromium/base/ObserverListTest.java

Issue 1297073004: Fix various errorprone warnings and errors. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove third_party/mojo/src change. 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: base/android/javatests/src/org/chromium/base/ObserverListTest.java
diff --git a/base/android/javatests/src/org/chromium/base/ObserverListTest.java b/base/android/javatests/src/org/chromium/base/ObserverListTest.java
index 973682b9328c0ff534bb58146387e395a4580311..347c842f2a9c6433a260e5496611818df0674bc7 100644
--- a/base/android/javatests/src/org/chromium/base/ObserverListTest.java
+++ b/base/android/javatests/src/org/chromium/base/ObserverListTest.java
@@ -70,6 +70,7 @@ public class ObserverListTest extends InstrumentationTestCase {
}
}
+ @SuppressWarnings("ElementsCountedInLoop")
nyquist 2015/08/19 09:51:33 Optional nit: I guess you could technically add an
mikecase (-- gone --) 2015/08/19 23:15:07 Done
private static <T> int getSizeOfIterable(Iterable<T> iterable) {
int num = 0;
for (T el : iterable) num++;

Powered by Google App Engine
This is Rietveld 408576698