Closed Bug 1415821 Opened 3 years ago Closed 3 years ago

[webvtt] support multi-line region parsing.

Categories

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

enhancement

Tracking

()

RESOLVED FIXED
mozilla59
Tracking Status
firefox59 --- fixed

People

(Reporter: bechen, Assigned: bechen)

References

Details

(Keywords: dev-doc-complete)

Attachments

(3 files)

https://www.w3.org/TR/webvtt1/#introduction-other-features example 10:

REGION
id:fred
width:40%
lines:3
regionanchor:0%,100%
viewportanchor:10%,90%
scroll:up
...

Now our parser only support only one line parsing.
Priority: -- → P3
Comment on attachment 8927752 [details]
Bug 1415821 - Modify .vtt REGION syntax.

https://reviewboard.mozilla.org/r/199022/#review204082
Attachment #8927752 - Flags: review?(alwu) → review+
Per offline discussion, will review the patch again after passing tests.
Comment on attachment 8927751 [details]
Bug 1415821 - Support multi-line parsing for REGION/STYLE block.

https://reviewboard.mozilla.org/r/199020/#review204786
Attachment #8927751 - Flags: review?(alwu) → review+
Comment on attachment 8928412 [details]
Bug 1415821 - Correct the behavior for BADCUE state.

https://reviewboard.mozilla.org/r/199636/#review204788

::: dom/media/webvtt/vtt.jsm:1360
(Diff revision 1)
>          }
>  
>          var nextIteration = false;
>          while (nextIteration || self.buffer) {
>            nextIteration = false;
> -          if (!line) {
> +          if (self.state == "BADCUE") {

Per offline discussed, should handle "BADCUE" in the following switch statement.
Comment on attachment 8928412 [details]
Bug 1415821 - Correct the behavior for BADCUE state.

https://reviewboard.mozilla.org/r/199636/#review204958
Attachment #8928412 - Flags: review?(alwu) → review+
Keywords: checkin-needed
Pushed by rgurzau@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0f276f1cc275
Modify .vtt REGION syntax. r=alwu
https://hg.mozilla.org/integration/autoland/rev/1786297a20df
Support multi-line parsing for REGION/STYLE block. r=alwu
https://hg.mozilla.org/integration/autoland/rev/5a9fbe6806cf
Correct the behavior for BADCUE state. r=alwu
Keywords: checkin-needed
Firefox 59 for developers now notes this change.
You need to log in before you can comment on or make changes to this bug.