Bug 1765201 - Remove mozPreservesPitch. r=emilio,devtools-reviewers,n…

…chevobbe Differential Revision: https://phabricator.services.mozilla.com/D182957
Thaodan · Jul 11, 2023 · 0c51148 · 0c51148
1 parent b9c61cf
commit 0c51148
Show file tree

Hide file tree

Showing 6 changed files with 1 addition and 19 deletions.
diff --git a/devtools/server/actors/webconsole/webidl-unsafe-getters-names.js b/devtools/server/actors/webconsole/webidl-unsafe-getters-names.js
@@ -9,7 +9,6 @@
 module.exports = [
   "InstallTrigger",
   "farthestViewportElement",
-  "mozPreservesPitch",
   "mozPressure",
   "nearestViewportElement",
   "onmouseenter",

diff --git a/dom/base/nsDeprecatedOperationList.h b/dom/base/nsDeprecatedOperationList.h
@@ -56,7 +56,6 @@ DEPRECATED_OPERATION(ElementSetCapture)
 DEPRECATED_OPERATION(ElementReleaseCapture)
 DEPRECATED_OPERATION(DocumentReleaseCapture)
 DEPRECATED_OPERATION(OffscreenCanvasToBlob)
-DEPRECATED_OPERATION(MozPreservesPitchDeprecatedPrefix)
 DEPRECATED_OPERATION(SVGNearestViewportElement)
 DEPRECATED_OPERATION(SVGFarthestViewportElement)
 DEPRECATED_OPERATION(SizeToContent)
diff --git a/dom/webidl/HTMLMediaElement.webidl b/dom/webidl/HTMLMediaElement.webidl
@@ -114,11 +114,6 @@ partial interface HTMLMediaElement {
 
   attribute MediaStream? srcObject;
 
-  // TODO: Remove mozPreservesPitch. (bug 1765201)
-  [Deprecated="MozPreservesPitchDeprecatedPrefix",
-   Pref="dom.media.mozPreservesPitch.enabled",
-   BinaryName="preservesPitch"]
-  attribute boolean mozPreservesPitch;
   attribute boolean preservesPitch;
 
   // NB: for internal use with the video controls:

diff --git a/modules/libpref/init/StaticPrefList.yaml b/modules/libpref/init/StaticPrefList.yaml
@@ -3134,12 +3134,6 @@
   value: true
   mirror: always
 
-# Whether HTMLMediaElement.mozPreservesPitch is enabled.
-- name: dom.media.mozPreservesPitch.enabled
-  type: bool
-  value: false
-  mirror: always
-
 # WebCodecs API
 - name: dom.media.webcodecs.enabled
   type: RelaxedAtomicBool

diff --git a/testing/web-platform/meta/html/semantics/embedded-content/media-elements/__dir__.ini b/testing/web-platform/meta/html/semantics/embedded-content/media-elements/__dir__.ini
@@ -1 +1 @@
-prefs: [dom.security.featurePolicy.experimental.enabled:true, dom.security.featurePolicy.header.enabled:true, dom.security.featurePolicy.webidl.enabled:true, dom.media.mozPreservesPitch.enabled:false]
+prefs: [dom.security.featurePolicy.experimental.enabled:true, dom.security.featurePolicy.header.enabled:true, dom.security.featurePolicy.webidl.enabled:true]
diff --git a/...ng/web-platform/tests/html/semantics/embedded-content/media-elements/preserves-pitch.html b/...ng/web-platform/tests/html/semantics/embedded-content/media-elements/preserves-pitch.html
@@ -12,9 +12,6 @@
     if ("preservesPitch" in HTMLAudioElement.prototype) {
         return audio.preservesPitch;
     }
-    if ("mozPreservesPitch" in HTMLAudioElement.prototype) {
-        return audio.mozPreservesPitch;
-    }
     if ("webkitPreservesPitch" in HTMLAudioElement.prototype) {
         return audio.webkitPreservesPitch;
     }
@@ -25,8 +22,6 @@
 function setPreservesPitch(audio, value) {
     if ("preservesPitch" in HTMLAudioElement.prototype) {
         audio.preservesPitch = value;
-    } else if ("mozPreservesPitch" in HTMLAudioElement.prototype) {
-        audio.mozPreservesPitch = value;
     } else if ("webkitPreservesPitch" in HTMLAudioElement.prototype) {
         audio.webkitPreservesPitch = value;
     }
Original file line number	Diff line number	Diff line change
		@@ -1 +1 @@
		prefs: [dom.security.featurePolicy.experimental.enabled:true, dom.security.featurePolicy.header.enabled:true, dom.security.featurePolicy.webidl.enabled:true, dom.media.mozPreservesPitch.enabled:false]
		prefs: [dom.security.featurePolicy.experimental.enabled:true, dom.security.featurePolicy.header.enabled:true, dom.security.featurePolicy.webidl.enabled:true]