From: Yen Chi Hsuan <yan12125@gmail.com>
Date: Sat, 24 Sep 2016 06:27:08 +0000 (+0800)
Subject: [openload] Support subtitles (closes #10625)
X-Git-Url: http://git.bitcoin.ninja/?a=commitdiff_plain;h=0711995bcac2f44e09a943521dceb1c54bf8ffb7;p=youtube-dl

[openload] Support subtitles (closes #10625)
---

diff --git a/ChangeLog b/ChangeLog
index ebe4ff0e8..766cc477b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -3,6 +3,9 @@ vesion <unreleased>
 Core
 + Improved support for HTML5 subtitles
 
+Extractors
++ [openload] Support subtitles (#10625)
+
 
 version 2016.09.24
 
diff --git a/youtube_dl/extractor/openload.py b/youtube_dl/extractor/openload.py
index b6e3ac250..4f5175136 100644
--- a/youtube_dl/extractor/openload.py
+++ b/youtube_dl/extractor/openload.py
@@ -24,6 +24,22 @@ class OpenloadIE(InfoExtractor):
             'title': 'skyrim_no-audio_1080.mp4',
             'thumbnail': 're:^https?://.*\.jpg$',
         },
+    }, {
+        'url': 'https://openload.co/embed/rjC09fkPLYs',
+        'info_dict': {
+            'id': 'rjC09fkPLYs',
+            'ext': 'mp4',
+            'title': 'movie.mp4',
+            'thumbnail': 're:^https?://.*\.jpg$',
+            'subtitles': {
+                'en': [{
+                    'ext': 'vtt',
+                }],
+            },
+        },
+        'params': {
+            'skip_download': True,  # test subtitles only
+        },
     }, {
         'url': 'https://openload.co/embed/kUEfGclsU9o/skyrim_no-audio_1080.mp4',
         'only_matching': True,
@@ -71,11 +87,17 @@ class OpenloadIE(InfoExtractor):
             'title', default=None) or self._html_search_meta(
             'description', webpage, 'title', fatal=True)
 
-        return {
+        entries = self._parse_html5_media_entries(url, webpage, video_id)
+        subtitles = entries[0]['subtitles'] if entries else None
+
+        info_dict = {
             'id': video_id,
             'title': title,
             'thumbnail': self._og_search_thumbnail(webpage, default=None),
             'url': video_url,
             # Seems all videos have extensions in their titles
             'ext': determine_ext(title),
+            'subtitles': subtitles,
         }
+
+        return info_dict