Merge pull request #18216 from Snuffleupagus/download-data

Change `DownloadManager.download` to use Uint8Array-data
This commit is contained in:
Tim van der Meij 2024-06-07 14:14:48 +02:00 committed by GitHub
commit 593ce96834
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
4 changed files with 13 additions and 13 deletions

View File

@ -1083,13 +1083,11 @@ const PDFViewerApplication = {
this._ensureDownloadComplete(); this._ensureDownloadComplete();
const data = await this.pdfDocument.getData(); const data = await this.pdfDocument.getData();
const blob = new Blob([data], { type: "application/pdf" }); this.downloadManager.download(data, url, filename, options);
await this.downloadManager.download(blob, url, filename, options);
} catch { } catch {
// When the PDF document isn't ready, or the PDF file is still // When the PDF document isn't ready, or the PDF file is still
// downloading, simply download using the URL. // downloading, simply download using the URL.
await this.downloadManager.downloadUrl(url, filename, options); this.downloadManager.downloadUrl(url, filename, options);
} }
}, },
@ -1106,9 +1104,7 @@ const PDFViewerApplication = {
this._ensureDownloadComplete(); this._ensureDownloadComplete();
const data = await this.pdfDocument.saveDocument(); const data = await this.pdfDocument.saveDocument();
const blob = new Blob([data], { type: "application/pdf" }); this.downloadManager.download(data, url, filename, options);
await this.downloadManager.download(blob, url, filename, options);
} catch (reason) { } catch (reason) {
// When the PDF document isn't ready, or the PDF file is still // When the PDF document isn't ready, or the PDF file is still
// downloading, simply fallback to a "regular" download. // downloading, simply fallback to a "regular" download.

View File

@ -113,8 +113,10 @@ class DownloadManager {
return false; return false;
} }
download(blob, url, filename, _options) { download(data, url, filename, _options) {
const blobUrl = URL.createObjectURL(blob); const blobUrl = URL.createObjectURL(
new Blob([data], { type: "application/pdf" })
);
download(blobUrl, filename); download(blobUrl, filename);
} }
} }

View File

@ -140,8 +140,10 @@ class DownloadManager {
return false; return false;
} }
download(blob, url, filename, options = {}) { download(data, url, filename, options = {}) {
const blobUrl = URL.createObjectURL(blob); const blobUrl = URL.createObjectURL(
new Blob([data], { type: "application/pdf" })
);
FirefoxCom.request("download", { FirefoxCom.request("download", {
blobUrl, blobUrl,

View File

@ -160,12 +160,12 @@ class IDownloadManager {
openOrDownloadData(data, filename, dest = null) {} openOrDownloadData(data, filename, dest = null) {}
/** /**
* @param {Blob} blob * @param {Uint8Array} data
* @param {string} url * @param {string} url
* @param {string} filename * @param {string} filename
* @param {Object} [options] * @param {Object} [options]
*/ */
download(blob, url, filename, options) {} download(data, url, filename, options) {}
} }
/** /**