Chromad: Use DM server reply to determine enrollment type

chrome/browser/chromeos/login/screens/host_pairing_screen.cc

Index: chrome/browser/chromeos/login/screens/host_pairing_screen.cc
diff --git a/chrome/browser/chromeos/login/screens/host_pairing_screen.cc b/chrome/browser/chromeos/login/screens/host_pairing_screen.cc
index 402cabbf36392fe4b95a34aa2703eea07219f510..8d8e296d960246660758f5a4cb1842eb9303d2ad 100644
--- a/chrome/browser/chromeos/login/screens/host_pairing_screen.cc
+++ b/chrome/browser/chromeos/login/screens/host_pairing_screen.cc
@@ -26,7 +26,6 @@ HostPairingScreen::HostPairingScreen(
       delegate_(delegate),
       actor_(actor),
       remora_controller_(remora_controller),
-      current_stage_(HostPairingController::STAGE_NONE),
       weak_ptr_factory_(this) {
   actor_->SetDelegate(this);
   remora_controller_->AddObserver(this);
@@ -146,7 +145,7 @@ void HostPairingScreen::EnrollHostRequested(const std::string& auth_token) {
           ->browser_policy_connector_chromeos()
           ->GetPrescribedEnrollmentConfig();
   enrollment_helper_ = EnterpriseEnrollmentHelper::Create(
-      this, enrollment_config, std::string());
+      this, this, enrollment_config, std::string());
   enrollment_helper_->EnrollUsingToken(auth_token);
   remora_controller_->OnEnrollmentStatusChanged(
       HostPairingController::ENROLLMENT_STATUS_ENROLLING);
@@ -200,4 +199,8 @@ void HostPairingScreen::OnAnyEnrollmentError() {
       HostPairingController::ENROLLMENT_STATUS_FAILURE);
 }
 
+void HostPairingScreen::JoinDomain(OnDomainJoinedCallback on_joined_callback) {
+  NOTREACHED();

[Thiemo Nagel, 2017/02/14 14:45:00]

Why do you inherit the Delegate and then implement it NOTREACHED()?  That seems
questionable design to me.

[Roman Sorokin (ftl), 2017/02/14 16:01:33]

Created FakeActiveDirectoryJoinDelegate

+}
+
 }  // namespace chromeos