InfoExtractors: use report_download_webpage in _request_webpage

Allows to show the warning when falling back on GenericIE
This commit is contained in:
Jaime Marquínez Ferrándiz 2013-04-24 22:11:57 +02:00
parent 320e26a0af
commit 0d173446ff

View file

@ -114,8 +114,8 @@ def IE_NAME(self):
def _request_webpage(self, url_or_request, video_id, note=None, errnote=None): def _request_webpage(self, url_or_request, video_id, note=None, errnote=None):
""" Returns the response handle """ """ Returns the response handle """
if note is None: if note is None:
note = u'Downloading video webpage' self.report_download_webpage(video_id)
if note is not False: elif note is not False:
self.to_screen(u'%s: %s' % (video_id, note)) self.to_screen(u'%s: %s' % (video_id, note))
try: try:
return compat_urllib_request.urlopen(url_or_request) return compat_urllib_request.urlopen(url_or_request)
@ -1318,7 +1318,7 @@ def report_download_webpage(self, video_id):
"""Report webpage download.""" """Report webpage download."""
if not self._downloader.params.get('test', False): if not self._downloader.params.get('test', False):
self._downloader.report_warning(u'Falling back on generic information extractor.') self._downloader.report_warning(u'Falling back on generic information extractor.')
self.to_screen(u'%s: Downloading webpage' % video_id) super(GenericIE, self).report_download_webpage(video_id)
def report_following_redirect(self, new_url): def report_following_redirect(self, new_url):
"""Report information extraction.""" """Report information extraction."""