[Webkit-unassigned] [Bug 160199] New: MediaFragmentURIParser rejects valid NPT strings if 'hours' is defined using 1 digit

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jul 26 09:36:42 PDT 2016


https://bugs.webkit.org/show_bug.cgi?id=160199

            Bug ID: 160199
           Summary: MediaFragmentURIParser rejects valid NPT strings if
                    'hours' is defined using 1 digit
    Classification: Unclassified
           Product: WebKit
           Version: WebKit Local Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Minor
          Priority: P2
         Component: Media Elements
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: fvallee at connected-labs.com

If a media uri contains a fragment, the MediaFragmentURIParser (Source/WebCore/html/MediaFragmentURIParser.cpp) is used to get the temporal clipping values (if any).

Based on https://www.w3.org/TR/media-frags/#rtsp: "Minutes and seconds must be specified as exactly two digits, hours and fractional seconds can be any number of digits".
As far as I understand the specifications, "t=0:02:00,121.5" is valid (it's an example from the W3C specs anyway) but this string is rejected by the MediaFragmentURIParser (Source/WebCore/html/MediaFragmentURIParser.cpp)

===================
Steps to reproduce:

The regression test LayoutTests/media/media-fragments/TC0078.html can PASS using "t=00:00:03,00:00:07" string, as defined in LayoutTests/media/media-fragments/media-fragments.js:
TC0078 : { start: 3, end: 7, valid: true, description: "NPT HH:MM:SS format", fragment: "t=00:00:03,00:00:07", comment: "The media is requested from a to b."},

If the fragment is changed to "t=0:00:03,00:00:07" (use only 1 digit for the hour), the test fails (based on my understanding "NPT H:MM:SS format" is valid and the test should pass).

===================
Note: 1-digit hour string ("0:00:03") is rejected because of https://trac.webkit.org/browser/trunk/Source/WebCore/html/MediaFragmentURIParser.cpp#L281

Please let me know if I'm missing something. thanks.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160726/629a2578/attachment.html>


More information about the webkit-unassigned mailing list