[Hotword] Make ConfirmDialog subclass from SettingsDialog

chrome/browser/resources/options/pref_ui.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 cr.define('options', function() {
 
   var Preferences = options.Preferences;
 
   /**
    * Allows an element to be disabled for several reasons.
@@ skipping 142 matching lines @@
   PrefCheckbox.prototype = {
     // Set up the prototype chain
     __proto__: PrefInputElement.prototype,
 
     /**
      * Initialization function for the cr.ui framework.
      */
     decorate: function() {
       PrefInputElement.prototype.decorate.call(this);
       this.type = 'checkbox';
+
+      // Prime the checkbox pref, as it may be default-checked.

[Dan Beam, 2014/03/27 21:06:43]

i have no idea what this means.

[rpetterson, 2014/03/27 21:26:00]

The preference may be default true.

[Dan Beam, 2014/03/28 00:00:41]

just remove the comment, imo

[rpetterson, 2014/03/28 00:05:35]

Done.

+      if (this.dialogPref)
+        this.updatePrefFromState_();
     },
 
     /**
      * Update the associated pref when when the user makes changes to the
      * checkbox state.
      * @private
      */
     updatePrefFromState_: function() {
       var value = this.inverted_pref ? !this.checked : this.checked;
       Preferences.setBooleanPref(this.pref, value,
@@ skipping 375 matching lines @@
     PrefNumber: PrefNumber,
     PrefRadio: PrefRadio,
     PrefRange: PrefRange,
     PrefSelect: PrefSelect,
     PrefTextField: PrefTextField,
     PrefPortNumber: PrefPortNumber,
     PrefButton: PrefButton
   };
 
 });