Bug 244852 – REGRESSION(252943@main): [GTK] Cannot display https://www.cbsnews.com/

• Status: RESOLVED
• Resolution: DUPLICATE
• Component: WebKitGTK

[Michael Catanzaro]

After some recent change, probably either WebKitGTK 2.37.91 or else perhaps a change in libsoup, we are no longer able to display https://www.cbsnews.com/. Instead we just see an empty page.

[Carlos Garcia Campos]

a697ed80d982828b800582af5c283f19ed6aba97 is the first bad commit
commit a697ed80d982828b800582af5c283f19ed6aba97
Date:   Thu Jul 28 23:43:19 2022 -0700

    link elements should be able to fire more than one load / error event
    https://bugs.webkit.org/show_bug.cgi?id=232309
    
    Reviewed by Chris Dumez and Darin Adler.
    
    Based on a patch written by Chris Dumez.
    
    This patch makes link element emit more than one load event each time resource is loaded,
    and fixes the bug that setting rel content attribute to the same value resulted in the resource to be reloaded.
    New behavior matches that of Chrome and Firefox.
    
    * LayoutTests/TestExpectations: Unskip now passing tests.
    * LayoutTests/fast/dom/HTMLLinkElement/link-preload-load-once-expected.txt: Added.
    * LayoutTests/fast/dom/HTMLLinkElement/link-preload-load-once.html: Added.
    * LayoutTests/fast/dom/HTMLLinkElement/link-stylesheet-load-once-expected.txt: Added.
    * LayoutTests/fast/dom/HTMLLinkElement/link-stylesheet-load-once.html: Added.
    * LayoutTests/imported/w3c/web-platform-tests/html/semantics/document-metadata/the-link-element/link-multiple-error-events-expected.txt:
    * LayoutTests/imported/w3c/web-platform-tests/html/semantics/document-metadata/the-link-element/link-multiple-load-events-expected.txt:
    * LayoutTests/platform/mac-wk1/fast/dom/HTMLLinkElement/link-preload-load-once-expected.txt:
    * LayoutTests/platform/mac-wk1/fast/dom/HTMLLinkElement/link-stylesheet-load-once-expected.txt:
    * LayoutTests/platform/win/fast/dom/HTMLLinkElement/link-preload-load-once-expected.txt:
    * LayoutTests/platform/win/fast/dom/HTMLLinkElement/link-stylesheet-load-once-expected.txt:
    
    * Source/WebCore/html/HTMLLinkElement.cpp:
    (WebCore::HTMLLinkElement::HTMLLinkElement):
    (WebCore::HTMLLinkElement::parseAttribute):
    (WebCore::HTMLLinkElement::notifyLoadedSheetAndAllCriticalSubresources):
    * Source/WebCore/html/HTMLLinkElement.h:
    * Source/WebCore/html/LinkRelAttribute.h:
    (WebCore::operator==): Added.
    
    Canonical link: https://commits.webkit.org/252943@main

[Carlos Garcia Campos]

The reason why page doesn't load is because the web process is at 100% CPU doing layouts all the time.

[Ryosuke Niwa]

This patch got reverted.

[Michael Catanzaro]

OK, just yesterday, in bug #244879. I'll add it to the backport list. And now you have another website to test on before trying to re-land. ;)

Thanks for bisecting it, Carlos.

[Adrian Perez]

I have already cherry-picked the revert in the 2.38 release branch,
it will be included in the next release.

[Fujii Hironori]

> <link rel="stylesheet" href="/fly/bundles/cbsnewscontent/css/responsive.min.css?v=8bc2e0f78408a4b8086da85d60d8c0a5" media="print" onload="this.media='all'">

'media' attribute is modified.

[Ryosuke Niwa]

*** This bug has been marked as a duplicate of bug 244879 ***