Bug 54567 – Drop "Windows;" (redundant) from user agent string

• Component: Platform
• Status: RESOLVED FIXED

[Peter Kasting]

This is about dropping the initial "Windows;" due to its redundancy with the subsequent "Windows NT 6.0;" (or similar other version identifier).  See blocked bug 54556 for motivation.

[Peter Kasting]

Created attachment 83710 [details]
patch v1

I have mailed the Mozilla guys on why "Macintosh;" was not also dropped (they explain why "X11;" was not dropped), but I bet the answer is going to be "web compat", as "Windows" is a string still found in the updated Windows-specific UA string (in the version identifier) but "Macintosh" would not be found in a similarly-updated Mac UA string (which uses "Mac OS" in the version identifier).

[Peter Kasting]

Response from Mozilla: the motivation for dropping "Windows;" is primarily to match IE better (as IE has never had this token), only secondarily because it's redundant.

This is a stronger motivation than just "the token could be removed" and also explains why "Macintosh" was not dropped.

[Peter Kasting]

Ping?  I'd really like to land this soon so all the UA string changes happen in a short window.

[James Robinson]

Comment on attachment 83710 [details]
patch v1

Let's do it.

[WebKit Commit Bot]

Comment on attachment 83710 [details]
patch v1

Rejecting attachment 83710 [details] from commit-queue.

Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=cr-jail-8', 'apply-..." exit_code: 2

Last 500 characters of output:
k #3 succeeded at 3727 (offset -1 lines).
patching file Source/WebKit/win/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file Source/WebKit/win/WebView.cpp
patching file LayoutTests/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file LayoutTests/http/tests/navigation/useragent-expected.txt
patching file LayoutTests/http/tests/navigation/useragent.php

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--reviewer', u'James Robinson', u'--f..." exit_code: 1

Full output: http://queues.webkit.org/results/8084070

[Peter Kasting]

Created attachment 84457 [details]
Updated patch

Updated to apply against current trunk.

[WebKit Commit Bot]

Comment on attachment 84457 [details]
Updated patch

Clearing flags on attachment: 84457

Committed r80218: <http://trac.webkit.org/changeset/80218>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.