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

Unified Diff: chrome/browser/resources/options2/pref_ui.js

Issue 9353019: Revert 120803 - [uber] Redoing the homepage selection UI (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 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/browser/resources/options2/pref_ui.js
===================================================================
--- chrome/browser/resources/options2/pref_ui.js (revision 120816)
+++ chrome/browser/resources/options2/pref_ui.js (working copy)
@@ -611,29 +611,6 @@
// Set up the prototype chain
__proto__: HTMLInputElement.prototype,
- prefValue_: null,
-
- savePrefState: function() {
- switch(this.dataType) {
- case 'number':
- Preferences.setIntegerPref(this.pref, this.value, this.metric);
- break;
- case 'double':
- Preferences.setDoublePref(this.pref, this.value, this.metric);
- break;
- case 'url':
- Preferences.setURLPref(this.pref, this.value, this.metric);
- break;
- default:
- Preferences.setStringPref(this.pref, this.value, this.metric);
- break;
- }
- },
-
- resetPrefState: function() {
- this.value = this.prefValue_;
- },
-
/**
* Initialization function for the cr.ui framework.
*/
@@ -647,16 +624,26 @@
event.value['value'] : event.value;
updateElementState_(self, event);
-
- self.prefValue_ = self.value;
});
// Listen to user events.
- if (!self.dialogPref) {
- this.addEventListener('change', function(e) {
- self.savePrefState.bind(self);
- });
- }
+ this.addEventListener('change',
+ function(e) {
+ switch(self.dataType) {
+ case 'number':
+ Preferences.setIntegerPref(self.pref, self.value, self.metric);
+ break;
+ case 'double':
+ Preferences.setDoublePref(self.pref, self.value, self.metric);
+ break;
+ case 'url':
+ Preferences.setURLPref(self.pref, self.value, self.metric);
+ break;
+ default:
+ Preferences.setStringPref(self.pref, self.value, self.metric);
+ break;
+ }
+ });
window.addEventListener('unload',
function() {
« no previous file with comments | « chrome/browser/resources/options2/options_page.css ('k') | chrome/browser/resources/options2/settings_dialog.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698