bug 229785 also allow "Content-Disposition: name=foo" headers, in addition to filename=foo

r=bzbarsky sr=darin

uriloader/base/nsURILoader.cpp

@@ -391,7 +391,9 @@ nsresult nsDocumentOpenInfo::DispatchContent(nsIRequest *request, nsISupports *
           // Broken sites just send
           // Content-Disposition: filename="file"
           // without a disposition token... screen those out.
-           !dispToken.EqualsIgnoreCase("filename", 8)))
+           !dispToken.EqualsIgnoreCase("filename", 8)) &&
+          // Also in use is Content-Disposition: name="file"
+           !dispToken.EqualsIgnoreCase("name", 4))
         // We have a content-disposition of "attachment" or unknown
         forceExternalHandling = PR_TRUE;
     }

uriloader/exthandler/nsExternalHelperAppService.cpp

@@ -224,7 +224,9 @@ static PRBool GetFilenameAndExtensionFromChannel(nsIChannel* aChannel,
         // Broken sites just send
         // Content-Disposition: filename="file"
         // without a disposition token... screen those out.
-        !dispToken.EqualsIgnoreCase("filename", 8))) 
+        !dispToken.EqualsIgnoreCase("filename", 8)) &&
+        // Also in use is Content-Disposition: name="file"
+        !dispToken.EqualsIgnoreCase("name", 4)) 
     {
       // We have a content-disposition of "attachment" or unknown
       handleExternally = PR_TRUE;