Add some comments
authorAntonio Ospite <ospite@studenti.unina.it>
Fri, 9 Dec 2011 23:13:47 +0000 (00:13 +0100)
committerAntonio Ospite <ospite@studenti.unina.it>
Sat, 10 Dec 2011 01:18:26 +0000 (02:18 +0100)
Signed-off-by: Antonio Ospite <ospite@studenti.unina.it>
direct_download_links.user.js

index b149698..eac46c9 100644 (file)
@@ -36,7 +36,8 @@
 /*
  * TODO:
  *  - find a way to use the same string as in the @include lines to match the
 /*
  * TODO:
  *  - find a way to use the same string as in the @include lines to match the
- *    current window.location
+ *    current window.location. Look for something like GM_testUrl() which builds
+ *    the regexp starting from a glob line.
  *  - use xpath instead of regexp like in http://a32.me/2009/11/greasemonkey/
  *  - use jquery, like shown in http://a32.me/2009/11/greasemonkey/
  */
  *  - use xpath instead of regexp like in http://a32.me/2009/11/greasemonkey/
  *  - use jquery, like shown in http://a32.me/2009/11/greasemonkey/
  */
@@ -123,7 +124,8 @@ function direct_download_link_add(pageURL, site) {
     return null;
   }
 
     return null;
   }
 
-
+  // This is used for sites adding the URL to the DOM after DOMContentLoaded,
+  // for example by some javascript library (like Silverlight.js on rai.tv).
   if (site.onEvent) {
     element.addEventListener(site.onEvent.evt, function(e) {
       if (site.onEvent.targetElement &&
   if (site.onEvent) {
     element.addEventListener(site.onEvent.evt, function(e) {
       if (site.onEvent.targetElement &&