XMLHttpRequest should not prevent entering the back/forward cache.
Created attachment 381245 [details] Patch
This is blocked on Bug 203106.
Created attachment 381378 [details] Patch
<rdar://problem/56438647>
Created attachment 381388 [details] Patch
Comment on attachment 381388 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=381388&action=review > Source/WebCore/ChangeLog:10 > + 1. We no longer cancel pending loads in the suspend() method as this may This means we will continue loads even if the page navigates away. If this is a large download or a load a la EventSource, we might spend some network resources for a suspended page. Is this ok? Do you know what other browsers are doing?
(In reply to youenn fablet from comment #6) > Comment on attachment 381388 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=381388&action=review > > > Source/WebCore/ChangeLog:10 > > + 1. We no longer cancel pending loads in the suspend() method as this may > > This means we will continue loads even if the page navigates away. > If this is a large download or a load a la EventSource, we might spend some > network resources for a suspended page. > Is this ok? Do you know what other browsers are doing? No, all loads are cancelled when entering the page cache.
Comment on attachment 381388 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=381388&action=review >>> Source/WebCore/ChangeLog:10 >>> + 1. We no longer cancel pending loads in the suspend() method as this may >> >> This means we will continue loads even if the page navigates away. >> If this is a large download or a load a la EventSource, we might spend some network resources for a suspended page. >> Is this ok? Do you know what other browsers are doing? > > No, all loads are cancelled when entering the page cache. We just don’t want to do it in the suspend() as it may fire events and we are not allowed to run script from inside suspend().
Comment on attachment 381388 [details] Patch Clearing flags on attachment: 381388 Committed r251366: <https://trac.webkit.org/changeset/251366>
All reviewed patches have been landed. Closing bug.