Skip to content

/ servo

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement a basic HTTP memory cache #4117

Closed
wants to merge 15 commits into from
+941 −57
Closed

Implement a basic HTTP memory cache #4117

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
3c0fb5a
Add a non-evicting, non-validating memory cache that keys on original…
jdm Nov 12, 2014
197dc35
Doom cache entries based on the initial response, and prevent matchin…
jdm Nov 13, 2014
5d95607
Evict cache entries that have passed their expiry date instead of mat…
jdm Nov 13, 2014
9815deb
Document the cache. Refactor incomplete entries to lessen Option-itis.
jdm Nov 14, 2014
1e70aa8
Revalidate expired cache entries instead of unconditionally evicting …
jdm Nov 15, 2014
0a314d0
Forbid missing docs in cache code.
jdm Nov 18, 2014
2208da3
Revalidate must-revalidate entries.
jdm Nov 18, 2014
a95d21b
Fetch content tests from a local HTTP server.
jdm Nov 18, 2014
195d5f1
Track requests made to the test HTTP server.
jdm Nov 18, 2014
09619f1
Add a simple test that a cached resource with no expiry is not revali…
jdm Nov 18, 2014
bcb6183
Fix incorrect revalidation logic that dropped the consumer channels o…
jdm Nov 18, 2014
8c602bc
Ensure that requests are cached based on their request headers.
jdm Nov 18, 2014
82a1a99
Run a separate http server instance for each test to avoid intermitte…
jdm Nov 27, 2014
4ea0e05
Add a test for uncacheable responses.
jdm Nov 27, 2014
f3074f5
Address review comments.
jdm Dec 16, 2014
File filter...
Filter file types
Clear filters
Jump to…
Jump to file
Failed to load files.

Always

Just for now

Prev Next

Ensure that requests are cached based on their request headers.

  • Loading branch information
@jdm
jdm committed Nov 27, 2014
commit 8c602bc0c33a991016f8eda4c20a5a1dfc95df48
7 tests/content/netharness.js
View file
Open in desktop
@@ -12,9 +12,14 @@ function reset_stats() {
is(x.status, 200, 'resetting stats should succeed');
}

function fetch(url) {
function fetch(url, headers) {
var x = new XMLHttpRequest();
x.open('GET', url, false);
if (headers) {
for (var i = 0; i < headers.length; i++) {
x.setRequestHeader(headers[i][0], headers[i][1]);
}
}
x.send();
is(x.status, 200, 'fetching ' + url + ' should succeed ');
}
14 tests/content/test_cached_headers_differ.html
View file
Open in desktop
@@ -0,0 +1,14 @@
<html>
<head>
<script src="harness.js"></script>
<script src="netharness.js"></script>
</head>
<body>
<script>
reset_stats();
fetch('resources/helper.html');
fetch('resources/helper.html', [['X-User', 'foo']]);
assert_requests_made('resources/helper.html', 2);
</script>
</body>
</html>
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.