diff options
author | aurimas <aurimas@chromium.org> | 2015-04-03 14:43:56 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-04-03 21:44:28 +0000 |
commit | 2827f2b320e8e00137fc6b6328f10edb229c1d4a (patch) | |
tree | 72573aeb039759990afcbeea36bb706333dc97b2 /base/android | |
parent | c2e6516482a66a39deff2915ff81e04dcd65f8d2 (diff) | |
download | chromium_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.java | 12 |
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); |