Index: chrome/browser/android/signin/signin_manager_android.cc |
diff --git a/chrome/browser/android/signin/signin_manager_android.cc b/chrome/browser/android/signin/signin_manager_android.cc |
index 9015d2126b30b7f9afe56e8607c84ead51e61b4e..b5d45468b3cec92817824bd2d608bbe963b2477f 100644 |
--- a/chrome/browser/android/signin/signin_manager_android.cc |
+++ b/chrome/browser/android/signin/signin_manager_android.cc |
@@ -20,6 +20,8 @@ |
#include "chrome/browser/browsing_data/browsing_data_helper.h" |
#include "chrome/browser/browsing_data/browsing_data_remover.h" |
#include "chrome/browser/browsing_data/browsing_data_remover_factory.h" |
+#include "chrome/browser/policy/cloud/user_cloud_policy_manager_factory.h" |
+#include "chrome/browser/policy/cloud/user_policy_signin_service_factory.h" |
#include "chrome/browser/policy/cloud/user_policy_signin_service_mobile.h" |
#include "chrome/browser/profiles/profile_manager.h" |
#include "chrome/browser/signin/account_tracker_service_factory.h" |
@@ -27,6 +29,10 @@ |
#include "chrome/browser/signin/profile_oauth2_token_service_factory.h" |
#include "chrome/browser/signin/signin_manager_factory.h" |
#include "components/bookmarks/browser/bookmark_model.h" |
+#include "components/policy/core/browser/browser_policy_connector.h" |
+#include "components/policy/core/common/cloud/cloud_policy_core.h" |
+#include "components/policy/core/common/cloud/cloud_policy_store.h" |
+#include "components/policy/core/common/cloud/user_cloud_policy_manager.h" |
#include "components/prefs/pref_service.h" |
#include "components/signin/core/browser/account_tracker_service.h" |
#include "components/signin/core/browser/profile_oauth2_token_service.h" |
@@ -34,20 +40,10 @@ |
#include "components/signin/core/browser/signin_metrics.h" |
#include "components/signin/core/common/profile_management_switches.h" |
#include "components/signin/core/common/signin_pref_names.h" |
+#include "google_apis/gaia/gaia_auth_util.h" |
#include "google_apis/gaia/gaia_constants.h" |
#include "jni/SigninManager_jni.h" |
- |
-#if defined(ENABLE_CONFIGURATION_POLICY) |
-#include "chrome/browser/policy/cloud/user_cloud_policy_manager_factory.h" |
-#include "chrome/browser/policy/cloud/user_policy_signin_service_factory.h" |
-#include "chrome/browser/policy/cloud/user_policy_signin_service_mobile.h" |
-#include "components/policy/core/browser/browser_policy_connector.h" |
-#include "components/policy/core/common/cloud/cloud_policy_core.h" |
-#include "components/policy/core/common/cloud/cloud_policy_store.h" |
-#include "components/policy/core/common/cloud/user_cloud_policy_manager.h" |
-#include "google_apis/gaia/gaia_auth_util.h" |
#include "net/url_request/url_request_context_getter.h" |
-#endif |
using bookmarks::BookmarkModel; |
@@ -103,7 +99,6 @@ void SigninManagerAndroid::CheckPolicyBeforeSignIn( |
JNIEnv* env, |
const JavaParamRef<jobject>& obj, |
const JavaParamRef<jstring>& username) { |
-#if defined(ENABLE_CONFIGURATION_POLICY) |
username_ = base::android::ConvertJavaStringToUTF8(env, username); |
policy::UserPolicySigninService* service = |
policy::UserPolicySigninServiceFactory::GetForProfile(profile_); |
@@ -113,20 +108,11 @@ void SigninManagerAndroid::CheckPolicyBeforeSignIn( |
.account_id, |
base::Bind(&SigninManagerAndroid::OnPolicyRegisterDone, |
weak_factory_.GetWeakPtr())); |
-#else |
- // This shouldn't be called when ShouldLoadPolicyForUser() is false. |
- NOTREACHED(); |
- base::android::ScopedJavaLocalRef<jstring> domain; |
- Java_SigninManager_onPolicyCheckedBeforeSignIn(env, |
- java_signin_manager_.obj(), |
- domain.obj()); |
-#endif |
} |
void SigninManagerAndroid::FetchPolicyBeforeSignIn( |
JNIEnv* env, |
const JavaParamRef<jobject>& obj) { |
-#if defined(ENABLE_CONFIGURATION_POLICY) |
if (!dm_token_.empty()) { |
policy::UserPolicySigninService* service = |
policy::UserPolicySigninServiceFactory::GetForProfile(profile_); |
@@ -141,7 +127,7 @@ void SigninManagerAndroid::FetchPolicyBeforeSignIn( |
client_id_.clear(); |
return; |
} |
-#endif |
+ |
// This shouldn't be called when ShouldLoadPolicyForUser() is false, or when |
// CheckPolicyBeforeSignIn() failed. |
NOTREACHED(); |
@@ -152,11 +138,9 @@ void SigninManagerAndroid::FetchPolicyBeforeSignIn( |
void SigninManagerAndroid::AbortSignIn( |
JNIEnv* env, |
const base::android::JavaParamRef<jobject>& obj) { |
-#if defined(ENABLE_CONFIGURATION_POLICY) |
policy::UserPolicySigninService* service = |
policy::UserPolicySigninServiceFactory::GetForProfile(profile_); |
service->ShutdownUserCloudPolicyManager(); |
-#endif |
} |
void SigninManagerAndroid::OnSignInCompleted( |
@@ -180,7 +164,6 @@ SigninManagerAndroid::GetManagementDomain(JNIEnv* env, |
const JavaParamRef<jobject>& obj) { |
base::android::ScopedJavaLocalRef<jstring> domain; |
-#if defined(ENABLE_CONFIGURATION_POLICY) |
policy::UserCloudPolicyManager* manager = |
policy::UserCloudPolicyManagerFactory::GetForBrowserContext(profile_); |
policy::CloudPolicyStore* store = manager->core()->store(); |
@@ -190,7 +173,6 @@ SigninManagerAndroid::GetManagementDomain(JNIEnv* env, |
base::android::ConvertUTF8ToJavaString( |
env, gaia::ExtractDomainName(store->policy()->username()))); |
} |
-#endif |
return domain; |
} |
@@ -208,8 +190,6 @@ void SigninManagerAndroid::WipeProfileData( |
weak_factory_.GetWeakPtr(), java_callback)); |
} |
-#if defined(ENABLE_CONFIGURATION_POLICY) |
- |
void SigninManagerAndroid::OnPolicyRegisterDone( |
const std::string& dm_token, |
const std::string& client_id) { |
@@ -238,8 +218,6 @@ void SigninManagerAndroid::OnPolicyFetchDone(bool success) { |
java_signin_manager_.obj()); |
} |
-#endif |
- |
void SigninManagerAndroid::OnBrowsingDataRemoverDone( |
const base::android::ScopedJavaGlobalRef<jobject>& callback) { |
BookmarkModel* model = BookmarkModelFactory::GetForProfile(profile_); |
@@ -307,13 +285,9 @@ static jboolean ShouldLoadPolicyForUser( |
JNIEnv* env, |
const JavaParamRef<jobject>& obj, |
const JavaParamRef<jstring>& j_username) { |
-#if defined(ENABLE_CONFIGURATION_POLICY) |
std::string username = |
base::android::ConvertJavaStringToUTF8(env, j_username); |
return !policy::BrowserPolicyConnector::IsNonEnterpriseUser(username); |
-#else |
- return false; |
-#endif |
} |
// static |