WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
263789
Extend Quirk shouldDisableContentChangeObserver() to the full YouTube.com
https://bugs.webkit.org/show_bug.cgi?id=263789
Summary
Extend Quirk shouldDisableContentChangeObserver() to the full YouTube.com
Karl Dubost
Reported
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
Attachments
Add attachment
proposed patch, testcase, etc.
Karl Dubost
Comment 1
2023-10-27 10:53:50 PDT
<
rdar://115566212
>
Karl Dubost
Comment 2
2023-10-27 11:00:32 PDT
Pull request:
https://github.com/WebKit/WebKit/pull/19647
Karl Dubost
Comment 3
2023-10-27 14:45:43 PDT
We may not need this Quirk, if YouTube is changing the way they listen to events.
EWS
Comment 4
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.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug