Issue metadata
Sign in to add a comment
|
Regression: Cursor is not visible after hitting 'Spacebar' for PDF page after adding 'PDF viewer' extension.
Reported by
rk...@etouch.net,
Jan 16 2017
|
||||||||||||||||||||||
Issue descriptionChrome Version: 57.0.2983.0 Revision 266a86c94d34f3c7c3d22ee00624636b801f80d3-refs/heads/master@{#443819} OS: Windows(7,8,10),Linux (14.04 LTS) URL: https://chrome.google.com/webstore/detail/pdf-viewer/oemmndcbldboiebfnladdacbdfmadadm?utm_source=chrome-app-launcher-info-dialog What steps will reproduce the problem? (1) Launch chrome, navigate to above url and click on 'ADD TO CHROME' (2) Then navigate to any PDF file(eg.www.orimi.com/pdf-test.pdf), click on extension icon. (3) Hitt Spacebar and observe. Actual: Nothing is happened after hitting Spacebar i.e. Cursor is not visible. Expected: Cursor should be seen after hitting Spacebar. This is a regression issue, broken in 'M-57', will soon update the other info: Good Build: 57.0.2979.0 Bad Build: 57.0.2980.0 Note: Issue is not seen on Mac OS.
,
Jan 17 2017
Adding release block label, please undo if not the case.
,
Jan 23 2017
Issue is still reproducible on the latest M-57 branched build(57.0.2987.6). Friendly ping to get an update on this.
,
Feb 1 2017
yosin@ Since this issue is marked as RB-Stable, could you please let us know is there any latest update available for this issue. Thanks!
,
Feb 8 2017
A friendly reminder that M57 Stable is launch is coming soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch ASAP so it gets enough baking time in Beta (before Stable promotion). Thank you!
,
Feb 15 2017
Able to reproduce this issue on Windows-7 using chrome latest Dev 58.0.3012.0. yosin@ We're getting closer to M57 early stable launch and this issue is marked as RB-Stable, so can we get any latest update on this issue? Thanks!
,
Feb 15 2017
My patch doesn't affect focus change. Rather my patch checks focus.
,
Feb 16 2017
A friendly reminder that M57 Stable is launch is coming VERY soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch (2987) ASAP so it gets enough baking time in Beta (before Stable promotion). Thank you!
,
Feb 22 2017
URGENT - PTAL ASAP. We're getting VERY close to M57 Stable promotion. And this issue is marked as M57 stable release blocker. Pls make sure to land the fix and get it merged into the release branch ASAP so it gets enough baking time in Beta (before Stable promotion). Know that this issue shouldn't block the release? Remove the ReleaseBlock-Stable label or move to M58. Thank you.
,
Feb 22 2017
While performing per-revision bisect getting the same suspect as mentioned in the comment #1. rkote@ Could you please check the same and provide chromium bisect from your end? Thanks!
,
Feb 22 2017
This is not the internal PDF viewer, it's an extension which displays PDFs as HTML5. Removing the Internals>Plugins>PDF label.
,
Feb 22 2017
The assigned owner "rkote@etouch.net" is not able to receive e-mails, please re-triage. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Feb 23 2017
rdevlin.cronin@, could you ptal please?
,
Feb 23 2017
The bisect from #1 is correct; I've confirmed that reverting it locally changes the behavior. This also makes sense with the cl description, "Keyboard event should not insert a character at selection if selection doesn't have focus". We would previously insert characters into the text field even if the field didn't have focus, and yosin@'s patch changed that behavior. It may be that the new behavior is desired; I'll leave that up to others to decide.
,
Feb 24 2017
Yes, this is expected behavior both logical, compatible to other browsers and W3C UI Event spec[1]. [1] https://www.w3.org/TR/uievents/ |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by hdodda@chromium.org
, Jan 16 2017Labels: hasbisect-per-revision M-57
Owner: yosin@chromium.org
Status: Assigned (was: Unconfirmed)