Bug 171823 - Update Chrome and Firefox versions in user agent quirks
Summary: Update Chrome and Firefox versions in user agent quirks
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: Other
Hardware: PC Linux
: P2 Normal
Assignee: Michael Catanzaro
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-05-08 13:16 PDT by Michael Catanzaro
Modified: 2017-05-09 11:44 PDT (History)
5 users (show)

See Also:


Attachments
Patch (1.95 KB, patch)
2017-05-08 13:20 PDT, Michael Catanzaro
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2017-05-08 13:16:55 PDT
Update Chrome and Firefox versions in user agent quirks
Comment 1 Michael Catanzaro 2017-05-08 13:19:08 PDT
(I wonder what this will break.)
Comment 2 Michael Catanzaro 2017-05-08 13:20:04 PDT
Created attachment 309404 [details]
Patch
Comment 3 Carlos Garcia Campos 2017-05-08 23:19:39 PDT
(In reply to Michael Catanzaro from comment #1)
> (I wonder what this will break.)

Why do we want this if you think it's going to break something? I don't think we should do this unless it fixes anything.
Comment 4 Michael Catanzaro 2017-05-09 06:38:37 PDT
Well we know websites are definitely going to break eventually if we don't do it. E.g. you'll get unsupported browser warnings in GMail. So we should do it.

But I would err on the side of not backporting it to 2.16 until we know it's needed to fix something.
Comment 5 Carlos Alberto Lopez Perez 2017-05-09 10:53:21 PDT
Gmail will stop working with chrome53 https://gsuiteupdates.googleblog.com/2017/02/gmail-no-longer-supporting-chrome-v53.html

Soon similar things will happen for chrome56, etc.

I think keeping updated the version to the last one (or almost) is something sane.

People running chrome or firefox are usually running the last version (they are evergreen browsers -- they update themselves automatically without asking the user).

So sites that test for the chrome or firefox version usually do this as way to see if you don't have an obsolete browser.
Comment 6 Carlos Alberto Lopez Perez 2017-05-09 10:54:35 PDT
Comment on attachment 309404 [details]
Patch

r=me, but please test that this doesn't break any major site
Comment 7 Michael Catanzaro 2017-05-09 11:40:57 PDT
(In reply to Carlos Alberto Lopez Perez from comment #6)
> Comment on attachment 309404 [details]
> Patch
> 
> r=me, but please test that this doesn't break any major site

Note these versions are only used for our quirks, so Firefox UA bump only affects Google and the Chrome UA bump only affects TypeKit and Slack (but see bug #171869).
Comment 8 WebKit Commit Bot 2017-05-09 11:44:38 PDT
Comment on attachment 309404 [details]
Patch

Clearing flags on attachment: 309404

Committed r216531: <http://trac.webkit.org/changeset/216531>
Comment 9 WebKit Commit Bot 2017-05-09 11:44:39 PDT
All reviewed patches have been landed.  Closing bug.