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

Unified Diff: testing/variations/fieldtrial_testing_config_android.json

Issue 1668523002: [Password Manager] Switch password manager code to use the Feature framework. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Changes to fieldtrial_testing_config_*.json Created 4 years, 10 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: testing/variations/fieldtrial_testing_config_android.json
diff --git a/testing/variations/fieldtrial_testing_config_android.json b/testing/variations/fieldtrial_testing_config_android.json
index 1e6f1a5df9acfa6dcd3881742a4d276939d6980f..95f9d31f784930cffba5be4664fa7efdb1871b8a 100644
--- a/testing/variations/fieldtrial_testing_config_android.json
+++ b/testing/variations/fieldtrial_testing_config_android.json
@@ -1,4 +1,12 @@
{
+ "AffiliationBasedMatching": [
vabr (Chromium) 2016/02/26 09:43:41 I don't think we want this feature turned off on t
Pritam Nikam 2016/02/26 12:42:23 Acknowledged.
+ {
+ "group_name": "Disabled",
+ "params": {
+ "propagate_password_changes_to_web": "Disabled"
+ }
+ }
+ ],
"AndroidSessionNotifications": [
{
"group_name": "Disabled"
@@ -41,6 +49,14 @@
"group_name": "Enabled"
}
],
+ "AutofillSyncCredential": [
vabr (Chromium) 2016/02/26 09:43:41 I also do not see a point in having this override
Pritam Nikam 2016/02/26 12:42:23 Acknowledged.
+ {
+ "group_name": "DisallowSyncCredentials"
+ },
+ {
+ "group_name": "DisallowSyncCredentialsForReauth"
+ }
+ ],
"ChromotingQUIC": [
{
"group_name": "Enabled"
@@ -208,6 +224,14 @@
"group_name": "SmartLockBrandingSavePromptOnly"
}
],
+ "PasswordManagerDropSyncCredential": [
vabr (Chromium) 2016/02/26 09:43:41 Please drop this one as well, for the same reasons
Pritam Nikam 2016/02/26 12:42:23 Acknowledged.
+ {
+ "group_name": "Enabled"
+ },
+ {
+ "group_name": "Disabled"
+ }
+ ],
"PhysicalWeb": [
{
"group_name": "Enabled"

Powered by Google App Engine
This is Rietveld 408576698