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 122 matching lines @@
   assertNotEquals(-1, signedInText.textContent.indexOf(custodianEmail));
 
   CreateProfileOverlay.updateSignedInStatus('');
   assertEquals('', CreateProfileOverlay.getInstance().signedInEmail_);
   assertTrue(signedInText.hidden);
   assertFalse(notSignedInText.hidden);
   assertFalse($('create-profile-managed').checked);
   assertTrue($('create-profile-managed').disabled);
 });
 
+function ManageProfileAsyncUITest() {}

[Bernhard Bauer, 2014/01/16 15:26:34]

Nit: Async test fixtures are usually named <thing>TestAsync.

[Adrian Kuegel, 2014/01/16 15:51:38]

Done.

+
+ManageProfileAsyncUITest.prototype = {
+  __proto__: ManageProfileUITest.prototype,
+
+  isAsync: true,
+};
+
 // The import link should show up if the user tries to create a profile with the
 // 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');
-
+TEST_F('ManageProfileAsyncUITest', 'CreateExistingManagedUser', function() {
   // Initialize the list of existing managed users.
-  var managedUserListData = options.ManagedUserListData.getInstance();
-  managedUserListData.managedUsers_ = [
+  var managedUsers = [
     {
       id: 'managedUser1',
       name: 'Rosalie',
       iconURL: 'chrome://path/to/icon/image',
       onCurrentDevice: false,
       needAvatar: false
     },
     {
       id: 'managedUser2',
       name: 'Fritz',
       iconURL: 'chrome://path/to/icon/image',
       onCurrentDevice: false,
       needAvatar: true
     },
     {
       id: 'managedUser3',
       name: 'Test',
       iconURL: 'chrome://path/to/icon/image',
       onCurrentDevice: true,
       needAvatar: false
+    },
+    {
+      id: 'managedUser4',
+      name: 'SameName',
+      iconURL: 'chrome://path/to/icon/image',
+      onCurrentDevice: false,
+      needAvatar: false
     }];
-  // Also add the name 'Test' to |profileNames_| to simulate that the profile
-  // exists on the device.
+  var promise = Promise.resolve(managedUsers);
+  options.ManagedUserListData.getInstance().promise_ = promise;
+
+  // Initialize the ManageProfileOverlay.
+  ManageProfileOverlay.getInstance().initializePage();
+  var custodianEmail = 'chrome.playpen.test@gmail.com';
+  CreateProfileOverlay.updateSignedInStatus(custodianEmail);
+  assertEquals(custodianEmail,
+               CreateProfileOverlay.getInstance().signedInEmail_);
+  this.setProfileManaged_(false, 'create');
+
+  // Also add the names 'Test' and 'Test2' to |profileNames_| to simulate that
+  // profiles with that names exist on the device.

[Bernhard Bauer, 2014/01/16 15:26:34]

Nit: "with those names"

[Adrian Kuegel, 2014/01/16 15:51:38]

Done.

   ManageProfileOverlay.getInstance().profileNames_.Test = true;
+  ManageProfileOverlay.getInstance().profileNames_.SameName = 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.
+  promise.then(function() {
+    assertTrue($('create-profile-ok').disabled);
+    assertFalse($('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 doesn't have an avatar yet.
+    nameField.value = 'Fritz';
+    ManageProfileOverlay.getInstance().onNameChanged_('create');
+    return options.ManagedUserListData.getInstance().promise_;
+  }).then(function() {
+    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');
+    return options.ManagedUserListData.getInstance().promise_;
+  }).then(function() {
+    assertTrue($('create-profile-ok').disabled);
+    assertTrue($('supervised-user-import') == null);
+
+    // A profile which does not exist on the device, but there is a profile with
+    // the same name already on the device.
+    nameField.value = 'SameName';
+    ManageProfileOverlay.getInstance().onNameChanged_('create');
+    return options.ManagedUserListData.getInstance().promise_;
+  }).then(function() {
+    assertTrue($('create-profile-ok').disabled);
+    assertFalse($('supervised-user-import') == null);
+
+    // A profile which does not exist yet.
+    nameField.value = 'NewProfileName';
+    ManageProfileOverlay.getInstance().onNameChanged_('create');
+    return options.ManagedUserListData.getInstance().promise_;
+  }).then(function() {
+    assertFalse($('create-profile-ok').disabled);
+    assertTrue($('supervised-user-import') == null);
+    testDone();
+  });
 });
 
 // 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 98 matching lines @@
   assertEquals('createProfile', OptionsPage.getTopmostVisiblePage().name);
 
   checkDialog('OneWord');
   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 + '...');
+  // Test elision. MAX_LENGTH = 50, minus 1 for the ellipsis.
+  var name49Characters = '0123456789012345678901234567890123456789012345678';
+  var name50Characters = name49Characters + '9';
+  var name51Characters = name50Characters + '0';
+  var name60Characters = name51Characters + '123456789';
+  checkDialog(name49Characters, name49Characters);
+  checkDialog(name50Characters, name50Characters);
+  checkDialog(name51Characters, name49Characters + '\u2026');
+  checkDialog(name60Characters, name49Characters + '\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', '>');
+  name49Characters = name49Characters.replace('0', '&').replace('1', '>');
   name60Characters = name60Characters.replace('0', '&').replace('1', '>');
-  var escaped = name47Characters.replace('&', '&amp;').replace('>', '&gt;');
-  checkDialog(name60Characters, name47Characters + '...', escaped + '...');
+  var escaped = name49Characters.replace('&', '&amp;').replace('>', '&gt;');
+  checkDialog(
+      name60Characters, name49Characters + '\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');