[Android] Incorporate findbugs into android builds.

tools/android/findbugs_plugin/test/run_findbugs_plugin_tests.py

 #!/usr/bin/env python
 #
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # This is used to test the findbugs plugin, it calls
 # build/android/pylib/utils/findbugs.py to analyze the classes in
 # org.chromium.tools.findbugs.plugin package, and expects to get the same
 # issue with those in expected_result.txt.
 #
 # Useful command line:
 # --rebaseline to generate the expected_result.txt, please make sure don't
 # remove the expected result of exsting tests.
 
 
-import optparse
+import argparse
 import os
 import sys
 
 sys.path.append(os.path.abspath(os.path.join(os.path.dirname(__file__),
                                              '..', '..', '..', '..',
                                              'build', 'android')))
 
 from pylib import constants
 from pylib.utils import findbugs
 
 
+_EXPECTED_WARNINGS = set([
+    findbugs.FindBugsWarning(
+        bug_type='CHROMIUM_SYNCHRONIZED_THIS',
+        class_name='org.chromium.tools.findbugs.plugin.SimpleSynchronizedThis',
+        method_name='synchronizedThis',
+        start_line=15,
+        end_line=15,
+        message="Shouldn't use synchronized(this)"),
+    findbugs.FindBugsWarning(
+        bug_type='CHROMIUM_SYNCHRONIZED_METHOD',
+        class_name=
+            'org.chromium.tools.findbugs.plugin.SimpleSynchronizedStaticMethod',
+        method_name='synchronizedStaticMethod',
+        start_line=14,
+        end_line=14,
+        message="Shouldn't use synchronized method"),
+    findbugs.FindBugsWarning(
+        bug_type='CHROMIUM_SYNCHRONIZED_METHOD',
+        class_name=
+            'org.chromium.tools.findbugs.plugin.SimpleSynchronizedMethod',
+        method_name='synchronizedMethod',
+        start_line=15,
+        end_line=15,
+        message="Shouldn't use synchronized method"),
+])
+
+
 def main(argv):
-  parser = findbugs.GetCommonParser()
 
-  options, _ = parser.parse_args()
+  parser = argparse.ArgumentParser()
+  parser.add_argument(
+      '-l', '--release-build', action='store_true', dest='release',
+      help='Run the release build of the findbugs plugin test.')
+  args = parser.parse_args()
 
-  if not options.known_bugs:
-    options.known_bugs = os.path.join(constants.DIR_SOURCE_ROOT, 'tools',
-                                      'android', 'findbugs_plugin', 'test',
-                                      'expected_result.txt')
+  test_jar_path = os.path.join(
+      constants.GetOutDirectory(
+          'Release' if args.release else 'Debug'),
+      'lib.java', 'findbugs_plugin_test.jar')
 
-  if not options.only_analyze:
-    options.only_analyze = 'org.chromium.tools.findbugs.plugin.*'
+  findbugs_command, findbugs_warnings = findbugs.Run(
+      None, 'org.chromium.tools.findbugs.plugin.*', None, None, None,
+      [test_jar_path])
 
-  # crbug.com/449101
-  # Temporary workaround to have the Android Clang Builder (dbg) bot
-  # pass the findbugs_tests step.
-  if not options.exclude:
-    options.exclude = os.path.join(constants.DIR_SOURCE_ROOT, 'build',
-                                   'android', 'findbugs_filter',
-                                   'findbugs_exclude.xml')
+  missing_warnings = _EXPECTED_WARNINGS.difference(findbugs_warnings)
+  if missing_warnings:
+    print 'Missing warnings:'
+    for w in missing_warnings:
+      print '  %s' % str(w)
 
-  return findbugs.Run(options)
+  unexpected_warnings = findbugs_warnings.difference(_EXPECTED_WARNINGS)
+  if unexpected_warnings:
+    print 'Unexpected warnings:'
+    for w in unexpected_warnings:
+      print '  %s' % str(w)
+
+  return len(unexpected_warnings) + len(missing_warnings)
 
 if __name__ == '__main__':
   sys.exit(main(sys.argv))