[chrome] add a few markdirtys

Author: velzie

packages/chrome/src/History.ts

@@ -3,6 +3,7 @@ import { browser } from "./Browser";
 import { StatefulClass } from "./StatefulClass";
 import type { Tab } from "./Tab";
 import { sendFrame } from "./proxy/ipc";
+import { markDirty } from "./storage";
 
 // history api emulation
 export class HistoryState extends StatefulClass {
@@ -105,6 +106,7 @@ export class History {
 		this.tab.canGoBack = this.canGoBack();
 		this.tab.canGoForward = this.canGoForward();
 
+		markDirty();
 		return this.states[this.index];
 	}
 	replace(
@@ -130,6 +132,7 @@ export class History {
 		this.tab.canGoBack = this.canGoBack();
 		this.tab.canGoForward = this.canGoForward();
 
+		markDirty();
 		return this.states[this.index];
 	}
 	go(delta: number, navigate: boolean = true): HistoryState {
@@ -164,6 +167,7 @@ export class History {
 		this.tab.canGoBack = this.canGoBack();
 		this.tab.canGoForward = this.canGoForward();
 
+		markDirty();
 		return newstate;
 	}
 	canGoBack(): boolean {

packages/chrome/src/components/TabStrip/TabStrip.tsx

@@ -15,6 +15,7 @@ import { setContextMenu } from "../Menu";
 import { browser, forceScreenshot, pushTab } from "../../Browser";
 import { defaultFaviconUrl } from "../../assets/favicon";
 import { DragTab } from "./DragTab";
+import { markDirty } from "../../storage";
 
 type VisualTab = {
 	tab: Tab;
@@ -186,6 +187,7 @@ export function TabStrip(
 
 		if (s.activetab != tab.tab) {
 			s.activetab = tab.tab;
+			markDirty();
 		}
 	};
 

packages/chrome/src/storage.ts

@@ -30,7 +30,13 @@ setInterval(() => {
 		saveBrowserState();
 		dirty = false;
 	}
-}, 5000);
+}, 3000);
+
+// just in case
+setInterval(() => {
+	saveBrowserState();
+	dirty = false;
+}, 10000);
 
 window.addEventListener("beforeunload", (e) => {
 	if (dirty) {

packages/sandbox/controller.html

@@ -1,33 +1,13 @@
 <title>Proxy Sandbox Controller</title>
 <script>
 	function wake(controller) {
-		console.log("wakling");
 		window.parent.postMessage({ $sandboxsw$type: "ready" }, "*");
 	}
 	navigator.serviceWorker
 		.register("/sw.js", {
 			scope: "./",
 		})
 		.then((registration) => {
-			console.log("Service Worker registered with scope:", registration.scope);
-			console.log("Registration installing:", registration.installing);
-			console.log("Registration waiting:", registration.waiting);
-			console.log("Registration active:", registration.active);
-
-			// Monitor the registration for changes
-			registration.addEventListener("updatefound", () => {
-				console.log("SW update found!");
-				const newWorker = registration.installing;
-				console.log("New worker state:", newWorker?.state);
-
-				newWorker?.addEventListener("statechange", () => {
-					console.log("SW state changed to:", newWorker.state);
-				});
-			});
-
-			// Check current controller
-			console.log("Current controller:", navigator.serviceWorker.controller);
-
 			if (navigator.serviceWorker.controller) {
 				// there's probably already a service worker
 				wake(navigator.serviceWorker.controller);

packages/sandbox/sw.js

@@ -15,7 +15,3 @@ self.addEventListener("activate", (e) => {
 });
 
 console.log("sw initialized");
-
-addEventListener("message", (e) => {
-	console.log(e);
-});