From 56e26df0c2ad2077f548a765aafc56b2ceccdbba Mon Sep 17 00:00:00 2001 From: Molly Howell Date: Thu, 13 Oct 2022 18:44:25 +0000 Subject: [PATCH] Bug 1790646 Part 2 - Switch to using the new string and remove the old one. r=pip-reviewers,fluent-reviewers,mconley,flod Differential Revision: https://phabricator.services.mozilla.com/D159290 --- .../tests/browser_nimbusMessageFirstTimePip.js | 2 +- .../pictureinpicture/tests/browser_nimbusShowIconOnly.js | 2 +- toolkit/content/widgets/videocontrols.js | 4 ++-- toolkit/locales/en-US/toolkit/global/videocontrols.ftl | 1 - 4 files changed, 4 insertions(+), 5 deletions(-) diff --git a/toolkit/components/pictureinpicture/tests/browser_nimbusMessageFirstTimePip.js b/toolkit/components/pictureinpicture/tests/browser_nimbusMessageFirstTimePip.js index e20278a54429..d5840841cd90 100644 --- a/toolkit/components/pictureinpicture/tests/browser_nimbusMessageFirstTimePip.js +++ b/toolkit/components/pictureinpicture/tests/browser_nimbusMessageFirstTimePip.js @@ -78,7 +78,7 @@ add_task(async function test_experiment_control() { ); let pipExplainerMessage = l10n.formatValueSync( - "videocontrols-picture-in-picture-explainer2" + "videocontrols-picture-in-picture-explainer3" ); await SimpleTest.promiseFocus(browser); diff --git a/toolkit/components/pictureinpicture/tests/browser_nimbusShowIconOnly.js b/toolkit/components/pictureinpicture/tests/browser_nimbusShowIconOnly.js index e9e3e5f377e7..5f3ad07ef3e8 100644 --- a/toolkit/components/pictureinpicture/tests/browser_nimbusShowIconOnly.js +++ b/toolkit/components/pictureinpicture/tests/browser_nimbusShowIconOnly.js @@ -78,7 +78,7 @@ add_task(async function test_experiment_control() { ); let pipExplainerMessage = l10n.formatValueSync( - "videocontrols-picture-in-picture-explainer2" + "videocontrols-picture-in-picture-explainer3" ); await SimpleTest.promiseFocus(browser); diff --git a/toolkit/content/widgets/videocontrols.js b/toolkit/content/widgets/videocontrols.js index 734929efc096..9bbce74249c4 100644 --- a/toolkit/content/widgets/videocontrols.js +++ b/toolkit/content/widgets/videocontrols.js @@ -2868,7 +2868,7 @@ this.VideoControlsImplWidget = class { -
+
@@ -3374,7 +3374,7 @@ this.NoControlsDesktopImplWidget = class { -
+
diff --git a/toolkit/locales/en-US/toolkit/global/videocontrols.ftl b/toolkit/locales/en-US/toolkit/global/videocontrols.ftl index d84dabcec244..adfb9d14e2b4 100644 --- a/toolkit/locales/en-US/toolkit/global/videocontrols.ftl +++ b/toolkit/locales/en-US/toolkit/global/videocontrols.ftl @@ -41,7 +41,6 @@ videocontrols-picture-in-picture-toggle-label2 = Pop out this video # This string is used as part of a variation of the Picture-in-Picture video toggle # button. When using this variation, this string appears below the toggle when the # mouse hovers the toggle. -videocontrols-picture-in-picture-explainer2 = More screens are more fun. Play this video in Picture-in-Picture while you browse. videocontrols-picture-in-picture-explainer3 = More screens are more fun. Play this video while you do other things. videocontrols-error-aborted = Video loading stopped.