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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/preferences/privacy/PrivacyPreferences.java

Issue 1618413002: Change the CBD dialog on Android to a PreferenceFragment (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed button, fixed tests Created 4 years, 11 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/java/src/org/chromium/chrome/browser/preferences/privacy/PrivacyPreferences.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/preferences/privacy/PrivacyPreferences.java b/chrome/android/java/src/org/chromium/chrome/browser/preferences/privacy/PrivacyPreferences.java
index 0f401926827bb9d733d30f077fd95e2fa316f10b..0d082606dedbd2508ec7ff3c2d09ae59871455ae 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/preferences/privacy/PrivacyPreferences.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/preferences/privacy/PrivacyPreferences.java
@@ -8,7 +8,6 @@ import android.os.Bundle;
import android.preference.CheckBoxPreference;
import android.preference.Preference;
import android.preference.Preference.OnPreferenceChangeListener;
-import android.preference.Preference.OnPreferenceClickListener;
import android.preference.PreferenceFragment;
import android.preference.PreferenceScreen;
import android.view.Menu;
@@ -20,12 +19,10 @@ import org.chromium.chrome.browser.contextualsearch.ContextualSearchFieldTrial;
import org.chromium.chrome.browser.help.HelpAndFeedback;
import org.chromium.chrome.browser.physicalweb.PhysicalWeb;
import org.chromium.chrome.browser.precache.PrecacheLauncher;
-import org.chromium.chrome.browser.preferences.ButtonPreference;
import org.chromium.chrome.browser.preferences.ChromeBaseCheckBoxPreference;
import org.chromium.chrome.browser.preferences.ManagedPreferenceDelegate;
import org.chromium.chrome.browser.preferences.NetworkPredictionOptions;
import org.chromium.chrome.browser.preferences.PrefServiceBridge;
-import org.chromium.chrome.browser.preferences.Preferences;
import org.chromium.chrome.browser.profiles.Profile;
/**
@@ -33,14 +30,6 @@ import org.chromium.chrome.browser.profiles.Profile;
*/
public class PrivacyPreferences extends PreferenceFragment
implements OnPreferenceChangeListener {
-
- /**
- * Set to true in the {@link Preferences#EXTRA_SHOW_FRAGMENT_ARGUMENTS} bundle to
- * trigger the clear browsing data dialog when showing the privacy preferences.
- */
- public static final String SHOW_CLEAR_BROWSING_DATA_EXTRA =
- "ShowClearBrowsingData";
-
private static final String PREF_NAVIGATION_ERROR = "navigation_error";
private static final String PREF_SEARCH_SUGGESTIONS = "search_suggestions";
private static final String PREF_SAFE_BROWSING_EXTENDED_REPORTING =
@@ -145,22 +134,6 @@ public class PrivacyPreferences extends PreferenceFragment
safeBrowsingPref.setOnPreferenceChangeListener(this);
safeBrowsingPref.setManagedPreferenceDelegate(mManagedPreferenceDelegate);
- ButtonPreference clearBrowsingData =
- (ButtonPreference) findPreference(PREF_CLEAR_BROWSING_DATA);
- clearBrowsingData.setOnPreferenceClickListener(new OnPreferenceClickListener() {
- @Override
- public boolean onPreferenceClick(Preference preference) {
- showClearBrowsingDialog();
- return true;
- }
- });
-
- if (getArguments() != null) {
- boolean showClearBrowsingData =
- getArguments().getBoolean(SHOW_CLEAR_BROWSING_DATA_EXTRA, false);
- if (showClearBrowsingData) showClearBrowsingDialog();
- }
-
if (!PhysicalWeb.featureIsEnabled()) {
preferenceScreen.removePreference(findPreference(PREF_PHYSICAL_WEB));
}
@@ -301,12 +274,6 @@ public class PrivacyPreferences extends PreferenceFragment
};
}
- private void showClearBrowsingDialog() {
- mClearBrowsingDataDialogFragment = new ClearBrowsingDataDialogFragment();
- mClearBrowsingDataDialogFragment.show(
- getFragmentManager(), ClearBrowsingDataDialogFragment.FRAGMENT_TAG);
- }
-
@Override
public void onDestroy() {
super.onDestroy();

Powered by Google App Engine
This is Rietveld 408576698