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

Unified Diff: chrome/browser/chromeos/settings/stub_install_attributes.h

Issue 2737733003: Add Active Directory login UI tests (Closed)
Patch Set: Update after review Created 3 years, 9 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/settings/stub_install_attributes.h
diff --git a/chrome/browser/chromeos/settings/stub_install_attributes.h b/chrome/browser/chromeos/settings/stub_install_attributes.h
index 2358c2c35e090eb2759e80c0d2ed7b6e58fcf7df..461668754fcbb4b6e947dbbce881e0b133d69eaa 100644
--- a/chrome/browser/chromeos/settings/stub_install_attributes.h
+++ b/chrome/browser/chromeos/settings/stub_install_attributes.h
@@ -27,6 +27,10 @@ class StubInstallAttributes : public InstallAttributes {
// Setup as enterprise enrolled. (Clears existing configuration.)
void SetEnterprise(const std::string& domain, const std::string& device_id);
Thiemo Nagel 2017/03/21 13:44:22 Nit: Maybe rename to something with Cloud to make
Roman Sorokin (ftl) 2017/03/22 15:19:25 Make sense!
+ // Setup as Active Directory enterprise enrolled. (Clears existing
+ // configuration.)
+ void SetActiveDirectoryEnterprise(const std::string& realm);
Thiemo Nagel 2017/03/21 13:44:22 For better or worse, AD install attributes contain
Roman Sorokin (ftl) 2017/03/22 15:19:25 Done.
+
private:
DISALLOW_COPY_AND_ASSIGN(StubInstallAttributes);
};
@@ -47,6 +51,10 @@ class ScopedStubInstallAttributes {
const std::string& domain,
const std::string& device_id);
+ // Factory for Active Directory enterprise-type ScopedStubInstallAttributes.
+ static ScopedStubInstallAttributes CreateActiveDirectoryEnterprise(
+ const std::string& realm);
+
private:
ScopedStubInstallAttributes();
};

Powered by Google App Engine
This is Rietveld 408576698