Merge pull request mozilla#784 from arturadib/fix-783

vingtetun · vingtetun · commit c15311c8ebc6 · 2011-11-08T17:04:43.000-08:00
Replacing URL flag format for download
diff --git a/extensions/firefox/components/pdfContentHandler.js b/extensions/firefox/components/pdfContentHandler.js
@@ -52,7 +52,7 @@ pdfContentHandler.prototype = {
     }
 
     let targetUrl = aRequest.URI.spec;
-    if (targetUrl.indexOf('?pdfjs.action=download') >= 0)
+    if (targetUrl.indexOf('#pdfjs.action=download') >= 0)
       throw NS_ERROR_WONT_HANDLE_CONTENT;
 
     aRequest.cancel(Cr.NS_BINDING_ABORTED);
diff --git a/web/viewer.js b/web/viewer.js
@@ -139,7 +139,7 @@ var PDFView = {
   },
 
   download: function pdfViewDownload() {
-    window.open(this.url + '?pdfjs.action=download', '_parent');
+    window.open(this.url + '#pdfjs.action=download', '_parent');
   },
 
   navigateTo: function pdfViewNavigateTo(dest) {

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ pdfContentHandler.prototype = {`
`52`	`52`	`}`
`53`	`53`
`54`	`54`	`let targetUrl = aRequest.URI.spec;`
`55`		`- if (targetUrl.indexOf('?pdfjs.action=download') >= 0)`
	`55`	`+ if (targetUrl.indexOf('#pdfjs.action=download') >= 0)`
`56`	`56`	`throw NS_ERROR_WONT_HANDLE_CONTENT;`
`57`	`57`
`58`	`58`	`aRequest.cancel(Cr.NS_BINDING_ABORTED);`