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_ = [
+  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
     }];
+  var promise = new Promise(function(resolve, reject) {

[Bernhard Bauer, 2014/01/13 14:33:50]

This is available as Promise.resolve(...).

[Adrian Kuegel, 2014/01/13 15:43:37]

Cool. Thanks for pointing that out :)

+    resolve(managedUsers);
+  });
+  options.ManagedUserListData.getInstance().promise_ = promise;
   // 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);
-
-  // A profile which does not exist yet.
-  nameField.value = 'NewProfileName';
-  ManageProfileOverlay.getInstance().onNameChanged_('create');
-  assertFalse($('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);
+  }).then(function() {

[Bernhard Bauer, 2014/01/13 14:33:50]

This will be called immediately afterwards, so you might as well remove it. The
reason for that is that `then(f)` itself returns a promise that will be
fulfilled with the return value of |f|, or if that return value itself is a
promise, when that promise is fulfilled. Savvy? :-) Basically, what it means is
that you can chain multiple asynchronous calls by returning a promise in your
callback function. If you don't return a promise (as in this case, where you
implicitly return undef), the returned promise will be immediately fulfilled, so
the next callback will run immediately (where "immediately" means "on the next
turn of the event loop").

I suspect what you want to do here is repeatedly change the name, wait for the
user list to get updated, then verify properties? In that case, you can simply
do the same thing as above: Set the name, get the promise out of the
ManagedProfileOverlay instance, and wait on that promise.

Also, it's very likely that all of this code doesn't run at all, because the
test ends before that (remember that the callbacks only run on the next turn of
the event loop!). What you should do here is mark the test as asynchronous (by
setting isAsync to true on the test fixture prototype), then calling testDone()
at the end when you're finished. Sorry this is getting more complicated, but
this way you can actually test the whole thing including the communication with
the browser.

[Adrian Kuegel, 2014/01/13 15:43:37]

Thank you very much for the detailed explanation. Makes perfect sense now to me.
I changed the test to async, and use a few nested then calls which wait for the
promise to resolve after each name change.

+    // A profile which doesn't have an avatar yet.
+    nameField.value = 'Fritz';
+    ManageProfileOverlay.getInstance().onNameChanged_('create');
+  }).then(function() {
+    assertTrue($('create-profile-ok').disabled);
+    assertFalse($('supervised-user-import') == null);
+  }).then(function() {
+    // A profile which already exists on the device.
+    nameField.value = 'Test';
+    ManageProfileOverlay.getInstance().onNameChanged_('create');
+  }).then(function() {
+    assertTrue($('create-profile-ok').disabled);
+    assertTrue($('supervised-user-import') == null);
+  }).then(function() {
+    // A profile which does not exist yet.
+    nameField.value = 'NewProfileName';
+    ManageProfileOverlay.getInstance().onNameChanged_('create');
+  }).then(function() {
+    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 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');