But it was only added 2 weeks ago by @dstftw in
b68a812ea8, so maybe it should be
handled differently? Remove the test? Mark it only_matching? Not sure.
Black screen in browsers.
youtube-dl reports:
[brightcove:legacy] ref:event-stream-356: Downloading webpage
ERROR: brightcove said: This player is not enabled for access without key.
I really reget not being able to use this compact syntax, as well as to align
with spaces:
if not account_id: account_id = attrs.get('data-account')
if not player_id: player_id = attrs.get('data-player')
if not embed: embed = attrs.get('data-embed')
but it really upsets flake8:
brightcove.py:519:30: E701 multiple statements on one line (colon)
brightcove.py:519:42: E221 multiple spaces before operator
brightcove.py:520:29: E701 multiple statements on one line (colon)
brightcove.py:520:30: E241 multiple spaces after ':'
brightcove.py:520:41: E221 multiple spaces before operator
brightcove.py:521:25: E701 multiple statements on one line (colon)
brightcove.py:521:26: E241 multiple spaces after ':'
brightcove.py:521:37: E221 multiple spaces before operator
Per @yan12125.
Instead of searching for <video> followed by <script>, now
search for <video> optionally followed by <script>. If we
find <script>, use it. Otherwise, extract_attributes() from the video
tag and use those.
Move stripping the 'ref': prefix of data-video-id="ref:12345" out of
regexp into code, since we now use extract_attributes().
Otherwise, if you screw up a playlist test by including a playlist
dictionary key, you'll be there for eons while it downloads all the
files before erroring out.