Tweak logging in incremental_install.py to know when each part is done

build/android/incremental_install.py

 #!/usr/bin/env python
 #
 # Copyright 2015 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.
 
 """Install *_incremental.apk targets as well as their dependent files."""
 
 import argparse
 import glob
@@ skipping 74 matching lines @@
   # Install .apk(s) if any of them have changed.
   def do_install():
     if args.splits:
       splits = []
       for split_glob in args.splits:
         splits.extend((f for f in glob.glob(split_glob)))
       device.InstallSplitApk(args.apk_path, splits, reinstall=True,
                              allow_cached_props=True)
     else:
       device.Install(args.apk_path, reinstall=True)
+    logging.info('Finished installing .apk')

[jbudorick, 2015/09/02 18:14:19]

I know this feeling. I'd recommend using -v/--verbose along with
devil.utils.run_tests_helper.SetLogLevel, though. Many people won't care about
the log messages, and flipping them on with a -v will let you opt in. 

[agrieve, 2015/09/02 18:34:04]

Was planning on adding this, but does seem like a good time. added!

 
   # Push .so files to the device (if they have changed).
   def do_push_libs():
     if args.lib_dir:
       device_lib_dir = posixpath.join(device_incremental_dir, 'lib')
       device.PushChangedFiles([(args.lib_dir, device_lib_dir)],
                               delete_device_stale=True)
+      logging.info('Finished pushing native libs')
+
   # Concurrency here speeds things up quite a bit, but DeviceUtils hasn't
   # been designed for multi-threading. Enabling only because this is a
   # developer-only tool.
   if args.no_threading:
     do_install()
     do_push_libs()
   else:
     reraiser_thread.RunAsync((do_install, do_push_libs))
   logging.info('Took %s seconds' % round(time.time() - start_time, 1))
 
 
 if __name__ == '__main__':
   sys.exit(main())