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

Unified Diff: chrome/browser/resources/options/browser_options.js

Issue 421653006: Update "Predict network actions" UI setting. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 6 years, 4 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/browser/resources/options/browser_options.js
diff --git a/chrome/browser/resources/options/browser_options.js b/chrome/browser/resources/options/browser_options.js
index d55284ee2e51e383c1dd229ebaa8b9aa184b8f51..4ee1c92fc03bb90a221bf8b9eb1910205b591a9c 100644
--- a/chrome/browser/resources/options/browser_options.js
+++ b/chrome/browser/resources/options/browser_options.js
@@ -9,6 +9,13 @@ cr.define('options', function() {
var ArrayDataModel = cr.ui.ArrayDataModel;
var RepeatingButton = cr.ui.RepeatingButton;
var HotwordSearchSettingIndicator = options.HotwordSearchSettingIndicator;
+ var NetworkPredictionOptions = {
+ ALWAYS: 0,
+ WIFI_ONLY: 1,
+ NEVER: 2,
+ UNSET: 3,
+ DEFAULT: 1
+ };
//
// BrowserOptions class
@@ -377,6 +384,17 @@ cr.define('options', function() {
updateMetricsRestartButton);
updateMetricsRestartButton();
}
+ $('networkPredictionOptions').onchange = function(event) {
+ var value = (event.target.checked ?
+ NetworkPredictionOptions.WIFI_ONLY :
+ NetworkPredictionOptions.NEVER);
+ var metric = event.target.metric;
+ Preferences.setIntegerPref(
+ 'net.network_prediction_options',
+ value,
+ true,
+ metric);
+ };
// Bluetooth (CrOS only).
if (cr.isChromeOS) {
@@ -1485,6 +1503,25 @@ cr.define('options', function() {
},
/**
+ * Set network prediction checkbox value.
+ *
+ * @param {Object} pref Information about network prediction options.
+ * @param {number} pref.value The value of network prediction options.
+ * @param {boolean} pref.disabled If the pref is not user modifiable.
+ * @private
+ */
+ setNetworkPredictionValue_: function(pref) {
+ var checkbox = $('networkPredictionOptions');
+ checkbox.disabled = pref.disabled;
+ if (pref.value == NetworkPredictionOptions.UNSET) {
+ checkbox.checked = !(NetworkPredictionOptions.DEFAULT ==
stevenjb 2014/08/05 15:51:56 DEFAULT != NEVER
Bence 2014/08/06 19:38:16 Done.
+ NetworkPredictionOptions.NEVER);
+ } else {
+ checkbox.checked = !(pref.value == NetworkPredictionOptions.NEVER);
stevenjb 2014/08/05 15:51:56 value != NEVER
Bence 2014/08/06 19:38:16 Done.
+ }
+ },
+
+ /**
* Set the font size selected item. This item actually reflects two
* preferences: the default font size and the default fixed font size.
*
@@ -1867,6 +1904,7 @@ cr.define('options', function() {
'setCanSetTime',
'setFontSize',
'setNativeThemeButtonEnabled',
+ 'setNetworkPredictionValue',
'setHighContrastCheckboxState',
'setMetricsReportingCheckboxState',
'setMetricsReportingSettingVisibility',

Powered by Google App Engine
This is Rietveld 408576698