fix some jank

src/client/dom/postmessage.ts

@@ -11,7 +11,22 @@ export default function (client: ScramjetClient) {
 			// if we were given any object that came from the real realm we can use that to get the real origin
 			// and this works in every case EXCEPT for the fact that all three arguments can be strings which are copied instead of cloned
 			// so we have to use `$setrealm` which will pollute this with an object from the real realm
-			const pollutant = ctx.this[POLLUTANT] || {};
+
+			let pollutant;
+
+			if (typeof ctx.args[0] === "object" && ctx.args[0] !== null) {
+				pollutant = ctx.args[0]; // try to use the first object we can find because it's more reliable
+			} else if (typeof ctx.args[2] === "object" && ctx.args[2] !== null) {
+				pollutant = ctx.args[2]; // next try to use transfer
+			} else if (
+				POLLUTANT in ctx.this &&
+				typeof ctx.this[POLLUTANT] === "object" &&
+				ctx.this[POLLUTANT] !== null
+			) {
+				pollutant = ctx.this[POLLUTANT]; // lastly try to use the object from $setrealm
+			} else {
+				pollutant = {}; // give up
+			}
 
 			// and now we can steal Function from the caller's realm
 			const {

src/client/shared/requests/beacon.ts

@@ -1,8 +1,10 @@
-export default function (client, self) {
-	// goodybye spyware~
+import { encodeUrl } from "../../../shared/rewriters/url";
+import { ScramjetClient } from "../../client";
+
+export default function (client: ScramjetClient, self) {
 	client.Proxy("navigator.sendBeacon", {
 		apply(ctx) {
-			ctx.return(true);
+			ctx.args[0] = encodeUrl(ctx.args[0], client.meta);
 		},
 	});
 }

src/client/shared/requests/xmlhttprequest.ts

@@ -1,8 +1,9 @@
 import { config, decodeUrl, encodeUrl, rewriteHeaders } from "../../../shared";
 import { ScramjetClient } from "../../client";
+const nativeworker = Worker;
 
 export default function (client: ScramjetClient, self: Self) {
-	const worker = new Worker(config.sync);
+	const worker = new nativeworker(config.sync);
 	const ARGS = Symbol("xhr original args");
 	const HEADERS = Symbol("xhr headers");
 

src/worker/fetch.ts

@@ -218,7 +218,17 @@ async function handleResponse(
 	// scramjet runtime can use features that permissions-policy blocks
 	delete responseHeaders["permissions-policy"];
 
-	if (crossOriginIsolated) {
+	if (
+		crossOriginIsolated &&
+		[
+			"document",
+			"iframe",
+			"worker",
+			"sharedworker",
+			"style",
+			"script",
+		].includes(destination)
+	) {
 		responseHeaders["Cross-Origin-Embedder-Policy"] = "require-corp";
 		responseHeaders["Cross-Origin-Opener-Policy"] = "same-origin";
 	}

src/worker/index.ts

@@ -151,10 +151,17 @@ export class ScramjetServiceWorker {
 				base: new URL(origin),
 			});
 
-			return new Response(rewritten, {
-				headers: {
+			const headers = {
 				"Content-Type": "application/javascript",
-				},
+			};
+
+			if (crossOriginIsolated) {
+				headers["Cross-Origin-Opener-Policy"] = "same-origin";
+				headers["Cross-Origin-Embedder-Policy"] = "require-corp";
+			}
+
+			return new Response(rewritten, {
+				headers,
 			});
 		}