Merge pull request #142 from marshallaf/issue139

fix double controller on Coursera
This commit is contained in:
Ilya Grigorik
2016-09-09 17:36:51 +00:00
committed by GitHub

View File

@@ -218,7 +218,15 @@ chrome.extension.sendMessage({}, function(response) {
function checkForVideo(node, parent, added) {
if (node.nodeName === 'VIDEO') {
if (added) {
if (!node.classList.contains('vsc-initialized')) {
if (!node.classList.contains('vsc-initialized') && !node.dataset['vscid']) {
new tc.videoController(node, parent);
}
// if the video has already been initialized, then it has been mutated
// we may need to update the controller location to reflect this
else {
let id = node.dataset['vscid'];
let ctrl = document.querySelector(`div[data-vscid="${id}"]`);
if (ctrl) ctrl.remove();
new tc.videoController(node, parent);
}
} else {