1648277 - MediaElementAudioSourceNode should take into account preservesPitch

Status: RESOLVED DUPLICATE of bug 1517199

(Core :: Audio/Video: Playback, defect)

Description

[tguilbert]

User Agent: Mozilla/5.0 (X11; CrOS x86_64 13020.67.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/83.0.4103.97 Safari/537.36

Steps to reproduce:

• Create an audio element, and a WebAudio MediaElementAudioSourceNode from it.
• Set preservesPitch (or mozPreservesPitch) to false, and playback rate to 2.0
• Play the audio element

Example test page:
https://tguilbert-google.github.io/preserves_pitch/pitch_detect.html

Actual results:

Pitch is preserved.

Expected results:

Pitch should be shifted.

While there is no formal spec for preservesPitch that describes this behavior, it would make sense to allow for pitch shifting for WebAudio.

I cannot test how this behaves on Safari due to https://bugs.webkit.org/show_bug.cgi?id=211394.

Chrome's behavior is (will be) to allow pitch shifting when preservesPitch=false.

Comment 1

[Dale Curtis]

The reason to do this is that it's impossible for WebAudio to get pitch preserved input from the media element otherwise.

Comment 2

[Gyula Palko[:gpalko]]

Setting the component in order to involve the development team to review this.
Thanks for reporting!

Comment 3

[Alastor Wu [:alwu]]

Hi, Paul,
Is this issue related with bug1517199? I think the reason of the pitch not being changed is because we didn't apply the playback rate change on MediaElementAudioSourceNode?
Thank you.

Comment 4

[Alastor Wu [:alwu]]

Sorry, I didn't mean to close this bug.

Comment 5

[Paul Adenot (:padenot)]

Yes, this is a dupe of bug 1517199, that covers both preservesPitch and playbackRate.

Dale, Thomas, thanks for filing.