Crawler frontend

chrome/content/crawler.js

 /*
  * This Source Code is subject to the terms of the Mozilla Public License
  * version 2.0 (the "License"). You can obtain a copy of the License at
  * http://mozilla.org/MPL/2.0/.
  */
 

[Wladimir Palant, 2012/09/21 15:36:18]

Site-note: I don't think that this manual UI will be all too useful beyond your
initial testing, I consider this temporary code. Screenshotter creates a new
Firefox profile, installs Adblock Plus and itself into that profile
automatically and runs Firefox then - this extension needs to run the same way.
Starting it in a random Firefox profile, potentially with other extensions and
non-default browser settings just isn't very useful.

 const Cu = Components.utils;
 
 Cu.import("resource://gre/modules/Services.jsm");
 
 let crawling = false;
 
 function require(module)
 {
   let result = {};
   result.wrappedJSObject = result;
   Services.obs.notifyObservers(result, "abpcrawler-require", module);
   return result.exports;
 }
 
 let {Crawler} = require("crawler");
 
 function onUnload()
 {
   const fields = ["backend-url", "parallel-tabs"];
   fields.forEach(function(field)
   {
     let control = document.getElementById(field);
     control.setAttribute("value", control.value);

[Wladimir Palant, 2012/09/21 15:36:18]

Are you trying to make the field values persist? Unfortunately, this won't
happen automatically if you change attributes manually. You need to add:

  document.persist(control.id, "value");

   });
 }
 
-function onClose()
-{
-  return !crawling;

[Wladimir Palant, 2012/09/21 15:36:18]

Maybe show a message why the window cannot be closed right now?

-}
-
 function getBackendUrl()
 {
   let backendUrlTextBox = document.getElementById("backend-url");
   return backendUrlTextBox.value;
 }
 
 function getParallelTabs()
 {
   let parallelTabsTextBox = document.getElementById("parallel-tabs");
   return parseInt(parallelTabsTextBox.value);
 }
 
 function onAccept()
 {
   let backendUrl = getBackendUrl();
   let parallelTabs = getParallelTabs();
-  let dialog = document.getElementById("abpcrawler-crawler");

[Wladimir Palant, 2012/09/21 15:36:18]

How about using document.documentElement here?

+  let dialog = document.documentElement;
   let acceptButton = dialog.getButton("accept");
   crawling = acceptButton.disabled = true;
-  Crawler.crawl(backendUrl, parallelTabs, window.opener, function()

[Wladimir Palant, 2012/09/21 15:36:18]

Of course we could have the condition that window.opener is null or
window.opener.closed is true - the user closed that browser window already. Add
a check and an error message maybe?

+
+  let mainWindow = window.opener;
+  if (!mainWindow || mainWindow.closed)
   {
+    alert( "Unable to find the main window, aborting.");
     crawling = acceptButton.disabled = false;
-  });
+  }
+  else
+    Crawler.crawl(backendUrl, parallelTabs, mainWindow, function()
+    {
+      crawling = acceptButton.disabled = false;
+    });
+
   return false;
 }
+
+function onCancel()
+{
+  let closingPossible = !crawling;
+  if (!closingPossible)
+    alert("Crawling still in progress.");
+  return closingPossible;
+}