Regression:[PDF]Page number entered in text field appears chopped
Reported by
vineetha...@etouch.net,
Sep 25
|
|||
Issue descriptionChrome version : 71.0.3561.0 (Official Build) Revision 59edfd1d195efd57c937c950c1fd2a708a83f1f0-refs/branch-heads/3561@{#1}(32/64-bit) OS : Win(7,8,8.1,10), Mac(10.12.6, 10.13.1, 10.14, 10.13.6) and Linux(14.04 LTS) OS Test PDF URL: http://www.pdf995.com/samples/pdf.pdf Steps to reproduce: 1. Launch chrome and navigate to above given test PDF. 2. Enter a valid page number in the page number text field and observe. Actual Result : Entered page number in text field appears chopped. Expected Result: Entered page number in text field should be seen properly. This is a regression issue broken in ‘M-71’ and below is the 'per-revision' bisect information: Good build: 71.0.3558.0(Revision: 593043) Bad build : 71.0.3559.0(Revision: 593399) You are probably looking for a change made after 593204 (known good), but no later than 593205 (first known bad). CHANGE-LOG URL: The script might not always return single CL as suspect as some perf builds might get missing due to failure. https://chromium.googlesource.com/chromium/src/+log/ad515f5fb43fed36bf72cf0001147a31cce0500c..a12770d1657f8823f5e867bd3e60e8d2c05065b6 Suspecting : https://chromium.googlesource.com/chromium/src/+/a12770d1657f8823f5e867bd3e60e8d2c05065b6 rbpotter@ : Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner. Kindly refer the attached screen-cast. Thank you..!
,
Oct 2
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6b111a069b509076f2dd822db652d1a69ede0494 commit 6b111a069b509076f2dd822db652d1a69ede0494 Author: rbpotter <rbpotter@chromium.org> Date: Tue Oct 02 02:42:23 2018 CrInput: Add a new mixin for new input container Add a new mixin for the new input container, which is needed by the PDF viewer. Also move background-color styling to the existing --cr-input-background-color so it is applied to both the container and the input. Bug: 888985 Change-Id: I60e1dfe6828ba2d198984ed9ca33d0753aaa1308 Reviewed-on: https://chromium-review.googlesource.com/1244831 Reviewed-by: Noel Gordon <noel@chromium.org> Reviewed-by: Scott Chen <scottchen@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#595704} [modify] https://crrev.com/6b111a069b509076f2dd822db652d1a69ede0494/chrome/browser/resources/media_router/elements/media_router_container/media_router_container.css [modify] https://crrev.com/6b111a069b509076f2dd822db652d1a69ede0494/chrome/browser/resources/pdf/elements/viewer-page-selector/viewer-page-selector.html [modify] https://crrev.com/6b111a069b509076f2dd822db652d1a69ede0494/chrome/browser/resources/settings/settings_page/settings_subpage_search.html [modify] https://crrev.com/6b111a069b509076f2dd822db652d1a69ede0494/ui/file_manager/file_manager/main.html [modify] https://crrev.com/6b111a069b509076f2dd822db652d1a69ede0494/ui/file_manager/gallery/gallery.html [modify] https://crrev.com/6b111a069b509076f2dd822db652d1a69ede0494/ui/webui/resources/cr_elements/cr_input/cr_input_style_css.html
,
Oct 2
vineetha.sarma@ please verify the fix in latest canary.
,
Oct 2
FYI, this has not landed in Canary yet. I am guessing it will land there tomorrow.
,
Oct 3
Update : Rechecked the above issue on Win(7,8,8.1,10) ,Mac(10.12.6 , 10.13.1 , 10.13.6, 10.14.1) and Linux(14.04 LTS)OS with Canary version #71.0.3569.0 and the issue is fixed. Kindly refer the attached screen cast.
,
Oct 3
|
|||
►
Sign in to add a comment |
|||
Comment 1 by nyerramilli@chromium.org
, Sep 25