WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
221845
[GTK] Need new user agent quirk for Google Docs and Google Drive
https://bugs.webkit.org/show_bug.cgi?id=221845
Summary
[GTK] Need new user agent quirk for Google Docs and Google Drive
Michael Catanzaro
Reported
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.
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
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Michael Catanzaro
Comment 1
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....
Michael Catanzaro
Comment 2
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.
Michael Catanzaro
Comment 3
2021-02-15 15:23:27 PST
Created
attachment 420386
[details]
Patch
Carlos Garcia Campos
Comment 4
2021-02-16 00:53:51 PST
It seems the unit tests is failing
Michael Catanzaro
Comment 5
2021-02-16 05:59:04 PST
Oops, they have to be updated whenever changing user agent quirks and I forgot to do so.
Michael Catanzaro
Comment 6
2021-02-16 07:33:38 PST
Created
attachment 420471
[details]
Patch
EWS
Comment 7
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]
.
Michael Catanzaro
Comment 8
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. :/
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug