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

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

Issue 2671743002: Separate state of basic and advanced tab in CBD dialog (Closed)
Patch Set: rebase and fix compilation Created 3 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: chrome/android/java/src/org/chromium/chrome/browser/preferences/PrefServiceBridge.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/preferences/PrefServiceBridge.java b/chrome/android/java/src/org/chromium/chrome/browser/preferences/PrefServiceBridge.java
index 9ae7199f24bcc8a1f86cc58e22b19dd4b125af21..94d9ee4f087f8a6c975cb20ef3a6218a4202ea7d 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/preferences/PrefServiceBridge.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/preferences/PrefServiceBridge.java
@@ -523,7 +523,7 @@ public final class PrefServiceBridge {
}
/**
- * @param Whether Contextual Search should be enabled.
+ * @param enabled Whether Contextual Search should be enabled.
*/
public void setContextualSearchState(boolean enabled) {
setContextualSearchPreference(enabled
@@ -545,7 +545,7 @@ public final class PrefServiceBridge {
}
/**
- * @param Whether Safe Browsing Extended Reporting should be enabled.
+ * @param enabled Whether Safe Browsing Extended Reporting should be enabled.
*/
public void setSafeBrowsingExtendedReportingEnabled(boolean enabled) {
nativeSetSafeBrowsingExtendedReportingEnabled(enabled);
@@ -566,7 +566,7 @@ public final class PrefServiceBridge {
}
/**
- * @param Whether Safe Browsing should be enabled.
+ * @param enabled Whether Safe Browsing should be enabled.
*/
public void setSafeBrowsingEnabled(boolean enabled) {
nativeSetSafeBrowsingEnabled(enabled);
@@ -663,38 +663,47 @@ public final class PrefServiceBridge {
* Checks the state of deletion preference for a certain browsing data type.
* @param dataType The requested browsing data type (from the shared enum
* {@link org.chromium.chrome.browser.browsing_data.BrowsingDataType}).
+ * @param clearBrowsingDataTab Indicates if this is a checkbox on the default, basic or advanced
+ * tab to apply the right preference.
* @return The state of the corresponding deletion preference.
*/
- public boolean getBrowsingDataDeletionPreference(int dataType) {
- return nativeGetBrowsingDataDeletionPreference(dataType);
+ public boolean getBrowsingDataDeletionPreference(int dataType, int clearBrowsingDataTab) {
+ return nativeGetBrowsingDataDeletionPreference(dataType, clearBrowsingDataTab);
}
/**
* Sets the state of deletion preference for a certain browsing data type.
* @param dataType The requested browsing data type (from the shared enum
* {@link org.chromium.chrome.browser.browsing_data.BrowsingDataType}).
+ * @param clearBrowsingDataTab Indicates if this is a checkbox on the default, basic or advanced
+ * tab to apply the right preference.
* @param value The state to be set.
*/
- public void setBrowsingDataDeletionPreference(int dataType, boolean value) {
- nativeSetBrowsingDataDeletionPreference(dataType, value);
+ public void setBrowsingDataDeletionPreference(
+ int dataType, int clearBrowsingDataTab, boolean value) {
+ nativeSetBrowsingDataDeletionPreference(dataType, clearBrowsingDataTab, value);
}
/**
* Gets the time period for which browsing data will be deleted.
+ * @param clearBrowsingDataTab Indicates if this is a timeperiod on the default, basic or
+ * advanced tab to apply the right preference.
* @return The currently selected browsing data deletion time period (from the shared enum
* {@link org.chromium.chrome.browser.browsing_data.TimePeriod}).
*/
- public int getBrowsingDataDeletionTimePeriod() {
- return nativeGetBrowsingDataDeletionTimePeriod();
+ public int getBrowsingDataDeletionTimePeriod(int clearBrowsingDataTab) {
+ return nativeGetBrowsingDataDeletionTimePeriod(clearBrowsingDataTab);
}
/**
* Sets the time period for which browsing data will be deleted.
+ * @param clearBrowsingDataTab Indicates if this is a timeperiod on the default, basic or
+ * advanced tab to apply the right preference.
* @param timePeriod The selected browsing data deletion time period (from the shared enum
* {@link org.chromium.chrome.browser.browsing_data.TimePeriod}).
*/
- public void setBrowsingDataDeletionTimePeriod(int timePeriod) {
- nativeSetBrowsingDataDeletionTimePeriod(timePeriod);
+ public void setBrowsingDataDeletionTimePeriod(int clearBrowsingDataTab, int timePeriod) {
+ nativeSetBrowsingDataDeletionTimePeriod(clearBrowsingDataTab, timePeriod);
}
/**
@@ -756,6 +765,14 @@ public final class PrefServiceBridge {
}
/**
+ * Migrate browsing data preferences when the new "clear browsing data" dialog with tabs is
+ * visited.
+ */
+ public void migrateBrowsingDataPreferences() {
+ nativeMigrateBrowsingDataPreferences();
+ }
+
+ /**
* @return Whether browser history can be deleted by the user.
*/
public boolean canDeleteBrowsingHistory() {
@@ -1112,15 +1129,19 @@ public final class PrefServiceBridge {
private native void nativeSetTranslateEnabled(boolean enabled);
private native void nativeResetTranslateDefaults();
private native void nativeMigrateJavascriptPreference();
- private native boolean nativeGetBrowsingDataDeletionPreference(int dataType);
- private native void nativeSetBrowsingDataDeletionPreference(int dataType, boolean value);
- private native int nativeGetBrowsingDataDeletionTimePeriod();
- private native void nativeSetBrowsingDataDeletionTimePeriod(int timePeriod);
+ private native boolean nativeGetBrowsingDataDeletionPreference(
+ int dataType, int clearBrowsingDataTab);
+ private native void nativeSetBrowsingDataDeletionPreference(
+ int dataType, int clearBrowsingDataTab, boolean value);
+ private native int nativeGetBrowsingDataDeletionTimePeriod(int clearBrowsingDataTab);
+ private native void nativeSetBrowsingDataDeletionTimePeriod(
+ int clearBrowsingDataTab, int timePeriod);
private native void nativeClearBrowsingData(int[] dataTypes, int timePeriod,
String[] blacklistDomains, int[] blacklistedDomainReasons, String[] ignoredDomains,
int[] ignoredDomainReasons);
private native int nativeGetLastClearBrowsingDataTab();
private native void nativeSetLastClearBrowsingDataTab(int lastTab);
+ private native void nativeMigrateBrowsingDataPreferences();
private native void nativeRequestInfoAboutOtherFormsOfBrowsingHistory(
OtherFormsOfBrowsingHistoryListener listener);
private native boolean nativeCanDeleteBrowsingHistory();

Powered by Google App Engine
This is Rietveld 408576698