Update FWMP in TPM

chrome/browser/chromeos/login/enrollment/auto_enrollment_controller.h

Index: chrome/browser/chromeos/login/enrollment/auto_enrollment_controller.h
diff --git a/chrome/browser/chromeos/login/enrollment/auto_enrollment_controller.h b/chrome/browser/chromeos/login/enrollment/auto_enrollment_controller.h
index 5dfada546c763ff65cc1d1c2c95342c595885949..240f72c8ff2159398c74c9783e42de76b6fa4692 100644
--- a/chrome/browser/chromeos/login/enrollment/auto_enrollment_controller.h
+++ b/chrome/browser/chromeos/login/enrollment/auto_enrollment_controller.h
@@ -16,6 +16,12 @@
 #include "chrome/browser/chromeos/policy/auto_enrollment_client.h"
 #include "chrome/browser/chromeos/settings/device_settings_service.h"
 
+namespace cryptohome {
+

[Daniel Erat, 2017/03/06 21:18:25]

nit: i think we usually omit these blank lines when forward-declaring something
in a namespace, e.g.

namespace cryptohome {
class BaseReply;
}  // namespace cryptohome

[Andrew T Wilson (Slow), 2017/03/07 11:16:30]

nit: this is OK as-is, but typically I avoid blank lines in namespace blocks

[igorcov, 2017/03/09 12:22:57]

Done.

+class BaseReply;
+
+}  // namespace cryptohome
+
 namespace chromeos {
 
 // Drives the forced re-enrollment check (for historical reasons called
@@ -86,6 +92,15 @@ class AutoEnrollmentController {
   // Sets |state_| and notifies |progress_callbacks_|.
   void UpdateState(policy::AutoEnrollmentState state);
 
+  // D-Bus call to cryptohome to remove the firmware management parameters from

[Daniel Erat, 2017/03/06 21:18:25]

nit: "Makes a D-Bus call ..."

[igorcov, 2017/03/09 12:22:56]

Done.

+  // TPM.
+  void RemoveFwmp();

[Daniel Erat, 2017/03/06 21:18:25]

mind also renaming this to RemoveFirmwareManagementParameters?

[igorcov, 2017/03/09 12:22:57]

Done.

+
+  // Callback after the firmware management parameters is removed.

[Daniel Erat, 2017/03/06 21:18:25]

nit: s/is/are/, but maybe just:

// Callback for RemoveFirmwareManagementParameters().

instead?

[Andrew T Wilson (Slow), 2017/03/07 11:16:30]

nit: is->are

[igorcov, 2017/03/09 12:22:56]

Done.

+  void OnFwmpRemoved(chromeos::DBusMethodCallStatus call_status,

[Daniel Erat, 2017/03/06 21:18:25]

ditto here

[igorcov, 2017/03/09 12:22:57]

Done.

+                     bool result,
+                     const cryptohome::BaseReply& reply);
+
   // Handles timeout of the safeguard timer and stops waiting for a result.
   void Timeout();