New issue
Advanced search Search tips

Issue 808902 link

Starred by 2 users
Status: Fixed
Owner:
npm@chromium.org
Closed: Feb 2018
Cc:
hnakashima@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

_Ignition!_ PDF displays fine in Preview but is missing pages in Chrome's PDF viewer

Project Member Reported by rdsmith@chromium.org, Feb 5 2018 
Chrome Version: 64.0.3282.140 (Official Build) beta (64-bit)
OS: Mac OS X 10.13.3

What steps will reproduce the problem?
(1) Go to http://library.sciencemadness.org/library/books/ignition.pdf in chrome
(2) Scroll to the second page of the preface (page 12 within the PDF)
(3)

What is the expected result?

There is text.

What happens instead?

The page is blank.

Please use labels and text to provide additional information.

If I download the PDF and open it Preview I can see the page.  The first paragraph of that page is:

"story, intelligence reports; all have contributed. Since this is not a for- mal history, but an informal attempt by an active participant to tell the story as it happened, I haven't attempted formal documentation. Particularly as in many cases such documentation would be embar- rassing—not to say hazardous! It's not only newsmen who have to pro- tect their sources."

For graphics-related bugs, please copy/paste the contents of the about:gpu
page at the end of this report.

Comment 1 by thestig@chromium.org, Feb 6 2018

Labels: OS-Chrome OS-Linux OS-Windows
I went back to r350000 on it showed the same problem, so probably not a regression. It happens with pdfium_test as well.

Comment 2 by hnakashima@chromium.org, Feb 8 2018

Labels: -Pri-3 Pri-1
Owner: hnakashima@chromium.org
Status: Available (was: Untriaged)
I am able to see the page on Mac, running Chrome 63.0.3239.132. Bumping priority as it's not clear now if it's a regression.
Screen Shot 2018-02-08 at 10.15.13 AM.png
952 KB View Download

Comment 3 by hnakashima@chromium.org, Feb 12 2018

Cc: hnakashima@chromium.org
Labels: -Type-Bug RegressedIn-64 Type-Bug-Regression
Owner: npm@chromium.org
It's a regression. Bisect over pdfium using pdfium_test points to: https://pdfium-review.googlesource.com/c/pdfium/+/20011

Comment 4 by npm@chromium.org, Feb 12 2018 

Ugh it seems that the cast to int was intentional. Ugh.

Comment 5 by npm@chromium.org, Feb 13 2018

Labels: Merge-Request-65
Fixed at https://pdfium-review.googlesource.com/c/pdfium/+/26470
This bug was present in Stable 64 so not high priority, but requesting merge for M65 because it is a safe merge. Letting merge owners decide if they want it merged or not.
Project Member

Comment 6 by sheriffbot@chromium.org, Feb 13 2018

Labels: -Merge-Request-65 Merge-Review-65 Hotlist-Merge-Review
This bug requires manual review: M65 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), bhthompson@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 7 by gov...@chromium.org, Feb 13 2018

Labels: -Merge-Review-65 Merge-Approved-65
Approving merge to M65 branch 3325 based on comment #5. Please merge ASAP so we can pick it up for tomorrow's beta release. Thank you.

Comment 8 by npm@chromium.org, Feb 13 2018

Status: Fixed (was: Available)
Merge: https://pdfium-review.googlesource.com/c/pdfium/+/26570

Comment 9 by gov...@chromium.org, Feb 13 2018

Labels: merge-merged-3325
If nothing is pending for M65, please remove "Merge-Approved-65" label. Thank you.

Comment 10 by npm@chromium.org, Feb 13 2018

Labels: -Merge-Approved-65
►

Sign in to add a comment