From 630b099fd6df9b59f6acf1228e3be8b812ce2b58 Mon Sep 17 00:00:00 2001 From: Ajay Ramachandran Date: Thu, 19 Dec 2019 00:36:35 -0500 Subject: [PATCH] Fixed formatting and removed whitelisting fix code. Whitelisting was now fixed in a different way. --- content.js | 20 ++++---------------- utils.js | 7 +++---- 2 files changed, 7 insertions(+), 20 deletions(-) diff --git a/content.js b/content.js index cf0ef27c..424b476f 100644 --- a/content.js +++ b/content.js @@ -474,14 +474,6 @@ function sponsorsLookup(id, channelIDPromise) { sponsorLookupRetries++; } - - // If the initial channel whitelist check finishes before - // the fetching of the sponsor times, then the sponsor times will be - // added again after the whitelist check clears them. Checking it here makes sure - // the whitelist will be applied (downside is `whitelistCheck()` gets called multiple times). - if (channelWhitelisted) { - whitelistCheck(); - } }); //add the event to run on the videos "ontimeupdate" @@ -530,8 +522,7 @@ function getChannelID() { channelContainers = document.getElementsByClassName("yt-user-info"); if (channelContainers.length != 0) { channelURLContainer = channelContainers[0].firstElementChild; - } - else if (onInvidious) { + } else if (onInvidious) { // Unfortunately, the Invidious HTML doesn't have much in the way of element identifiers... channelContainers = document.querySelector("body > div > div.pure-u-1.pure-u-md-20-24 div.pure-u-1.pure-u-lg-3-5 > div > a"); if (channelContainers.length != 0) { @@ -550,12 +541,10 @@ function getChannelID() { let currentTitle = ""; if (titleInfoContainer != null) { currentTitle = titleInfoContainer.firstElementChild.firstElementChild.querySelector(".title").firstElementChild.innerText; - } - else if (onInvidious) { + } else if (onInvidious) { // Unfortunately, the Invidious HTML doesn't have much in the way of element identifiers... currentTitle = document.querySelector("body > div > div.pure-u-1.pure-u-md-20-24 div.pure-u-1.pure-u-lg-3-5 > div > a > div > span").textContent; - } - else { + } else { //old YouTube theme currentTitle = document.getElementById("eow-title").innerText; } @@ -760,8 +749,7 @@ function getControls() { // The invidious video element's controls element controls = document.getElementsByClassName("vjs-control-bar"); return (!controls || controls.length === 0) ? false : controls[controls.length - 1]; - } - else { + } else { return controls[controls.length - 1]; } }; diff --git a/utils.js b/utils.js index dceedeb8..4373d69c 100644 --- a/utils.js +++ b/utils.js @@ -32,11 +32,10 @@ function getYouTubeVideoID(url) { } //Check if valid hostname - if(!["www.youtube.com", "www.youtube-nocookie.com", ...supportedInvidiousInstances].includes(urlObject.host)) { - return false; - } - else if (supportedInvidiousInstances.includes(urlObject.host)) { + if (supportedInvidiousInstances.includes(urlObject.host)) { onInvidious = true; + } else if (!["www.youtube.com", "www.youtube-nocookie.com"].includes(urlObject.host)) { + return false } //Get ID from searchParam