Don't allow profile deletion in Metro mode.

chrome/browser/resources/options/browser_options.js

 // Copyright (c) 2012 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.exportPath('options');
 
 /**
  * @typedef {{actionLinkText: (string|undefined),
  *            childUser: (boolean|undefined),
  *            hasError: (boolean|undefined),
@@ skipping 306 matching lines @@
           this.setDefaultSearchEngine_);
 
       // Users section.
       if (loadTimeData.valueExists('profilesInfo')) {
         $('profiles-section').hidden = false;
         this.maybeShowUserSection_();
 
         var profilesList = $('profiles-list');
         options.browser_options.ProfileList.decorate(profilesList);
         profilesList.autoExpands = true;
+        profilesList.canDeleteItems = true;

[tapted, 2015/01/26 23:36:07]

I'm not a webui guru, but this is probably unnecessary, since it's done by an
~initializer in browser_options_profile_list.js  (and if it _was_ necessary the
profileIsSupervised clause would be broken :).

[noms (inactive), 2015/01/28 02:20:13]

Done.

 
         // The profiles info data in |loadTimeData| might be stale.
         this.setProfilesInfo_(loadTimeData.getValue('profilesInfo'));
         chrome.send('requestProfilesInfo');
 
         profilesList.addEventListener('change',
             this.setProfileViewButtonsStatus_);
         $('profiles-create').onclick = function(event) {
           ManageProfileOverlay.showCreateDialog();
         };
         if (OptionsPage.isSettingsApp()) {
           $('profiles-app-list-switch').onclick = function(event) {
             var selectedProfile = self.getSelectedProfileItem_();
             chrome.send('switchAppListProfile', [selectedProfile.filePath]);
           };
         }
         $('profiles-manage').onclick = function(event) {
           ManageProfileOverlay.showManageDialog();
         };
         $('profiles-delete').onclick = function(event) {
           var selectedProfile = self.getSelectedProfileItem_();
           if (selectedProfile)
             ManageProfileOverlay.showDeleteDialog(selectedProfile);
         };
         if (loadTimeData.getBoolean('profileIsSupervised')) {
           $('profiles-create').disabled = true;
           $('profiles-delete').disabled = true;
           $('profiles-list').canDeleteItems = false;
         }
+        if (!loadTimeData.getBoolean('allowProfileDeletion')) {
+          $('profiles-delete').disabled = true;
+          $('profiles-list').canDeleteItems = false;
+        }
       }
 
       if (cr.isChromeOS) {
         // Username (canonical email) of the currently logged in user or
         // |kGuestUser| if a guest session is active.
         this.username_ = loadTimeData.getString('username');
 
         this.updateAccountPicture_();
 
         $('account-picture').onclick = this.showImagerPickerOverlay_;
@@ skipping 1033 matching lines @@
      * Helper function to set the status of profile view buttons to disabled or
      * enabled, depending on the number of profiles and selection status of the
      * profiles list.
      * @private
      */
     setProfileViewButtonsStatus_: function() {
       var profilesList = $('profiles-list');
       var selectedProfile = profilesList.selectedItem;
       var hasSelection = selectedProfile != null;
       var hasSingleProfile = profilesList.dataModel.length == 1;
-      var isSupervised = loadTimeData.getBoolean('profileIsSupervised');
       $('profiles-manage').disabled = !hasSelection ||
           !selectedProfile.isCurrentProfile;
       if (hasSelection && !selectedProfile.isCurrentProfile)
         $('profiles-manage').title = loadTimeData.getString('currentUserOnly');
       else
         $('profiles-manage').title = '';
-      $('profiles-delete').disabled = isSupervised ||
+      $('profiles-delete').disabled = !profilesList.canDeleteItems_ ||

[tapted, 2015/01/26 23:36:07]

Does this need an accessor (i.e. trailing underscore => "private")? (It's also
@private, and I think dbeam's working on running all this through closure)

[noms (inactive), 2015/01/28 02:20:13]

Done.

                                       (!hasSelection && !hasSingleProfile);
       if (OptionsPage.isSettingsApp()) {
         $('profiles-app-list-switch').disabled = !hasSelection ||
             selectedProfile.isCurrentProfile;
       }
       var importData = $('import-data');
       if (importData) {
         importData.disabled = $('import-data').disabled = hasSelection &&
           !selectedProfile.isCurrentProfile;
       }
@@ skipping 775 matching lines @@
       }
       button.textContent = loadTimeData.getString(strId);
     };
   }
 
   // Export
   return {
     BrowserOptions: BrowserOptions
   };
 });