Bug 1548422 - disable activity stream during the app update tests since it periodically errors in getScreenshot. r=bytesized
authorRobert Strong <robert.bugzilla@gmail.com>
Thu, 02 May 2019 15:35:00 +0000
changeset 472341 47e694cadae621df8240af8bd4446a988d7631b2
parent 472340 92678c01ead789b2951fe1880c30d5251d2e5785
child 472342 7376b80d14d78fcaa3fd81d052df06ae0c95e2ff
push id113004
push userrgurzau@mozilla.com
push dateFri, 03 May 2019 04:25:36 +0000
treeherdermozilla-inbound@a9d7796acb8e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbytesized
bugs1548422
milestone68.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 1548422 - disable activity stream during the app update tests since it periodically errors in getScreenshot. r=bytesized The failures when running the app update tests during TV runs print an activity stream error to the console and this will at the least eliminate it as a cause for the failure and possibly will prevent these tests from failing during TV runs. Differential Revision: https://phabricator.services.mozilla.com/D29612
toolkit/mozapps/update/tests/browser/head.js
--- a/toolkit/mozapps/update/tests/browser/head.js
+++ b/toolkit/mozapps/update/tests/browser/head.js
@@ -52,16 +52,19 @@ requestLongerTimeout(10);
  */
 add_task(async function setupTestCommon() {
   await SpecialPowers.pushPrefEnv({
     set: [
       [PREF_APP_UPDATE_DOWNLOAD_ATTEMPTS, 0],
       [PREF_APP_UPDATE_DOWNLOAD_MAXATTEMPTS, 2],
       [PREF_APP_UPDATE_LOG, gDebugTest],
       [PREF_APP_UPDATE_SERVICE_ENABLED, false],
+      // Disable activity stream to prevent errors when opening pages during
+      // TV runs. See bug 1548422 for an example.
+      ["browser.library.activity-stream.enabled", false],
     ],
   });
 
   setUpdateTimerPrefs();
   removeUpdateFiles(true);
   AppMenuNotifications.removeNotification(/.*/);
   // Most app update mochitest-browser-chrome tests expect auto update to be
   // enabled. Those that don't will explicitly change this.