Closed Bug 943277 Opened 11 years ago Closed 11 years ago

Add test for disable cache option in option panel

Categories

(DevTools :: Framework, defect)

27 Branch
x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: miker, Assigned: miker)

References

Details

Attachments

(1 file, 1 obsolete file)

Due to difficulties in testing whether pages are cached we split this bug off from bug 864098
Attached patch The start of a test - WIP (obsolete) (deleted) — Splinter Review
The starts of a test are here but we think that the pageShow event is returning the wrong information in it's "persistent" property. Maybe we are best to use a .sjs file as we can force caching and read a unique value to check caching.
Assignee: nobody → mratcliffe
Status: NEW → ASSIGNED
It is simple to use Etags etc to cache the page but detecting whether the page is cached is a completely different kettle of fish. This is made even more difficult because the cache toggling works for the current tab so any cache checks must be in the context of the current tab.
Assignee: mratcliffe → nobody
Assignee: nobody → mratcliffe
Attached patch test-disable-cache-943277.patch (deleted) — Splinter Review
Using a combination of Etags and content checking works. This means that using a ^headers^ file doesn't work in order to make the browser cache test resource files. Try: https://tbpl.mozilla.org/?tree=Try&rev=283021e0dd56
Attachment #8338399 - Attachment is obsolete: true
Attachment #8338472 - Flags: review?(paul)
Folding into patch for bug 864098
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → INVALID
Attachment #8338472 - Flags: review?(paul)
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: