Remove HTMLMediaElement.webkitPreservesPitch

Remove HTMLMediaElement.webkitPreservesPitch

It is not implemented, so remove it until there is a spec, at
which point it can be implemented without a prefix.

BUG=305554
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=159246

[philipj_slow, 2013-10-09 09:52:58]

There's a non-zero compat risk here, since the property is visible and it can be
used to sniff for... something. However, since it doesn't actually work, it
seems better to not have it at all.

[acolwell GONE FROM CHROMIUM, 2013-10-09 14:26:31]

lgtm

[adamk, 2013-10-09 16:29:13]

I'm not an IDL owner, passing off to eseidel...

But I'm curious if we have any interest in actually implementing preservesPitch
in Blink. I believe it's in the spec, and Mozilla has implemented it (I looked
at this a few months ago and simplified the innards of the code without removing
the web-exposed bit).

[eseidel, 2013-10-09 16:30:49]

lgtm

remove all teh codez.

[commit-bot: I haz the power, 2013-10-09 16:31:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/philipj@opera.com/26679002/1

[commit-bot: I haz the power, 2013-10-09 18:11:59]

Change committed as 159246

[philipj_slow, 2013-10-09 18:30:03]

On 2013/10/09 16:29:13, adamk wrote:
> I'm not an IDL owner, passing off to eseidel...
> 
> But I'm curious if we have any interest in actually implementing
preservesPitch
> in Blink. I believe it's in the spec, and Mozilla has implemented it (I looked
> at this a few months ago and simplified the innards of the code without
removing
> the web-exposed bit).

There's no pitch-related property on HTMLMediaElement:
http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element...

I looks Mozilla has indeed added mozPreservesPitch, but the bug
<https://www.w3.org/Bugs/Public/show_bug.cgi?id=15318> they filed to standardize
it was rejected because Hixie thinks you should simply always preserve pitch. If
that isn't what we already do, I guess that would be a start.