Cache API should reject 206 Partial Responses

third_party/WebKit/LayoutTests/http/tests/cachestorage/script-tests/cache-add.js

Index: third_party/WebKit/LayoutTests/http/tests/cachestorage/script-tests/cache-add.js
diff --git a/third_party/WebKit/LayoutTests/http/tests/cachestorage/script-tests/cache-add.js b/third_party/WebKit/LayoutTests/http/tests/cachestorage/script-tests/cache-add.js
index c3fa2d0d1e4446a9cdf1e8e0af265aceca095cb7..3446a2361a2202101d2e6a1ac7ed5bf1a086b8d0 100644
--- a/third_party/WebKit/LayoutTests/http/tests/cachestorage/script-tests/cache-add.js
+++ b/third_party/WebKit/LayoutTests/http/tests/cachestorage/script-tests/cache-add.js
@@ -103,6 +103,22 @@ cache_test(function(cache, test) {
     return promise_rejects(
       test,
       new TypeError(),
+      cache.add('../resources/fetch-status.php?status=206'),
+      'Cache.add should reject on partial response');
+  }, 'Cache.add with request that results in a status of 206');
+
+cache_test(function(cache, test) {
+    return promise_rejects(
+      test,
+      new TypeError(),
+      cache.addAll(['../resources/fetch-status.php?status=206']),

[jkarlin, 2016/08/02 13:47:51]

Can you put two requests in the addAll, where the first isn't a 206? We should
make sure the whole operation fails.

[jsbell, 2016/08/02 19:54:10]

Good idea - done!

(I didn't verify that the cache is untouched on failure, since that's covered by
other tests. I don't think every permutation is necessary, but can be persuaded
otherwise.)

+      'Cache.addAll should reject on partial response');
+  }, 'Cache.addAll with request that results in a status of 206');
+
+cache_test(function(cache, test) {
+    return promise_rejects(
+      test,
+      new TypeError(),
       cache.addAll(),
       'Cache.addAll with no arguments should throw TypeError.');
   }, 'Cache.addAll with no arguments');