Intent to Remove: AudioBufferSourceNode.looping

44 views
Skip to first unread message

Philip Jägenstedt

unread,
Jan 16, 2014, 10:50:30 AM1/16/14
to blink-dev

Primary eng (and PM) emails

phi...@opera.com


Summary

Remove the looping boolean attribute from the AudioBufferSourceNode.


Motivation

This attribute was replaced by "loop" in the spec:

It has been deprecated in Blink since April 2013 (r149184).


Usage information from UseCounter

This was on pdr's list <http://crbug.com/332191> of features which are "unused or have extremely low usage numbers". I don't have the exact numbers. The use counter is old enough to have been in the stable channel for a while now.

Compatibility Risk

Scripts that set .looping to true won't actually have looping audio.

Row on feature dashboard?

No.


Requesting approval to remove too?

Yes, I'd like to remove the feature immediately with no deprecation period.



Eric Seidel

unread,
Jan 16, 2014, 11:00:51 AM1/16/14
to Philip Jägenstedt, blink-dev
Lgtm

Adam Barth

unread,
Jan 16, 2014, 11:47:56 AM1/16/14
to Eric Seidel, Philip Jägenstedt, blink-dev
LGTM

Dimitri Glazkov

unread,
Jan 16, 2014, 11:55:42 AM1/16/14
to Adam Barth, Eric Seidel, Philip Jägenstedt, blink-dev
L G T M.
Reply all
Reply to author
Forward
0 new messages