Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[CRX] Respect download requests in main frame

Browse Source 
When the URL contains "=download", respect the Content-Disposition
header in the Chrome extension.
This commit is contained in:
Rob Wu 2015-07-06 17:13:44 +02:00
parent 9ad6af4a3f
commit ffbe669730
1 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
extensions/chromium/pdfHandler.js
View File

@ -34,9 +34,15 @@ function isPdfDownloadable(details) {
if (details.url.indexOf('pdfjs.action=download') >= 0) { if (details.url.indexOf('pdfjs.action=download') >= 0) {
return true; return true;
} }
// Display the PDF viewer regardless of the Content-Disposition header // Display the PDF viewer regardless of the Content-Disposition header if the
// if the file is displayed in the main frame. // file is displayed in the main frame, since most often users want to view
if (details.type === 'main_frame') { // a PDF, and servers are often misconfigured.
// If the query string contains "=download", do not unconditionally force the
// viewer to open the PDF, but first check whether the Content-Disposition
// header specifies an attachment. This allows sites like Google Drive to
// operate correctly (#6106).
if (details.type === 'main_frame' &&
details.url.indexOf('=download') === -1) {
return false; return false;
} }
var cdHeader = (details.responseHeaders && var cdHeader = (details.responseHeaders &&