WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
284510
Remove Quirk needsIPadMiniUserAgent for cctv.com
https://bugs.webkit.org/show_bug.cgi?id=284510
Summary
Remove Quirk needsIPadMiniUserAgent for cctv.com
Karl Dubost
Reported
2024-12-11 21:36:31 PST
Currently on regular iPad, cctv.com is receiving iPad Mini User Agent instead of the traditional desktop user agent. Initially this Quirk was added through
Bug 198484
on 2019-06-04
https://github.com/WebKit/WebKit/commit/4bb8f0f34179bd8ed4dcafeda205f65b5b8d6c87#diff-d2eb30e968da05615b61df7746f0d13b708da1629b083d455e1f98de80367bebR1293-R1295
then moved later on to
https://github.com/WebKit/WebKit/blob/ba1fbaadee0af044adad5a31c2eca663007c44b2/Source/WebCore/page/Quirks.cpp#L1677
The reason for adding it is at the time this site is that receiving the desktop UA, some sites were sending the Flash Player to the users which was not supported on iOS devices. The Flah player end of life was programmed for 2020. This needs to be tested with Quirks disabled. If the site is working without the iPad Mini UA, then we can remove this quirk.
Attachments
video page failing
(1.56 MB, image/jpeg)
2024-12-11 21:57 PST
,
Karl Dubost
no flags
Details
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2024-12-11 21:36:51 PST
<
rdar://problem/141335507
>
Karl Dubost
Comment 2
2024-12-11 21:50:08 PST
With the iPad normal UA string (aka desktop), we can see a perfectly normal website. * both portrait and landscape fit exactly the window width Most of the homepage links open to subsites such as
https://tv.cctv.com/2024/12/11/VIDE74ZoCjjwl5XcUn2aAjb3241211.shtml
The video is not starting. They are using this player.
https://player.cntv.cn/h5vod/vodh5player.min.js
``` [Log] ENV: domain:tv.cctv.com,DOMAIN:player.cntv.cn/h5vod/,startMuted:false,isSafari:true,RELEASE:true,LOCAL:undefined,DEBUG:false (vodh5player.min.js, line 5) [Log] (ns.createVodPlayer) H5vodplayer Release Wrapper:3.2.4.24.07.15 (vodh5player.min.js, line 5) [Error] Failed to load resource: the server responded with a status of 404 (Not Found) (
hlsp2p_cntv@1.6.40.js.map
, line 0) [Error] VJS: – "ERROR:" – "(CODE:3 MEDIA_ERR_DECODE)" – "The media playback was aborted due to a corruption problem or because the media used features your browser did not support." wt code: 3 message: "The media playback was aborted due to a corruption problem or because the media used features your browser did not support." wt Prototype (anonymous function) (video_novhs.min.js:15:1557) (anonymous function) (video_novhs.min.js:23:243642) (anonymous function) (video_novhs.min.js:23:225787) (anonymous function) (video_novhs.min.js:15:13917) ``` The same url on a desktop computer
https://tv.cctv.com/2024/12/11/VIDE74ZoCjjwl5XcUn2aAjb3241211.shtml
loads the video after advertisement has played. Another test the video is working with iPad Mini UA ```
> window.location.href
< "
https://tv.cctv.com/2024/12/12/VIDEKtwzgIt3xO6Dw3uiIb8a241212.shtml?spm=C28340.PImrkkn9UZ9h.EHuWA8qIunFk.1
"
> navigator.userAgent
< "Mozilla/5.0 (iPad; CPU OS 18_2 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/18.2 Mobile/15E148 Safari/604.1" ``` But as soon as I switch to desktop UA for iPad, it doesn't work anymore.
Karl Dubost
Comment 3
2024-12-11 21:57:00 PST
Created
attachment 473547
[details]
video page failing
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