• Friedrich Horschig's avatar
    Revert "Enroll device into Demo Mode when derelict" · 9f5af185
    Friedrich Horschig authored
    This reverts commit 7d8cd352.
    
    Reason for revert: The test DemoSetupDerelictTest.DerelictSetup failed two consecutive times right after it landed: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Linux%20ChromiumOS%20MSan%20Tests/8241 and
    https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Linux%20ChromiumOS%20MSan%20Tests/8242
    
    Original change's description:
    > Enroll device into Demo Mode when derelict
    > 
    > When a device is "derelict" (idle at OOBE), we currently auto-launch a
    > very old Kiosk demo app. This CL changes the idle detector to enroll
    > into offline Demo Mode instead if Demo Mode and offline Demo Mode are
    > enabled.
    > 
    > More specifically, "derelict" is defined as:
    > * Spending a total of 8 hours on the Welcome screen, followed by
    > * Being idle on the Welcome screen for 5 minutes
    > 
    > So a device that has progressed past the Welcome screen shouldn't be
    > considered derelict; nor should a device that goes idle for 5 minutes
    > but has only been in OOBE for <8 hours.
    > 
    > We will follow this CL  with actual derelict policies; until then it
    > will fail to enroll and simply return to the Welcome screen.
    > 
    > Bug: 869273
    > Test: browser_tests, and manually at OOBE with the following command line:
    >       --enable-demo-mode --enable-offline-demo-mode --derelict-idle-timeout=1
    >       --derelict-detection-timeout=1 --oobe-timer-interval=1
    >       (manual testing requires copying policies from
    >       /usr/share/chromeos-assets/demo_mode_resources/policy/ to
    >       /usr/share/chromeos-assets/demo_mode_resources/derelict_policy/)
    > 
    > Change-Id: I81572e447ce5082c0bf534ab32b0c4b34f0570a0
    > Reviewed-on: https://chromium-review.googlesource.com/1164398
    > Commit-Queue: Michael Giuffrida <michaelpg@chromium.org>
    > Reviewed-by: Alexander Alekseev <alemate@chromium.org>
    > Reviewed-by: Aga Wronska <agawronska@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#583927}
    
    TBR=alemate@chromium.org,michaelpg@chromium.org,agawronska@chromium.org
    
    Change-Id: Ieeeb3973303588be8f84443b221fe560a52dc105
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 869273
    Reviewed-on: https://chromium-review.googlesource.com/1179622Reviewed-by: default avatarFriedrich Horschig <fhorschig@chromium.org>
    Commit-Queue: Friedrich Horschig <fhorschig@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#584017}
    9f5af185
wizard_controller.cc 74.3 KB