1519855 - The server should support the BFCache

Status: RESOLVED FIXED

(DevTools :: Debugger, enhancement, P2)

Description

[Jason Laster [:jlast]]

The client has some fragile logic to support the BFCache on navigation:

1. wait 100ms
2. fetch sources, if it has not received any sources yet

This fails for heavy sites or slow computers that load sites in > 100ms. It is also an implementation detail the client should not have to know about.

The server should support the BFCache case and always stream sources down to the client.

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1519855 - Try sending all existing source when navigating to bfcached document.

Comment 2

[Jason Laster [:jlast]]

removing dbg-69 as we can work on this when we focus on dbg-source-requests

Comment 3

[Brian Hackett [Laid off!]]

This logic in the client is causing the last failure on try that is preventing bug 1538056 from landing. The problem is that when a page reloads, there is a window of time between when the client has been notified of the reload and when the server's new window finishes being created, at which point the server resets its sources information. If the client sends requests to the server in this time period, it gets info about the old sources but associates it with the new page. This is an existing problem but seems asymptomatic without bug 1538056. With bug 1538056, the sources information include HTML file contents, and the client can get stale data from those sources and show it to the user after reloading.

Fixing the server to handle the BFCache and removing this odd logic from the client seems like the best way to fix this. In particular, if the BFCache is being used and the client sends a request to get the sources before the window is ready, the client won't get the new sources from the server when they appear later on.

Comment 4

[Brian Hackett [Laid off!]]

Attachment: Bug 1519855 - Notify client about sources after BFCache navigations, r=jdescottes.

Comment 5

[Brian Hackett [Laid off!]]

Attachment: Bug 1519855 - Notify client about sources after BFCache navigations, r=jdescottes.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f76e143931db
Notify client about sources after BFCache navigations, r=jdescottes.

Comment 7

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/f76e143931db