Chromium Code Reviews| Index: build/android/buildbot/bb_device_status_check.py |
| diff --git a/build/android/buildbot/bb_device_status_check.py b/build/android/buildbot/bb_device_status_check.py |
| index ae0bb4a7b18b1aed4be9c850c78651166b296bee..9ce2b82a21f0861eba5e04395ddd880e49254367 100755 |
| --- a/build/android/buildbot/bb_device_status_check.py |
| +++ b/build/android/buildbot/bb_device_status_check.py |
| @@ -41,7 +41,6 @@ def DeviceInfo(serial, options): |
| device_build_type = device_adb.GetBuildType() |
| device_product_name = device_adb.GetProductName() |
| - setup_wizard_disabled = device_adb.GetSetupWizardStatus() == 'DISABLED' |
|
mkosiba (inactive)
2013/08/30 13:13:33
if you've taken the assert out in GetSetupWizardSt
navabi
2013/08/30 15:53:10
That would work too. But we would still be checkin
|
| battery = device_adb.GetBatteryInfo() |
| install_output = GetCmdOutput( |
| ['%s/build/android/adb_install_apk.py' % constants.DIR_SOURCE_ROOT, '--apk', |
| @@ -76,9 +75,10 @@ def DeviceInfo(serial, options): |
| errors = [] |
| if battery_level < 15: |
| errors += ['Device critically low in battery. Turning off device.'] |
| - if (not setup_wizard_disabled and device_build_type != 'user' and |
| - not options.no_provisioning_check): |
| - errors += ['Setup wizard not disabled. Was it provisioned correctly?'] |
| + if not options.no_provisioning_check: |
| + setup_wizard_disabled = device_adb.GetSetupWizardStatus() == 'DISABLED' |
| + if not setup_wizard_disabled and device_build_type != 'user': |
| + errors += ['Setup wizard not disabled. Was it provisioned correctly?'] |
| if device_product_name == 'mantaray' and ac_power != 'true': |
| errors += ['Mantaray device not connected to AC power.'] |
| # TODO(navabi): Insert warning once we have a better handle of what install |