Prevent settings page from closing when someone clicks an <option> in a <select>.

chrome/browser/resources/options/options_page.js

 // Copyright (c) 2011 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() {
   /////////////////////////////////////////////////////////////////////////////
   // OptionsPage class:
 
   /**
    * Base class for options page.
@@ skipping 681 matching lines @@
    * closing subpages and/or stopping event propagation.
    * @return {Event} a mousedown or click event.
    * @private
    */
   OptionsPage.bodyMouseEventHandler_ = function(event) {
     // Do nothing if a subpage isn't showing.
     var topPage = this.getTopmostVisiblePage();
     if (!topPage || topPage.isOverlay || !topPage.parentPage)
       return;
 
+    // Don't close subpages if a user is clicking in a select element.
+    // This is necessary because WebKit sends click events with strange
+    // coordinates when a user selects a new entry in a select element.

[Evan Stade, 2011/06/23 02:05:05]

add crbug link

+    if (event.srcElement.nodeName == 'SELECT')
+      return;
+
     // Do nothing if the client coordinates are not within the source element.
     // This occurs if the user toggles a checkbox by pressing spacebar.
     // This is a workaround to prevent keyboard events from closing the window.
     // See: crosbug.com/15678
     if (event.clientX == -document.body.scrollLeft &&
         event.clientY == -document.body.scrollTop) {
       return;
     }
 
     // Don't interfere with navbar clicks.
@@ skipping 291 matching lines @@
     canShowPage: function() {
       return true;
     },
   };
 
   // Export
   return {
     OptionsPage: OptionsPage
   };
 });