|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
59 bytes, text/x-review-board-request
|Details | Review|
We will access the file path by giving the key "ContentTmpD", but it's only work on content process. On non-e10s we can't save pdf to file now.
Hi Bruce, Could you please review the patch? Thanks. Since the file path we retrieve from key Tmpd is different between content process and jsplugin process, we pass the absolute path from jsplugin process to content process instead of the key and file name. Also it seems no one is using contentTmpD in script file so we change it to TmpD on all platform.
Comment on attachment 8867588 [details] Bug 1363291 - Handle save to file when printing pdf in non-e10s. https://reviewboard.mozilla.org/r/139138/#review142380 LGTM
Attachment #8867588 - Flags: review?(brsun) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/2eea6d27f05f Handle save to file when printing pdf in non-e10s. r=brsun
Status: ASSIGNED → RESOLVED
Last Resolved: 11 months ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
You need to log in before you can comment on or make changes to this bug.