Disable and turn off autofill for guest mode.

chrome/browser/resources/options/personal_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() {
 
   var OptionsPage = options.OptionsPage;
 
   // State variables.
   var syncEnabled = false;
@@ skipping 60 matching lines @@
         if ($('themes-GTK-button')) {
           $('themes-GTK-button').onclick = function(event) {
             chrome.send('themesSetGTK');
           };
         }
       } else {
         $('change-picture-button').onclick = function(event) {
           OptionsPage.navigateToPage('changePicture');
         };
         chrome.send('loadAccountPicture');
-      }
 
-      if (cr.commandLine.options['--bwsi']) {
-        // Disable the screen lock checkbox for the guest mode.
-        $('enable-screen-lock').disabled = true;
+        if (cr.commandLine.options['--bwsi']) {
+          // Disable the screen lock checkbox and change-picture-button in
+          // guest mode.
+          $('enable-screen-lock').disabled = true;
+          $('change-picture-button').disabled = true;
+        }
       }
 
       if (PersonalOptions.disablePasswordManagement()) {
         $('passwords-offersave').disabled = true;
         $('passwords-neversave').disabled = true;
         $('passwords-offersave').value = false;
         $('passwords-neversave').value = true;
         $('manage-passwords').disabled = true;
       }
+
+      if (PersonalOptions.disableAutofillManagement()) {
+        $('autofill-enabled').disabled = true;

[achuithb, 2011/05/20 23:38:01]

Curious - is it better to cache the autofill-enabled dom element? Otherwise you
call getElementById which presumably searches the whole DOM tree, right? We're
doing this everywhere so is it safe to assume getElementById is efficient?

[xiyuan, 2011/05/20 23:52:39]

I think you are right that use $() will incur a performance hit.

Not sure how big the improvement would be though so not changing it in all
places.

+        $('autofill-settings').disabled = true;
+
+        // Turn off autofill.
+        $('autofill-enabled').checked = false;
+        cr.dispatchSimpleEvent($('autofill-enabled'), 'change');
+      }
     },
 
     showStopSyncingOverlay_: function() {
       AlertOverlay.show(localStrings.getString('stop_syncing_title'),
                         localStrings.getString('stop_syncing_explanation'),
                         localStrings.getString('stop_syncing_confirm'),
                         localStrings.getString('cancel'),
                         function() { chrome.send('stopSyncing'); });
     },
 
@@ skipping 102 matching lines @@
 
   /**
    * Returns whether the user should be able to manage (view and edit) their
    * stored passwords. Password management is disabled in guest mode.
    * @return {boolean} True if password management should be disabled.
    */
   PersonalOptions.disablePasswordManagement = function() {
     return cr.commandLine.options['--bwsi'];
   };
 
+  /**
+   * Returns whether the user should be able to manage autofill settings.
+   * @return {boolean} True if password management should be disabled.
+   */
+  PersonalOptions.disableAutofillManagement = function() {
+    return cr.commandLine.options['--bwsi'];
+  };
+
   // Forward public APIs to private implementations.
   [
     'setSyncEnabled',
     'setSyncSetupCompleted',
     'setAccountPicture',
     'setSyncStatus',
     'setSyncStatusErrorVisible',
     'setSyncActionLinkEnabled',
     'setSyncActionLinkLabel',
     'setProfilesSectionVisible',
@@ skipping 10 matching lines @@
       PersonalOptions.getInstance()[name + '_'](value);
     };
   });
 
   // Export
   return {
     PersonalOptions: PersonalOptions
   };
 
 });