Pages using Google Tag Manager Google's anti-flicker optimization may take ~5 seconds to do initial paint

[Anthony Ricaud]

This is a followup of bug 199173.

Visible on https://www.formula1.com/en.html

This page uses the other snippet to load Google Analytics. I think this is now the main snippet given by Google.


<!-- Page hiding snippet (recommended)  -->
<style>.async-hide { opacity: 0 !important} </style>
<script>(function(a,s,y,n,c,h,i,d,e)
{s.className+=' '+y;h.start=1*new Date; h.end=i=function()
{s.className=s.className.replace(RegExp(' ?'+y),'')};
(a[n]=a[n]||[]).hide=h;setTimeout(function(){i();h.end=null},c);h.timeout=c;
})(window,document.documentElement,'async-hide','dataLayer',4000,
{'GTM-KB77S58':true});
</script>
<!-- Google Tag Manager -->
<script>
    (function(w,d,s,l,i){w[l]=w[l]||[];w[l].push({'gtm.start':
    new Date().getTime(),event:'gtm.js'});var f=d.getElementsByTagName(s)[0],
    j=d.createElement(s),dl=l!='dataLayer'?'&l='+l:'';j.async=true;j.src='https://www.googletagmanager.com/gtm.js?id='+i+dl;f.parentNode.insertBefore(j,f);
})(window,document,'script','dataLayer','GTM-NTR4PPW');
</script>
<!-- End Google Tag Manager -->

[Radar WebKit Bug Importer]

<rdar://problem/52966461>

[Chris Dumez]

Created attachment 373943 [details]
Patch

[Geoffrey Garen]

Comment on attachment 373943 [details]
Patch

r=me

[Chris Dumez]

Comment on attachment 373943 [details]
Patch

Clearing flags on attachment: 373943

Committed r247361: <https://trac.webkit.org/changeset/247361>

[Chris Dumez]

All reviewed patches have been landed.  Closing bug.

[Anthony Ricaud]

Thank you very much for the quick turnaround!

[Chris Dumez]

(In reply to Anthony Ricaud from comment #6)
> Thank you very much for the quick turnaround!

No problem, thanks for letting us know of this other case.