We should always claim to be running on Intel regardless of actual CPU type. See discussion in bug #162548 for details. In particular, we can never advertise ARM because it causes dumb websites to send mobile pages.
Created attachment 289946 [details] Patch
Comment on attachment 289946 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=289946&action=review > Source/WebCore/platform/gtk/UserAgentGtk.cpp:96 > - static NeverDestroyed<const String> uaOSVersion(String::format("%s Mac OS X", cpuDescriptionForUAString())); > + static NeverDestroyed<const String> uaOSVersion(ASCIILiteral("Intel Mac OS X")); So, we should update the comment to also explain why we use Intel unconditionally. > Source/WebCore/platform/gtk/UserAgentGtk.cpp:121 > + uaString.appendLiteral("Intel"); > uaString.appendLiteral(" Mac OS X"); These could be a single appendLiteral call now.
Created attachment 290080 [details] Patch
Comment on attachment 290080 [details] Patch Clearing flags on attachment: 290080 Committed r206517: <http://trac.webkit.org/changeset/206517>
All reviewed patches have been landed. Closing bug.