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

Issue 716363 link

Starred by 2 users
Status: Started
Owner:
hcarmona@chromium.org
Cc:
nyerramilli@chromium.org
kkaluri@chromium.org
█ msrchandra@chromium.org
█ ranjitkan@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression : Unwanted extra blank space is observed on 'Manage spell check' sub page.

Reported by aiman.an...@etouch.net, Apr 28 2017 
Chrome: 60.0.3083.0 (Official Build) (cohort: win_58_81)1423a66d68562cda92de55d3f247ae1c8a155241-refs/heads/master@{#467821}(32/64-bit)

OS: Windows(7,8,8.1,10), Linux(14.04 LTS).

Steps to reproduce:
1. Launch chrome, go to chrome://settings/editDictionary, Add 50-55 words, click on back iron-icon for Manage spell check.
2. Expand spell check, click on Custom Spelling and observe

Actual Result: Unwanted extra blank space is observed on 'Manage Spell Check' sub page.
Expected Result:  'Manage Spell Check' sub page should appear properly without any extra blank space.

This is Regression issue broken in M-59,  will soon update other info

Manual Bisect:
Good Build: 59.0.3057.0
Bad Build: 59.0.3059.0

Note: Issue is not seen on MAC OS.
Actual Result.mp4
3.6 MB View Download
Expected Result.mp4
2.1 MB View Download

Comment 1 by kkaluri@chromium.org, Apr 28 2017

Cc: kkaluri@chromium.org
Labels: hasbisect-per-revision
Owner: dschuyler@chromium.org
Status: Assigned (was: Unconfirmed)
Bisect Info:
===========
Good build : 59.0.3057.0,  Revision Range- 460966
Bad build  : 59.0.3059.0,  Revision Range- 461269

After executing the per-revision-bisect script, i got the following CL's between good and bad build versions
============================================
https://chromium.googlesource.com/chromium/src/+log/1bafcfdfcc6867b9be120e54c73b0ea526cd886e..b87ba7ea439809c7a7ccd76a4ce55b1aae3aeeed

The suspecting Change Log is :
-----------
https://chromium.googlesource.com/chromium/src/+/b87ba7ea439809c7a7ccd76a4ce55b1aae3aeeed


Review-Url: https://codereview.chromium.org/2792463002

dschuyler@- Could you please look into this issue, if it's related to your change?  if not could you please help us to reassign this issue to the right owner.


Thank You...

Comment 2 by dschuyler@chromium.org, May 11 2017 

The above CL changed the list to use a scroll target, but the issue appears to be older than that. It may be an issue between iron-list and how we update the contents.

Dan found that the list can be reset by setting the list contents to a copy of itself (Object.assign([], 'items')) or by notifying the list that the items have changed.
Hector found that using notify resize will not refresh the contents.

It happens on ToT as well.

Comment 3 by dschuyler@chromium.org, May 11 2017 

Reproducing this bug requires going to the list, then back to the main page, then back to the list; which is an unusual set of steps. Refreshing the page will also cause the list to display correctly.

Comment 4 by dbeam@chromium.org, May 11 2017

Labels: -M-59 M-60

Comment 5 by hcarmona@chromium.org, May 16 2017

Owner: hcarmona@chromium.org
Status: Started (was: Assigned)
I've been investigating this
►

Sign in to add a comment