New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 648116 link

Starred by 2 users
Status: Fixed
Owner:
█ dschuyler@chromium.org
Last visit > 30 days ago
Closed: Sep 2016
Cc:
nyerramilli@chromium.org
█ raymes@chromium.org
dbeam@chromium.org
█ tsergeant@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Page number text box is shifted upwards on PDF pages.

Reported by rk...@etouch.net, Sep 19 2016 
Chrome Version: 55.0.2864.0 Revision 533dcff13b0b002494a3040d2f90de937a93e76d-refs/heads/master@{#419385} 
OS:Windows(7,8,10), Mac(10.10.5,10.11.4)

What steps will reproduce the problem?
(1) Launch chrome, navigate to http://www.orimi.com/pdf-test.pdf
(2) Observe the Page number text box

Actual: Page number text box is shifted upwards on PDF pages.

Expected: Page number text box should seen properly on PDF pages.

This is a regression issue, broken in 'M-55'. will soon update the other info:

Good Build: 55.0.2862.0
Bad Build: 55.0.2864.0

Comment 1 by rk...@etouch.net, Sep 19 2016

Screenshot.png
6.9 KB View Download

Comment 2 by msrchandra@chromium.org, Sep 19 2016

Labels: ReleaseBlock-Stable
Owner: dschuyler@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build: 55.0.2862.0 (Revision: 419063).
Bad build: 55.0.28464.0 (Revision: 419385).

You are probably looking for a change made after 419344 (known good), but no later than 419345 (first known bad).
CHANGELOG URL:
  https://chromium.googlesource.com/chromium/src/+log/7f8965f2d8ef4ac323cc4654769e853174267f6f..a75f7d2fb6a32656361607654107dfa6dd4f1719

From the CL above, assigning the issue to the concern owner --
@dschuyler -- Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to concern owner.

Note: Adding RB Label as this is a recent Regression. Please remove if not required.
Thank You.

Comment 3 by msrchandra@chromium.org, Sep 19 2016

Labels: hasbisect-per-revison

Comment 4 by thestig@chromium.org, Sep 20 2016

Cc: raymes@chromium.org dbeam@chromium.org tsergeant@chromium.org

Comment 5 by ligim...@chromium.org, Sep 27 2016

Labels: -hasbisect-per-revison hasbisect-per-revision
Fixing the spelling error in label.

Comment 6 by ajha@chromium.org, Sep 27 2016 

Issue is still reproducible on the latest canary(55.0.2872.0) on Windows 10.

dschuyler@: Could you please take a look at this.
Project Member

Comment 7 by bugdroid1@chromium.org, Sep 28 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d2752a183c0b1bc7c6ed6bdca17ac5d8eccf8b96

commit d2752a183c0b1bc7c6ed6bdca17ac5d8eccf8b96
Author: dschuyler <dschuyler@chromium.org>
Date: Tue Sep 27 23:04:49 2016

[PDF viewer] Align page N of M input field

This CL sets the alignment of the N part of the: page N / M at the top
of the PDF viewer. The N was rendering too high and this change brings
it down to the baseline with the " / M" part of the line.

BUG= 648116 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2366113002
Cr-Commit-Position: refs/heads/master@{#421376}

[modify] https://crrev.com/d2752a183c0b1bc7c6ed6bdca17ac5d8eccf8b96/chrome/browser/resources/pdf/elements/viewer-page-selector/viewer-page-selector.css

Comment 8 by thestig@chromium.org, Sep 28 2016

Status: Fixed (was: Assigned)

Comment 9 by tkonch...@chromium.org, Oct 4 2016

Labels: TE-Verified-M55 TE-Verified-55.0.2880.0
Tested the issue on win10 and mac 10.11.6 chrome version 55.0.2880.0 - page number is properly displayed as shown in the screenshot

Fix works as expected. Hencea dding TE Verified labels
648116.png
541 bytes View Download
►

Sign in to add a comment