• navabi@google.com's avatar
    This is a less aggressive patch than the description below. The original was... · 4694e3a7
    navabi@google.com authored
    This is a less aggressive patch than the description below. The original was reverted for breaking the Android tests bot (https://codereview.chromium.org/16299003).
    
    Removes check install step and moves it into the device status check. This will allow us to examine Android install speeds, which have recently caused timeout problems when installing large APK's. Unlike the original CL, low install speed only causes a warning (i.e. does not fail the step). The step fails if the battery of an online device is less than 5%.
    
    BUG=230970, 224004, 242237
    
    ---------------------------------------------------
    Original CL:
    Move CheckInstall to device status and fail on low install speed and battery.
    
    Removes the recently added CheckInstall step and moves the checking to the
    device status check step. Recently, devices will install very slowly causing
    tests to fail with device problems. We have also seen battery issues. The new
    device status check step will fail if any devices install unreasonably slow
    (i.e. < 800 KB/s) or have critically low battery level (i.e. < 5%).
    
    BUG=230970, 224004, 242237
    
    Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=203587
    
    Review URL: https://chromiumcodereview.appspot.com/16110005
    
    git-svn-id: svn://svn.chromium.org/chrome/trunk/src@204013 0039d316-1c4b-4281-b951-d872f2087c98
    4694e3a7
bb_device_steps.py 12.7 KB