Move screen sharing error check into platform

Because on the browser it depends on HTTPS, on electron it doesn't
matter.
pull/21833/head
David Baker 2017-01-10 18:37:57 +00:00
parent 085f134e84
commit a16aae1124
2 changed files with 16 additions and 0 deletions

View File

@ -73,4 +73,13 @@ export default class BasePlatform {
getAppVersion() {
throw new Error("getAppVersion not implemented!");
}
/*
* If it's not expected that capturing the screen will work
* with getUserMedia, return a string explaining why not.
* Otherwise, return null.
*/
screenCaptureErrorString() {
return "Not implemented";
}
}

View File

@ -52,6 +52,7 @@ limitations under the License.
*/
var MatrixClientPeg = require('./MatrixClientPeg');
var PlatformPeg = require("./PlatformPeg");
var Modal = require('./Modal');
var sdk = require('./index');
var Matrix = require("matrix-js-sdk");
@ -187,6 +188,12 @@ function _onAction(payload) {
);
}
else if (payload.type === 'screensharing') {
const screenCapErrorString = PlatformPeg.get().screenCaptureErrorString();
if (screenCapErrorString) {
_setCallState(undefined, newCall.roomId, "ended");
console.log("Can't capture screen: " + screenCapErrorString);
return;
}
newCall.placeScreenSharingCall(
payload.remote_element,
payload.local_element