summaryrefslogtreecommitdiffstats
path: root/base/android
diff options
context:
space:
mode:
authoraurimas <aurimas@chromium.org>2015-04-03 14:43:56 -0700
committerCommit bot <commit-bot@chromium.org>2015-04-03 21:44:28 +0000
commit2827f2b320e8e00137fc6b6328f10edb229c1d4a (patch)
tree72573aeb039759990afcbeea36bb706333dc97b2 /base/android
parentc2e6516482a66a39deff2915ff81e04dcd65f8d2 (diff)
downloadchromium_src-2827f2b320e8e00137fc6b6328f10edb229c1d4a.zip
chromium_src-2827f2b320e8e00137fc6b6328f10edb229c1d4a.tar.gz
chromium_src-2827f2b320e8e00137fc6b6328f10edb229c1d4a.tar.bz2
Enable NeedsBraces check and fix some checkstyle issues.
NeedBraces will now check for {} in if, while, and for statements. It will still allow one line inline statements. BUG=None Review URL: https://codereview.chromium.org/1054203002 Cr-Commit-Position: refs/heads/master@{#323823}
Diffstat (limited to 'base/android')
-rw-r--r--base/android/javatests/src/org/chromium/base/ObserverListTest.java12
1 files changed, 4 insertions, 8 deletions
diff --git a/base/android/javatests/src/org/chromium/base/ObserverListTest.java b/base/android/javatests/src/org/chromium/base/ObserverListTest.java
index c28d51d..973682b 100644
--- a/base/android/javatests/src/org/chromium/base/ObserverListTest.java
+++ b/base/android/javatests/src/org/chromium/base/ObserverListTest.java
@@ -72,8 +72,7 @@ public class ObserverListTest extends InstrumentationTestCase {
private static <T> int getSizeOfIterable(Iterable<T> iterable) {
int num = 0;
- for (T el : iterable)
- num++;
+ for (T el : iterable) num++;
return num;
}
@@ -91,8 +90,7 @@ public class ObserverListTest extends InstrumentationTestCase {
observerList.addObserver(a);
observerList.addObserver(b);
- for (Observer obs : observerList)
- obs.observe(10);
+ for (Observer obs : observerList) obs.observe(10);
// Removing an observer not in the list should do nothing.
observerList.removeObserver(e);
@@ -101,8 +99,7 @@ public class ObserverListTest extends InstrumentationTestCase {
observerList.addObserver(c);
observerList.addObserver(d);
- for (Observer obs : observerList)
- obs.observe(10);
+ for (Observer obs : observerList) obs.observe(10);
// observe should be called twice on a.
assertEquals(20, a.mTotal);
@@ -129,8 +126,7 @@ public class ObserverListTest extends InstrumentationTestCase {
observerList.addObserver(a);
observerList.addObserver(b);
- for (Observer obs : observerList)
- obs.observe(10);
+ for (Observer obs : observerList) obs.observe(10);
assertTrue(observerList.hasObserver(c));
assertEquals(10, a.mTotal);