Bail out from CallbackPromiseAdapter::OnErrorAdapter::OnError if context is not valid

caseq · chromium-wpt-export-bot · commit 9331d00053da · 2025-02-27T15:16:47.000-08:00
This fixes a DCHECK() in case this is called in a detached context. Bug: 398910931 Change-Id: Icf2a9b2e51facaa691e470b3af775a2d4253c6fc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6307064 Reviewed-by: Nate Chapin <japhet@chromium.org> Commit-Queue: Andrey Kosyakov <caseq@chromium.org> Cr-Commit-Position: refs/heads/main@{#1425987}
diff --git a/service-workers/service-worker/detached-register-crash.https.html b/service-workers/service-worker/detached-register-crash.https.html
@@ -0,0 +1,14 @@
+<!DOCTYPE html>
+<meta charset="utf-8">
+<title>Assures navigator.serviceWorker.register() doesn't crash when rejecting being called in a detached frame</title>
+<link rel="author" href="mailto:caseq@chromium.org">
+<link rel="help" href="http://crbug.com/398910931">
+<body>
+<iframe></iframe>
+<script>
+const frame = document.getElementsByTagName("iframe")[0];
+const container = frame.contentWindow.navigator.serviceWorker;
+frame.remove();
+container.register("data:,");
+</script>
+</body>
