Changes between Initial Version and Version 1 of Ticket #6595, comment 5


Ignore:
Timestamp:
06/14/2018 06:02:44 PM (14 months ago)
Author:
sebastian
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #6595, comment 5

    initial v1  
    1 The problem appears to be that we update the document structure on `onComitted` here, which is triggered when the browser starts showing the new document, but some requests might have already been sent at this point. For documents that are served right from the web we work around that issue by updating the document structure already on `onHeadersReceived`, however this event isn't triggered for documents served by Service Workers. Otherwise, the change here wouldn't have been necessary in the first place. It seems this edge case is a limitation we cannot work around. 
     1The problem appears to be that we update the document structure on `onCommitted` here, which is triggered when the browser starts showing the new document, but some requests might have already been sent at this point. For documents that are served right from the web we work around that issue by updating the document structure already on `onHeadersReceived`, however this event isn't triggered for documents served by Service Workers. Otherwise, the change here wouldn't have been necessary in the first place. It seems this edge case is a limitation we cannot work around.