Issue metadata
Sign in to add a comment
|
PDF plugin preview looks good but printout is missing content
Reported by
goo...@herrmannsdorfer.de,
Oct 24 2017
|
||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13) AppleWebKit/604.1.38 (KHTML, like Gecko) Version/11.0 Safari/604.1.38 Example URL: Steps to reproduce the problem: 1. open attached PDF file 2. preview content 3. print What is the expected behavior? WYSIWYG What went wrong? Printout is missing content. This is independent of printer or printer driver. Was working in Chrome 61. Is working in current Adobe Reader. Attached sample is a scanned image inside a PDF file. Other attachments are a scan of the printout and a screenshot of the print preview. Does it occur on multiple sites: N/A Is it a problem with a plugin? Yes PDF internal Did this work before? Yes 61 Does this work in other browsers? Yes Chrome version: 62.0.3202.62 Channel: n/a OS Version: 7 Flash Version: Tested with several printers. It is not a driver issue. And no, toner status is fine :-)
,
Oct 24 2017
,
Oct 24 2017
Printed sample.pdf and it looks fine on paper - see a scan of my printout attached. Version is 61.0.3163.100 (Official Build) (64-bit) Looks like a printing issue
,
Oct 24 2017
I'll take a look. Might be related to the large number of image masks in the PDF. Were there issues with this in Chrome 60?
,
Oct 25 2017
Chrome 60 works, as does 61. In 62 printing is broken. Tested with 62.0.3202.62 official build, 64bit.
,
Oct 25 2017
Thank you for providing more feedback. Adding requester "rbpotter@chromium.org" to the cc list and removing "Needs-Feedback" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Oct 25 2017
The following revision refers to this bug: https://pdfium.googlesource.com/pdfium/+/78b334a824c42138f5e98060638b1643b0279602 commit 78b334a824c42138f5e98060638b1643b0279602 Author: rbpotter <rbpotter@chromium.org> Date: Wed Oct 25 23:16:37 2017 Temporarily disable mask rendering again Bug: chromium:777837 Change-Id: I1830dc37b9c039a95509627026461134d8f2c16d Reviewed-on: https://pdfium-review.googlesource.com/16710 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> [modify] https://crrev.com/78b334a824c42138f5e98060638b1643b0279602/fpdfsdk/fpdfview.cpp
,
Oct 27 2017
Able to reproduce issue on reported version 62.0.3202.62 and Latest beta 63.0.3239.18 but its not reproducible on latest Canary M-64(64.0.3251.0) using Win 10, looks like the issue is fixed on Latest Canary 64.0.3251.0. Hence adding TE-Verified-M64 label Please find the attached screenshot. Adding the M-62 RB-Stable in case we plan any stable refresh to get the fix merged to M-62
,
Oct 27 2017
Attaching screenshot of printed sample PDF of both 62.0.3202.62(content missing) and 64.0.3251.0(content seen)
,
Oct 30 2017
rbpotter@, thank you for the fix. Please go ahead and request a merge ASAP if this fix needs to be on M62 as we'll have M62 re-spin (most likely) this week.
,
Oct 30 2017
Requesting M-63 merge for now.
,
Oct 30 2017
This bug requires manual review: M63 has already been promoted to the beta branch, so this requires manual review Please contact the milestone owner if you have questions. Owners: cmasso@(Android), cmasso@(iOS), gkihumba@(ChromeOS), govind@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Oct 30 2017
Approving merge to M63 branch 3239 based on comment #8. Please merge ASAP so we cna take in for this week Beta release. Thank you.
,
Oct 30 2017
[Bulk Edit] URGENT - PTAL. M63 Stable promotion is coming soon and your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch ASAP. Thank you.
,
Oct 30 2017
The following revision refers to this bug: https://pdfium.googlesource.com/pdfium/+/a8caf1c23a8376705d8cb08500d82aa6e948a082 commit a8caf1c23a8376705d8cb08500d82aa6e948a082 Author: Lei Zhang <thestig@chromium.org> Date: Mon Oct 30 21:58:04 2017 M63: Temporarily disable mask rendering again Bug: chromium:777837 TBR=rbpotter@chromium.org Change-Id: I1830dc37b9c039a95509627026461134d8f2c16d Reviewed-on: https://pdfium-review.googlesource.com/16710 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> (cherry picked from commit 78b334a824c42138f5e98060638b1643b0279602) Reviewed-on: https://pdfium-review.googlesource.com/17190 Commit-Queue: Lei Zhang <thestig@chromium.org> [modify] https://crrev.com/a8caf1c23a8376705d8cb08500d82aa6e948a082/fpdfsdk/fpdfview.cpp
,
Oct 30 2017
My recommendation right now is to wait until M63 since issue doesn't seem quite as prevalent. Removing RBS label for now; but we can re-visit if this issue spikes as we ramp-up M62.
,
Oct 30 2017
+RBS. On a second thought, adding RBS for now for tracking.
,
Nov 1 2017
,
Nov 6 2017
M63 Stable promotion is coming soon and your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and request a merge to M63 ASAP. Thank you.
,
Nov 6 2017
Fix was merged to M63 already. See comment 15.
,
Nov 6 2017
Yeah, sorry it was a bulk edit. Could you pls mark the bug as fixed if nothing is pending?
,
Nov 6 2017
Marking fixed since this is fixed in M63.
,
Nov 10 2017
We have customer reporting the same issue, is there a chance to fix this in M62, as M63 release is not that close.
,
Nov 10 2017
+ abdulsyed@ (Chrome Desktop M62 TPM), PTAL comment #23. Thank you.
,
Nov 10 2017
This is a relatively small change that reverts to M59 (and earlier) behavior, and has been in Canary/Dev/Beta for over a week so it should be safe to merge. Given issue reports seem to have increased with the ramp-up of M-62, requesting merge for M62.
,
Nov 11 2017
Thanks rbpotter@ for the fix. Upon more examination, we have received only 3 reports, we've been at 25% for more than a week, and it's affecting only a very small subset of pdfs. There is a clear workaround (download the PDF and use a PDF reader). Rejecting this merge for M62 since we're already ramping up and this is not an absolutely critical P0 issue. M63 will contain the fix. In the meantime, if customers face this issue, please provide workaround solution.
,
Nov 15 2017
Issue 785380 has been merged into this issue.
,
Nov 15 2017
You can add me to the list of people requesting for this to be fixed sooner than later. We have seen this with many different PDFs within our organization, and it is affecting a lot of users. Is there any way to revert to the older version and keep it from updating itself? Thank you.
,
Nov 20 2017
,
Nov 21 2017
Issue 787052 has been merged into this issue.
,
Nov 21 2017
Please add me to those requesting an earlier fix. This issue is compromising our ability to transition our teachers, staff, and students to use Google Chrome instead of Internet Explorer, as a number of our instructional programs rely heavily on the ability to print cloud-hosted PDFs.
,
Nov 21 2017
I already requested to be included with those asking for an earlier fix, but I just wanted to add this: there seems to be a low priority placed on this fix based on comments above, and I do not understand this. Based on what the comment above me said, and what I am seeing myself, this is a major issue. Many of our users are unable to use Chrome effectively without a workaround, and instructing them all in the use of workarounds may work for some, but for others it can be quite a confusing issue. Without a simple way to fix it for everyone now, we are starting to question our choice to use Chrome as our main browser here. I don't think that's what Google wants is for issues like these to make people question their choice in browsers.
,
Nov 22 2017
Separate from the discussion on whether the merge of the fix to M62 should have happened or not, I would like to point out it is really hard to test the combination of all possible PDFs with all possible printers. If you are responsible for an organization with lots of users, it would be extremely helpful if you run Chrome Beta or Dev channel, and try to use that channel for every day tasks that are deemed business critical. If you notice something is wrong and file a bug, we will have a better chance of catching it before it hits Stable channel. Please see this "Chrome release best practices" article for more details: https://support.google.com/chrome/a/answer/6025002
,
Nov 22 2017
Just to update the latest behavior on beta. Tested this issue on Windows 7 using chrome latest beta-63.0.3239.59 as per C#0. Able to print above attached (sample.pdf) clearly without any missing content.Also tried below sample URL's which are also printed text without any text missing issue. http://css4.pub/2015/textbook/somatosensory.pdf http://www.pdf995.com/samples/pdf.pdf Thanks..!
,
Nov 28 2017
,
Nov 30 2017
Issue 789536 has been merged into this issue. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by dtapu...@chromium.org
, Oct 24 2017