Bug 1200488 - browser_UITour_heartbeat.js: requestLongerTimeout(2) and add logging to see where/if the test is hanging. r=test-only
authorMatthew Noorenberghe <mozilla@noorenberghe.ca>
Fri, 25 Sep 2015 15:50:20 -0700
changeset 297840 ade25495a4817e9bdb6ffff4a3f4b8f7687f9659
parent 297839 a142bd33d9fa039a24cf229e93b265a801a8a498
child 297841 e597556a843367405cfc557b7a9d84d9fbd48800
push id5392
push userraliiev@mozilla.com
push dateMon, 14 Dec 2015 20:08:23 +0000
treeherdermozilla-beta@16ce8562a975 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerstest-only
bugs1200488
milestone44.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1200488 - browser_UITour_heartbeat.js: requestLongerTimeout(2) and add logging to see where/if the test is hanging. r=test-only
browser/components/uitour/test/browser_UITour_heartbeat.js
browser/components/uitour/test/head.js
--- a/browser/components/uitour/test/browser_UITour_heartbeat.js
+++ b/browser/components/uitour/test/browser_UITour_heartbeat.js
@@ -4,16 +4,17 @@
 "use strict";
 
 var gTestTab;
 var gContentAPI;
 var gContentWindow;
 
 function test() {
   UITourTest();
+  requestLongerTimeout(2);
 }
 
 function getHeartbeatNotification(aId, aChromeWindow = window) {
   let notificationBox = aChromeWindow.document.getElementById("high-priority-global-notificationbox");
   // UITour.jsm prefixes the notification box ID with "heartbeat-" to prevent collisions.
   return notificationBox.getNotificationWithValue("heartbeat-" + aId);
 }
 
--- a/browser/components/uitour/test/head.js
+++ b/browser/components/uitour/test/head.js
@@ -246,38 +246,41 @@ function UITourTest() {
       gBrowser.removeTab(gTestTab);
     delete window.gTestTab;
     Services.prefs.clearUserPref("browser.uitour.enabled", true);
     Services.perms.remove(testHttpsUri, "uitour");
     Services.perms.remove(testHttpUri, "uitour");
   });
 
   function done() {
+    info("== Done test, doing shared checks before teardown ==");
     executeSoon(() => {
       if (gTestTab)
         gBrowser.removeTab(gTestTab);
       gTestTab = null;
 
       let highlight = document.getElementById("UITourHighlightContainer");
       is_element_hidden(highlight, "Highlight should be closed/hidden after UITour tab is closed");
 
       let tooltip = document.getElementById("UITourTooltip");
       is_element_hidden(tooltip, "Tooltip should be closed/hidden after UITour tab is closed");
 
       ok(!PanelUI.panel.hasAttribute("noautohide"), "@noautohide on the menu panel should have been cleaned up");
       ok(!PanelUI.panel.hasAttribute("panelopen"), "The panel shouldn't have @panelopen");
       isnot(PanelUI.panel.state, "open", "The panel shouldn't be open");
       is(document.getElementById("PanelUI-menu-button").hasAttribute("open"), false, "Menu button should know that the menu is closed");
 
+      info("Done shared checks");
       executeSoon(nextTest);
     });
   }
 
   function nextTest() {
     if (tests.length == 0) {
+      info("finished tests in this file");
       finish();
       return;
     }
     let test = tests.shift();
     info("Starting " + test.name);
     waitForFocus(function() {
       loadUITourTestPage(function() {
         test(done);