Turn on session restore by default for mac on Lion

chrome/browser/resources/options/browser_options.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() {
   const OptionsPage = options.OptionsPage;
   const ArrayDataModel = cr.ui.ArrayDataModel;
 
   //
   // BrowserOptions class
@@ skipping 45 matching lines @@
       // Call base class implementation to start preference initialization.
       OptionsPage.prototype.initializePage.call(this);
 
       // Wire up controls.
       $('startupUseCurrentButton').onclick = function(event) {
         chrome.send('setStartupPagesToCurrentPages');
       };
       $('toolbarShowBookmarksBar').onchange = function() {
         chrome.send('toggleShowBookmarksBar');
       };
+      var items = document.querySelectorAll('input[name=startup]')
+      for (i=0; i < items.length; i++) {

[Robert Sesek, 2011/07/28 18:30:22]

nit: spaces around operators

[dhollowa, 2011/07/28 20:05:35]

Done.

+        items[i].onchange = function(event) {
+          chrome.send('setRestoreOnStartup', [event.target.value]);
+          chrome.send('coreOptionsUserMetricsAction',
+              [event.target.getAttribute("metric")]);
+        };
+      }
       $('defaultSearchManageEnginesButton').onclick = function(event) {
         OptionsPage.navigateToPage('searchEngines');
         chrome.send('coreOptionsUserMetricsAction',
             ['Options_ManageSearchEngines']);
       };
       $('defaultSearchEngine').onchange = this.setDefaultSearchEngine_;
 
       var self = this;
       $('instantEnabledCheckbox').customChangeHandler = function(event) {
         if (this.checked) {
@@ skipping 146 matching lines @@
      * be enabled.
      * @returns {boolean} Whether the startup page controls should be
      *     enabled.
      */
     shouldEnableCustomStartupPageControls: function(pages) {
       return $('startupShowPagesButton').checked &&
           !this.startup_pages_pref_.controlledBy;
     },
 
     /**
+     * Updates the startup radio group with the given value.
+     * @param {number} value the preference value being set.
+     * @private
+     */
+    updateRestoreOnStartup_: function(value) {
+      var items = document.querySelectorAll('input[name=startup]')
+      for (i=0; i < items.length; i++) {
+        if (items[i].value == value)
+          items[i].checked = true

[Robert Sesek, 2011/07/28 18:30:22]

nit: missing semicolon

[dhollowa, 2011/07/28 20:05:35]

Done.

+      }
+    },
+
+    /**
      * Updates the startup pages list with the given entries.
      * @param {Array} pages List of startup pages.
      * @private
      */
     updateStartupPages_: function(pages) {
       var model = new ArrayDataModel(pages);
       // Add a "new page" row.
       model.push({
         'modelIndex': '-1'
       });
@@ skipping 232 matching lines @@
                                                               canBeDefault);
     }
   };
 
   BrowserOptions.updateSearchEngines = function(engines, defaultValue,
                                                 defaultManaged) {
     BrowserOptions.getInstance().updateSearchEngines_(engines, defaultValue,
                                                       defaultManaged);
   };
 
+  BrowserOptions.updateRestoreOnStartup = function(value) {
+    BrowserOptions.getInstance().updateRestoreOnStartup_(value);
+  };
+
   BrowserOptions.updateStartupPages = function(pages) {
     BrowserOptions.getInstance().updateStartupPages_(pages);
   };
 
   BrowserOptions.updateAutocompleteSuggestions = function(suggestions) {
     BrowserOptions.getInstance().updateAutocompleteSuggestions_(suggestions);
   };
 
   // Export
   return {
     BrowserOptions: BrowserOptions
   };
 
 });