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

Issue 791494 link

Starred by 3 users
Status: Started
Owner:
einbinder@chromium.org
Cc:
nyerramilli@chromium.org
rbasuvula@chromium.org
█ msrchandra@chromium.org
█ ranjitkan@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression : Cursor disappears after clicking in 'Filter' search field in 'Element' section of devtools.

Reported by rp...@etouch.net, Dec 4 2017 
Version: 65.0.3284.0 46c535d207b3648ddefdf3f9a588b69ac5ef31f2-refs/heads/master@{#521227}
OS: Windows (7,8,8.1,10),Linux (14.04 LTS),Mac OS X(10.12.6,10.13.2)

What steps will reproduce the problem?
1. Launch chrome, navigate to NTP and open devtools.
2. Now under 'Element' section click on 'New style Rule' icon and then immediately click on 'Filter' search field,observe

Actual: Cursor disappears after clicking in 'Filter' search field
Expected: Cursor should stay in 'Filter' search field after clicking in it.

This is regression issue, broken in ‘M 62’ and will soon update other info :
Good build:62.0.3181.0
Bad build: 62.0.3182.0
Actual_video.mov
2.0 MB Download
Expected_video.mov
2.6 MB Download

Comment 1 by rp...@etouch.net, Dec 4 2017

Labels: hasbisect-per-revision
Owner: einbinder@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build: 62.0.3181.0 (Revision: 493198).
Bad build: 62.0.3182.0 (Revision: 493619).

You are probably looking for a change made after 493603 (known good), but no lat
er than 493604 (first known bad).

CHANGELOG URL:

The script might not always return single CL as suspect as some perf builds might get missing due to failure.

https://chromium.googlesource.com/chromium/src/+log/2326c3a7b988ed9cc668dd00e398cc103615d53d..857bee5e96261ac2da484ff0c54d6838f6c951dd

From the CL above, assigning the issue to the concern owner 

@einbinder- 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.

Suspect : https://chromium.googlesource.com/chromium/src/+/857bee5e96261ac2da484ff0c54d6838f6c951dd

Thanks!

Comment 2 by einbinder@chromium.org, Dec 4 2017

Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 23 2018 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b9364568bd9d6c10f7eed68e32e3847376fc022b

commit b9364568bd9d6c10f7eed68e32e3847376fc022b
Author: Joel Einbinder <einbinder@chromium.org>
Date: Tue Jan 23 01:53:44 2018

DevTools: Don't steal focus after editing

Bug: 791494
Change-Id: Icd5f9a5f379758b481f373af71b17a6fa8f3feeb
Reviewed-on: https://chromium-review.googlesource.com/876929
Reviewed-by: Andrey Lushnikov <lushnikov@chromium.org>
Commit-Queue: Andrey Lushnikov <lushnikov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531126}
[modify] https://crrev.com/b9364568bd9d6c10f7eed68e32e3847376fc022b/third_party/WebKit/Source/devtools/front_end/elements/StylesSidebarPane.js

Comment 4 by rp...@etouch.net, Jan 23 2018

Labels: TE-Verified-M66 TE-Verified-66.0.3329.0
Rechecked the above issue on OS: Windows (7,8,8.1,10),Linux (14.04 LTS),Mac OS X(10.12.6,10.13.1,10.13.3) with latest canary chrome version: 66.0.3329.0 and the issue is fixed.Kindly refer attached screen cast for reference.
Actual_video.mov
1.7 MB View Download
►

Sign in to add a comment