Bug 201544 – REGRESSION(r249142): [GTK] Epiphany delayed page loads continue indefinitely

• Component: WebKitGTK
• Status: RESOLVED FIXED

[Michael Catanzaro]

When starting Epiphany, it loads a blank page in all web views except the one that's initially-visited using webkit_web_view_load_alternate_html().

In 2.25.4 and earlier, the load completes.

In 2.25.92, the load seems to continue forever. It's easy to see in Epiphany because all tabs you haven't visited yet have a spinner that spins forever until you visit the tab.

I haven't investigated yet, but it's probably not receiving a load finished event.

[Michael Catanzaro]

ca18a70558b52ae21287b5e0003c36085baa59e7 is the first bad commit
commit ca18a70558b52ae21287b5e0003c36085baa59e7
Author: [e-mail] <[e-mail]@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Tue Aug 27 16:08:48 2019 +0000

    Do not clear the pending api request when there's no navigation ID
    https://bugs.webkit.org/show_bug.cgi?id=201175
    
    Patch by Carlos Garcia Campos <[e-mail]> on 2019-08-27
    Reviewed by Chris Dumez.
    
    After r247851, the pending API request URL is cleared for subresources in some cases.
    
    * UIProcess/WebPageProxy.cpp:
    (WebKit::WebPageProxy::didStartProvisionalLoadForFrameShared):
    (WebKit::WebPageProxy::decidePolicyForNavigationAction):
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@249142 268f45cc-cd09-0410-ab3c-d52691b4dbfc

[Carlos Garcia Campos]

Created attachment 378727 [details]
Patch

[Michael Catanzaro]

Comment on attachment 378727 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=378727&action=review

Nice test. Tests are r=me. The actual code change needs an owner, of course.

> Tools/TestWebKitAPI/Tests/WebKitGLib/TestLoaderClient.cpp:439
> +    test->checkURIAtState(ViewURITrackingTest::State::Commited, "/normal");

Preexisting type: this should be Committed

[Chris Dumez]

Comment on attachment 378727 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=378727&action=review

> Source/WebKit/UIProcess/WebPageProxy.cpp:4050
> +    bool fromAlternateHTMLAPI = !unreachableURL.isEmpty() && unreachableURL == m_pageLoadState.pendingAPIRequestURL();

WK2 Owner LGTM.

[Carlos Garcia Campos]

Committed r249890: <https://trac.webkit.org/changeset/249890>