[Android] Switch all subprocess.Popen calls to use a temporary file instead of PIPE.

build/android/pylib/utils/findbugs.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.
 
 import optparse
 import os
 import re
 import shlex
 import subprocess
 import sys
 
+from pylib import cmd_helper
 from pylib import constants
 
 
 def _PrintMessage(warnings, title, action, known_bugs_file):
   if warnings:
     print
     print '*' * 80
     print '%s warnings.' % title
     print '%s %s' % (action, known_bugs_file)
     print '-' * 80
@@ skipping 42 matching lines @@
       print >>known_bugs, warning
   return 0
 
 
 def _GetChromeClasses(release_version):
   version = 'Debug'
   if release_version:
     version = 'Release'
   path = os.path.join(constants.CHROME_DIR, 'out', version)
   cmd = 'find %s -name "*.class"' % path
-  proc = subprocess.Popen(shlex.split(cmd),
-                          stdout=subprocess.PIPE, stderr=subprocess.PIPE)
-  out, err = proc.communicate()
-
+  out = cmd_helper.GetCmdOutput(shlex.split(cmd))
   if not out:
     print 'No classes found in %s' % path
   return out
 
 
 def _Run(exclude, known_bugs, classes_to_analyze, auxiliary_classes,
         rebaseline, release_version, findbug_args):
   """Run the FindBugs.
 
   Args:
@@ skipping 136 matching lines @@
 
 def main(argv):
   parser = GetCommonParser()
   options, _ = parser.parse_args()
 
   return Run(options)
 
 
 if __name__ == '__main__':
   sys.exit(main(sys.argv))