Merge branch 'exists-calls-too-frequent' into development

Author: Simounet

wallabagger/js/background.js

@@ -123,9 +123,8 @@ function onTabCreatedListener (tab) {
 }
 
 function onTabUpdatedListener (tabId, changeInfo, tab) {
-    if (changeInfo.status === 'loading' && tab.active) {
-        saveExistFlag(tab.url, existStates.notexists);
-        requestExists(tab.url);
+    if ((changeInfo.status === 'loading') && tab.active) {
+        checkExist(tab.url);
     }
 }
 
@@ -457,8 +456,9 @@ function savePageToWallabag (url, resetIcon) {
 
 const GotoWallabag = (part) => api.checkParams() && browser.tabs.create({ url: `${api.data.Url}/${part}/list` });
 
-const checkExist = (url) => {
-    if (isServicePage(url)) { return; }
+const checkExist = (dirtyUrl) => {
+    if (isServicePage(dirtyUrl)) { return; }
+    const url = dirtyUrl.split('#')[0];
     if (existCache.check(url)) {
         const existsFlag = existCache.get(url);
         if (existsFlag === existStates.exists) {
@@ -491,7 +491,7 @@ const saveExistFlag = (url, exists) => {
     existCache.set(url, exists);
 };
 
-const isServicePage = (url) => RegExp('((chrome|about|browser):(.*)|' + api.data.Url + ')', 'g').test(url);
+const isServicePage = (url) => !/^https?:\/\/.+/.test(url) || RegExp('^' + api.data.Url).test(url);
 
 const addToAllTags = (tags) => {
     if (tags.length === 0) { return; }