From b1443be1e043be74df95f7280e86762a4abcb8f4 Mon Sep 17 00:00:00 2001 From: Shan Zhuang Date: Fri, 16 Jun 2017 16:24:12 -0400 Subject: [PATCH] rephrase the comment --- src/master-playlist-controller.js | 2 +- test/videojs-contrib-hls.test.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/master-playlist-controller.js b/src/master-playlist-controller.js index 975e05e33..19580b014 100644 --- a/src/master-playlist-controller.js +++ b/src/master-playlist-controller.js @@ -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; diff --git a/test/videojs-contrib-hls.test.js b/test/videojs-contrib-hls.test.js index f52fa55ff..39f116f03 100644 --- a/test/videojs-contrib-hls.test.js +++ b/test/videojs-contrib-hls.test.js @@ -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. ' +