Fixes from PR, removing local event emitter

This commit is contained in:
Robert Smallwood 2015-12-14 15:23:07 -07:00 committed by Mike Girard
parent 6bc94d1692
commit 383afbc66d
2 changed files with 4 additions and 5 deletions

View File

@ -33,8 +33,6 @@ var firefoxExtInstalled = null;
*/
var reDetectFirefoxExtension = false;
var eventEmitter = new EventEmitter();
/**
* Handles obtaining a stream from a screen capture on different browsers.
*/
@ -185,7 +183,8 @@ function isUpdateRequired(minVersion, extVersion) {
return false;
}
catch (e) {
eventEmitter.emit(DesktopSharingEventTypes.EXTENSION_VERSION_ERROR);
this.eventEmitter
.emit(DesktopSharingEventTypes.EXTENSION_VERSION_ERROR);
console.error("Failed to parse extension version", e);
return true;
}
@ -282,7 +281,7 @@ function obtainScreenFromExtension(streamCallback, failCallback) {
}, 500);
},
function (arg) {
eventEmitter
this.eventEmitter
.emit(DesktopSharingEventTypes.EXTENSION_INSTALLATION_ERROR);
console.log("Failed to install the extension", arg);
failCallback(arg);

View File

@ -95,7 +95,7 @@ var Settings = {
setLanguage: function (lang) {
language = lang;
window.localStorage.language = lang;
},
}
};
module.exports = Settings;