Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(494)

Unified Diff: LayoutTests/http/tests/cachestorage/script-tests/cache-put.js

Issue 1295633003: Cache Storage: replace assert_object_equals w/ assert_response_equals (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Review feedback and rebased Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: LayoutTests/http/tests/cachestorage/script-tests/cache-put.js
diff --git a/LayoutTests/http/tests/cachestorage/script-tests/cache-put.js b/LayoutTests/http/tests/cachestorage/script-tests/cache-put.js
index b6e9275465c68f06710eee8578b9c09a27ad4b46..6389b60fd6ef15fe49200ebfa66546a35b34d5e1 100644
--- a/LayoutTests/http/tests/cachestorage/script-tests/cache-put.js
+++ b/LayoutTests/http/tests/cachestorage/script-tests/cache-put.js
@@ -30,9 +30,9 @@ cache_test(function(cache) {
return cache.match(test_url);
})
.then(function(result) {
- assert_object_equals_fixed(result, response,
- 'Cache.put should update the cache with ' +
- 'new request and response.');
+ assert_response_equals(result, response,
+ 'Cache.put should update the cache with ' +
+ 'new request and response.');
return result.text();
})
.then(function(body) {
@@ -75,9 +75,9 @@ cache_test(function(cache) {
return cache.match(test_url);
})
.then(function(result) {
- assert_object_equals_fixed(result, response,
- 'Cache.put should update the cache with ' +
- 'new Request and Response.');
+ assert_response_equals(result, response,
+ 'Cache.put should update the cache with ' +
+ 'new Request and Response.');
});
}, 'Cache.put with a Response containing an empty URL');
@@ -118,9 +118,9 @@ cache_test(function(cache) {
return cache.match(test_url);
})
.then(function(result) {
- assert_object_equals_fixed(result, response,
- 'Cache.put should update the cache with ' +
- 'new request and response.');
+ assert_response_equals(result, response,
+ 'Cache.put should update the cache with ' +
+ 'new request and response.');
return result.text();
})
.then(function(body) {
@@ -142,9 +142,9 @@ cache_test(function(cache) {
return cache.match(test_url);
})
.then(function(result) {
- assert_object_equals_fixed(result, alternate_response,
- 'Cache.put should replace existing ' +
- 'response with new response.');
+ assert_response_equals(result, alternate_response,
+ 'Cache.put should replace existing ' +
+ 'response with new response.');
return result.text();
})
.then(function(body) {
@@ -168,9 +168,9 @@ cache_test(function(cache) {
return cache.match(test_url);
})
.then(function(result) {
- assert_object_equals_fixed(result, alternate_response,
- 'Cache.put should replace existing ' +
- 'response with new response.');
+ assert_response_equals(result, alternate_response,
+ 'Cache.put should replace existing ' +
+ 'response with new response.');
return result.text();
})
.then(function(body) {
@@ -248,9 +248,9 @@ cache_test(function(cache) {
return cache.match(new URL('relative-url', location.href).href);
})
.then(function(result) {
- assert_object_equals_fixed(result, response,
- 'Cache.put should accept a relative URL ' +
- 'as the request.');
+ assert_response_equals(result, response,
+ 'Cache.put should accept a relative URL ' +
+ 'as the request.');
});
}, 'Cache.put with a relative URL');

Powered by Google App Engine
This is Rietveld 408576698