2017-08-04 62 views
1

我有一些编程知识,但是我没有在Javascript中进行广泛的工作或者之前处理过Chrome扩展。我期望构建一个Chrome扩展程序,在新选项卡中打开选定的链接(这是我想实现的最初目标,因为我的最终目标是打开相同选项卡的框架中的链接 - 如果可能)为了保存这些链接中的图像,类似于Firefox的Save Images扩展所做的)。在Chrome扩展程序中关闭弹出框后记住数组

我有一个问题,但弹出窗口关闭后(由于打开新标签),我弹出的选定链接不被“记住”以便使用延迟打开它们。我想这是关于不做一个后台脚本来执行操作,而不是popup.js?也许在这两个Javascipt文件之间传递一些消息?只是猜测这里...

这是到目前为止我的代码:

manifest.json的

{ 
    "name": "Get links", 
    "version": "1.0", 
    "manifest_version": 2, 
    "description": "Get links from a page", 
    "browser_action": { 
    "default_icon": "icon.png", 
    "default_popup": "popup.html" 
    }, 
    "permissions": ["tabs", "downloads", "<all_urls>"] 
} 

popup.html

<!DOCTYPE html> 
<html> 
    <head> 
    <script src="popup.js"></script> 
    </head> 
    <body style="width:400px;"> 
    <form id="linksform"> 
     <p> 
     <label for="matchregex" style="margin-left: 7px;">RegEx Match: </label> 
     <input id="regextext" type="text" style="width:90px;" name="matchregex" value="" /> 
     <input id="togglematches" type="button" value="Toggle Matches" /> 
     <input id="openselected" type="button" value="Open Selected" /> 
     </p> 
     <table id='links'> 
     <th></th> 
     </table> 
    </form> 
    </body> 
</html> 

popup.js

var alllinks = []; 
var visiblelinks = []; 
var selectedlinks = []; 
var delay = 1000; 

// Display all visible links. 

function showlinks() 
    { 
    var togglematchesbutton = document.getElementById('togglematches'); 
    var openselectedbutton = document.getElementById('openselected'); 
    var linkstable = document.getElementById('links'); 
    while (linkstable.children.length > 1) 
    { 
    linkstable.removeChild(linkstable.children[linkstable.children.length - 1]) 
    } 
    for (var i = 0; i < visiblelinks.length; ++i) 
    { 
    var row = document.createElement('tr'); 
    var col0 = document.createElement('td'); 
    var col1 = document.createElement('td'); 
    var checkbox = document.createElement('input'); 
    checkbox.type = 'checkbox'; 
    checkbox.id = 'check' + i; 
    checkbox.checked = false; 
    checkbox.value = visiblelinks[i]; 
    col0.appendChild(checkbox); 
    col1.innerText = visiblelinks[i]; 
    col1.style.whiteSpace = 'nowrap'; 
    col1.onclick = function() 
     { 
     checkbox.checked = !checkbox.checked; 
     } 
    row.appendChild(col0); 
    row.appendChild(col1); 
    linkstable.appendChild(row); 
    } 
    togglematchesbutton.onclick = function() 
    { 
    var regex = new RegExp(document.getElementById("regextext").value); 
    var getinputs = document.getElementsByTagName("input"); 
    for (var i = 0, max = getinputs.length; i < max; i++) 
     { 
     if ((getinputs[i].type === 'checkbox') && (regex.test(getinputs[i].value))) getinputs[i].checked = !getinputs[i].checked; 
     } 
    } 
    openselectedbutton.onclick = function() 
    { 
    var getinputs = document.getElementsByTagName("input"); 
    for (var i = 0, max = getinputs.length; i < max; i++) 
     { 
     if ((getinputs[i].type === 'checkbox') && (getinputs[i].checked)) 
     { 
     selectedlinks.push(getinputs[i].value); 
     } 
     } 
    for (var i = 0, max = selectedlinks.length; i < max; i++) 
     { 
     window.setTimeout(function() {chrome.tabs.create({url: selectedlinks[i]});}, delay); 
     //chrome.tabs.create({url: selectedlinks[i]}); 
     } 
    } 
    } 

