disable two handlers, improve twitch, add mainclose setting

external-video.js

@@ -1,11 +1,9 @@
 var targetPages = [
-  "*://*.youtube.com/watch*",
+  "*://*.youtube.com/watch?*",
   "*://*.twitch.tv/*",
   "*://*.vimeo.com/*",
   "*://*.streamable.com/*",
   "*://*.liveleak.com/view*",
-  "*://*.vid.me/*",
-  "*://*.funnyordie.com/videos/*",
   "*://*.dailymotion.com/video/*"
 ];
 
@@ -34,28 +32,47 @@ function restoreSettings() {
 }
 
 function openInMpv(request) {
+  if (!(request.type == "main_frame")) {
+    return { cancel: false };
+  }
+
   var lockedTabIndex = tabsLock.lastIndexOf(request.tabId);
 
+  if (!(lockedTabIndex == -1)) {
+    return { cancel: false };
+  }
+
   function closeTab(data) {
+    if (settings.mainClose) {
+      browser.tabs.remove(data.id);
+    }
     if (!data.active) {
       browser.tabs.remove(data.id);
     }
   }
 
-  if (request.type === "main_frame" && lockedTabIndex === -1) {
-    var command = `${request.url} --force-window=immediate ${settings.args}`;
-
+  function mpvRun(data) {
+    var command = `${data.url} --force-window=immediate`;
     browser.runtime.sendNativeMessage("mpv", command);
 
-    var querying = browser.tabs.get(request.tabId);
-    querying.then(closeTab);
-
     browser.history.addUrl({
-      url: request.url
+      url: data.url
     });
 
-    return { cancel: true };
+    var querying = browser.tabs.get(data.tabId);
+    querying.then(closeTab);
+  }
+
+  if (request.url == "https://www.twitch.tv/") {
+    return { cancel: false };
   }
+
+  if (request.url.includes("twitch.tv/directory")) {
+    return { cancel: false };
+  }
+
+  mpvRun(request);
+  return { cancel: true };
 }
 
 chrome.contextMenus.create({

settings/options.html

@@ -13,9 +13,16 @@
          Options given to MPV
          <input type="text" id="args" style="width:50%;">
        </label>
+       <br>
+       <label>
+         Still close tab to be viewed if it's focused?
+         <input type="checkbox" id="mainClose">
+       </label>
+       <br>
        <button type="submit">Save</button>
    </form>
 
+
    <script src="options.js"></script>
 
   </body>

settings/options.js

@@ -1,16 +1,25 @@
 function saveOptions(e) {
-  browser.storage.local.set({
-    args: document.querySelector("#args").value
-  });
+    browser.storage.local.set({
+        args: document.querySelector("#args").value
+    });
+    browser.storage.local.set({
+        mainClose: document.querySelector("#mainClose").checked
+    });
+    restoreOptions;
 }
 
 function restoreOptions() {
-  function setCurrentChoice(result) {
-    document.querySelector("#args").value = result.args || "--pause --ytdl-format='bestvideo[height<=?1080]+bestaudio/best'";
+    function setArgs(result) {
+        document.querySelector("#args").value = result.args || "--pause --ytdl-format='bestvideo[height<=?1080]+bestaudio/best'";
   }
+    function setMainClose(result) {
+        document.querySelector("#mainClose").checked = result.mainClose;
+    }
 
-  var getting = browser.storage.local.get("args");
-  getting.then(setCurrentChoice);
+    var getting = browser.storage.local.get("args");
+    getting.then(setArgs);
+    var getting = browser.storage.local.get("mainClose");
+    getting.then(setMainClose);
 }
 
 document.addEventListener("DOMContentLoaded", restoreOptions);