Issue metadata
Sign in to add a comment
|
Regression: Unnecessary touch keyboard is seen after tap/touch on down/up arrow key of find text box.
Reported by
jshan...@etouch.net,
May 17 2016
|
||||||||||||||||||||
Issue descriptionChrome Version: 52.0.2739.0 (Official Build) e63177a3d418d078a8777d83f817c11e30831d7b-refs/heads/master@{#393996}-32/64 bit OS: Windows 10(Touch device) Steps: 1. Launch Chrome and open Dev-tool on NTP. 2. Go to source and open any file using 'Ctrl'+'P' keys and then press Ctrl+F key to find in file. 3. Type any letter like 'c', tap/touch on down or up arrow key of find text box and observe. Actual: Unnecessary touch keyboard is seen after tap/touch on down or up arrow key of find text box. Expected: No touch keyboard should be seen after tap/touch on down or up arrow key of find text box. This is a regression issue broken in M-52, below is bisect info. Good build: 52.0.2706.0 Bad build: 52.0.2707.0 Narrow bisect: https://chromium.googlesource.com/chromium/src/+log/8e4a0b36ac3989db1afb69dff442fdc85af047da..44a0b274c777c0a6aee6f5ba722c435e71d54273?pretty=fuller&n=100 Suspecting: r386700 ? Please help to re-assign if your change is not the cause for this issue. Note: This is touch device specific issue, same works fine on mouse click.
,
May 17 2016
I don't think my patch causes this. We display the virtual keyboard here https://code.google.com/p/chromium/codesearch#chromium/src/content/browser/renderer_host/render_view_host_impl.cc&q=DisplayVirtualK&sq=package:chromium&l=1303 This code runs in the browser. From what I can see it looks like when you touch the down or up arrow in the Find box in the dev tools inspector, blink is telling us that the node is editable, which is not correct. Please assign this to someone from the dev tools team.
,
May 24 2016
@pfeldman: Hey, would you mind helping us in assigning it to correct owner as per comment#2 & from Narrow bisect in the original bug ? I really appreciate your help. Thank you!
,
May 30 2016
,
Jun 3 2016
,
Jun 4 2016
Moving this nonessential bug to the next milestone. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jun 6 2016
==================================== Good Build: 52.0.2706.0 Base Position: 386523 Bad Build: 52.0.2707.0 Base Position: 386876 ===================================== Able to repro this issue on Windows 10 [Pro] for the Google Chrome Canary Version - 53.0.2760.0 This is a regression issue broken in M52, below mentioned is the bisect info: CHANGELOG URL: https://chromium.googlesource.com/chromium/src/+log/8e4a0b36ac3989db1afb69dff442fdc85af047da..44a0b274c777c0a6aee6f5ba722c435e71d54273?pretty=fuller&n=100 MANUAL CHANGELOG URL: https://chromium.googlesource.com/chromium/src/+log/52.0.2706.0..52.0.2707.0?pretty=fuller&n=10000 Suspecting Below: ======================================================= Commit: e16154fa1a98b487a9e04feaad87fd31a3ae3fa4 Review URL: https://codereview.chromium.org/1869853002 ------------------------------------------------------- Commit: 25c52c3f3c407fd9f5678c0079c877ccef34c090 Review URL: https://codereview.chromium.org/1868363002 ======================================================= @alexmos/danakj: Could you please look into the issue, and if it has nothing to do with your changes and if possible please do assign it to the concerned owner. Thank you.
,
Jun 6 2016
,
Jun 20 2016
Sorry for missing this earlier. I don't think my r386833 caused this - it only moves tests around and has no effect on browser behavior. Reassigning to danakj@ to take a look at the second suspect.
,
Jun 21 2016
My patch is a rename of "scoped_ptr" to "std::unique_ptr", sorry but not applicable. => sadrul do you know who owns virtual keyboard things?
,
Jun 21 2016
I think this has more to do with issue 479301 (or something similar to that) than with the virtual keyboard. I don't really know who does text-input in blink land these days. Perhaps tdresser@ knows?
,
Jun 21 2016
Over to dtapuska@.
,
Jun 22 2016
,
Jul 10 2016
This issue is Pri-1 but has already been moved once. Lowering the priority and moving to the next milestone. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Oct 3 2016
Note: Above issue is reproducible on latest canary version: 55.0.2878.0
,
Sep 21 2017
,
Nov 2 2017
|
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by ashej...@chromium.org
, May 17 2016