PDF: Plugin does not response to DidChangeFocus |
||||||
Issue descriptionThis results in inconsistent UI. For example, if a window is showing some web content with a focused input box, the caret stops blinking if the window loses focus. In the pdf case, the caret would keep blinking.
,
Nov 15 2016
Yes, that's correct. Also, selected text should also turn some type of inactive color when losing focus, and back to some active when gaining focus.
,
Nov 15 2016
,
Nov 16 2017
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Nov 16 2017
,
Nov 16 2017
,
Nov 16 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/7259b50e35328ecbf4f3b8966e5817a1cfab9952 commit 7259b50e35328ecbf4f3b8966e5817a1cfab9952 Author: Henrique Nakashima <hnakashima@chromium.org> Date: Thu Nov 16 21:06:17 2017 In PDF viewer, kill form focus when window loses focus. Bug: 665554 Change-Id: I96b716d38a51c13a5d94595c2b568f06471a33aa Reviewed-on: https://chromium-review.googlesource.com/774980 Reviewed-by: dsinclair <dsinclair@chromium.org> Commit-Queue: Henrique Nakashima <hnakashima@chromium.org> Cr-Commit-Position: refs/heads/master@{#517190} [modify] https://crrev.com/7259b50e35328ecbf4f3b8966e5817a1cfab9952/pdf/out_of_process_instance.cc [modify] https://crrev.com/7259b50e35328ecbf4f3b8966e5817a1cfab9952/pdf/out_of_process_instance.h [modify] https://crrev.com/7259b50e35328ecbf4f3b8966e5817a1cfab9952/pdf/pdf_engine.h [modify] https://crrev.com/7259b50e35328ecbf4f3b8966e5817a1cfab9952/pdf/pdfium/pdfium_engine.cc [modify] https://crrev.com/7259b50e35328ecbf4f3b8966e5817a1cfab9952/pdf/pdfium/pdfium_engine.h
,
Nov 16 2017
|
||||||
►
Sign in to add a comment |
||||||
Comment 1 by thestig@chromium.org
, Nov 15 2016