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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/preferences/password/SavePasswordsPreferencesTest.java

Issue 1877213003: [Smart Lock, UI] Fix dissapeared Auto-signin setting. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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/android/javatests/src/org/chromium/chrome/browser/preferences/password/SavePasswordsPreferencesTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/preferences/password/SavePasswordsPreferencesTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/preferences/password/SavePasswordsPreferencesTest.java
index 8b2d3b1a63dd9388c0cfdae5b70df626fab0d21d..32ebcece89211786e15578606ff81542ef81e949 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/preferences/password/SavePasswordsPreferencesTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/preferences/password/SavePasswordsPreferencesTest.java
@@ -6,16 +6,16 @@ package org.chromium.chrome.browser.preferences.password;
import android.test.suitebuilder.annotation.SmallTest;
-import org.chromium.base.CommandLine;
import org.chromium.base.ThreadUtils;
+import org.chromium.base.test.util.CommandLineFlags;
import org.chromium.base.test.util.Feature;
+import org.chromium.chrome.browser.ChromeFeatureList;
import org.chromium.chrome.browser.preferences.ChromeBaseCheckBoxPreference;
import org.chromium.chrome.browser.preferences.ChromeSwitchPreference;
import org.chromium.chrome.browser.preferences.PrefServiceBridge;
import org.chromium.chrome.browser.preferences.Preferences;
import org.chromium.chrome.browser.preferences.PreferencesTest;
import org.chromium.content.browser.test.NativeLibraryTestBase;
-import org.chromium.content.common.ContentSwitches;
/**
* Tests for the "Save Passwords" settings screen.
@@ -25,7 +25,6 @@ public class SavePasswordsPreferencesTest extends NativeLibraryTestBase {
@Override
protected void setUp() throws Exception {
super.setUp();
- CommandLine.getInstance().appendSwitch(ContentSwitches.ENABLE_CREDENTIAL_MANAGER_API);
loadNativeLibraryAndInitBrowserProcess();
}
@@ -86,6 +85,7 @@ public class SavePasswordsPreferencesTest extends NativeLibraryTestBase {
* Ensure that the "Auto Sign-in" switch in "Save Passwords" settings actually enables and
* disables auto sign-in.
*/
+ @CommandLineFlags.Add({SafePasswordPreferences.ENABLE_CREDENTIAL_MANAGER_API})
@SmallTest
@Feature({"Preferences"})
public void testAutoSignInCheckbox() throws Exception {
@@ -102,8 +102,10 @@ public class SavePasswordsPreferencesTest extends NativeLibraryTestBase {
ThreadUtils.runOnUiThreadBlocking(new Runnable() {
@Override
public void run() {
- assertTrue(CommandLine.getInstance().hasSwitch(
- ContentSwitches.ENABLE_CREDENTIAL_MANAGER_API));
+ if (!ChromeFeatureList.isEnabled(
Bernhard Bauer 2016/04/14 13:57:48 This shouldn't be necessary now. In fact, if for s
+ SavePasswordsPreferences.ENABLE_CREDENTIAL_MANAGER_API)) {
+ return;
+ }
SavePasswordsPreferences passwordPrefs =
(SavePasswordsPreferences) preferences.getFragmentForTest();
ChromeBaseCheckBoxPreference onOffSwitch =
@@ -127,8 +129,10 @@ public class SavePasswordsPreferencesTest extends NativeLibraryTestBase {
ThreadUtils.runOnUiThreadBlocking(new Runnable() {
@Override
public void run() {
- assertTrue(CommandLine.getInstance().hasSwitch(
- ContentSwitches.ENABLE_CREDENTIAL_MANAGER_API));
+ if (!ChromeFeatureList.isEnabled(
+ SafePasswordPreferences.ENABLE_CREDENTIAL_MANAGER_API)) {
+ return;
+ }
SavePasswordsPreferences passwordPrefs =
(SavePasswordsPreferences) preferences2.getFragmentForTest();
ChromeBaseCheckBoxPreference onOffSwitch =

Powered by Google App Engine
This is Rietveld 408576698