Chromium Code Reviews| 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 = |