New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 612405 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug-Regression



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 description

Chrome 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.

 
Actual_video.mp4
897 KB Download
Expected_video.mp4
554 KB Download
Labels: ReleaseBlock-Stable
Marking the above issue as RB-Stable as this is a recent regression.

Thank you!

Comment 2 by ananta@chromium.org, May 17 2016

Cc: ananta@chromium.org
Owner: ----
Status: Available (was: Assigned)
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.


Cc: pfeldman@chromium.org
@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!
Labels: Needs-triage
Labels: -ReleaseBlock-Stable Needs-Bisect
Project Member

Comment 6 by sheriffbot@chromium.org, Jun 4 2016

Labels: -M-52 M-53 MovedFrom-52
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: dcheng@chromium.org danakj@chromium.org
Labels: -Needs-triage -Needs-Bisect
Owner: alex...@chromium.org
Status: Assigned (was: Available)
====================================

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.

Cc: rnimmagadda@chromium.org
Cc: -danakj@chromium.org alex...@chromium.org
Owner: danakj@chromium.org
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.
Owner: sadrul@chromium.org
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?
Owner: tdres...@chromium.org
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?
Owner: dtapu...@chromium.org
Over to dtapuska@.
Cc: sadrul@chromium.org
Project Member

Comment 14 by sheriffbot@chromium.org, Jul 10 2016

Labels: -M-53 -Pri-1 M-54 MovedFrom-53 Pri-2
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
Labels: -M-54 M-55
Note: Above issue is reproducible on latest canary version: 55.0.2878.0
Components: -Platform>DevTools
Cc: -ashej...@chromium.org

Sign in to add a comment