Bug 1925823 - Stop disabling screenshots component in mochitest r=urlbar-reviewers,niklas

gregorypappas · gregorypappas · commit efde7db23aef · 2025-02-03T14:55:59.000Z
Differential Revision: https://phabricator.services.mozilla.com/D226241
diff --git a/browser/base/content/test/contextMenu/browser_contextmenu.js b/browser/base/content/test/contextMenu/browser_contextmenu.js
@@ -72,7 +72,6 @@ add_setup(async function () {
   await SpecialPowers.pushPrefEnv({
     set: [
       ["browser.search.separatePrivateDefault.ui.enabled", true],
-      ["extensions.screenshots.disabled", false],
       ["privacy.query_stripping.strip_on_share.canDisable", false],
     ],
   });
diff --git a/browser/base/content/test/contextMenu/browser_contextmenu_input.js b/browser/base/content/test/contextMenu/browser_contextmenu_input.js
@@ -298,6 +298,10 @@ add_task(
           null,
           "---",
           null,
+          "context-take-screenshot",
+          true,
+          "---",
+          null,
           "context-viewsource",
           true,
         ],
diff --git a/browser/base/content/test/performance/browser.toml b/browser/base/content/test/performance/browser.toml
@@ -17,9 +17,6 @@ prefs = [
   # For perfomance tests do not enable the remote control cue, which gets set
   # when Marionette is enabled, but users normally don't see.
   "browser.chrome.disableRemoteControlCueForTests=true",
-  # The Screenshots extension is disabled by default in Mochitests. We re-enable
-  # it here, since it's a more realistic configuration.
-  "extensions.screenshots.disabled=false",
   # Need to keep the revamped sidebar off until Bug 1905212 is done
   "sidebar.revamp=false",
 ]
diff --git a/browser/base/content/test/performance/io/browser.toml b/browser/base/content/test/performance/io/browser.toml
@@ -14,7 +14,6 @@ skip-if = [
 prefs = [
   "browser.startup.record=true",
   "gfx.canvas.willReadFrequently.enable=true",
-  "extensions.screenshots.disabled=false", # The Screenshots extension is disabled by default in Mochitests. We re-enable it here, since it's a more realistic configuration.
 ]
 environment = [
   "GNOME_ACCESSIBILITY=0",
diff --git a/browser/components/customizableui/test/browser.toml b/browser/components/customizableui/test/browser.toml
@@ -341,6 +341,7 @@ skip-if = ["(verify && debug && (os == 'linux' || os == 'mac'))"]
 ["browser_reset_dom_events.js"]
 
 ["browser_screenshot_button_disabled.js"]
+prefs = ["extensions.screenshots.disabled=true"]
 
 ["browser_searchbar_removal.js"]
 
diff --git a/browser/components/enterprisepolicies/tests/browser/disable_fxscreenshots/browser.toml b/browser/components/enterprisepolicies/tests/browser/disable_fxscreenshots/browser.toml
@@ -1,7 +1,6 @@
 [DEFAULT]
 prefs = [
   "browser.policies.alternatePath='<test-root>/browser/components/enterprisepolicies/tests/browser/disable_fxscreenshots/config_disable_fxscreenshots.json'",
-  "extensions.screenshots.disabled=false",
 ]
 support-files = ["config_disable_fxscreenshots.json"]
 
diff --git a/browser/components/screenshots/tests/browser/browser.toml b/browser/components/screenshots/tests/browser/browser.toml
@@ -14,7 +14,6 @@ support-files = [
 ]
 
 prefs = [
-  "extensions.screenshots.disabled=false",
   "screenshots.browser.component.enabled=true",
   "sidebar.revamp=false",
 ]
diff --git a/browser/components/screenshots/tests/browser/browserSidebarRevamp.toml b/browser/components/screenshots/tests/browser/browserSidebarRevamp.toml
@@ -14,7 +14,6 @@ support-files = [
 ]
 
 prefs = [
-  "extensions.screenshots.disabled=false",
   "screenshots.browser.component.enabled=true",
   "sidebar.revamp=true",
 ]
diff --git a/browser/components/urlbar/tests/browser/browser.toml b/browser/components/urlbar/tests/browser/browser.toml
@@ -12,7 +12,6 @@ skip-if = [
 prefs = [
   "browser.bookmarks.testing.skipDefaultBookmarksImport=true",
   "browser.urlbar.trending.featureGate=false",
-  "extensions.screenshots.disabled=false",
   "screenshots.browser.component.enabled=true",
   "sidebar.revamp=false",
 ]
diff --git a/browser/components/urlbar/tests/browser/browserSidebarRevamp.toml b/browser/components/urlbar/tests/browser/browserSidebarRevamp.toml
@@ -13,7 +13,6 @@ skip-if = [
 prefs = [
   "browser.bookmarks.testing.skipDefaultBookmarksImport=true",
   "browser.urlbar.trending.featureGate=false",
-  "extensions.screenshots.disabled=false",
   "screenshots.browser.component.enabled=true",
   "sidebar.revamp=true",
 ]
diff --git a/browser/extensions/screenshots/test/browser/browser.toml b/browser/extensions/screenshots/test/browser/browser.toml
@@ -1,7 +1,5 @@
 [DEFAULT]
-# The Screenshots extension is disabled by default in Mochitests. We re-enable it here, since it's a more realistic configuration.
 prefs = [
- "extensions.screenshots.disabled=false",
  "screenshots.browser.component.enabled=false",
 ]
 
diff --git a/testing/profiles/unittest-required/user.js b/testing/profiles/unittest-required/user.js
@@ -106,8 +106,6 @@ user_pref("extensions.getAddons.search.browseURL", "http://{server}/extensions-d
 user_pref("extensions.hotfix.url", "http://{server}/extensions-dummy/hotfixURL");
 // Disable intalling any distribution add-ons
 user_pref("extensions.installDistroAddons", false);
-// Disable Screenshots by default for now
-user_pref("extensions.screenshots.disabled", true);
 user_pref("extensions.systemAddon.update.url", "http://{server}/dummy-system-addons.xml");
 user_pref("extensions.systemAddon.update.enabled", false);
 user_pref("extensions.update.background.url", "http://{server}/extensions-dummy/updateBackgroundURL");

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,6 @@ skip-if = [`
`14`	`14`	`prefs = [`
`15`	`15`	`"browser.startup.record=true",`
`16`	`16`	`"gfx.canvas.willReadFrequently.enable=true",`
`17`		`- "extensions.screenshots.disabled=false", # The Screenshots extension is disabled by default in Mochitests. We re-enable it here, since it's a more realistic configuration.`
`18`	`17`	`]`
`19`	`18`	`environment = [`
`20`	`19`	`"GNOME_ACCESSIBILITY=0",`
Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,6 @@`
`1`	`1`	`[DEFAULT]`
`2`	`2`	`prefs = [`
`3`	`3`	`"browser.policies.alternatePath='<test-root>/browser/components/enterprisepolicies/tests/browser/disable_fxscreenshots/config_disable_fxscreenshots.json'",`
`4`		`- "extensions.screenshots.disabled=false",`
`5`	`4`	`]`
`6`	`5`	`support-files = ["config_disable_fxscreenshots.json"]`
`7`	`6`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,6 @@ support-files = [`
`14`	`14`	`]`
`15`	`15`
`16`	`16`	`prefs = [`
`17`		`- "extensions.screenshots.disabled=false",`
`18`	`17`	`"screenshots.browser.component.enabled=true",`
`19`	`18`	`"sidebar.revamp=false",`
`20`	`19`	`]`
Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,6 @@ skip-if = [`
`12`	`12`	`prefs = [`
`13`	`13`	`"browser.bookmarks.testing.skipDefaultBookmarksImport=true",`
`14`	`14`	`"browser.urlbar.trending.featureGate=false",`
`15`		`- "extensions.screenshots.disabled=false",`
`16`	`15`	`"screenshots.browser.component.enabled=true",`
`17`	`16`	`"sidebar.revamp=false",`
`18`	`17`	`]`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,6 @@ skip-if = [`
`13`	`13`	`prefs = [`
`14`	`14`	`"browser.bookmarks.testing.skipDefaultBookmarksImport=true",`
`15`	`15`	`"browser.urlbar.trending.featureGate=false",`
`16`		`- "extensions.screenshots.disabled=false",`
`17`	`16`	`"screenshots.browser.component.enabled=true",`
`18`	`17`	`"sidebar.revamp=true",`
`19`	`18`	`]`