1393669 - Remove useless attributes for SourceBuffer.webidl

Status: RESOLVED INVALID

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

Description

[Alastor Wu [:alwu]]

According to [1], those comment could be removed, they're not the part of the source buffer.

[1] https://www.w3.org/TR/media-source/#sourcebuffer

Comment 1

[Alastor Wu [:alwu]]

Attachment: Bug 1393669 - remove useless attributes of SourceBuffer.webidl.

According to [1], those comment could be removed, they're not the part of the source buffer.

[1] https://www.w3.org/TR/media-source/#sourcebuffer

Review commit: https://reviewboard.mozilla.org/r/172482/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/172482/

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8901014 [details]
Bug 1393669 - remove useless attributes of SourceBuffer.webidl.

https://reviewboard.mozilla.org/r/172482/#review177848

::: commit-message-39f53:5
(Diff revision 1)
> +Bug 1393669 - remove useless attributes of SourceBuffer.webidl.
> +
> +According to [1], those comment could be removed, they're not the part of the source buffer.
> +
> +[1] https://www.w3.org/TR/media-source/#sourcebuffer

Please don't use /TR/ specs, but the up-to-date specs, in this case
https://w3c.github.io/media-source/

::: dom/webidl/SourceBuffer.webidl
(Diff revision 1)
>    readonly attribute boolean updating;
>    [Throws]
>    readonly attribute TimeRanges buffered;
>    [SetterThrows]
>    attribute double timestampOffset;
> -  //readonly attribute AudioTrackList audioTracks;

I definitely see the following in the spec
    readonly attribute AudioTrackList      audioTracks;
    readonly attribute VideoTrackList      videoTracks;
    readonly attribute TextTrackList       textTracks;

::: dom/webidl/SourceBuffer.webidl
(Diff revision 1)
>    attribute EventHandler onabort;
>    [Throws]
>    void appendBuffer(ArrayBuffer data);
>    [Throws]
>    void appendBuffer(ArrayBufferView data);
> -  //[Throws]

This is fine.

Comment 3

[Alastor Wu [:alwu]]

(In reply to Olli Pettay [:smaug] from comment #2)
> ::: commit-message-39f53:5
> (Diff revision 1)
> > +Bug 1393669 - remove useless attributes of SourceBuffer.webidl.
> > +
> > +According to [1], those comment could be removed, they're not the part of the source buffer.
> > +
> > +[1] https://www.w3.org/TR/media-source/#sourcebuffer
> 
> Please don't use /TR/ specs, but the up-to-date specs, in this case
> https://w3c.github.io/media-source/

What's the difference between TR and non-TR?
Because the left bar on the TR spec said that it's "W3C Recommendation", and the one in non-TR said that it's W3C "Editor's Draft".
Is the non-TR one always better than TR one?
Thanks!

Comment 4

[Jean-Yves Avenard [:jya]]

Comment on attachment 8901014 [details]
Bug 1393669 - remove useless attributes of SourceBuffer.webidl.

https://reviewboard.mozilla.org/r/172482/#review177856

::: dom/webidl/SourceBuffer.webidl
(Diff revision 1)
>    readonly attribute boolean updating;
>    [Throws]
>    readonly attribute TimeRanges buffered;
>    [SetterThrows]
>    attribute double timestampOffset;
> -  //readonly attribute AudioTrackList audioTracks;

those aren't useless. they are valid attributes in the spec.

we don't do it at this stage (and we should).

So the comment "useless" isn't appropriate.

::: dom/webidl/SourceBuffer.webidl
(Diff revision 1)
>    [Throws]
>    void appendBuffer(ArrayBuffer data);
>    [Throws]
>    void appendBuffer(ArrayBufferView data);
> -  //[Throws]
> -  //void appendStream(Stream stream, [EnforceRange] optional unsigned long long maxSize);

this one got removed from the V1 spec, but there's strong push to re-add them in V2.
We didn't support Fetch Stream before, which is why we couldn't implement those specs.

Now we can

Comment 5

[Olli Pettay [:smaug][[e-mail]]]

(In reply to Alastor Wu [:alwu][please needinfo me][GMT+8] from comment #3)
> What's the difference between TR and non-TR?
TR is some old static version of the spec (may it called Recommendation or whatever), and non-TR is the up-to-date version including spec bug fixes etc.


In this case though it is not too clear which one is better since both are a bit old, from last November.
But in general non-TR should be newer.

Non-TR is basically Living Standard usually.

Comment 6

[Alastor Wu [:alwu]]

According to the comment4, set it as INVALID.