DOMUI: Use dispatchSimpleEvent instead of programatically clicking the checkbox

chrome/browser/resources/options/personal_options.js

 // Copyright (c) 2010 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 OptionsPage = options.OptionsPage;
 
   // State variables.
   var syncEnabled = false;
@@ skipping 82 matching lines @@
     },
 
     /**
      * Updates the sync datatype checkboxes based on the selected sync option.
      * @private
      */
     updateSyncSelection_: function() {
       var idx = $('sync-select').selectedIndex;
       var syncCheckboxes = $('sync-table').getElementsByTagName('input');
       if (idx == 0) {
+        // 'Choose what to sync.'
         for (var i = 0; i < syncCheckboxes.length; i++) {
           syncCheckboxes[i].disabled = false;
         }
       } else if (idx == 1) {
+        // 'Everything' is synced.
         for (var i = 0; i < syncCheckboxes.length; i++) {
-          // Merely setting checked = true is not enough to trigger the pref
-          // being set; thus, we simulate the click.
-          if (!syncCheckboxes[i].checked)
-            syncCheckboxes[i].click();
+          if (!syncCheckboxes[i].checked) {
+            syncCheckboxes[i].checked = true;
+
+            // Merely setting checked = true is not enough to trigger the pref
+            // being set; thus, we dispatch a change event to notify
+            // PrefCheckbox |checked| has changed.
+            cr.dispatchSimpleEvent(syncCheckboxes[i], 'change');

[arv (Not doing code reviews), 2011/01/21 23:57:29]

This seems backwards.

We should not dispatch fake events with the purpose to get the handler invoked.
Instead we should just call the method we care about.

+          }
 
           syncCheckboxes[i].disabled = true;
         }
       }
     },
 
     showStopSyncingOverlay_: function(event) {
       AlertOverlay.show(localStrings.getString('stop_syncing_title'),
                         localStrings.getString('stop_syncing_explanation'),
                         localStrings.getString('stop_syncing_confirm'),
@@ skipping 116 matching lines @@
       PersonalOptions.getInstance()[name + '_'](value);
     };
   });
 
   // Export
   return {
     PersonalOptions: PersonalOptions
   };
 
 });