Bug 1586105 - Request longer timeout for page info tests r=canaltinova
authorJulien Wajsberg <felash@gmail.com>
Wed, 13 Nov 2019 01:08:32 +0000
changeset 501730 a46ef03516e66a6bd02531c651578dff1ae781fc
parent 501729 360c8b9f1566c66e8890f8bda2891a508ad36cc9
child 501731 0a3045be7a72a15be26196a4401d0b695931387c
push id114172
push userdluca@mozilla.com
push dateTue, 19 Nov 2019 11:31:10 +0000
treeherdermozilla-inbound@b5c5ba07d3db [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerscanaltinova
bugs1586105
milestone72.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 1586105 - Request longer timeout for page info tests r=canaltinova Differential Revision: https://phabricator.services.mozilla.com/D52696
tools/profiler/tests/browser/browser_test_profile_multi_frame_page_info.js
tools/profiler/tests/browser/browser_test_profile_single_frame_page_info.js
--- a/tools/profiler/tests/browser/browser_test_profile_multi_frame_page_info.js
+++ b/tools/profiler/tests/browser/browser_test_profile_multi_frame_page_info.js
@@ -1,12 +1,19 @@
 /* This Source Code Form is subject to the terms of the Mozilla Public
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
+if (Services.prefs.getBoolPref("fission.autostart")) {
+  // Bug 1586105: these tests could time out in some extremely slow conditions,
+  // when fission is enabled.
+  // Requesting a longer timeout should make it pass.
+  requestLongerTimeout(2);
+}
+
 add_task(async function test_profile_multi_frame_page_info() {
   // Requesting the complete log to be able to debug Bug 1586105.
   SimpleTest.requestCompleteLog();
   if (!AppConstants.MOZ_GECKO_PROFILER) {
     return;
   }
   Assert.ok(!Services.profiler.IsActive());
   info("Clear the previous pages just in case we still some open tabs.");
--- a/tools/profiler/tests/browser/browser_test_profile_single_frame_page_info.js
+++ b/tools/profiler/tests/browser/browser_test_profile_single_frame_page_info.js
@@ -1,12 +1,19 @@
 /* This Source Code Form is subject to the terms of the Mozilla Public
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
+if (Services.prefs.getBoolPref("fission.autostart")) {
+  // Bug 1586105: these tests could time out in some extremely slow conditions,
+  // when fission is enabled.
+  // Requesting a longer timeout should make it pass.
+  requestLongerTimeout(2);
+}
+
 add_task(async function test_profile_single_frame_page_info() {
   // Requesting the complete log to be able to debug Bug 1586105.
   SimpleTest.requestCompleteLog();
   if (!AppConstants.MOZ_GECKO_PROFILER) {
     return;
   }
   Assert.ok(!Services.profiler.IsActive());
   info("Clear the previous pages just in case we still some open tabs.");