We need tests covering OOBE.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b284829a639691cc7e2513c2194ce3164b4354cd commit b284829a639691cc7e2513c2194ce3164b4354cd Author: Alexander Alekseev <alemate@chromium.org> Date: Fri Dec 14 00:54:36 2018 Chrome OS tests: Generalize JsConditionWaiter. This Cl replaces JsConditionWaiter with more general TestConditionWaiter, and removes some code duplication. Bug: 880803 Change-Id: Iaa43837542305c1a7ed0ec0c5c761d7818db9d65 Reviewed-on: https://chromium-review.googlesource.com/c/1375713 Commit-Queue: Alexander Alekseev <alemate@chromium.org> Reviewed-by: Jacob Dufault <jdufault@chromium.org> Cr-Commit-Position: refs/heads/master@{#616532} [modify] https://crrev.com/b284829a639691cc7e2513c2194ce3164b4354cd/chrome/browser/chromeos/BUILD.gn [modify] https://crrev.com/b284829a639691cc7e2513c2194ce3164b4354cd/chrome/browser/chromeos/login/demo_mode/demo_setup_browsertest.cc [modify] https://crrev.com/b284829a639691cc7e2513c2194ce3164b4354cd/chrome/browser/chromeos/login/kiosk_browsertest.cc [modify] https://crrev.com/b284829a639691cc7e2513c2194ce3164b4354cd/chrome/browser/chromeos/login/oobe_interactive_ui_test.cc [modify] https://crrev.com/b284829a639691cc7e2513c2194ce3164b4354cd/chrome/browser/chromeos/login/sync_consent_interactive_ui_test.cc [add] https://crrev.com/b284829a639691cc7e2513c2194ce3164b4354cd/chrome/browser/chromeos/login/test/test_condition_waiter.cc [add] https://crrev.com/b284829a639691cc7e2513c2194ce3164b4354cd/chrome/browser/chromeos/login/test/test_condition_waiter.h [modify] https://crrev.com/b284829a639691cc7e2513c2194ce3164b4354cd/chrome/test/BUILD.gn
Comment 1 by alemate@chromium.org
, Sep 5