Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(842)

Unified Diff: build/android/pylib/remote/device/remote_device_environment.py

Issue 950893004: Add exit code for infra failures to testrunner script. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Made CL more general. BaseError now has is_infra_error property. Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: build/android/pylib/remote/device/remote_device_environment.py
diff --git a/build/android/pylib/remote/device/remote_device_environment.py b/build/android/pylib/remote/device/remote_device_environment.py
index 256118098f409c1f622a76300e51be53817f4095..3d76efda0383b6cbd8420be68a25fd6ae29f8075 100644
--- a/build/android/pylib/remote/device/remote_device_environment.py
+++ b/build/android/pylib/remote/device/remote_device_environment.py
@@ -302,7 +302,8 @@ class RemoteDeviceEnvironment(environment.Environment):
def _NoDeviceFound(self):
self._PrintAvailableDevices(self._GetDeviceList())
- raise remote_device_helper.RemoteDeviceError('No device found.')
+ raise remote_device_helper.RemoteDeviceError('No device found.',
jbudorick 2015/02/23 22:13:00 Either: - put a parameter on the first line and i
mikecase (-- gone --) 2015/02/23 22:29:27 Done.
+ is_infra_error=True)
@property
def collect(self):

Powered by Google App Engine
This is Rietveld 408576698