When Mobile Safari is set to request the desktop website in iOS and iPadOS, it reports a user agent string with the macOS version as 10.15.6: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_6) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/15.5 Safari/605.1.15 On the other hand, Safari on Mac always reports the macOS version as 10.15.7: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/15.5 Safari/605.1.15 This inconsistency should be fixed (i.e. Mobile Safari should report 10.15.7 in desktop mode) because it could make it easier for a website to detect whether the visitor is using Mac Safari or is using Mobile Safari, defeating the purpose of desktop mode.
Created attachment 460042 [details] Patch
This has repeatedly bugged me (after all, it provides another way to sniff for iOS v. macOS), but I've never taken the small amount of time to actually fix this. (Thanks Nitin!)
Thank you for the patch! Would you be willing to re-submit it with a commit message? It cannot be landed without one. `webkit-patch upload` would include the commit message, and you can confirm that its format matches existing landed commits. It looks like https://webkit.org/contributing-code/ is currently out of date, we need to fix it ASAP.
Created attachment 460050 [details] Patch
(In reply to Alexey Proskuryakov from comment #3) > Would you be willing to re-submit it with a commit message? It cannot be > landed without one. `webkit-patch upload` would include the commit message, > and you can confirm that its format matches existing landed commits. done!
Sorry, it's not quite it yet (needs a "Reviewed by NOBODY (OOPS!)" line, otherwise tooling that replaces it with an actual reviewer name won't work). I think that if you reset, run `git-webkit setup`, and re-commit, the correct template should be generated.
Created attachment 460054 [details] Patch
(In reply to Alexey Proskuryakov from comment #6) > Sorry, it's not quite it yet (needs a "Reviewed by NOBODY (OOPS!)" line, > otherwise tooling that replaces it with an actual reviewer name won't work). > > I think that if you reset, run `git-webkit setup`, and re-commit, the > correct template should be generated. Should be fixed now.
Comment on attachment 460054 [details] Patch Let's see what happens!
Committed r295305 (251347@main): <https://commits.webkit.org/251347@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 460054 [details].
<rdar://problem/94474123>