Closed
Bug 943277
Opened 11 years ago
Closed 11 years ago
Add test for disable cache option in option panel
Categories
(DevTools :: Framework, defect)
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: miker, Assigned: miker)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
Details | Diff | Splinter Review |
Due to difficulties in testing whether pages are cached we split this bug off from bug 864098
Assignee | ||
Comment 1•11 years ago
|
||
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
Assignee | ||
Comment 2•11 years ago
|
||
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 | ||
Updated•11 years ago
|
Assignee: nobody → mratcliffe
Assignee | ||
Comment 3•11 years ago
|
||
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)
Assignee | ||
Comment 4•11 years ago
|
||
Folding into patch for bug 864098
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → INVALID
Updated•10 years ago
|
Attachment #8338472 -
Flags: review?(paul)
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•