browser: split page start from page navigate

src/cdp/page.zig

@@ -331,8 +331,9 @@ fn navigate(
     // TODO: noop event, we have no env context at this point, is it necesarry?
     try sendEvent(alloc, ctx, "Runtime.executionContextsCleared", void, {}, input.sessionId);
 
-    // Launch navigate
-    const p = try ctx.browser.session.createPage();
+    // Launch navigate, the page must have been created by a
+    // target.createTarget.
+    var p = ctx.browser.session.page orelse return error.NoPage;
     ctx.state.executionContextId += 1;
     const auxData = try std.fmt.allocPrint(
         alloc,

src/cdp/target.zig

@@ -344,6 +344,14 @@ fn createTarget(
     ctx.state.loaderID = LoaderID;
     ctx.state.sessionID = msg.sessionId;
 
+    // TODO stop the previous page instead?
+    if (ctx.browser.session.page != null) return error.pageAlreadyExists;
+
+    // create the page
+    const p = try ctx.browser.session.createPage();
+    // start the js env
+    try p.start();
+
     // send targetCreated event
     const created = TargetCreated{
         .sessionId = cdp.ContextSessionID,
@@ -440,6 +448,8 @@ fn closeTarget(
         null,
     );
 
+    if (ctx.browser.session.page != null) ctx.browser.session.page.?.end();
+
     return "";
 }