Fix various errorprone warnings and errors.

chrome/android/javatests/src/org/chromium/chrome/browser/widget/ToolbarProgressBarTest.java

Index: chrome/android/javatests/src/org/chromium/chrome/browser/widget/ToolbarProgressBarTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/widget/ToolbarProgressBarTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/widget/ToolbarProgressBarTest.java
index 8407d32c1e1dc02add810aacbb885582d9805529..b8c6572a6009e1a1d0d700bbe9744269d9f11719 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/widget/ToolbarProgressBarTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/widget/ToolbarProgressBarTest.java
@@ -44,6 +44,7 @@ public class ToolbarProgressBarTest extends ChromeActivityTestCaseBase<ChromeAct
     @MediumTest
     @Restriction(RESTRICTION_TYPE_PHONE)
     @SuppressFBWarnings({"WA_NOT_IN_LOOP", "UW_UNCOND_WAIT"})
+    @SuppressWarnings("WaitNotInLoop")

[nyquist, 2015/08/19 09:51:33]

This seems dubious to me. Given the possible spurious wakeups, it's not
guaranteed that the wait() call will in fact ensure that the condition is
satisfied. Shouldn't we instead also set a flag in onAnimationEnd(...)? And
AtomicBoolean for example should be able to be used easily, and then in the
wait()-places, you'd add a loop to verify the state (and a timeout I guess).

[mikecase (-- gone --), 2015/08/19 23:15:08]

Done

     public void testProgressBarDisappearsAfterFastShowHide() throws InterruptedException {
         // onAnimationEnd will be signaled on progress bar showing/hiding animation end.
         final Object onAnimationEnd = new Object();