Regression: Marquee Text in 'News and Update' section is not moving upwards in www.vidyalakshmi.com
Reported by
adha...@etouch.net,
Mar 16 2016
|
|||
Issue descriptionChrome version:50.0.2661.37 (Official Build)4cf5e769930d697e8f617cff6a12f0d27dbfee89-refs/branch-heads/2661@{#250}(32/64 bit) OS:Windows(Aero enabled),Mac Steps to reproduce the issue: 1. Launch chrome and navigate to https://www.vidyalakshmi.co.in/Students/ 2. Observe the 'News and Updates' section on the page. Actual:Marquee text in 'News and Update' section is not moving upwards. Expected:Marquee Text in 'News and Update' section should be moving upwards. This is a Non regression issue broken in M-41 ,below is the bisect information: Narrow bisect: https://chromium.googlesource.com/chromium/src/+log/289e745590ec9bf78f6cd6f546cf1801c36bb6d9..85fb6f102205ae33f4fde9d3afe66f626d5459b4?pretty=fuller&n=100 Blink CL: https://chromium.googlesource.com/chromium/blink/+log/7a69af2..36b9619?pretty=fuller&n=100 Suspecting: 185567 ? from blink Good build:41.0.2223.3 Bad build:41.0.2224.0 Could you please help to reassign if your change is not the cause for this change?
,
Mar 16 2016
Will update Linux info soon.
,
Mar 17 2016
,
Mar 17 2016
Looks like the Marquee code on the page is racy and buggy. It is trying to get the dimensions of the element to be scrolled using getComputedStyle before layout has happened (getting NaN for contentWidth, contentHeight and marqueeWidth). It then tries to call animate with -NaNpx using the invalid dimensions. My change must made it fail deterministically where it would have been racy and intermittent before. If we want to fix this and allow for it on the browser side we will need to make getComputedStyle force a synchronous layout (and that will need to go to the Blink layout team). |
|||
►
Sign in to add a comment |
|||
Comment 1 by ranjitkan@chromium.org
, Mar 16 2016