Added the --screenshot option.

build/android/pylib/local/device/local_device_instrumentation_test_run.py

 # 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.
 
 import logging
 import re
 import time
 
 from devil.android import device_errors
 from devil.android import flag_changer
@@ skipping 246 matching lines @@
           r.SetName(test_display_name)
     if DidPackageCrashOnDevice(self._test_instance.test_package, device):
       for r in results:
         if r.GetType() == base_test_result.ResultType.UNKNOWN:
           r.SetType(base_test_result.ResultType.CRASH)
 
     if any(r.GetType() not in (base_test_result.ResultType.PASS,
                                base_test_result.ResultType.SKIP)
            for r in results):
       logging.info('detected failure in %s. raw output:', test_display_name)
+      if self._test_instance.screenshot_failures:

[mikecase (-- gone --), 2016/05/16 23:40:50]

I can't imagine this takes a screenshot before Chrome closes after the test
failure. Will these screenshots be mostly of the Android homescreen

[BigBossZhiling, 2016/05/18 17:46:11]

You are right. lol. It is only useful in a few occasions, but taking a
screenshot in java is not trivial, will upgrade the feature in another cl.

+        device.TakeScreenshot()

[mikecase (-- gone --), 2016/05/16 23:40:50]

This should probably specify the host_path arg for TakeScreenshot even if it is
just something like
out/failure_screenshots/.
Or maybe change the arg from screenshot_failures to screenshot_failures_dir?

[jbudorick, 2016/05/17 00:01:16]

+1, we should be able to specify this.

[BigBossZhiling, 2016/05/18 17:46:12]

Done.

[BigBossZhiling, 2016/05/18 17:46:12]

Done.

       for l in output:
         logging.info('  %s', l)
       if (not self._env.skip_clear_data
           and self._test_instance.package_info):
         permissions = (
             self._test_instance.apk_under_test.GetPermissions()
             if self._test_instance.apk_under_test
             else None)
         device.ClearApplicationState(self._test_instance.package_info.package,
                                      permissions=permissions)
@@ skipping 24 matching lines @@
         timeout = v
         break
     else:
       logging.warning('Using default 1 minute timeout for %s', test_name)
       timeout = 60
 
     timeout *= cls._GetTimeoutScaleFromAnnotations(annotations)
 
     return timeout