Skip to content
This repository has been archived by the owner on Jan 12, 2019. It is now read-only.

Commit

Permalink
rephrase the comment
Browse files Browse the repository at this point in the history
  • Loading branch information
Shan Zhuang committed Jun 16, 2017
1 parent 6275f13 commit b1443be
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/master-playlist-controller.js
Original file line number Diff line number Diff line change
Expand Up @@ -1078,7 +1078,7 @@ export class MasterPlaylistController extends videojs.EventTarget {
// Since we're on the final non-blacklisted playlist, and we're about to blacklist it,
// instead of erring the player or retrying this playlist, we clear out the current blacklist
// so the other playlists may be attempted in case any have been fixed.
videojs.log.warn('Clearing blacklist for every playlist because last rendition ' +
videojs.log.warn('Removing all playlists from the blacklist because the last rendition ' +
'is about to be blacklisted.');
playlists.forEach((playlist) => {
delete playlist.excludeUntil;
Expand Down
2 changes: 1 addition & 1 deletion test/videojs-contrib-hls.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -1141,7 +1141,7 @@ QUnit.test('playlist 404 should blacklist media', function(assert) {
assert.ok(media.excludeUntil > 0, 'second media was blacklisted after playlist 404');
assert.equal(this.env.log.warn.calls, 2, 'warning logged for blacklist');
assert.equal(this.env.log.warn.args[1],
'Clearing blacklist for every playlist because last rendition is about to be blacklisted.',
'Removing all playlists from the blacklist because the last rendition is about to be blacklisted.',
'log generic error message');
assert.equal(this.env.log.warn.args[2],
'Problem encountered with the current HLS playlist. HLS playlist request error at URL: media1.m3u8. ' +
Expand Down

0 comments on commit b1443be

Please sign in to comment.