Replace own callback handling with Promises.

chrome/browser/ui/webui/options/manage_profile_browsertest.js

 // Copyright 2013 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.
 
 // None of these tests is relevant for Chrome OS.
 GEN('#if !defined(OS_CHROMEOS)');
 
 GEN('#include "base/command_line.h"');
 GEN('#include "chrome/common/chrome_switches.h"');
 
@@ skipping 133 matching lines @@
 // same name as an existing managed user profile.
 TEST_F('ManageProfileUITest', 'CreateExistingManagedUser', function() {
   ManageProfileOverlay.getInstance().initializePage();
   var custodianEmail = 'chrome.playpen.test@gmail.com';
   CreateProfileOverlay.updateSignedInStatus(custodianEmail);
   assertEquals(custodianEmail,
                CreateProfileOverlay.getInstance().signedInEmail_);
   this.setProfileManaged_(false, 'create');
 
   // Initialize the list of existing managed users.
-  var managedUserListData = options.ManagedUserListData.getInstance();
-  managedUserListData.managedUsers_ = [

[Bernhard Bauer, 2014/01/10 13:52:08]

Would this work if you inject a fulfilled promise here?

[Adrian Kuegel, 2014/01/10 16:52:01]

Yes, it does. Thanks for the suggestion.

+  var managedUsers = [
     {
       id: 'managedUser1',
       name: 'Rosalie',
       iconURL: 'chrome://path/to/icon/image',
       onCurrentDevice: false,
+      nameConflict: false,
       needAvatar: false
     },
     {
       id: 'managedUser2',
       name: 'Fritz',
       iconURL: 'chrome://path/to/icon/image',
       onCurrentDevice: false,
+      nameConflict: false,
       needAvatar: true
     },
     {
       id: 'managedUser3',
       name: 'Test',
       iconURL: 'chrome://path/to/icon/image',
       onCurrentDevice: true,
+      nameConflict: true,
       needAvatar: false
     }];
+  options.ManagedUserListData.receiveExistingManagedUsers(managedUsers);
   // Also add the name 'Test' to |profileNames_| to simulate that the profile
   // exists on the device.
   ManageProfileOverlay.getInstance().profileNames_.Test = true;
 
   // Initially, the ok button should be enabled and the import link should not
   // exist.
   assertFalse($('create-profile-ok').disabled);
   assertTrue($('supervised-user-import') == null);
 
   // Now try to create profiles with the names of existing supervised users.
   $('create-profile-managed').checked = true;
   var nameField = $('create-profile-name');
   // A profile which already has an avatar.
   nameField.value = 'Rosalie';
   ManageProfileOverlay.getInstance().onNameChanged_('create');
-  assertTrue($('create-profile-ok').disabled);
-  assertFalse($('supervised-user-import') == null);
-  // A profile which doesn't have an avatar yet.
-  nameField.value = 'Fritz';
-  ManageProfileOverlay.getInstance().onNameChanged_('create');
-  assertTrue($('create-profile-ok').disabled);
-  assertFalse($('supervised-user-import') == null);
-  // A profile which already exists on the device.
-  nameField.value = 'Test';
-  ManageProfileOverlay.getInstance().onNameChanged_('create');
-  assertTrue($('create-profile-ok').disabled);
-  assertTrue($('supervised-user-import') == null);
+  // Need to wait until the promise resolves for the first time. The first time
+  // resolves asynchronously even if the resolve method was already called. Bug?
+  options.ManagedUserListData.getInstance().promise_.then(function() {
+    assertTrue($('create-profile-ok').disabled);
+    assertFalse($('supervised-user-import') == null);
+    // A profile which doesn't have an avatar yet.
+    nameField.value = 'Fritz';
+    ManageProfileOverlay.getInstance().onNameChanged_('create');
+    assertTrue($('create-profile-ok').disabled);
+    assertFalse($('supervised-user-import') == null);
+    // A profile which already exists on the device.
+    nameField.value = 'Test';
+    ManageProfileOverlay.getInstance().onNameChanged_('create');
+    assertTrue($('create-profile-ok').disabled);
+    assertTrue($('supervised-user-import') == null);
 
-  // A profile which does not exist yet.
-  nameField.value = 'NewProfileName';
-  ManageProfileOverlay.getInstance().onNameChanged_('create');
-  assertFalse($('create-profile-ok').disabled);
-  assertTrue($('supervised-user-import') == null);
+    // A profile which does not exist yet.
+    nameField.value = 'NewProfileName';
+    ManageProfileOverlay.getInstance().onNameChanged_('create');
+    assertFalse($('create-profile-ok').disabled);
+    assertTrue($('supervised-user-import') == null);
+  });
 });
 
 // Managed users should not be able to edit their profile names, and the initial
 // focus should be adjusted accordingly.
 TEST_F('ManageProfileUITest', 'EditManagedUserNameAllowed', function() {
   var nameField = $('manage-profile-name');
 
   this.setProfileManaged_(false, 'manage');
   ManageProfileOverlay.showManageDialog();
   expectFalse(nameField.disabled);
@@ skipping 101 matching lines @@
   checkDialog('Multiple Words');
   checkDialog('It\'s "<HTML> injection" & more!',
               'It\'s "<HTML> injection" & more!',
               // The innerHTML getter doesn't escape quotation marks,
               // independent of whether they were escaped in the setter.
               'It\'s "&lt;HTML&gt; injection" &amp; more!');
 
   // Test elision. MAX_LENGTH = 50, minus 3 for the ellipsis.
   var name47Characters = '01234567890123456789012345678901234567890123456';
   var name60Characters = name47Characters + '0123456789012';
-  checkDialog(name60Characters, name47Characters + '...');
+  checkDialog(name60Characters, name47Characters + '\u2026');
 
   // Test both elision and HTML escaping. The allowed string length is the
   // visible length, not the length including the entity names.
   name47Characters = name47Characters.replace('0', '&').replace('1', '>');
   name60Characters = name60Characters.replace('0', '&').replace('1', '>');
   var escaped = name47Characters.replace('&', '&amp;').replace('>', '&gt;');
-  checkDialog(name60Characters, name47Characters + '...', escaped + '...');
+  checkDialog(
+      name60Characters, name47Characters + '\u2026', escaped + '\u2026');
 });
 
 // An additional warning should be shown when deleting a managed user.
 TEST_F('ManageProfileUITest', 'DeleteManagedUserWarning', function() {
   var addendum = $('delete-managed-profile-addendum');
 
   ManageProfileOverlay.showDeleteDialog(this.testProfileInfo_(true));
   assertFalse(addendum.hidden);
 
   ManageProfileOverlay.showDeleteDialog(this.testProfileInfo_(false));
@@ skipping 113 matching lines @@
       chromeSendMessages.push(message);
     };
     $('delete-profile-ok').onclick();
     // Restore the original function so the test framework can use it.
     chrome.send = originalChromeSend;
     return chromeSendMessages;
   }
 
   this.setProfileManaged_(false, 'manage');
   var messages = clickAndListen();
-  assertEquals(2, messages.length);
+  assertEquals(1, messages.length);
   assertEquals('deleteProfile', messages[0]);
-  assertEquals('requestManagedUserImportUpdate', messages[1]);
   assertEquals('settings', OptionsPage.getTopmostVisiblePage().name);
 
   ManageProfileOverlay.showDeleteDialog(this.testProfileInfo_(false));
   this.setProfileManaged_(true, 'manage');
   messages = clickAndListen();
   assertEquals(0, messages.length);
   assertEquals('settings', OptionsPage.getTopmostVisiblePage().name);
 });
 
 GEN('#endif  // OS_CHROMEOS');