Chromad: Increase D-Bus timeout for device policy fetch

chromeos/dbus/auth_policy_client.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 #include "chromeos/dbus/auth_policy_client.h"
 
 #include "base/bind.h"
 #include "base/memory/weak_ptr.h"
 #include "components/signin/core/account_id/account_id.h"
 #include "dbus/bus.h"
 #include "dbus/message.h"
 #include "dbus/object_proxy.h"
 
 namespace chromeos {
 
 namespace {
 
+// The first device policy fetch after joining Active Directory can be very slow
+// because machine credentials need to propagate through the AD deployment.
+const int kRefreshDevicePolicyTimeoutMilliseconds = 90000;
+
 authpolicy::ErrorType GetErrorFromReader(dbus::MessageReader* reader) {
   int32_t int_error;
   if (!reader->PopInt32(&int_error)) {
     DLOG(ERROR) << "AuthPolicyClient: Failed to get an error from the response";
     return authpolicy::ERROR_DBUS_FAILURE;
   }
   if (int_error < 0 || int_error >= authpolicy::ERROR_COUNT)
     return authpolicy::ERROR_UNKNOWN;
   return static_cast<authpolicy::ErrorType>(int_error);
 }
@@ skipping 30 matching lines @@
     writer.AppendFileDescriptor(password_fd);
     proxy_->CallMethod(&method_call, dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
                        base::Bind(&AuthPolicyClientImpl::HandleAuthCallback,
                                   weak_ptr_factory_.GetWeakPtr(), callback));
   }
 
   void RefreshDevicePolicy(const RefreshPolicyCallback& callback) override {
     dbus::MethodCall method_call(authpolicy::kAuthPolicyInterface,
                                  authpolicy::kAuthPolicyRefreshDevicePolicy);
     proxy_->CallMethod(
-        &method_call, dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
+        &method_call, kRefreshDevicePolicyTimeoutMilliseconds,
         base::Bind(&AuthPolicyClientImpl::HandleRefreshPolicyCallback,
                    weak_ptr_factory_.GetWeakPtr(), callback));
   }
 
   void RefreshUserPolicy(const AccountId& account_id,
                          const RefreshPolicyCallback& callback) override {
     DCHECK(account_id.GetAccountType() == AccountType::ACTIVE_DIRECTORY);
     dbus::MethodCall method_call(authpolicy::kAuthPolicyInterface,
                                  authpolicy::kAuthPolicyRefreshUserPolicy);
     dbus::MessageWriter writer(&method_call);
@@ skipping 65 matching lines @@
 AuthPolicyClient::AuthPolicyClient() {}
 
 AuthPolicyClient::~AuthPolicyClient() {}
 
 // static
 AuthPolicyClient* AuthPolicyClient::Create() {
   return new AuthPolicyClientImpl();
 }
 
 }  // namespace chromeos