When Process Swap on Navigation is enabled, web pages that seem to have subresources from multiple origins hang. Some pages hang after updating the page title and address bar. Sometimes there appears to be a quick flash of web content. These sites hang: - amazon.com - facebook.com - ebay.com - wikipedia.org after searching for something - cnn.com - netflix.com
rdar://problem/39162236
Created attachment 337231 [details] Work in Progress Patch
Comment on attachment 337231 [details] Work in Progress Patch View in context: https://bugs.webkit.org/attachment.cgi?id=337231&action=review I had fixed this locally, not knowing someone else was working on it. My patch was simpler: > Source/WebKit/UIProcess/WebPageProxy.cpp:2354 > if (action == PolicyAction::Use && navigation) { > - auto proposedProcess = process().processPool().processForNavigation(*this, *navigation, action); > + auto proposedProcess = process().processPool().processForNavigation(*this, *navigation, action, frame.isMainFrame()); Instead of adding the new parameter, I just changed 2353 to: - if (action == PolicyAction::Use && navigation) { + if (action == PolicyAction::Use && navigation && frame.isMainFrame()) {
That said, while the main frame patch definitely fixes things in MiniBrowser, for example... Safari still has problems with defersLoading. Sometimes. That may or may not be related, so we should fix it separately.
Created attachment 337304 [details] Patch
Attachment 337304 [details] did not pass style-queue: ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:700: Multi-line string ("...") found. This lint script doesn't do well with such strings, and may give bogus warnings. They're ugly and unnecessary, and you should use concatenation instead". [readability/multiline_string] [5] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:702: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:710: Multi-line string ("...") found. This lint script doesn't do well with such strings, and may give bogus warnings. They're ugly and unnecessary, and you should use concatenation instead". [readability/multiline_string] [5] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:712: Multi-line string ("...") found. This lint script doesn't do well with such strings, and may give bogus warnings. They're ugly and unnecessary, and you should use concatenation instead". [readability/multiline_string] [5] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:714: Multi-line string ("...") found. This lint script doesn't do well with such strings, and may give bogus warnings. They're ugly and unnecessary, and you should use concatenation instead". [readability/multiline_string] [5] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:717: Multi-line string ("...") found. This lint script doesn't do well with such strings, and may give bogus warnings. They're ugly and unnecessary, and you should use concatenation instead". [readability/multiline_string] [5] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:721: Multi-line string ("...") found. This lint script doesn't do well with such strings, and may give bogus warnings. They're ugly and unnecessary, and you should use concatenation instead". [readability/multiline_string] [5] Total errors found: 7 in 4 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 337304 [details] Patch Clearing flags on attachment: 337304 Committed r230315: <https://trac.webkit.org/changeset/230315>
All reviewed patches have been landed. Closing bug.