b215290 - Alt+Click does not follow download folder preferences

- make the call to saveURL from the Alt+Click handling pass the 'skipPrompt' flag, and also make the code in contentAreaUtils.js not overwrite existing files similar to 215194.


git-svn-id: svn://10.0.0.236/trunk@145675 18797224-902f-48f8-a5cc-f745e15eee43

mozilla/browser/base/content/browser.js

@@ -3899,7 +3899,7 @@ function handleLinkClick(event, href, linkNode)
       }
       
       if (event.altKey) {
-        saveURL(href, linkNode ? gatherTextUnder(linkNode) : "");
+        saveURL(href, linkNode ? gatherTextUnder(linkNode) : "", null, true, true);
         return true;
       }
  

mozilla/browser/base/content/contentAreaUtils.js

@@ -346,6 +346,27 @@ function foundHeaderInfo(aSniffer, aData, aSkipPrompt)
     // bugs. 
     dir.append(defaultString);
     file = dir;
+    
+    // Since we're automatically downloading, we don't get the file picker's 
+    // logic to check for existing files, so we need to do that here.
+    //
+    // Note - this code is identical to that in 
+    //   browser/components/downloads/content/nsHelperAppDlg.js. 
+    // If you are updating this code, update that code too! We can't share code
+    // here since that code is called in a js component. 
+    while (file.exists()) {
+      var parts = /.+-(\d+)(\..*)?$/.exec(file.leafName);
+      if (parts) {
+        file.leafName = file.leafName.replace(/((\d+)\.)/, 
+                                              function (str, p1, part, s) { 
+                                                return (parseInt(part) + 1) + "."; 
+                                              });
+      }
+      else {
+        file.leafName = file.leafName.replace(/\./, "-1$&");
+      }
+    }
+    
   }
 
   if (isDocument)