subparse: accept WebVTT timestamps without an hour component

https://www.w3.org/TR/webvtt1/#webvtt-timestamp

Change-Id: I5e87fdcda7f150c12fc08857ce04c378001321bd
mm:ss,000 is a valid WebVTT timestamp

diff --git a/gst/subparse/gstsubparse.c b/gst/subparse/gstsubparse.c
index 2115da8..c702a33 100644 (file)
--- a/gst/subparse/gstsubparse.c
+++ b/gst/subparse/gstsubparse.c
@@ -999,8 +999,23 @@ parse_subrip_time (const gchar * ts_string, GstClockTime * t)
 
   GST_LOG ("parsing timestamp '%s'", s);
   if (sscanf (s, "%u:%u:%u,%u", &hour, &min, &sec, &msec) != 4) {
+#ifdef TIZEN_FEATURE_UPSTREAM
+    /* https://www.w3.org/TR/webvtt1/#webvtt-timestamp
+     *
+     * The hours component is optional with webVTT, for example
+     * mm:ss,500 is a valid webVTT timestamp. When not present,
+     * hours is 0.
+     */
+    hour = 0;
+
+    if (sscanf (s, "%u:%u,%u", &min, &sec, &msec) != 3) {
+      GST_WARNING ("failed to parse subrip timestamp string '%s'", s);
+      return FALSE;
+    }
+#else
     GST_WARNING ("failed to parse subrip timestamp string '%s'", s);
     return FALSE;
+#endif
   }
 
   *t = ((hour * 3600) + (min * 60) + sec) * GST_SECOND + msec * GST_MSECOND;