New issue
Advanced search Search tips

Issue 912494 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Regression: Unnecessary 'Passwords' page gets auto scrolled on clicking first password entry.

Project Member Reported by cdin...@virtusa.com, Dec 6

Issue description

Chrome Version: 73.0.3632.0 (Official Build) Revision	914f5b8f721ff5a734bb22a96363191c20c8c3f2-refs/branch-heads/3632@{#1} (32/64-bit)
OS: Windows (7, 8, 8.1, 10), Mac(10.13.1 , 10.13.6 , 10.14.2) and Linux(14.04 LTS).

Pre-condition: At-least 7-8 password entry must be present under 'Saved Passwords' section so that vertical scroll-bar appears on 'Passwords' page.

What steps will reproduce the problem?
1. Launch Chrome and navigate to chrome://settings/passwords page.
2. Scroll down the page and click on first password entry under 'Saved Password' section and observe.

Actual: Unnecessary 'Passwords' page scrolled up after clicking on first password entry.
Expected: 'Passwords' page should stay on their position after clicking on first password entry.

This is a regression issue, broken in M-61 series, below is manual regression range:

Good build: 61.0.3142.0 (Revision: 482492)
Bad build: 61.0.3143.0 (Revision: 482833)

You are probably looking for a change made after 482751 (known good), but no later than 482762 (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/57c6dff1214ac4d327d0705d89534412a1743f3c..b962595bafd843504c3489fe8dbbf5e4c518528b

Suspecting: https://chromium.googlesource.com/chromium/src/+/522d72aec37a711f65b180102cc42f99496fbb8a

@dschuyler: 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: 
1. Provided suspect through 'Chromium bisect' script because unable to perform bisect using 'per-revision' bisect script.
2. Tried performing 'per revision' bisect on multiple Windows and Mac machines but unable to perform the same since getting "RuntimeError: We don't have enough builds to bisect." error.
3. This issue is also reproducible on Stable #71.0.3578.80, Beta #71.0.3578.80 & Dev #72.0.3626.7
4.  This issue is only reproducible after clicking on first password entry and every time we have to reload the page to reproduce this issue.

Kindly review the attached screen-cast for reference.

Thank You..!
 
Actual behavior.mp4
919 KB View Download
Expected behavior.mp4
774 KB View Download
Owner: jdoerrie@chromium.org
Components: UI>Browser>Passwords
Labels: -Pri-1 Pri-2
De-prioritizing this a bit, given that this is present since M61 already.

Sign in to add a comment