ProximaEntertainment/plyr
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Removed redundant wrappers within the adsmanager promises.

Browse Source
This commit is contained in:
Arthur Hulsman 2018-03-09 14:29:37 +01:00
parent 7adc2bc6c8
commit 6a2ca534d2
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/js/plugins/ads.js
View File

@ -68,9 +68,9 @@ class Ads {
// Setup a promise to resolve when the IMA manager is ready // Setup a promise to resolve when the IMA manager is ready
this.managerPromise = new Promise((resolve, reject) => { this.managerPromise = new Promise((resolve, reject) => {
// The ad is pre-loaded and ready // The ad is pre-loaded and ready
this.on('ADS_MANAGER_LOADED', () => resolve()); this.on('ADS_MANAGER_LOADED', resolve);
// Ads failed // Ads failed
this.on('ERROR', () => reject()); this.on('ERROR', reject);
}); });
} }
@ -503,7 +503,7 @@ class Ads {
// Re-set our adsManager promises // Re-set our adsManager promises
this.managerPromise = new Promise(resolve => { this.managerPromise = new Promise(resolve => {
this.on('ADS_MANAGER_LOADED', () => resolve()); this.on('ADS_MANAGER_LOADED', resolve);
this.player.debug.log(this.manager); this.player.debug.log(this.manager);
}); });