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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2640533002: cros: Make sure PIN is disabled if the user is enterprise enrolled on m56. (Closed)

Created:
3 years, 11 months ago by jdufault
Modified:
3 years, 11 months ago
Reviewers:
sammiequon, stevenjb
CC:
chromium-reviews, alemate+watch_chromium.org, achuith+watch_chromium.org, oshima+watch_chromium.org, davemoore+watch_chromium.org
Project:
chromium
Visibility:
Public.

Description

cros: Make sure PIN is disabled if the user is enterprise enrolled on m56. This patch will only land on m56. BUG=681958

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M chrome/browser/chromeos/login/quick_unlock/quick_unlock_utils.cc View 1 chunk +6 lines, -0 lines 1 comment Download

Messages

Total messages: 6 (3 generated)
jdufault
stevenjb@, sammiequon@, PTAL. Thanks.
3 years, 11 months ago (2017-01-17 20:43:41 UTC) #3
stevenjb
lgtm https://codereview.chromium.org/2640533002/diff/1/chrome/browser/chromeos/login/quick_unlock/quick_unlock_utils.cc File chrome/browser/chromeos/login/quick_unlock/quick_unlock_utils.cc (right): https://codereview.chromium.org/2640533002/diff/1/chrome/browser/chromeos/login/quick_unlock/quick_unlock_utils.cc#newcode55 chrome/browser/chromeos/login/quick_unlock/quick_unlock_utils.cc:55: return false; {}
3 years, 11 months ago (2017-01-17 20:58:21 UTC) #5
sammiequon
3 years, 11 months ago (2017-01-17 21:15:30 UTC) #6 
lgtm

Powered by Google App Engine
This is Rietveld 408576698