Allow pages using EventSource to enter the back/forward cache.
<rdar://problem/55853142>
Created attachment 379853 [details] Patch
Created attachment 379904 [details] Patch
Created attachment 379931 [details] Patch
Created attachment 379936 [details] Patch
Created attachment 379950 [details] Patch
Patch is ready for review.
ping review?
Comment on attachment 379950 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=379950&action=review r=me > Source/WebCore/page/EventSource.cpp:427 > + if (std::exchange(m_shouldDispatchErrorEventOnResume, false)) { > + scriptExecutionContext()->postTask([this, pendingActivity = makePendingActivity(*this)](ScriptExecutionContext&) { > + if (!isContextStopped()) > + dispatchErrorEvent(); > + }); Would it be crazy just to silently reconnect instead?
Comment on attachment 379950 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=379950&action=review >> Source/WebCore/page/EventSource.cpp:427 >> + }); > > Would it be crazy just to silently reconnect instead? I can give it a shot. It would be nice.
Created attachment 380268 [details] Patch
Requesting review again since I changed the approach to reconnect the EventSource when restoring from page cache.
Comment on attachment 380268 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=380268&action=review r=me > Source/WebCore/page/EventSource.cpp:429 > + scheduleReconnect(); Shiny! :P
Comment on attachment 380268 [details] Patch Clearing flags on attachment: 380268 Committed r250761: <https://trac.webkit.org/changeset/250761>
All reviewed patches have been landed. Closing bug.