Silence adb command output by default

build/android/android_commands.py

Index: build/android/android_commands.py
diff --git a/build/android/android_commands.py b/build/android/android_commands.py
index 209ffeda30e4350aca3ff062362e28c0faf6ed9e..3f370f2552b8d99ecd3c7f40f94ca040f5c9150c 100755
--- a/build/android/android_commands.py
+++ b/build/android/android_commands.py
@@ -466,8 +466,8 @@ class AndroidCommands(object):
                          '(?P<time>[^\s]+)\s+'
                          '(?P<filename>[^\s]+)$')
     return _GetFilesFromRecursiveLsOutput(
-        path, self.RunShellCommand('ls -lR %s' % path), re_file,
-        self.RunShellCommand('date +%z')[0])
+        path, self.RunShellCommand('ls -lR %s' % path, log_result=False),

[bulach, 2012/06/25 09:47:51]

long story super short: in the past, we had various problems that were only
caught with the logs. things like parsing the output of a given command not
being robust enough to deal with unexpected output or errors, etc..

I think we're now in a much better situation, and overall all the parsers and
callers for RunShellCommand are much more reliable...

my suggestion would be: instead of sprinkling these with log_result=False, let's
make the default False, and then only the lines that actually need, would turn
it on...

since this is such a high impact change though, I'd suggest doing it downstream
first, wait a few days, then upstream..
(and yes, I know it's locked down :) but unfortunately we don't have yet the
same coverage level yet for such broad changes upstream first..).

wdyt?

+        re_file, self.RunShellCommand('date +%z')[0])
 
   def SetupPerformanceTest(self):
     """Sets up performance tests."""