Bug 263789 - Extend Quirk shouldDisableContentChangeObserver() to the full YouTube.com
Summary: Extend Quirk shouldDisableContentChangeObserver() to the full YouTube.com
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: Safari 17
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Karl Dubost
URL: http://youtube.com/
Keywords: InRadar
Depends on: 195295 249740
Blocks: 264234
  Show dependency tree / graph
 
Reported: 2023-10-27 10:53 PDT by Karl Dubost
Modified: 2023-11-05 17:17 PST (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Karl Dubost 2023-10-27 10:53:08 PDT 
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
Comment 1 Karl Dubost 2023-10-27 10:53:50 PDT 
<rdar://115566212>
Comment 2 Karl Dubost 2023-10-27 11:00:32 PDT 
Pull request: https://github.com/WebKit/WebKit/pull/19647
Comment 3 Karl Dubost 2023-10-27 14:45:43 PDT 
We may not need this Quirk, if YouTube is changing the way they listen to events.
Comment 4 EWS 2023-10-28 07:02:37 PDT 
Committed 269888@main (3aa4d2baaa6f): <https://commits.webkit.org/269888@main>

Reviewed commits have been landed. Closing PR #19647 and removing active labels.