Помните массив после закрытия всплывающего окна в расширении Chrome

1

У меня есть некоторые знания в области программирования, однако я не работал много в Javascript или раньше рассматривал расширения Chrome. Я хочу создать расширение Chrome, которое открывает выбранные ссылки на новых вкладках (это первая цель, которую я хочу достичь, поскольку моей конечной целью является открытие ссылок в рамках одной и той же вкладки - если возможно - в чтобы сохранить изображения из этих ссылок, аналогично тому, что делает расширение "Сохранить изображения" для Firefox).

У меня есть проблема: выбранные ссылки из моего всплывающего окна не запоминаются, чтобы открыть их с помощью задержки, после того, как всплывающее окно закрыто (из-за открытия новых вкладок). Думаю, это связано с тем, что вместо выполнения фонового сценария для выполнения операции вместо 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 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-адреса этих вкладок не соответствуют выбранным значениям (в основном, они пусты). Я хочу, чтобы последнее произошло, но открывая выбранные ссылки вместо пустых вкладок.

Не могли бы вы указать, где моя ошибка, и предложить изменения кода, чтобы сделать эту работу? Если это выполнимо, сохранив существующий код как можно ближе к опубликованной версии (так же, как только незначительные изменения в источнике). Спасибо.

Теги:
arrays
background
google-chrome
popup

1 ответ

0
Лучший ответ

Nevermind, я решил проблему. Оказывается, по какой-то причине Chrome не помнил переменную i внутри цикла for (возможно, из-за того, что settimeout() работает асинхронно по сравнению с циклом for из-за задержки?), Поэтому он не мог ссылаться на правильный элемент массива.

Все, что мне нужно было сделать, это заменить выше

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

с

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

и он работал безупречно. В принципе, вместо ссылки на элемент массива с использованием переменной i обновленный код всегда ссылается на первый элемент массива, одновременно удаляя его, таким образом, "перемещаясь" по всему массиву и устраняя элементы, для которых были созданы вкладки,

Поделиться

Ещё вопросы

Сообщество Overcoder
Наверх
Меню