Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[iOS] "A problem repeatedly occurred" error page when safari resuming from background #2085

Conversation

cdumez
Copy link
Contributor

@cdumez cdumez commented Jul 5, 2022

5798b62

[iOS] "A problem repeatedly occurred" error page when safari resuming from background
https://bugs.webkit.org/show_bug.cgi?id=242352

Reviewed by Darin Adler.

Follow-up to 252108@main to take care of more cases where we might try to reuse a
terminated WebProcess during crash recovery, because we have not received the crash
notification for the back up process yet.

252108@main took care of processes reused from a related WKWebView. This patch takes
care of processes taken from the WebProcess cache or from a suspended page in the
back/forward list.

* Source/WebKit/UIProcess/SuspendedPageProxy.cpp:
(WebKit::SuspendedPageProxy::findReusableSuspendedPageProcess):
* Source/WebKit/UIProcess/WebProcessCache.cpp:
(WebKit::WebProcessCache::takeProcess):
* Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:
(-[PSONNavigationDelegate webViewWebContentProcessDidTerminate:]):

Canonical link: https://commits.webkit.org/252177@main

@webkit-early-warning-system webkit-early-warning-system added the merging-blocked Applied to prevent a change from being merged label Jul 5, 2022
@cdumez cdumez added merge-queue Applied to send a pull request to merge-queue and removed merging-blocked Applied to prevent a change from being merged labels Jul 6, 2022
… from background

https://bugs.webkit.org/show_bug.cgi?id=242352

Reviewed by Darin Adler.

Follow-up to 252108@main to take care of more cases where we might try to reuse a
terminated WebProcess during crash recovery, because we have not received the crash
notification for the back up process yet.

252108@main took care of processes reused from a related WKWebView. This patch takes
care of processes taken from the WebProcess cache or from a suspended page in the
back/forward list.

* Source/WebKit/UIProcess/SuspendedPageProxy.cpp:
(WebKit::SuspendedPageProxy::findReusableSuspendedPageProcess):
* Source/WebKit/UIProcess/WebProcessCache.cpp:
(WebKit::WebProcessCache::takeProcess):
* Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:
(-[PSONNavigationDelegate webViewWebContentProcessDidTerminate:]):

Canonical link: https://commits.webkit.org/252177@main
@webkit-commit-queue
Copy link
Collaborator

Committed 252177@main (5798b62): https://commits.webkit.org/252177@main

Reviewed commits have been landed. Closing PR #2085 and removing active labels.

@webkit-early-warning-system webkit-early-warning-system merged commit 5798b62 into WebKit:main Jul 6, 2022
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Jul 6, 2022
@cdumez cdumez deleted the 242352_problem_repeatedly_occurred branch July 6, 2022 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WebKit2 Bugs relating to the WebKit2 API layer
Projects
None yet
4 participants