Bug 263789
| Summary: | Extend Quirk shouldDisableContentChangeObserver() to the full YouTube.com | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Karl Dubost <karlcow> |
| Component: | WebKit Misc. | Assignee: | Karl Dubost <karlcow> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | Safari 17 | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| URL: | http://youtube.com/ | ||
| Bug Depends on: | 195295, 249740 | ||
| Bug Blocks: | 264234 | ||
Karl Dubost
Currently there is a Quirk on YouTube.com which disables ContentChangeObserver on /results page and /watch pages
The homepage of YouTube is failing to work properly on iPad Pro
where time to time a tap on a thumbnail will not initiate a navigation to the proper video page.
There is a threshold of 32ms but that is breaking YouTube, because anything that would be faster when changing the DOM would not be caught.
YouTube is agreeing to apply the Quirk to the full Website.
https://searchfox.org/wubkat/rev/d686d8cda3e314c96462325828985ba322c9bf25/Source/WebCore/page/Quirks.cpp#196-206
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Karl Dubost
<rdar://115566212>
Karl Dubost
Pull request: https://github.com/WebKit/WebKit/pull/19647
Karl Dubost
We may not need this Quirk, if YouTube is changing the way they listen to events.
EWS
Committed 269888@main (3aa4d2baaa6f): <https://commits.webkit.org/269888@main>
Reviewed commits have been landed. Closing PR #19647 and removing active labels.