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

Remove DownloadManager dependency

Browse Source 
We can pass it in using the options object. Note that that this also avoids creating a DownloadManager object for each separate link (instead, having only one is enough).
This commit is contained in:
Tim van der Meij 2015-01-27 22:11:04 +01:00
parent 733882ac25
commit 4c6ca1c78f
2 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
web/pdf_attachment_view.js
View File

@ -14,7 +14,7 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
/* globals DownloadManager, getFileName */
/* globals getFileName */
'use strict';
@ -22,6 +22,7 @@ var PDFAttachmentView = (function PDFAttachmentViewClosure() {
function PDFAttachmentView(options) {
this.container = options.container;
this.attachments = options.attachments;
this.downloadManager = options.downloadManager;
}
PDFAttachmentView.prototype = {
@ -34,12 +35,11 @@ var PDFAttachmentView = (function PDFAttachmentViewClosure() {
_bindLink: function PDFAttachmentView_bindLink(button, item) {
button.onclick = function downloadFile(e) {
var downloadManager = new DownloadManager();
var content = item.content;
var filename = item.filename;
downloadManager.downloadData(content, getFileName(filename), '');
this.downloadManager.downloadData(content, getFileName(filename), '');
return false;
};
}.bind(this);
},
render: function PDFAttachmentView_render() {

3
web/viewer.js
View File

@ -988,7 +988,8 @@ var PDFViewerApplication = {
var container = document.getElementById('attachmentsView');
self.attachments = new PDFAttachmentView({
container: container,
attachments: attachments
attachments: attachments,
downloadManager: new DownloadManager()
});
self.attachments.render();
document.getElementById('viewAttachments').disabled = !attachments;