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

Unified Diff: chrome/browser/chromeos/policy/blocking_login_browsertest.cc

Issue 2403053003: Remove unused email from install attributes and add support for AD. (Closed)
Patch Set: Address Maksim's comments. Created 4 years, 2 months 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
Index: chrome/browser/chromeos/policy/blocking_login_browsertest.cc
diff --git a/chrome/browser/chromeos/policy/blocking_login_browsertest.cc b/chrome/browser/chromeos/policy/blocking_login_browsertest.cc
index bac4402597538297aafedfd48dfac92a15be5840..ca55ceb4a52fece580eb523032b9d8ff6834d778 100644
--- a/chrome/browser/chromeos/policy/blocking_login_browsertest.cc
+++ b/chrome/browser/chromeos/policy/blocking_login_browsertest.cc
@@ -138,11 +138,11 @@ class BlockingLoginTest
->browser_policy_connector_chromeos();
}
- void EnrollDevice(const std::string& username) {
+ void EnrollDevice(const std::string& domain) {
base::RunLoop loop;
InstallAttributes::LockResult result;
browser_policy_connector()->GetInstallAttributes()->LockDevice(
- username, policy::DEVICE_MODE_ENTERPRISE, "100200300",
+ policy::DEVICE_MODE_ENTERPRISE, domain, std::string(), "100200300",
base::Bind(&CopyLockResult, &loop, &result));
loop.Run();
EXPECT_EQ(InstallAttributes::LOCK_SUCCESS, result);
@@ -249,7 +249,7 @@ IN_PROC_BROWSER_TEST_P(BlockingLoginTest, LoginBlocksForUser) {
// Enroll the device, if enrollment is enabled for this test instance.
if (GetParam().enroll_device) {
- EnrollDevice(kUsername);
+ EnrollDevice(kDomain);
EXPECT_FALSE(user_manager->IsUserLoggedIn());
EXPECT_TRUE(browser_policy_connector()->IsEnterpriseManaged());

Powered by Google App Engine
This is Rietveld 408576698