Question

我有一些编程知识，但是之前我没有在Javascript中广泛使用或处理过Chrome扩展程序。我希望构建一个Chrome扩展程序，在新标签页中打开所选链接（这是我想要实现的初始目标，因为我的最终目标是在相同标签的框架中打开链接 - 如果可能的话 - ，为了保存这些链接中的图像，类似于Firefox的Save Images扩展程序。

我遇到了一个问题：我弹出窗口中的选定链接不是＆＃34;记住＆＃34;在弹出窗口关闭后（由于打开新选项卡），使用延迟打开它们。我想这与不制作后台脚本来执行操作有关，而不是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行进行注释并取消注释以下行，则它可以正常工作（例如，扩展程序会成功打开选项卡），但它不会在打开连续选项卡之间使用延迟 - 这是避免＆＃39所必需的;请求太多＆＃39;某些网站上的错误。如果我按原样使用它，它会打开它应该打开的标签数量（使用指定的延迟），但这些标签的网址不匹配所选的值（基本上，它们是＆＃39; re空白）。我想要的是后者发生，但打开选定的链接而不是空白标签。

请指出我的错误，并建议修改代码以使其工作？如果可行的话，通过保持现有代码尽可能接近发布的版本（也就是对源只进行微小的修改）。谢谢。

Answer 1

没关系，我解决了这个问题。事实证明，出于某种原因，Chrome并不记得for循环中的 i 变量（可能是因为settimeout()与for循环相比，由于延迟而异步工作？），因此它无法引用正确的数组元素。

我所要做的就是替换上面的

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

与

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

它完美无缺。基本上，不是使用 i 变量引用数组元素，而是更新的代码始终引用第一个数组元素，同时将其删除，因此＆＃34 ;导航＆＃34;通过整个数组并消除已创建选项卡的元素。

弹出窗口在Chrome扩展程序中关闭后，请记住数组

1 个答案: