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

Issue 811659 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression:Default cursor highlight is missing at password field in 'Connect to a network' overlay

Project Member Reported by rkalavakuntla@chromium.org, Feb 13 2018

Issue description

Chrome Version:66.0.3344.0/10399.0.0 Falco,Candy,Peppy
OS:Chrome OS

What steps will reproduce the problem?
(1)Recover Build >>In OOBE screen, go to 'Connect to a network' screen
(2)Click to connect to any network and Observe

Actual:Default cursor highlight is missing at password field
Expected:Default cursor highlight should be seen at password field

This is a Regression issue as same is working fine in 66.0.3341.0/10382.0.0 dev 

Note: Issue is not seen on Linux,Windows OS

@stevenjb : Please confirm the issue

Attached the screencast for reference..



 
Actual.jpg
3.4 MB View Download
Expected.jpg
4.7 MB View Download
Actualcursormissing.mp4
7.6 MB View Download
Expectedcursor.mp4
5.0 MB View Download

Comment 1 Deleted

Comment 2 Deleted

Comment 3 Deleted

OOps, these Cls above weren't meant for this CL, but for issue 811569. Sorry.
Labels: -M-66 M-65
Status: Started (was: Assigned)
I don't think this is a regression in 66, I think it has always been an issue for configured networks. (New networks work OK).

This affects 65 as well and I would like to fix it there (the fix is pretty small).

Project Member

Comment 6 by bugdroid1@chromium.org, Feb 16 2018

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

commit abd2d4ed8622d96e84a4a78ae3daae86df98a5e9
Author: Steven Bennetts <stevenjb@chromium.org>
Date: Fri Feb 16 01:20:54 2018

Network config: Fix focus for configured networks

Bug:  811659 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Id8937bcbd6828c8fc38b0e27e6505582ce6c25ca
Reviewed-on: https://chromium-review.googlesource.com/920647
Reviewed-by: Toni Barzic <tbarzic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537107}
[modify] https://crrev.com/abd2d4ed8622d96e84a4a78ae3daae86df98a5e9/ui/webui/resources/cr_components/chromeos/network/network_config.js

Labels: Merge-Request-65
Status: Fixed (was: Started)
Project Member

Comment 8 by sheriffbot@chromium.org, Feb 17 2018

Labels: -Merge-Request-65 Merge-Review-65 Hotlist-Merge-Review
This bug requires manual review: M65 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Hotlist-Merge-Review -Merge-Review-65 Merge-Approved-65
Project Member

Comment 10 by bugdroid1@chromium.org, Feb 21 2018

Labels: -merge-approved-65 merge-merged-3325
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6c93a1be5ceff71e67c204725802da6b75723bca

commit 6c93a1be5ceff71e67c204725802da6b75723bca
Author: Steven Bennetts <stevenjb@chromium.org>
Date: Wed Feb 21 19:10:26 2018

Network config: Fix focus for configured networks

TBR=stevenjb@chromium.org

(cherry picked from commit abd2d4ed8622d96e84a4a78ae3daae86df98a5e9)

Bug:  811659 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Id8937bcbd6828c8fc38b0e27e6505582ce6c25ca
Reviewed-on: https://chromium-review.googlesource.com/920647
Reviewed-by: Toni Barzic <tbarzic@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#537107}
Reviewed-on: https://chromium-review.googlesource.com/929346
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Cr-Commit-Position: refs/branch-heads/3325@{#540}
Cr-Branched-From: bc084a8b5afa3744a74927344e304c02ae54189f-refs/heads/master@{#530369}
[modify] https://crrev.com/6c93a1be5ceff71e67c204725802da6b75723bca/ui/webui/resources/cr_components/chromeos/network/network_config.js

Sign in to add a comment