Commit 40a5264a authored by Eduardo Cordeiro's avatar Eduardo Cordeiro Committed by Commit Bot

Cleaning up usage of ChildAccountDetection experiment, replacing it with...

Cleaning up usage of ChildAccountDetection experiment, replacing it with target OS checks to enforce that the feature is always enabled on ChromeOS and Android, and disabled elsewhere

Bug: 795857
Change-Id: I2cd752637bc0cb5549606062c52c3b539d08677d
Reviewed-on: https://chromium-review.googlesource.com/1002972Reviewed-by: default avatarRobert Kaplow <rkaplow@chromium.org>
Reviewed-by: default avatarRahul Chaturvedi <rkc@chromium.org>
Reviewed-by: default avatarMarc Treib <treib@chromium.org>
Commit-Queue: Eduardo Cordeiro <escordeiro@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550163}
parent 83bd7656
...@@ -37,10 +37,6 @@ ...@@ -37,10 +37,6 @@
#include "chrome/browser/chromeos/profiles/profile_helper.h" #include "chrome/browser/chromeos/profiles/profile_helper.h"
#endif #endif
#if !defined(OS_ANDROID)
const char kChildAccountDetectionFieldTrialName[] = "ChildAccountDetection";
#endif
const char kGaiaCookieManagerSource[] = "child_account_service"; const char kGaiaCookieManagerSource[] = "child_account_service";
// Normally, re-check the family info once per day. // Normally, re-check the family info once per day.
...@@ -90,15 +86,13 @@ ChildAccountService::~ChildAccountService() { ...@@ -90,15 +86,13 @@ ChildAccountService::~ChildAccountService() {
// static // static
bool ChildAccountService::IsChildAccountDetectionEnabled() { bool ChildAccountService::IsChildAccountDetectionEnabled() {
// Child account detection is always enabled on Android. // Child account detection is always enabled on Android and ChromeOS, and
#if !defined(OS_ANDROID) // disabled in other platforms.
const std::string group_name = #if defined(OS_ANDROID) || defined(OS_CHROMEOS)
base::FieldTrialList::FindFullName(kChildAccountDetectionFieldTrialName);
if (group_name == "Disabled")
return false;
#endif
return true; return true;
#else
return false;
#endif
} }
void ChildAccountService::RegisterProfilePrefs( void ChildAccountService::RegisterProfilePrefs(
......
...@@ -711,18 +711,6 @@ ...@@ -711,18 +711,6 @@
] ]
} }
], ],
"ChildAccountDetection": [
{
"platforms": [
"win"
],
"experiments": [
{
"name": "Disabled"
}
]
}
],
"ChromeDashboard": [ "ChromeDashboard": [
{ {
"platforms": [ "platforms": [
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment