New issue
Advanced search Search tips

Issue 700328 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Flex items render out of parent flex display element

Reported by marek.fr...@gmail.com, Mar 10 2017

Issue description

UserAgent: Mozilla/5.0 (Windows NT 6.3; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3036.0 Safari/537.36

Steps to reproduce the problem:
1. go on http://codepen.io/panntau/pen/evveoe in chrome 58 it display it correctly
2. go on http://codepen.io/panntau/pen/evveoe in chrome 59 it display it wrong, the flex items (li) starts after the end of it parent flex displayed element

What is the expected behavior?
I expect it to work as it work until version 59

What went wrong?
I think in chrome 59 do not know exact number of build.

Did this work before? Yes  58.0.3029.6 (64-bit)

Chrome version: 59.0.3036.0  Channel: canary
OS Version: 6.3
Flash Version:

 
I think it has something to do with the clearfix that bootstrap using with before and after pseudo elements.

Comment 2 by ajha@chromium.org, Mar 14 2017

Labels: Needs-Triage-M59 Needs-Bisect
Cc: jfernan...@igalia.com jmukthavaram@chromium.org
Labels: -Pri-2 -Needs-Bisect -Needs-Triage-M59 hasbisect-per-revision Pri-1
Owner: jfernandez@chromium.org
Status: Assigned (was: Unconfirmed)
Able to reproduce this issue on windows 7 with Chrome canary version-59.0.3040.0 and reported verison-58.0.3029.6.
Manual Bisect:
-------------
Good-59.0.3030.0 -Revision-454724
Bad-59.0.3032.0  -Revision-454806

Per revision biect info:
------------------------
You are probably looking for a change made after 454742 (known good), but no later than 454743 (first known bad).
CHANGELOG URL:
The script might not always return single CL as suspectas some perf builds might get missing due to failure.
https://chromium.googlesource.com/chromium/src/+log/a9d4fa4f3133c0fbd0a48d01846ec8968cfe2926..a90dba1f2dd094e51229dd2e5e059c6b53cee93a

Possible suspect:
----------------
https://chromium.googlesource.com/chromium/src/+/a90dba1f2dd094e51229dd2e5e059c6b53cee93a

jfernandez@Kindly take a look and please help us to reassign this issue to a right owner if not with respect to this change.
Note:No issue observed on Mac 10.12.3 & Linux Ubuntu 14.04
Thanks.!
Cc: -jfernan...@igalia.com
Owner: jfernan...@igalia.com
Yes, I'm aware of that commit causing many regressions. 
I reverted the change in https://crrev.com/2740063003 to this issue should be already fixed in ToT 

Could you please verify it's fixed on versions over refs/heads/master@{#455719} ?
Yes now in latest Canary (I have 59.0.3043.0 canary (64-bit)) it looks OK.
Status: Fixed (was: Assigned)

Sign in to add a comment