Skip to content

Commit

Permalink
Add comment to AutoEnrollmentController
Browse files Browse the repository at this point in the history
BUG=none

Review-Url: https://codereview.chromium.org/2906873002
Cr-Commit-Position: refs/heads/master@{#474984}
  • Loading branch information
thiemonagel authored and Commit bot committed May 26, 2017
1 parent ca6d7cb commit 17a106a
Showing 1 changed file with 6 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -324,6 +324,12 @@ void AutoEnrollmentController::OnFirmwareManagementParametersRemoved(
}

void AutoEnrollmentController::Timeout() {
// When tightening the FRE flows, as a cautionary measure (to prevent
// interference with consumer devices) timeout was chosen to only enforce FRE
// for EXPLICTLY_REQUIRED.
// TODO(igorcov): Investigate the remaining causes of hitting timeout and
// potentially either remove the timeout altogether or enforce FRE in the
// REQUIRED case as well.
// TODO(mnissler): Add UMA to track results of auto-enrollment checks.
if (client_start_weak_factory_.HasWeakPtrs() &&
fre_requirement_ != EXPLICITLY_REQUIRED) {
Expand Down

0 comments on commit 17a106a

Please sign in to comment.