Report an exception in certain error cases for <script type="webbundle">

myrzakereyms · chromium-wpt-export-bot · commit ec9d2073bc0e · 2022-03-17T13:17:59.000-07:00
When creating a WebBundle using <script type="webbundle"> API, we can encounter cases when parsing of JSON rule fails, or a ResourceFetcher is not present for the script element's document; in both cases we want to report an exception instead of throwing an error event. Bug: 1263783 Change-Id: I8d32da13b2fde5d9566b373efdb4f4bde50150de Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3283909 Reviewed-by: Yutaka Hirano <yhirano@chromium.org> Reviewed-by: Kunihiko Sakamoto <ksakamoto@chromium.org> Reviewed-by: Hayato Ito <hayato@chromium.org> Commit-Queue: Miras Myrzakerey <myrzakereyms@google.com> Cr-Commit-Position: refs/heads/main@{#982352}
diff --git a/web-bundle/subresource-loading/script-invalid-json.https.tentative.html b/web-bundle/subresource-loading/script-invalid-json.https.tentative.html
@@ -0,0 +1,33 @@
+<!DOCTYPE html>
+<title>WebBundle subresource loading with script API and invalid JSON</title>
+<link
+  rel="help"
+  href="https://github.com/WICG/webpackage/blob/main/explainers/subresource-loading.md"
+/>
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<body>
+  <script>
+    setup(() => {
+      assert_true(HTMLScriptElement.supports('webbundle'));
+    }, {allow_uncaught_exception: true});
+  </script>
+  <script>
+    promise_test((t) => {
+      const script = document.createElement("script");
+      script.type = "webbundle";
+      const json_rule  = {"resources": []};
+      script.textContent = JSON.stringify(json_rule);
+      return new Promise(((resolve, reject) => {
+          script.onload = () => reject(script);
+          script.onerror = () => reject(script);
+          window.onerror = function (message, url, line, col, error) {
+            assert_equals(message, "Uncaught SyntaxError: Failed to parse web bundle: invalid JSON");
+            assert_equals(error.name, "SyntaxError");
+            resolve(script);
+          }
+          document.body.appendChild(script);
+      }));
+    }, "Invalid JSON rule should make the script loader report an exception and throw error on the window.");
+  </script>
+</body>
