Merging upstream version 125.0.1.

Signed-off-by: Daniel Baumann <daniel.baumann@progress-linux.org>

1 files changed, 6 insertions, 6 deletions

diff --git a/toolkit/actors/PictureInPictureChild.sys.mjs b/toolkit/actors/PictureInPictureChild.sys.mjs
index 62dc6cdfff..2a415df32d 100644
--- a/toolkit/actors/PictureInPictureChild.sys.mjs
+++ b/toolkit/actors/PictureInPictureChild.sys.mjs
@@ -1225,11 +1225,11 @@ export class PictureInPictureToggleChild extends JSWindowActorChild {
     let shadowRoot = video.openOrClosedShadowRoot;
 
     if (shadowRoot.firstChild && video != oldOverVideo) {
-      if (video.getTransformToViewport().a == -1) {
-        shadowRoot.firstChild.setAttribute("flipped", true);
-      } else {
-        shadowRoot.firstChild.removeAttribute("flipped");
-      }
+      // TODO: Maybe this should move to videocontrols.js somehow.
+      shadowRoot.firstChild.toggleAttribute(
+        "flipped",
+        video.getTransformToViewport().a == -1
+      );
     }
 
     // It seems from automated testing that if it's still very early on in the
@@ -1812,7 +1812,7 @@ export class PictureInPictureChild extends JSWindowActorChild {
    *  4) all active cues with VTTCue.line integer have VTTCue.snapToLines = true
    *  5) all active cues with VTTCue.line percentage have VTTCue.snapToLines = false
    *
-   * vtt.jsm currently sets snapToLines to false if line is a percentage value, but
+   * vtt.sys.mjs currently sets snapToLines to false if line is a percentage value, but
    * cues are still ordered by line. In most cases, snapToLines is set to true by default,
    * unless intentionally overridden.
    * @param allCuesArray {Array<VTTCue>} array of active cues