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

Issue 11361187: Merge 166784 - Check for login screen availibility before showing spinner. (Closed)

Created:
8 years, 1 month ago by zel
Modified:
8 years, 1 month ago
Reviewers:
rkc
CC:
chromium-reviews, oshima+watch_chromium.org, stevenjb+watch_chromium.org, nkostylev+watch_chromium.org, davemoore+watch_chromium.org
Visibility:
Public.

Description

Merge 166784 - Check for login screen availibility before showing spinner. In retail mode, if for any reason, the login UI underneath the screensaver goes away (possibly due to a login happening while the screensaver was still starting up), we currently crash. Instead in that case, we want to just not try to show a spinner and close the screensaver and log the user in instead. R=nkostylev@chromium.org BUG=159433 Review URL: https://chromiumcodereview.appspot.com/11363109 TBR=rkc@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=166935

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -15 lines) Patch
M chrome/browser/chromeos/kiosk_mode/kiosk_mode_screensaver.cc View 1 chunk +20 lines, -15 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
zel
8 years, 1 month ago (2012-11-09 18:33:11 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698