Fix several findbugs issues

content/public/android/javatests/src/org/chromium/content/browser/LongPressDetectorTest.java

Index: content/public/android/javatests/src/org/chromium/content/browser/LongPressDetectorTest.java
diff --git a/content/public/android/javatests/src/org/chromium/content/browser/LongPressDetectorTest.java b/content/public/android/javatests/src/org/chromium/content/browser/LongPressDetectorTest.java
index f3348176fa6c01df636fad16f577374de30b317c..9fd2ca116eeda639d92ecdc968dee6be1c4921a0 100644
--- a/content/public/android/javatests/src/org/chromium/content/browser/LongPressDetectorTest.java
+++ b/content/public/android/javatests/src/org/chromium/content/browser/LongPressDetectorTest.java
@@ -11,12 +11,12 @@ import android.test.suitebuilder.annotation.SmallTest;
 import android.view.MotionEvent;
 import android.view.ViewConfiguration;
 
-import java.util.concurrent.CountDownLatch;
-import java.util.concurrent.TimeUnit;
-
 import org.chromium.base.test.util.Feature;
 import org.chromium.base.test.util.ScalableTimeout;
 
+import java.util.concurrent.CountDownLatch;
+import java.util.concurrent.TimeUnit;
+
 /**
  * Test suite for LongPressDetector.
  */
@@ -159,6 +159,7 @@ public class LongPressDetectorTest extends InstrumentationTestCase {
                 LongPressDetector longPressDetector = new LongPressDetector(
                         getInstrumentation().getTargetContext(),
                         new LongPressDetector.LongPressDelegate() {
+                            @Override
                             public void onLongPress(MotionEvent event) {
                                 longPressCalled.countDown();
                             }
@@ -166,7 +167,7 @@ public class LongPressDetectorTest extends InstrumentationTestCase {
 
                 final long downTime = SystemClock.uptimeMillis();
                 final long eventTime = SystemClock.uptimeMillis();

[joth, 2013/03/14 20:30:56]

micro-nit: to retain the exact behavior as before, I'd say "final long eventTime
= downTime"

[Yaron, 2013/03/14 20:34:04]

Err, I assumed this was inadvertent but maybe I'm mistaken. At least it doesn't
match the other test cases. +Yusuf to comment

[joth, 2013/03/14 20:35:38]

o i c. we do this all over the place in this file. agree, do it as you wrote it
then.

thanks

-                MotionEvent event = motionEvent(MotionEvent.ACTION_DOWN, downTime, downTime);
+                MotionEvent event = motionEvent(MotionEvent.ACTION_DOWN, downTime, eventTime);
                 longPressDetector.startLongPressTimerIfNeeded(event);
             }
         });