[uber] Redoing the homepage selection UI

chrome/browser/resources/options2/home_page_overlay.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() {
   const OptionsPage = options.OptionsPage;
   const SettingsDialog = options.SettingsDialog;
 
   /**
    * HomePageOverlay class
@@ skipping 11 matching lines @@
     __proto__: SettingsDialog.prototype,
 
     /**
      * Initialize the page.
      */
     initializePage: function() {
       // Call base class implementation to start preference initialization.
       SettingsDialog.prototype.initializePage.call(this);
 
       var self = this;
-      $('homepage-use-ntp').onchange = this.updateHomePageInput_.bind(this);
-      $('homepage-use-url').onchange = this.updateHomePageInput_.bind(this);
-
       $('homepageURL').addEventListener('keydown', function(event) {
         // Focus the 'OK' button when the user hits enter since people expect
         // feedback indicating that they are done editing.
         if (event.keyIdentifier == 'Enter')
           $('home-page-confirm').focus();
       });
 
       // TODO(jhawkins): Refactor BrowserOptions.autocompleteList and use it
       // here.
-    },
 
-    /**
-     * @inheritDoc
-     */
-    didShowPage: function() {
-      // Set initial state.
-      this.updateHomePageInput_();
-    },
+      // Extra functionality for the OK/cancel buttons, separate from what the
+      // SettingsDialog provides
 
-    /**
-     * Updates the state of the homepage text input. The input is enabled only
-     * if the |homepageUseURLBUtton| radio is checked.
-     * @private
-     */
-    updateHomePageInput_: function() {
-      var homepageInput = $('homepageURL');
-      var homepageUseURL = $('homepage-use-url');
-      homepageInput.disabled = !homepageUseURL.checked;
+      // TODO(tbreisacher): This doesn't quite work because users can also
+      // cancel by pressing 'ESC'. We could just override the handleCancel_
+      // method from SettingsDialog, but it's private.
+
+      $('home-page-cancel').addEventListener('click', function() {
+        BrowserOptions.getInstance().updateHomePageSelector();
+      });
+
+      $('home-page-confirm').addEventListener('click', function() {
+        BrowserOptions.getInstance().homePageSelectUrl();
+      });
+

[csilv, 2012/02/01 20:37:29]

My preference is to not do this... and instead let browserOptions listen for
changes to the homepage preference and adjust accordingly.

[Tyler Breisacher (Chromium), 2012/02/01 21:49:38]

If the user cancels then the preference doesn't change, so that doesn't quite
work, I think. For the OK button, it does, but we need to explicitly change the
prefs as in patch set 6.

     },
   };
 
   // Export
   return {
     HomePageOverlay: HomePageOverlay
   };
 });