Commit 536540ec authored by Natalie Chouinard's avatar Natalie Chouinard Committed by Commit Bot

[Android] Rename Main settings fragment

Following the naming conventions decided in
go/clank-preferences-refactor, this change replaces the "Preferences"
suffix with "Settings".

Bug: 1030446
Change-Id: Icc4f86b119194cc92716b5452711ea8dfafd8883
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2014623Reviewed-by: default avatarGang Wu <gangwu@chromium.org>
Reviewed-by: default avatarSky Malice <skym@chromium.org>
Reviewed-by: default avatarGanggui Tang <gogerald@chromium.org>
Commit-Queue: Natalie Chouinard <chouinard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#735132}
parent 27bce439
...@@ -1366,7 +1366,7 @@ chrome_java_sources = [ ...@@ -1366,7 +1366,7 @@ chrome_java_sources = [
"java/src/org/chromium/chrome/browser/settings/HyperlinkPreference.java", "java/src/org/chromium/chrome/browser/settings/HyperlinkPreference.java",
"java/src/org/chromium/chrome/browser/settings/LearnMorePreference.java", "java/src/org/chromium/chrome/browser/settings/LearnMorePreference.java",
"java/src/org/chromium/chrome/browser/settings/LocationSettings.java", "java/src/org/chromium/chrome/browser/settings/LocationSettings.java",
"java/src/org/chromium/chrome/browser/settings/MainPreferences.java", "java/src/org/chromium/chrome/browser/settings/MainSettings.java",
"java/src/org/chromium/chrome/browser/settings/NfcSystemLevelSetting.java", "java/src/org/chromium/chrome/browser/settings/NfcSystemLevelSetting.java",
"java/src/org/chromium/chrome/browser/settings/SettingsActivity.java", "java/src/org/chromium/chrome/browser/settings/SettingsActivity.java",
"java/src/org/chromium/chrome/browser/settings/SettingsLauncher.java", "java/src/org/chromium/chrome/browser/settings/SettingsLauncher.java",
......
...@@ -42,7 +42,7 @@ import java.util.Map; ...@@ -42,7 +42,7 @@ import java.util.Map;
/** /**
* The main settings screen, shown when the user first opens Settings. * The main settings screen, shown when the user first opens Settings.
*/ */
public class MainPreferences extends PreferenceFragmentCompat public class MainSettings extends PreferenceFragmentCompat
implements TemplateUrlService.LoadListener, ProfileSyncService.SyncStateChangedListener, implements TemplateUrlService.LoadListener, ProfileSyncService.SyncStateChangedListener,
SigninManager.SignInStateObserver { SigninManager.SignInStateObserver {
public static final String PREF_ACCOUNT_SECTION = "account_section"; public static final String PREF_ACCOUNT_SECTION = "account_section";
...@@ -62,7 +62,7 @@ public class MainPreferences extends PreferenceFragmentCompat ...@@ -62,7 +62,7 @@ public class MainPreferences extends PreferenceFragmentCompat
private final Map<String, Preference> mAllPreferences = new HashMap<>(); private final Map<String, Preference> mAllPreferences = new HashMap<>();
private SignInPreference mSignInPreference; private SignInPreference mSignInPreference;
public MainPreferences() { public MainSettings() {
setHasOptionsMenu(true); setHasOptionsMenu(true);
mManagedPreferenceDelegate = createManagedPreferenceDelegate(); mManagedPreferenceDelegate = createManagedPreferenceDelegate();
} }
......
...@@ -102,7 +102,7 @@ public class SettingsActivity extends ChromeBaseAppCompatActivity ...@@ -102,7 +102,7 @@ public class SettingsActivity extends ChromeBaseAppCompatActivity
// If savedInstanceState is non-null, then the activity is being // If savedInstanceState is non-null, then the activity is being
// recreated and super.onCreate() has already recreated the fragment. // recreated and super.onCreate() has already recreated the fragment.
if (savedInstanceState == null) { if (savedInstanceState == null) {
if (initialFragment == null) initialFragment = MainPreferences.class.getName(); if (initialFragment == null) initialFragment = MainSettings.class.getName();
Fragment fragment = Fragment.instantiate(this, initialFragment, initialArguments); Fragment fragment = Fragment.instantiate(this, initialFragment, initialArguments);
getSupportFragmentManager() getSupportFragmentManager()
......
...@@ -44,7 +44,7 @@ public class AutofillProfileEditorPreference extends Preference { ...@@ -44,7 +44,7 @@ public class AutofillProfileEditorPreference extends Preference {
protected void onClick() { protected void onClick() {
Bundle extras = getExtras(); Bundle extras = getExtras();
// We know which profile to edit based on the GUID stuffed in our extras // We know which profile to edit based on the GUID stuffed in our extras
// by MainPreferences. // by MainSettings.
mGUID = extras.getString(AutofillEditorBase.AUTOFILL_GUID); mGUID = extras.getString(AutofillEditorBase.AUTOFILL_GUID);
prepareEditorDialog(); prepareEditorDialog();
prepareAddressEditor(); prepareAddressEditor();
......
...@@ -32,7 +32,7 @@ public class AutofillServerProfileFragment ...@@ -32,7 +32,7 @@ public class AutofillServerProfileFragment
getActivity().setTitle(R.string.autofill_edit_profile); getActivity().setTitle(R.string.autofill_edit_profile);
// We know which card to display based on the GUID stuffed in // We know which card to display based on the GUID stuffed in
// our extras by MainPreferences. // our extras by MainSettings.
Bundle extras = getArguments(); Bundle extras = getArguments();
if (extras != null) { if (extras != null) {
mGUID = extras.getString(AutofillEditorBase.AUTOFILL_GUID); mGUID = extras.getString(AutofillEditorBase.AUTOFILL_GUID);
......
...@@ -45,7 +45,7 @@ public class PasswordViewingTypeTest { ...@@ -45,7 +45,7 @@ public class PasswordViewingTypeTest {
@Rule @Rule
public AccountManagerTestRule mAccountManagerTestRule = new AccountManagerTestRule(); public AccountManagerTestRule mAccountManagerTestRule = new AccountManagerTestRule();
private MainPreferences mMainPreferences; private MainSettings mMainSettings;
private ChromeBasePreference mPasswordsPref; private ChromeBasePreference mPasswordsPref;
private Context mContext; private Context mContext;
private MockSyncContentResolverDelegate mSyncContentResolverDelegate; private MockSyncContentResolverDelegate mSyncContentResolverDelegate;
...@@ -57,11 +57,11 @@ public class PasswordViewingTypeTest { ...@@ -57,11 +57,11 @@ public class PasswordViewingTypeTest {
setupTestAccount(); setupTestAccount();
mSyncContentResolverDelegate = new MockSyncContentResolverDelegate(); mSyncContentResolverDelegate = new MockSyncContentResolverDelegate();
mContext = InstrumentationRegistry.getTargetContext(); mContext = InstrumentationRegistry.getTargetContext();
mMainPreferences = (MainPreferences) startMainPreferences( mMainSettings = (MainSettings) startMainSettings(
InstrumentationRegistry.getInstrumentation(), mContext) InstrumentationRegistry.getInstrumentation(), mContext)
.getMainFragment(); .getMainFragment();
mPasswordsPref = (ChromeBasePreference) mMainPreferences.findPreference( mPasswordsPref = (ChromeBasePreference) mMainSettings.findPreference(
MainPreferences.PREF_SAVED_PASSWORDS); MainSettings.PREF_SAVED_PASSWORDS);
AndroidSyncSettings.overrideForTests(mSyncContentResolverDelegate, null); AndroidSyncSettings.overrideForTests(mSyncContentResolverDelegate, null);
mAuthority = AndroidSyncSettings.get().getContractAuthority(); mAuthority = AndroidSyncSettings.get().getContractAuthority();
AndroidSyncSettings.get().updateAccount(mAccount); AndroidSyncSettings.get().updateAccount(mAccount);
...@@ -80,13 +80,13 @@ public class PasswordViewingTypeTest { ...@@ -80,13 +80,13 @@ public class PasswordViewingTypeTest {
} }
/** /**
* Launches the main preferences. * Launches the main settings.
*/ */
private static SettingsActivity startMainPreferences( private static SettingsActivity startMainSettings(
Instrumentation instrumentation, final Context mContext) { Instrumentation instrumentation, final Context mContext) {
Intent intent = SettingsLauncher.getInstance().createIntentForSettingsPage( Intent intent = SettingsLauncher.getInstance().createIntentForSettingsPage(
mContext, MainPreferences.class.getName()); mContext, MainSettings.class.getName());
Activity activity = (SettingsActivity) instrumentation.startActivitySync(intent); Activity activity = instrumentation.startActivitySync(intent);
return (SettingsActivity) activity; return (SettingsActivity) activity;
} }
...@@ -159,7 +159,7 @@ public class PasswordViewingTypeTest { ...@@ -159,7 +159,7 @@ public class PasswordViewingTypeTest {
overrideProfileSyncService(true); overrideProfileSyncService(true);
Assert.assertEquals( Assert.assertEquals(
SavePasswordsPreferences.class.getCanonicalName(), mPasswordsPref.getFragment()); SavePasswordsPreferences.class.getCanonicalName(), mPasswordsPref.getFragment());
Assert.assertNotNull(mMainPreferences.getActivity().getIntent()); Assert.assertNotNull(mMainSettings.getActivity().getIntent());
} }
/** /**
......
...@@ -66,7 +66,7 @@ public class SettingsActivityTest { ...@@ -66,7 +66,7 @@ public class SettingsActivityTest {
}); });
SettingsActivityTest.startSettingsActivity( SettingsActivityTest.startSettingsActivity(
InstrumentationRegistry.getInstrumentation(), MainPreferences.class.getName()); InstrumentationRegistry.getInstrumentation(), MainSettings.class.getName());
onView(withText(R.string.prefs_saved_passwords_title)).perform(click()); onView(withText(R.string.prefs_saved_passwords_title)).perform(click());
onView(withText(R.string.prefs_saved_passwords)).check(matches(isDisplayed())); onView(withText(R.string.prefs_saved_passwords)).check(matches(isDisplayed()));
......
...@@ -21,7 +21,7 @@ import org.chromium.base.test.util.FlakyTest; ...@@ -21,7 +21,7 @@ import org.chromium.base.test.util.FlakyTest;
import org.chromium.base.test.util.RetryOnFailure; import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.browser.init.ChromeBrowserInitializer; import org.chromium.chrome.browser.init.ChromeBrowserInitializer;
import org.chromium.chrome.browser.search_engines.TemplateUrlServiceFactory; import org.chromium.chrome.browser.search_engines.TemplateUrlServiceFactory;
import org.chromium.chrome.browser.settings.MainPreferences; import org.chromium.chrome.browser.settings.MainSettings;
import org.chromium.chrome.browser.settings.ManagedPreferenceDelegate; import org.chromium.chrome.browser.settings.ManagedPreferenceDelegate;
import org.chromium.chrome.browser.settings.SettingsActivity; import org.chromium.chrome.browser.settings.SettingsActivity;
import org.chromium.chrome.browser.settings.SettingsActivityTest; import org.chromium.chrome.browser.settings.SettingsActivityTest;
...@@ -143,11 +143,11 @@ public class SearchEngineSettingsTest { ...@@ -143,11 +143,11 @@ public class SearchEngineSettingsTest {
SettingsActivity settingsActivity = ActivityUtils.waitForActivity( SettingsActivity settingsActivity = ActivityUtils.waitForActivity(
InstrumentationRegistry.getInstrumentation(), SettingsActivity.class); InstrumentationRegistry.getInstrumentation(), SettingsActivity.class);
final MainPreferences mainPreferences = final MainSettings mainSettings =
ActivityUtils.waitForFragmentToAttach(settingsActivity, MainPreferences.class); ActivityUtils.waitForFragmentToAttach(settingsActivity, MainSettings.class);
final Preference searchEnginePref = final Preference searchEnginePref =
waitForPreference(mainPreferences, MainPreferences.PREF_SEARCH_ENGINE); waitForPreference(mainSettings, MainSettings.PREF_SEARCH_ENGINE);
CriteriaHelper.pollUiThread(Criteria.equals(null, new Callable<Object>() { CriteriaHelper.pollUiThread(Criteria.equals(null, new Callable<Object>() {
@Override @Override
...@@ -157,7 +157,7 @@ public class SearchEngineSettingsTest { ...@@ -157,7 +157,7 @@ public class SearchEngineSettingsTest {
})); }));
TestThreadUtils.runOnUiThreadBlocking(() -> { TestThreadUtils.runOnUiThreadBlocking(() -> {
ManagedPreferenceDelegate managedPrefDelegate = ManagedPreferenceDelegate managedPrefDelegate =
mainPreferences.getManagedPreferenceDelegateForTest(); mainSettings.getManagedPreferenceDelegateForTest();
Assert.assertTrue(managedPrefDelegate.isPreferenceControlledByPolicy(searchEnginePref)); Assert.assertTrue(managedPrefDelegate.isPreferenceControlledByPolicy(searchEnginePref));
}); });
} }
......
...@@ -38,7 +38,7 @@ import org.chromium.chrome.browser.bookmarks.BookmarkBridge; ...@@ -38,7 +38,7 @@ import org.chromium.chrome.browser.bookmarks.BookmarkBridge;
import org.chromium.chrome.browser.preferences.Pref; import org.chromium.chrome.browser.preferences.Pref;
import org.chromium.chrome.browser.preferences.PrefServiceBridge; import org.chromium.chrome.browser.preferences.PrefServiceBridge;
import org.chromium.chrome.browser.profiles.Profile; import org.chromium.chrome.browser.profiles.Profile;
import org.chromium.chrome.browser.settings.MainPreferences; import org.chromium.chrome.browser.settings.MainSettings;
import org.chromium.chrome.browser.settings.SettingsActivity; import org.chromium.chrome.browser.settings.SettingsActivity;
import org.chromium.chrome.browser.settings.sync.AccountManagementFragment; import org.chromium.chrome.browser.settings.sync.AccountManagementFragment;
import org.chromium.chrome.browser.settings.sync.SignInPreference; import org.chromium.chrome.browser.settings.sync.SignInPreference;
...@@ -279,8 +279,8 @@ public class SigninTest { ...@@ -279,8 +279,8 @@ public class SigninTest {
// Verify that its preference UI is enabled. // Verify that its preference UI is enabled.
SettingsActivity settingsActivity = mActivityTestRule.startSettingsActivity(null); SettingsActivity settingsActivity = mActivityTestRule.startSettingsActivity(null);
MainPreferences mainPrefs = getMainPreferences(settingsActivity); MainSettings mainPrefs = getMainSettings(settingsActivity);
Preference passwordPref = mainPrefs.findPreference(MainPreferences.PREF_SAVED_PASSWORDS); Preference passwordPref = mainPrefs.findPreference(MainSettings.PREF_SAVED_PASSWORDS);
Assert.assertNotNull(passwordPref); Assert.assertNotNull(passwordPref);
// This preference opens a new fragment when clicked. // This preference opens a new fragment when clicked.
Assert.assertNotNull(passwordPref.getFragment()); Assert.assertNotNull(passwordPref.getFragment());
...@@ -353,16 +353,16 @@ public class SigninTest { ...@@ -353,16 +353,16 @@ public class SigninTest {
InstrumentationRegistry.getInstrumentation().waitForIdleSync(); InstrumentationRegistry.getInstrumentation().waitForIdleSync();
} }
private static MainPreferences getMainPreferences(SettingsActivity settingsActivity) { private static MainSettings getMainSettings(SettingsActivity settingsActivity) {
Fragment fragment = settingsActivity.getMainFragment(); Fragment fragment = settingsActivity.getMainFragment();
Assert.assertNotNull(fragment); Assert.assertNotNull(fragment);
Assert.assertTrue(fragment instanceof MainPreferences); Assert.assertTrue(fragment instanceof MainSettings);
return (MainPreferences) fragment; return (MainSettings) fragment;
} }
private static void clickSigninPreference(SettingsActivity settingsActivity) { private static void clickSigninPreference(SettingsActivity settingsActivity) {
MainPreferences mainPrefs = getMainPreferences(settingsActivity); MainSettings mainPrefs = getMainSettings(settingsActivity);
Preference signinPref = mainPrefs.findPreference(MainPreferences.PREF_SIGN_IN); Preference signinPref = mainPrefs.findPreference(MainSettings.PREF_SIGN_IN);
Assert.assertNotNull(signinPref); Assert.assertNotNull(signinPref);
Assert.assertTrue(signinPref instanceof SignInPreference); Assert.assertTrue(signinPref instanceof SignInPreference);
Assert.assertNotNull(signinPref.getOnPreferenceClickListener()); Assert.assertNotNull(signinPref.getOnPreferenceClickListener());
......
...@@ -7,16 +7,13 @@ package org.chromium.chrome.test.pagecontroller.controllers.codelab; ...@@ -7,16 +7,13 @@ package org.chromium.chrome.test.pagecontroller.controllers.codelab;
import org.chromium.chrome.test.pagecontroller.controllers.PageController; import org.chromium.chrome.test.pagecontroller.controllers.PageController;
import org.chromium.chrome.test.pagecontroller.utils.IUi2Locator; import org.chromium.chrome.test.pagecontroller.utils.IUi2Locator;
// TODO: Implement page controller for MainPreferences.java. Read // TODO: Implement page controller for MainSettings.java. Read
// documentation in the PageController class and refer to implemented Page // documentation in the PageController class and refer to implemented Page
// Controllers in the pagecontroller directory for examples. // Controllers in the pagecontroller directory for examples.
/** /**
* Settings Menu Page Controller for the Code Lab, representing * Settings Menu Page Controller for the Code Lab, representing
* MainPreferences.java. * {@link org.chromium.chrome.browser.settings.MainSettings}.
*
* @see <a
* href="https://chromium.googlesource.com/chromium/src/+/HEAD/chrome/android/java/src/org/chromium/chrome/browser/preferences/MainPreferences.java">MainPreferences.java</a>
*/ */
public class SettingsControllerForCodelab extends PageController { public class SettingsControllerForCodelab extends PageController {
// TODO: Replace null with a an actual locator. Hint, see // TODO: Replace null with a an actual locator. Hint, see
......
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