// Add links to alllinks and visiblelinks, sort and show them. Sendlinks.js is 
// injected into all frames of the active tab, so this listener may be called 
// multiple times. 

chrome.extension.onMessage.addListener 
    (
    function(links) 
    { 
    for (var index in links) 
     { 
     alllinks.push(links[index]); 
     } 
    alllinks.sort(); 
    visiblelinks = alllinks; 
    //console.log(links); 
    showlinks(); 
    } 
); 

// Set up event handlers and inject sendlinks.js into all frames in the active 
// tab. 

window.onload = function() 
    { 
    chrome.windows.getCurrent 
    (
    function (currentWindow) 
     { 
     chrome.tabs.query 
     (
     {active: true, windowId: currentWindow.id}, 
     function(activeTabs) 
      { 
      chrome.tabs.executeScript(activeTabs[0].id, {file: 'sendlinks.js', allFrames: true}); 
      } 
     ); 
     } 
    ); 
    }; 

sendlinks.js

// Send back to the popup a sorted deduped list of valid link URLs on this page. 
// The popup injects this script into all frames in the active tab. 
console.log("Injected"); 

var links = [].slice.apply(document.getElementsByTagName('a')); 
console.log(links); 
links = links.map 
    (
    function(element) 
    { 
    // Return an anchor's href attribute, stripping any URL fragment (hash '#'). 
    // If the html specifies a relative path, chrome converts it to an absolute 
    // URL. 
    var href = element.href; 
    var hashIndex = href.indexOf('#'); 
    if (hashIndex >= 0) 
     { 
     href = href.substr(0, hashIndex); 
     } 
    return href; 
    } 
); 

links.sort(); 

// Remove duplicates and invalid URLs. 
var kBadPrefix = 'javascript'; 
for (var i = 0; i < links.length;) 
    { 
    if (((i > 0) && (links[i] == links[i - 1])) || (links[i] == '') || (kBadPrefix == links[i].toLowerCase().substr(0, kBadPrefix.length))) 
    { 
    links.splice(i, 1); 
    } 
    else 
    { 
    ++i; 
    } 
    } 
console.log(links); 

chrome.extension.sendMessage(links); 

注:这段代码的大部分是从其他地方取,并已被修改,以满足我的需求。

的主要问题是从popup.js此代码段:

for (var i = 0, max = selectedlinks.length; i < max; i++) 
    { 
    window.setTimeout(function() {chrome.tabs.create({url: selectedlinks[i]});}, delay); 
    //chrome.tabs.create({url: selectedlinks[i]}); 
    } 

如果我评论的setTimeout的线,并取消对以下行,它的工作原理(如扩展打开的选项卡成功),但它不会在打开连续的选项卡之间使用延迟 - 这是避免某些网站上的“请求太多”错误所必需的。如果我放过它,它会打开它应该打开的标签数量(使用指定的延迟),但这些标签的URL不匹配选定的值(基本上它们是空白的)。我想要的是后者发生,但打开选定的链接,而不是空白标签。

请您指出我的错误在哪里,并建议修改代码以使其工作?如果可行,通过保持现有代码尽可能接近发布的版本(也就是只对源代码进行小修改)。谢谢。

回答

0

没关系,我解决了这个问题。事实证明,由于某种原因,Chrome不记得for循环内部的变量(可能由于settimeout()for循环相比异步工作,由于延迟?),所以它不能引用正确的数组元素。

我所要做的就是与

window.setTimeout(function() {chrome.tabs.create({url: selectedlinks.shift()});}, delay); 

替换上面

window.setTimeout(function() {chrome.tabs.create({url: selectedlinks[i]});}, delay); 

,它完美地工作。基本上,代替使用变量引用数组元素,更新后的代码始终引用第一个数组元素,同时将其删除,从而“遍历”整个数组,并消除为其设置的元素选项卡已创建。