Chromad: Switch AuthPolicyClient to use cros_system_api enums.

chrome/browser/ui/webui/chromeos/login/enrollment_screen_handler.cc

Index: chrome/browser/ui/webui/chromeos/login/enrollment_screen_handler.cc
diff --git a/chrome/browser/ui/webui/chromeos/login/enrollment_screen_handler.cc b/chrome/browser/ui/webui/chromeos/login/enrollment_screen_handler.cc
index 5121bfe826e674b74a31b65be374346dfd6a17d7..e6a790bf273f9e8f9f17f4df5b5f5381d220d10b 100644
--- a/chrome/browser/ui/webui/chromeos/login/enrollment_screen_handler.cc
+++ b/chrome/browser/ui/webui/chromeos/login/enrollment_screen_handler.cc
@@ -585,13 +585,16 @@ void EnrollmentScreenHandler::OnPasswordPipeReady(
 void EnrollmentScreenHandler::HandleAdDomainJoin(
     const std::string& machine_name,
     const std::string& user_name,
-    int code) {
-  if (code == 0) {
-    controller_->OnAdJoined(gaia::ExtractDomainName(user_name));
-    return;
+    authpolicy::ADJoinErrorType code) {

[Alexander Alekseev, 2016/12/21 14:45:30]

nit: please document here that code may be outside of
authpolicy::ADJoinErrorType domain because this value actually comes from DBUS
integer value . (Probably this is the reason you are using default: clause
here.)

[stevenjb, 2016/12/21 18:12:11]

authpolicy::ADJoinErrorType is a DBus constant, so that shouldn't be possible.
If we are concerned about that, we could test for that when we cast the integer
in the dbus code. We shouldn't have to document that in the implementation code,
that kind of defeats the purpose of using an enum.

[Roman Sorokin (ftl), 2016/12/23 10:03:04]

I added _COUNT member to the enum and check if the number we got from D-BUS is
within range.

+  switch (code) {
+    case authpolicy::AD_JOIN_ERROR_NONE:
+      controller_->OnAdJoined(gaia::ExtractDomainName(user_name));
+      return;
+    default:
+      // TODO(rsorokin): Add passing/displaying error codes. (see
+      // crbug.com/659984)
+      CallJS("invalidateAd", machine_name, user_name);
   }
-  // TODO(rsorokin): Add passing/displaying error codes. (see crbug.com/659984)
-  CallJS("invalidateAd", machine_name, user_name);
 }
 
 void EnrollmentScreenHandler::HandleRetry() {