1722805 - browser_storage_cache_navigation.js fails with server targets

Status: RESOLVED FIXED

(DevTools :: Storage Inspector, defect)

Description

[Alexandre Poirot [:ochameau]]

Fails with the following timeout:

 0:06.77 PASS Table row has an entry for: http://example.com/browser/devtools/client/storage/test/storage-blank.html with status: OK - 
 0:06.77 INFO Cleaning up cache…
 0:06.78 INFO Navigating to "http://example.net/document-builder.sjs?html=%3Ch1%3Eexample.net%3C/h1%3E%3Cscript%3E%0A%20%20%20%20%20%20%20%20caches.open(%22foo%22).then(cache%20=%3E%20%7B%0A%20%20%20%20%20%20%20%20%20%20cache.add(%22http://example.net/browser/devtools/client/storage/test/storage-blank.html%22);%0A%20%20%20%20%20%20%20%20%7D);%0A%20%20%20%20%20%20%20%20function%20clear()%20%7B%0A%20%20%20%20%20%20%20%20%20%20caches.delete(%22foo%22);%0A%20%20%20%20%20%20%20%20%7D%0A%20%20%20%20%20%20%3C/script%3E"
 0:06.78 INFO Waiting for page to be loaded…
 0:06.92 INFO → page loaded
 0:06.92 INFO Waiting for target switch…
 0:06.93 INFO Console message: [JavaScript Error: "The character encoding of the HTML document was not declared. The document will render with garbled text in some browser configurations if the document contains characters from outside the US-ASCII range. The character encoding of the page must be declared in the document or in the transfer protocol." {file: "http://example.net/document-builder.sjs?html=%3Ch1%3Eexample.net%3C/h1%3E%3Cscript%3E%0A%20%20%20%20%20%20%20%20caches.open(%22foo%22).then(cache%20=%3E%20%7B%0A%20%20%20%20%20%20%20%20%20%20cache.add(%22http://example.net/browser/devtools/client/storage/test/storage-blank.html%22);%0A%20%20%20%20%20%20%20%20%7D);%0A%20%20%20%20%20%20%20%20function%20clear()%20%7B%0A%20%20%20%20%20%20%20%20%20%20caches.delete(%22foo%22);%0A%20%20%20%20%20%20%20%20%7D%0A%20%20%20%20%20%20%3C/script%3E" line: 0}]
 0:06.99 INFO → switched-target emitted
 0:06.99 INFO Waiting for storage tree to update…

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

It looks like we're hitting Bug 1712757 here: with server side target switching the storage actor gets created before the cache of the test page is created. When server side target switching isn't enabled (but Fission is), the creation of the storage actor happens later which buy us some time for the cache to be created.

Comment 2

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1722805 - [devtools] Explicitely disable server side target switching in browser_storage_cache_navigation. r=jdescottes.

The test times out with server side target switching as the
storage actor gets created earlier than the cache and the
storage inspector doesn't update when CacheStorage is
updated (see Bug 1712757).

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/45aec0134284
[devtools] Explicitely disable server side target switching in browser_storage_cache_navigation. r=jdescottes.

Comment 4

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/45aec0134284