RESOLVED FIXED 191166
[PSON] Reuse SuspendedPages' process when possible, for performance 
https://bugs.webkit.org/show_bug.cgi?id=191166
Summary [PSON] Reuse SuspendedPages' process when possible, for performance
Chris Dumez
Reported 2018-11-01 13:31:21 PDT
Reuse SuspendedPages' process when possible, for performance.
Attachments
Patch (10.86 KB, patch)
2018-11-01 16:44 PDT, Chris Dumez 		ggaren: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2018-11-01 16:44:55 PDT
Created attachment 353660 [details] Patch
Geoffrey Garen
Comment 2 2018-11-01 21:29:29 PDT
Comment on attachment 353660 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=353660&action=review r=me > Source/WebKit/UIProcess/WebProcessPool.cpp:2219 > + // FIXME: If the SuspendedPage is for this page, then we need to destroy the suspended page as we do not support having > + // multiple WebPages with the same ID in a given WebProcess currently. > + if (&(*it)->page() == &page) > + m_suspendedPages.remove(it); This FIXME seems worth a bug report.
Chris Dumez
Comment 3 2018-11-02 08:59:29 PDT
Committed r237735: <https://trac.webkit.org/changeset/237735>
Radar WebKit Bug Importer
Comment 4 2018-11-02 09:00:40 PDT
<rdar://problem/45763153>
Note You need to log in before you can comment on or make changes to this bug.