Findbugs fixes, removing fields that are never read from

Findbugs fixes, removing fields that are never read from


BUG=


Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=176625

[nilesh, 2013-01-11 15:53:12]

LGTM

[commit-bot: I haz the power, 2013-01-11 18:49:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kristianm@chromium.org/11860005/1

[commit-bot: I haz the power, 2013-01-11 18:49:24]

Failed to apply patch for build/android/findbugs_filter/findbugs_known_bugs.txt:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file build/android/findbugs_filter/findbugs_known_bugs.txt
  Hunk #1 FAILED at 111.
  1 out of 1 hunk FAILED -- saving rejects to file
build/android/findbugs_filter/findbugs_known_bugs.txt.rej

Patch:       build/android/findbugs_filter/findbugs_known_bugs.txt
Index: build/android/findbugs_filter/findbugs_known_bugs.txt
diff --git a/build/android/findbugs_filter/findbugs_known_bugs.txt
b/build/android/findbugs_filter/findbugs_known_bugs.txt
index
a589e7c09fc090e839c736569f32e98f594e4b43..8bf6ecffbcfd385d6a99970cbe4c91649fb2535e
100644
--- a/build/android/findbugs_filter/findbugs_known_bugs.txt
+++ b/build/android/findbugs_filter/findbugs_known_bugs.txt
@@ -111,13 +111,6 @@ M P SS: Unread field:
org.chromium.android_webview.test.util.ImagePageGenerator.
 M P SS: Unread field:
org.chromium.android_webview.test.util.ImagePageGenerator.IMAGE_SUFFIX; should
this field be static?  At ImagePageGenerator.java
 M P SS: Unread field:
org.chromium.native_test.ChromeNativeTestActivity.EXTRA_RUN_IN_SUB_THREAD;
should this field be static?  At ChromeNativeTestActivity.java
 M P SS: Unread field: org.chromium.native_test.ChromeNativeTestActivity.TAG;
should this field be static?  At ChromeNativeTestActivity.java
-M P UrF: Unread field:
org.chromium.content.browser.ContentViewGestureHandlerTest$MockListener.mLastFling2
 At ContentViewGestureHandlerTest.java
-M P UrF: Unread field:
org.chromium.content.browser.ContentViewGestureHandlerTest$MockListener.mLastScroll1
 At ContentViewGestureHandlerTest.java
-M P UrF: Unread field:
org.chromium.content.browser.ContentViewGestureHandlerTest$MockListener.mLastScroll2
 At ContentViewGestureHandlerTest.java
-M P UrF: Unread field:
org.chromium.content.browser.ContentViewGestureHandlerTest$MockListener.mLastScrollDistanceX
 At ContentViewGestureHandlerTest.java
-M P UrF: Unread field:
org.chromium.content.browser.ContentViewGestureHandlerTest$MockListener.mLastScrollDistanceY
 At ContentViewGestureHandlerTest.java
-M P UrF: Unread field: org.chromium.content.browser.HandleView.mHeight  At
HandleView.java
-M P UuF: Unused field:
org.chromium.content.browser.HandleView.mLongPressCallback  In HandleView.java
 M P UuF: Unused field:
org.chromium.content.browser.JavaBridgeBasicsTest$19.privateField  In
JavaBridgeBasicsTest.java
 M P UuF: Unused field:
org.chromium.content.browser.JavaBridgeBasicsTest$23.field  In
JavaBridgeBasicsTest.java
 M V EI2: new org.chromium.chrome.browser.FindMatchRectsDetails(int, RectF[],
RectF) may expose internal representation by storing an externally mutable
object into FindMatchRectsDetails.rects  At FindMatchRectsDetails.java

[commit-bot: I haz the power, 2013-01-11 21:07:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kristianm@chromium.org/11860005/6001

[commit-bot: I haz the power, 2013-01-13 08:28:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kristianm@chromium.org/11860005/18002

[commit-bot: I haz the power, 2013-01-14 01:19:33]

Change committed as 176625