Bug 250871 – Remove HTMLMediaElement.webkitPreservesPitch

• Component: Media
• Status: RESOLVED FIXED

[Ahmad Saleem]

Hi Team,

While going through Blink's commit, I came across another potential removal of non-standard.

Blink Commit - https://chromium.googlesource.com/chromium/blink/+/79ddcbce61e61f4cfb09e4563c6deca94de44e6c

WebKit Source - https://searchfox.org/wubkat/source/Source/WebCore/html/HTMLMediaElement.h#1053 & https://searchfox.org/wubkat/source/Source/WebCore/html/HTMLMediaElement.cpp#3892

Just wanted to get input whether it is something we can get rid of now or we still need it?

Thanks!

[Radar WebKit Bug Importer]

<rdar://problem/104451631>

[Karl Dubost]

I don't see a lot of usage on GitHub
https://github.com/search?q=webkitpreservespitch+language%3AJavaScript&type=code&l=JavaScript


BUT still I can find things like
https://github.com/scottschiller/ArmorAlley/blob/da1d98a542bf55e5595b456983776890cb27a9b5/script/soundmanager2.js#L2196-L2199

            // 01/2022: prefixes for best compatibility
            if (audioContext && sm2.usePlaybackRate) {
              a.preservesPitch = a.mozPreservesPitch = a.webkitPreservesPitch = !!s._iO.preservesPitch;
            }
            a.play();

On the side of WPT:
https://wpt.fyi/results/html/semantics/embedded-content/media-elements/preserves-pitch.html?label=master&label=experimental&aligned


MDN 
https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement/preservesPitch
refers to Bug 214922


So I don't think it can be removed without first unprefixing it.

[Karl Dubost]

On the other hand the live test is passing 5 tests on the 9
http://wpt.live/html/semantics/embedded-content/media-elements/preserves-pitch.html

It requires user interaction. So Probably the failure in WPT is an infra issue. 

one of the tests is to test if it has been unprefixed.
the 3 remaining FAIL are about a difference in between the range of expected values


expected 440 +/- 23.46041055718475 but got 469.208211143695
expected 880 +/- 23.46041055718475 but got 469.208211143695
expected 220 +/- 23.46041055718475 but got 469.208211143695

[Ian Harris]

I just ran into this. Obviously I'll still need to use the `webkit` prefix for now, but it would be nice to have `preservesPitch` be the only property to deal with for all browsers :)

[EWS]

Committed 274551@main (4ed49192487b): <https://commits.webkit.org/274551@main>

Reviewed commits have been landed. Closing PR #24237 and removing active labels.