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

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.