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

Issue 83293004: kiosk: Show enrollment screen first if needed. (Closed)

Created:
7 years ago by xiyuan
Modified:
7 years ago
Reviewers:
Nikita (slow), zel
CC:
chromium-reviews, arv+watch_chromium.org, stevenjb+watch_chromium.org, davemoore+watch_chromium.org, oshima+watch_chromium.org, nkostylev+watch_chromium.org
Visibility:
Public.

Description

kiosk: Show enrollment screen first if needed. - Show enrollment screen by default after OOBE if the device is set to auto enrollment and is not yet enrolled; - Update screen loading code to defer Initialize() call for async loaded screen until the assets are loaded; BUG=322666 TEST=LoginUITest.ShowEnrollmentFirst Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=238262

Patch Set 1 #

Patch Set 2 : fix names #

Patch Set 3 : rebase, fix LOG(INFO) #

Messages

Total messages: 7 (0 generated)
xiyuan
7 years ago (2013-11-22 21:25:43 UTC) #1
xiyuan
ping?
7 years ago (2013-12-02 19:07:31 UTC) #2
zel
lgtm
7 years ago (2013-12-02 19:12:33 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/xiyuan@chromium.org/83293004/20001
7 years ago (2013-12-02 22:26:45 UTC) #4
commit-bot: I haz the power
Retried try job too often on chromium_presubmit for step(s) presubmit http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_presubmit&number=38954
7 years ago (2013-12-02 22:43:57 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/xiyuan@chromium.org/83293004/210001
7 years ago (2013-12-02 22:48:49 UTC) #6
commit-bot: I haz the power
7 years ago (2013-12-03 01:25:58 UTC) #7
Message was sent while issue was closed.
Change committed as 238262

Powered by Google App Engine
This is Rietveld 408576698