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

Issue 696894 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regresion : Unnecessarily cursor appears in omnibox after clicking inside the 'Search box' on NTP.

Reported by yfulgaon...@etouch.net, Feb 28 2017

Issue description

Chrome version : 58.0.3025.6 (Official Build) 6c1332290fc0f9c3b59345fde8c6476367c31395-refs/branch-heads/3025@{#9} 32/64 bit
OS : Windows (7,8,10), Linux (14.04 LTS)

What steps will reproduce the problem?
1. Launch chrome, open NTP and click inside the 'Search box'.
2. Observe the cursor in omnibox and 'Search box'.

Actual : Unnecessarily cursor appears in omnibox after clicking inside the 'Search box' on NTP (i.e both the cursors are seen at the same time)
Expected : Instead, the cursor should be seen only in the 'Search box' after clicking inside it and cursor should vanish from the omnibox.

This is a regression issue broken in ‘M-58’, below is the Manual Regression range and will soon update other info.
Good build : 58.0.3014.0
Bad build : 58.0.3015.0

Note : 
1. Above issue is reproducible on Windows (7,8,8.1,10), Linux (14.04 LTS) OS and it is working fine on Mac (10.11.6, 10.12.1, 10.12) OS.
2. Issue is also seen in #58.0.3026.0 build.
 
Actual_Result.mp4
639 KB View Download
Expected_Result.mp4
678 KB View Download
Labels: hasbisect-per-revision ReleaseBlock-Stable
Owner: yiyix@chromium.org
Status: Assigned (was: Unconfirmed)
Bisect Information:
---------------------
Using the per-revision bisect providing the bisect results,

You are probably looking for a change made after 451178 (known good), but no later than 451179 (first known bad).

CHANGE-LOG URL:
-----------------
https://chromium.googlesource.com/chromium/src/+log/a255f7468a598ea8cf8a5ab0e21ab0047f52568a..a33c8f5acb10c229da14134b9987e3b16eaa74f4

From the CL above, assigning the issue to the concern owner
Review-Url: https://codereview.chromium.org/2660593002

yiyix@ Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner.

Note: Since this is a recent regression adding RB-Stable, please feel free to edit if this is not the case.

Thanks!

Comment 2 by treib@chromium.org, Feb 28 2017

Cc: treib@chromium.org

Comment 3 by yiyix@chromium.org, Feb 28 2017

Cc: yiyix@chromium.org mpear...@chromium.org
 Issue 696096  has been merged into this issue.

Comment 4 by yiyix@chromium.org, Feb 28 2017

Status: Started (was: Assigned)
I have tried and it is caused by my changes. I am looking into fixing now. 

Comment 6 by yiyix@chromium.org, Mar 2 2017

Labels: Merge-Request-58
Project Member

Comment 7 by sheriffbot@chromium.org, Mar 2 2017

Labels: -Merge-Request-58 Merge-Review-58 Hotlist-Merge-Review
This bug requires manual review: We don't branch M58 until 2017-03-02.
Please contact the milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), bhthompson@(cros), govind@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 8 by yiyix@chromium.org, Mar 6 2017

Status: Fixed (was: Started)
Labels: -Merge-Review-58
M58 (3029) is branched at chromium revision #454471. Cl listed at #5 landed before the branch so no merge is needed here.

Comment 10 by ajha@chromium.org, Mar 7 2017

Labels: TE-Verified-58.0.3029.6 TE-Verified-M58
Verified the fix on the latest M-58(58.0.3029.6) on Windows-10 and Linux Ubuntu 14.04. This is working as intended and cursor is seen only inside the omnibox or inside the NTP search box when clicked.


Sign in to add a comment