Respect policy for list of startup pages

chrome/browser/resources/options2/browser_options.js

Index: chrome/browser/resources/options2/browser_options.js
diff --git a/chrome/browser/resources/options2/browser_options.js b/chrome/browser/resources/options2/browser_options.js
index 5bbc3bc1a27ce392f0e7272c5692a1257175defe..4784307d06bbf7f39390eb1c2dd712afaae17018 100644
--- a/chrome/browser/resources/options2/browser_options.js
+++ b/chrome/browser/resources/options2/browser_options.js
@@ -80,8 +80,19 @@ cr.define('options', function() {
       }
 
       // On Startup section.
-      $('startup-set-pages').onclick = function() {
-        OptionsPage.navigateToPage('startup');
+      var startupSetPagesLink = $('startup-set-pages');
+      const showPagesValue = Number($('startup-show-pages').value);
+
+      Preferences.getInstance().addEventListener(
+          'session.restore_on_startup',
+          function(event) {
+        startupSetPagesLink.disabled =

[Evan Stade, 2012/03/02 03:38:35]

this indent can't be right.

[Tyler Breisacher (Chromium), 2012/03/02 04:10:46]

Is this better?

+            event.value['disabled'] && event.value['value'] != showPagesValue;
+      });
+
+      startupSetPagesLink.onclick = function() {
+        if (!startupSetPagesLink.disabled)

[Evan Stade, 2012/03/02 03:38:35]

is this required? disabled buttons can't be clicked...

[Tyler Breisacher (Chromium), 2012/03/02 04:10:46]

I forgot it's actually a button, not a link. Done.

+          OptionsPage.navigateToPage('startup');
       };
 
       this.sessionRestoreEnabled_ = templateData.enable_restore_session_state;