Issue metadata
Sign in to add a comment
|
Regression:Unnecessary zoom icon is seen after pinch to zoom in for pdf page.
Reported by
adha...@etouch.net,
Dec 8 2016
|
||||||||||||||||||||||
Issue descriptionChrome Version: 56.0.2924.21 (Official Build)4d0fd85f092f947ef72046f62e953ecf44d5e64a-refs/branch-heads/2924@{#401} 32/64-bit. OS: Windows 10(Touch device) TEST URL:https://www.ets.org/Media/Tests/GRE/pdf/gre_research_validity_data.pdf What steps will reproduce the problem? (1)Launch chrome and navigate to the above url. (2) pinch to zoom in and observe the bookmark overlay.(Kindly refer the video) Actual:Unnecessary zoom icon is seen after pinch to zoom in. Expected:No such zoom icon should be seen after pinch to zoom in. This is a Regression issue broken in M-56,below is the Narrow bisect info. https://chromium.googlesource.com/chromium/src/+log/b5bf18f7c7200fbf8b886f32e39e6f355d455d5a..454b6067a00e9291cd94eab5af499b9f1941d815?pretty=fuller&n=100 Suspecting:r430981? Good build:56.0.2914.0 Bad build:56.0.2915.0 Kindly help to re-assign if your change is not the cause for this issue. Note: This is touch device specific issue, same works fine via mouse click
,
Dec 8 2016
Adding release block label, please undo if not the case.
,
Dec 8 2016
P3, because this is not a data loss or information leakage. Also, definitely not release-blocking. The CL referenced in #1 is a docs change. It didn't touch the code, so it is (almost) impossible that the CL caused the behavior change in the bug description. I looked through the list of changes, and none jumped at me as a likely candidate. Can you please bisect to a narrower range?
,
Dec 9 2016
Re-bisected and below is the Narrow bisect range: https://chromium.googlesource.com/chromium/src/+log/99ce6308c09c342dbf6cdabda0bdbc1452ee036d..b5bf18f7c7200fbf8b886f32e39e6f355d455d5a?pretty=fuller&n=100 Suspecting:r430973? Kindly help to re-assign if your change is not the cause for this issue.
,
Dec 9 2016
Works as intended as part of implementing pinch zoom in the PDF viewer. Unlike with web pages, there is no meaningful distinction between different zooming mechanisms, since PDFs don't reflow. So showing the zoom icon is appropriate for (non-embedded) PDFs. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by ranjitkan@chromium.org
, Dec 8 2016