bluetooth: Fix flaky bluetooth internals browser test.

chrome/test/data/webui/bluetooth_internals_browsertest.js

 // Copyright 2016 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.
 
 /**
  * @fileoverview Tests for chrome://bluetooth-internals
  */
 
 /** @const {string} Path to source root. */
 var ROOT_PATH = '../../../../';
@@ skipping 207 matching lines @@
    */
   fakeDeviceInfo3: function() {
     return {
       address: "CC:CC:84:96:92:84",
       name: "CCC",
       name_for_display: "CCC",
     };
   },
 };
 
-// Fails on Win/Mac. https://crbug.com/676227
-GEN('#if defined(OS_MACOSX) || defined(OS_WIN)');
-GEN('#define MAYBE_Startup_BluetoothInternals \\');
-GEN('    DISABLED_Startup_BluetoothInternals');
-GEN('#else');
-GEN('#define MAYBE_Startup_BluetoothInternals Startup_BluetoothInternals');
-GEN('#endif');
-
-TEST_F('BluetoothInternalsTest',
-       'MAYBE_Startup_BluetoothInternals', function() {
+TEST_F('BluetoothInternalsTest', 'Startup_BluetoothInternals', function() {
   var adapterFactory = null;
   var deviceTable = null;
   var sidebarNode = null;
 
   var fakeDeviceInfo1 = this.fakeDeviceInfo1;
   var fakeDeviceInfo2 = this.fakeDeviceInfo2;
   var fakeDeviceInfo3 = this.fakeDeviceInfo3;
 
   // Before tests are run, make sure setup completes.
   var setupPromise = this.setupResolver.promise.then(function() {
@@ skipping 222 matching lines @@
     });
 
     test('Sidebar_CloseTwice', function() {
       // Multiple calls to close shouldn't change the state.
       sidebarObj.close();
       sidebarObj.close();
       expectFalse(sidebarNode.classList.contains('open'));
     });
 
     /* Snackbar Tests */
-    function finishSnackbarTest(done) {
-      // Let event queue finish.
-      setTimeout(function() {
-        expectEquals(0, $('snackbar-container').children.length);
-        expectFalse(!!snackbar.Snackbar.current_);
-        done();
-      }, 10);
+
+    /**
+     * Checks snackbar showing status and returns a Promise that resolves when
+     * |pendingSnackbar| is shown. If the snackbar is already showing, the
+     * Promise resolves immediately.
+     * @param {!snackbar.Snackbar} pendingSnackbar
+     * @return {!Promise}
+     */
+    function whenSnackbarShows(pendingSnackbar) {
+      return new Promise(function(resolve) {
+        if (pendingSnackbar.classList.contains('open')) {
+          resolve();
+        } else {
+          pendingSnackbar.addEventListener('showed', function() { resolve(); });
+        }

[Dan Beam, 2017/01/04 01:48:33]

no curlies

[mbrunson, 2017/01/04 03:46:19]

Done.

+      });
+    }
+
+    /**
+     * Performs final checks for snackbar tests.
+     * @return {!Promise} Promise is fulfilled when the checks finish.
+     */
+    function finishSnackbarTest() {
+      return new Promise(function(resolve) {
+        // Let event queue finish.
+        setTimeout(function() {
+          expectEquals(0, $('snackbar-container').children.length);
+          expectFalse(!!snackbar.Snackbar.current_);
+          resolve();
+        }, 50);
+      });
     }
 
     test('Snackbar_ShowTimeout', function(done) {
       var snackbar1 = snackbar.Snackbar.show('Message 1');
       assertEquals(1, $('snackbar-container').children.length);
 
       snackbar1.addEventListener('dismissed', function() {
-        finishSnackbarTest(done);
+        finishSnackbarTest().then(done);
       });
     });
 
     test('Snackbar_ShowDismiss', function(done) {
       var snackbar1 = snackbar.Snackbar.show('Message 1');
       assertEquals(1, $('snackbar-container').children.length);
-      snackbar1.addEventListener('dismissed', function() {
-        finishSnackbarTest(done);
-      });
 
-      snackbar.Snackbar.dismiss();
+      function finish() {
+        snackbar.Snackbar.dismiss().then(finishSnackbarTest).then(done);
+      }
+
+      whenSnackbarShows(snackbar1).then(finish);

[Dan Beam, 2017/01/04 01:48:33]

if you drop the |done| argument and just return a Promise, the test will wait
for the returned Promise (and any chained promise).

I think something like this might work:

return whenSnackbarShows(snackbar1).then(function() {
  return snackbar.Snackbar.dismiss();
}).then(function() {
  return finishSnackbarTest();
});

[mbrunson, 2017/01/04 03:46:19]

Done.

     });
 
     test('Snackbar_QueueThreeDismiss', function(done) {
       var expectedCalls = 3;
       var actualCalls = 0;
 
       var snackbar1 = snackbar.Snackbar.show('Message 1');
       var snackbar2 = snackbar.Snackbar.show('Message 2');
       var snackbar3 = snackbar.Snackbar.show('Message 3');
 
       assertEquals(1, $('snackbar-container').children.length);
       expectEquals(2, snackbar.Snackbar.queue_.length);
 
       function next() {
         actualCalls++;
-        snackbar.Snackbar.dismiss();
+        return snackbar.Snackbar.dismiss();
       }
 
-      snackbar1.addEventListener('dismissed', next);
-      snackbar2.addEventListener('dismissed', next);
-      snackbar3.addEventListener('dismissed', function() {
-        next();
-        expectEquals(expectedCalls, actualCalls);
-        finishSnackbarTest(done);
-      });
+      function finish() {
+        next().then(function() {
+          expectEquals(expectedCalls, actualCalls);
+        }).then(finishSnackbarTest).then(done);
+      }
 
-      snackbar.Snackbar.dismiss();
+      whenSnackbarShows(snackbar1).then(next);
+      whenSnackbarShows(snackbar2).then(next);
+      whenSnackbarShows(snackbar3).then(finish);
     });
 
     test('Snackbar_QueueThreeDismissAll', function(done) {
       var expectedCalls = 1;
       var actualCalls = 0;
 
       var snackbar1 = snackbar.Snackbar.show('Message 1');
       var snackbar2 = snackbar.Snackbar.show('Message 2');
       var snackbar3 = snackbar.Snackbar.show('Message 3');
 
       assertEquals(1, $('snackbar-container').children.length);
       expectEquals(2, snackbar.Snackbar.queue_.length);
 
       function next() {
         assertTrue(false);
       }
 
-      snackbar1.addEventListener('dismissed', function() {
+      whenSnackbarShows(snackbar2).then(next);
+      snackbar2.addEventListener('dismissed', next);
+      whenSnackbarShows(snackbar3).then(next);
+      snackbar3.addEventListener('dismissed', next);
+
+      whenSnackbarShows(snackbar1).then(function() {
+        return snackbar.Snackbar.dismiss(true);
+      }).then(function() {
         expectEquals(0, snackbar.Snackbar.queue_.length);
         expectFalse(!!snackbar.Snackbar.current_);
-        snackbar.Snackbar.dismiss();
-
-        finishSnackbarTest(done);
-      });
-      snackbar2.addEventListener('dismissed', next);
-      snackbar3.addEventListener('dismissed', next);
-
-      snackbar.Snackbar.dismiss(true);
+      }).then(finishSnackbarTest).then(done);
     });
   });
 
 
   // Run all registered tests.
   mocha.run();
 });