Issue metadata
Sign in to add a comment
|
Timer on nytimes crossword is broken |
||||||||||||||||||||
Issue descriptionChrome Version: 61.0.3150.0 OS: Android 7.1.2 Pixel XL NJH47D What steps will reproduce the problem? (1) open the nytimes mini crossword (2) solve (3) time says 0sec when it should have accurate time What is the expected result? Shows time What happens instead? Shows zero Please use labels and text to provide additional information. Works in stable. 3071.125 so must be a regression. For graphics-related bugs, please copy/paste the contents of the about:gpu page at the end of this report.
,
Jul 11 2017
What the bug looks like
,
Jul 11 2017
Thanks for the report! Good build: 61.0.3137.3 Bad build: 61.0.3138.0 Regression range: https://chromium.googlesource.com/chromium/src/+log/61.0.3137.3..61.0.3138.0?pretty=fuller&n=10000 Good commit: 481207 Bad commit: 481208 Culprit CL: https://chromium.googlesource.com/chromium/src/+/4cef659a8009c5cb4f5708336ae37654d23653b3
,
Jul 12 2017
Not possible to do a per-rev bisect?
,
Jul 12 2017
Sorry didn't read the last comment fully, thanks for the bisect.
,
Jul 12 2017
This is actually broken in FireFox as well. It appears onTouchStart they start looking for scrolling reaching a maximum threshold and then take over scrolling. They set a repeating timer checking the scroll position on the body element: document.body.scrollTop > this.startPosition + 40 && (window.clearInterval(this.scrollChecker), Obviously this is not true anymore for ScrollTopLeftInterop enabled. But is easily fixable with using: document.scrollingElement This doesn't appear to be in a library but in the game logic itself. I'll try to reach out to nytimes.
,
Jul 14 2017
,
Jul 14 2017
We've reached out to nytimes contacts and we have been advised they are aware of the issue and working on a fix.
,
Jul 14 2017
BTW I want to point out that this is the sort of site compat issue that makes doing breaking changes like issue 157855 . Previously Firefox users were penalized here because the site was expecting non-standard behavior while Firefox was following the spec. With our change the site will be updated and now work correctly cross-browser. Yay! Thanks for driving this Dave!
,
Aug 3 2017
,
Aug 3 2017
,
Aug 3 2017
,
Aug 3 2017
verified this worked fine on 2017/08/03 mini crossword on chrome canary 62.0.3175.0 |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by wfh@chromium.org
, Jul 11 2017