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

Issue 697778 link

Starred by 1 user
Status: Fixed
Owner:
█ scottchen@chromium.org
Last visit > 30 days ago
Closed: Mar 2017
Cc:
nyerramilli@chromium.org
rbasuvula@chromium.org
█ msrchandra@chromium.org
█ ranjitkan@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression : Unnecessary movement of contents is seen on "Manage passwords" sub page.

Reported by yfulgaon...@etouch.net, Mar 2 2017 
Chrome Version : 58.0.3028.0 (Official Build) c49428ef69cfd173c5d7eaeb39982f44ca68da78-refs/heads/master@{#454140} 32/64 bit
OS : Windows (7,8,8.1,10), Linux (14.04 LTS), Mac (10.11.6, 10.12.1, 10.12)

What steps will reproduce the problem?
1. Launch chrome, navigate to chrome://md-settings/passwords
2. Click inside "Search passwords" text field and hit Space bar, observe the page contents.

Actual : Unnecessary movement of contents is seen on "Manage passwords" sub page after typing keywords in search field.
Expected : Contents of "Manage passwords" should not move after typing keywords in search field.

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.3026.0
Bad build : 58.0.3027.0
Actual_Result.mp4
687 KB View Download
Expected_Result.mp4
921 KB View Download

Comment 1 by rbasuvula@chromium.org, Mar 2 2017

Cc: rbasuvula@chromium.org
Labels: hasbisect-per-revision
Owner: scottchen@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build: 58.0.3026.0 (Revision: 453454).
Bad build: 58.0.3027.0 (Revision: 453800).

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

CHANGE-LOG URL:
---------------
https://chromium.googlesource.com/chromium/src/+log/5165a30947f7e933c1cc3b5d902b7bbae1873976..aa40908dd04900048559b86f7eeae484e6623e11

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

@scottchen : Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to concern owner.

Review-Url:https://codereview.chromium.org/2715863002
Note :Able to reproduce the issue in Win 10.0 and Mac 10.12.3 Not in Ubuntu 14.04 and Able to reproduce in latest Canary #58.0.3028.0

Comment 2 by rbasuvula@chromium.org, Mar 2 2017 

Correction: In comment #1 in Note mentioned Not in Ubuntu 14.04.Able to reproduce on Ubuntu 14.04 also.
Thank you!

Comment 3 by scottchen@chromium.org, Mar 3 2017

Status: Started (was: Assigned)
Project Member

Comment 4 by bugdroid1@chromium.org, Mar 6 2017 

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

commit 3ed5d998cc82797b116a949f75da144ec3465a6d
Author: scottchen <scottchen@chromium.org>
Date: Mon Mar 06 20:27:59 2017

MD Settings: re-fix relative paper-input-label font-size.

In a previous attempt to make paper-input-label font-size relative (https://codereview.chromium.org/2715863002), I actually shrunk the size by a couple pixels comparing to the original design. This new CL aims to fix the unintended change, and also bring one other hard-coded label in line.

BUG= 697726 , 697341 , 697778 , 697340 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2726113003
Cr-Commit-Position: refs/heads/master@{#454940}

[modify] https://crrev.com/3ed5d998cc82797b116a949f75da144ec3465a6d/chrome/browser/resources/settings/appearance_page/appearance_page.html
[modify] https://crrev.com/3ed5d998cc82797b116a949f75da144ec3465a6d/chrome/browser/resources/settings/languages_page/edit_dictionary_page.html
[modify] https://crrev.com/3ed5d998cc82797b116a949f75da144ec3465a6d/chrome/browser/resources/settings/passwords_and_forms_page/address_edit_dialog.html
[modify] https://crrev.com/3ed5d998cc82797b116a949f75da144ec3465a6d/chrome/browser/resources/settings/settings_page/settings_subpage_search.html
[modify] https://crrev.com/3ed5d998cc82797b116a949f75da144ec3465a6d/chrome/browser/resources/settings/settings_vars_css.html

Comment 5 by scottchen@chromium.org, Mar 6 2017

Status: Fixed (was: Started)
►

Sign in to add a comment