Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3482)

Unified Diff: chrome/browser/policy/cloud_policy_browsertest.cc

Issue 11415094: Split UserCloudPolicyManager implementation. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix DeviceCloudPolicyManagerChromeOSTest.EnrolledDevice failure. Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/policy/browser_policy_connector.cc ('k') | chrome/browser/policy/cloud_policy_manager.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/policy/cloud_policy_browsertest.cc
diff --git a/chrome/browser/policy/cloud_policy_browsertest.cc b/chrome/browser/policy/cloud_policy_browsertest.cc
index 38629adc69a085bf96a23bd936b6377810c78347..f0fa0f9f510b06b8c7ce881a432f1fc6836b8587 100644
--- a/chrome/browser/policy/cloud_policy_browsertest.cc
+++ b/chrome/browser/policy/cloud_policy_browsertest.cc
@@ -19,7 +19,6 @@
#include "chrome/browser/policy/policy_service.h"
#include "chrome/browser/policy/proto/chrome_settings.pb.h"
#include "chrome/browser/policy/proto/cloud_policy.pb.h"
-#include "chrome/browser/policy/user_cloud_policy_manager.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/ui/browser.h"
#include "chrome/common/chrome_notification_types.h"
@@ -37,7 +36,10 @@
#if defined(OS_CHROMEOS)
#include "chrome/browser/chromeos/login/user_manager.h"
+#include "chrome/browser/policy/user_cloud_policy_manager_chromeos.h"
#else
+#include "chrome/browser/policy/user_cloud_policy_manager.h"
+#include "chrome/browser/policy/user_cloud_policy_manager_factory.h"
#include "chrome/browser/signin/signin_manager.h"
#include "chrome/browser/signin/signin_manager_factory.h"
#endif
@@ -125,25 +127,29 @@ void SetUpNewStackBeforeCreatingBrowser() {
}
void SetUpNewStackAfterCreatingBrowser(Browser* browser) {
-#if !defined(OS_CHROMEOS)
+ BrowserPolicyConnector* connector =
+ g_browser_process->browser_policy_connector();
+ connector->ScheduleServiceInitialization(0);
+
+#if defined(OS_CHROMEOS)
+ connector->InitializeUserPolicy(GetTestUser(), true);
+ UserCloudPolicyManagerChromeOS* policy_manager =
+ connector->GetUserCloudPolicyManager();
+ ASSERT_TRUE(policy_manager);
+#else
// Mock a signed-in user. This is used by the UserCloudPolicyStore to pass the
// username to the UserCloudPolicyValidator.
SigninManager* signin_manager =
SigninManagerFactory::GetForProfile(browser->profile());
ASSERT_TRUE(signin_manager);
signin_manager->SetAuthenticatedUsername(GetTestUser());
-#endif
-
- BrowserPolicyConnector* connector =
- g_browser_process->browser_policy_connector();
- connector->ScheduleServiceInitialization(0);
UserCloudPolicyManager* policy_manager =
- browser->profile()->GetUserCloudPolicyManager();
+ UserCloudPolicyManagerFactory::GetForProfile(browser->profile());
ASSERT_TRUE(policy_manager);
policy_manager->Initialize(g_browser_process->local_state(),
- connector->device_management_service(),
- policy::USER_AFFILIATION_MANAGED);
+ connector->device_management_service());
+#endif // defined(OS_CHROMEOS)
ASSERT_TRUE(policy_manager->cloud_policy_client());
base::RunLoop run_loop;
« no previous file with comments | « chrome/browser/policy/browser_policy_connector.cc ('k') | chrome/browser/policy/cloud_policy_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698