New issue
Advanced search Search tips

Issue 740844 link

Starred by 4 users

Issue metadata

Status: Verified
Owner:
Closed: Aug 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug-Regression

Blocked on:
issue 157855



Sign in to add a comment

Timer on nytimes crossword is broken

Project Member Reported by wfh@chromium.org, Jul 11 2017

Issue description

Chrome 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.

 

Comment 1 by wfh@chromium.org, Jul 11 2017

Labels: ReleaseBlock-Stable Needs-Bisect M-61
Works in beta 60.0.3112.50

Does not work in dev 61.0.3142.0

Comment 2 by wfh@chromium.org, Jul 11 2017

What the bug looks like
Screenshot_20170711-095450.png
583 KB View Download
Labels: -Needs-Bisect hasbisect-per-revision
Owner: dtapu...@chromium.org
Status: Assigned (was: Untriaged)
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

Comment 4 by wfh@chromium.org, Jul 12 2017

Cc: candr...@chromium.org
Not possible to do a per-rev bisect?

Comment 5 by wfh@chromium.org, Jul 12 2017

Sorry didn't read the last comment fully, thanks for the bisect.
Cc: rbyers@chromium.org
Labels: -Pri-1 -ReleaseBlock-Stable Pri-2
Status: ExternalDependency (was: Assigned)
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.

Comment 7 by rbyers@chromium.org, Jul 14 2017

Blockedon: 157855
We've reached out to nytimes contacts and we have been advised they are aware of the issue and working on a fix.

Comment 9 by rbyers@chromium.org, Jul 14 2017

Cc: foolip@chromium.org
Labels: Hotlist-Interop
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!
Blockedon: 751965
Blockedon: -751965
Status: Fixed (was: ExternalDependency)

Comment 13 by wfh@chromium.org, Aug 3 2017

Status: Verified (was: Fixed)
verified this worked fine on 2017/08/03 mini crossword on chrome canary 62.0.3175.0

Sign in to add a comment