Replace GetHostSize with a cross-platform implementation.

build/android/pylib/android_commands.py

 # 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.
 
 """Provides an interface to communicate with the device via the adb command.
 
 Assumes adb binary is currently on system path.
 """
 # pylint: disable-all
 
@@ skipping 983 matching lines @@
 
     # See if the file on the host changed since the last push (if any) and
     # return early if it didn't. Note that this shortcut assumes that the tests
     # on the device don't modify the files.
     if not os.path.isdir(host_path):
       if host_path in self._push_if_needed_cache:
         host_path_mtime = self._push_if_needed_cache[host_path]
         if host_path_mtime == os.stat(host_path).st_mtime:
           return
 
-    def GetHostSize(path):
-      return int(cmd_helper.GetCmdOutput(['du', '-sb', path]).split()[0])
-
-    size = GetHostSize(host_path)
+    size = os.stat(host_path).st_size
     self._pushed_files.append(device_path)
     self._potential_push_size += size
 
     changed_files = self.GetFilesChanged(host_path, device_path)
     logging.info('Found %d files that need to be pushed to %s',
         len(changed_files), device_path)
     if not changed_files:
       return
 
     def Push(host, device):
@@ skipping 14 matching lines @@
           retry += 1
           wait_time = 5 * retry
           logging.error('Push failed, retrying in %d seconds: %s' %
                         (wait_time, output))
           time.sleep(wait_time)
         else:
           raise Exception('Push failed: %s' % output)
 
     diff_size = 0
     if len(changed_files) <= MAX_INDIVIDUAL_PUSHES:
-      diff_size = sum(GetHostSize(f[0]) for f in changed_files)
+      diff_size = sum(os.stat(f[0]).st_size for f in changed_files)

[craigdh, 2014/03/24 19:19:54]

This doesn't work for directories.

 
     # TODO(craigdh): Replace this educated guess with a heuristic that
     # approximates the push time for each method.
     if len(changed_files) > MAX_INDIVIDUAL_PUSHES or diff_size > 0.5 * size:
       self._actual_push_size += size
       if os.path.isdir(host_path):
         self.RunShellCommand('mkdir -p %s' % device_path)
       Push(host_path, device_path)
     else:
       for f in changed_files:
@@ skipping 790 matching lines @@
   """
   def __init__(self, output):
     self._output = output
 
   def write(self, data):
     data = data.replace('\r\r\n', '\n')
     self._output.write(data)
 
   def flush(self):
     self._output.flush()