Bug 221845 - [GTK] Need new user agent quirk for Google Docs and Google Drive
Summary: [GTK] Need new user agent quirk for Google Docs and Google Drive
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: PC Linux
: P2 Normal
Assignee: Michael Catanzaro
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-02-12 13:48 PST by Michael Catanzaro
Modified: 2021-03-09 08:33 PST (History)
4 users (show)

See Also:


Attachments
Patch (6.90 KB, patch)
2021-02-15 15:23 PST, Michael Catanzaro
no flags Details | Formatted Diff | Diff
Patch (11.88 KB, patch)
2021-02-16 07:33 PST, 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 2021-02-12 13:48:49 PST
In bug #219278 I determined that the only user agent quirk compatible with WebKit was an Internet Explorer quirk. Sadly I was wrong about how long it would work for:

"""
Google Docs won't support Internet Explorer after March 15th, 2021 Please upgrade to a
supported browser
"""

So yeah, time to find yet another quirk. We will probably have to select a fairly risky Firefox or Chrome quirk and hope for the best.
Comment 1 Michael Catanzaro 2021-02-12 13:53:53 PST
(In reply to Michael Catanzaro from comment #0)
> We will probably have to select a
> fairly risky Firefox or Chrome quirk and hope for the best.

See bug #215845 for what happened last time I tried a Firefox quirk....
Comment 2 Michael Catanzaro 2021-02-15 15:17:56 PST
As far as I can tell, Google Drive is no longer displaying unsupported browser warnings at all, except for Internet Explorer. That's a welcome change. I think it is a very recent change, because I tested it when creating the Internet Explorer quirk and determined it was really required. Anyway, I will remove our quirk for Google Drive.

Even better news: it seems we no longer need to send our Google Docs quirk to accounts.youtube.com. That was previously required for the quirk to take effect, but now it's sufficient to limit the quirk to docs.google.com. I think that was actually our fault, not Google's: it was probably an artifact of bug #191858. This is a *huge* relief because it means that if our new quirk does break something in the future, the scope will be limited to only Google Docs: it shouldn't blow up youtube.com or mail.google.com or accounts.google.com, all of which has happened to us in the past.

I discovered that a Chrome quirk seems to work on Google Docs without breaking anything. So for now, we can safely go with that. It's a risky quirk and could easily break in the future, but we don't have many options here.
Comment 3 Michael Catanzaro 2021-02-15 15:23:27 PST
Created attachment 420386 [details]
Patch
Comment 4 Carlos Garcia Campos 2021-02-16 00:53:51 PST
It seems the unit tests is failing
Comment 5 Michael Catanzaro 2021-02-16 05:59:04 PST
Oops, they have to be updated whenever changing user agent quirks and I forgot to do so.
Comment 6 Michael Catanzaro 2021-02-16 07:33:38 PST
Created attachment 420471 [details]
Patch
Comment 7 EWS 2021-02-16 12:20:26 PST
Committed r272921: <https://commits.webkit.org/r272921>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 420471 [details].
Comment 8 Michael Catanzaro 2021-02-16 13:28:53 PST
(In reply to Michael Catanzaro from comment #0)
> """
> Google Docs won't support Internet Explorer after March 15th, 2021 Please
> upgrade to a
> supported browser
> """

Note there is a time bomb on how long we have to release this change before users start to complain. Sorry about this, especially that I didn't notice until the day after the 2.30.5 release. :/