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

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

Issue 2862003002: Enable policy-imported root CA certificates for kiosk and AD sessions (Closed)
Patch Set: Added browsertests. Created 3 years, 7 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
« no previous file with comments | « no previous file | chrome/browser/chromeos/policy/user_network_configuration_updater_factory_browsertest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/policy/user_network_configuration_updater_factory.cc
diff --git a/chrome/browser/chromeos/policy/user_network_configuration_updater_factory.cc b/chrome/browser/chromeos/policy/user_network_configuration_updater_factory.cc
index e13ed617a4b8da8a2fe01406cc2e16205422294b..9beac92a3603c5a85a4614c4a2d12386efb4f76c 100644
--- a/chrome/browser/chromeos/policy/user_network_configuration_updater_factory.cc
+++ b/chrome/browser/chromeos/policy/user_network_configuration_updater_factory.cc
@@ -72,7 +72,9 @@ KeyedService* UserNetworkConfigurationUpdaterFactory::BuildServiceInstanceFor(
if (user != user_manager::UserManager::Get()->GetPrimaryUser())
return NULL;
- const bool allow_trusted_certs_from_policy = user->HasGaiaAccount();
+ // Disallow trusted root certs for public sessions.
+ const bool allow_trusted_certs_from_policy =
+ user->GetType() != user_manager::USER_TYPE_PUBLIC_ACCOUNT;
Andrew T Wilson (Slow) 2017/05/17 15:03:54 To be clear - what behavior does this expose for g
pmarko 2017/05/17 15:18:07 This is only relevant for user policy, and guest s
ProfilePolicyConnector* profile_connector =
ProfilePolicyConnectorFactory::GetForBrowserContext(context);
« no previous file with comments | « no previous file | chrome/browser/chromeos/policy/user_network_configuration_updater_factory_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698