handle no params

Author: sjorsdonkers

src/cdp/domains/network.zig

@@ -132,6 +132,7 @@ fn deleteCookies(cmd: anytype) !void {
 }
 
 fn clearBrowserCookies(cmd: anytype) !void {
+    if (try cmd.params(struct {}) != null) return error.InvalidParams;
     const bc = cmd.browser_context orelse return error.BrowserContextNotLoaded;
     bc.session.cookie_jar.clearRetainingCapacity();
     return cmd.sendResult(null, .{});
@@ -161,11 +162,10 @@ fn setCookies(cmd: anytype) !void {
     try cmd.sendResult(null, .{});
 }
 
+const getCookiesParam = struct { urls: ?[]const []const u8 = null };
 fn getCookies(cmd: anytype) !void {
     const bc = cmd.browser_context orelse return error.BrowserContextNotLoaded;
-    const params = (try cmd.params(struct {
-        urls: ?[]const []const u8 = null,
-    })) orelse return error.InvalidParams;
+    const params = (try cmd.params(getCookiesParam)) orelse getCookiesParam{};
 
     // If not specified, use the URLs of the page and all of its subframes. TODO subframes
     const page_url = if (bc.session.page) |*page| page.url.raw else null; // @speed: avoid repasing the URL

src/cdp/domains/storage.zig

@@ -38,11 +38,11 @@ pub fn processMessage(cmd: anytype) !void {
     }
 }
 
+const BrowserContextParam = struct { browserContextId: ?[]const u8 = null };
+
 fn clearCookies(cmd: anytype) !void {
     const bc = cmd.browser_context orelse return error.BrowserContextNotLoaded;
-    const params = (try cmd.params(struct {
-        browserContextId: ?[]const u8 = null,
-    })) orelse return error.InvalidParams;
+    const params = (try cmd.params(BrowserContextParam)) orelse BrowserContextParam{};
 
     if (params.browserContextId) |browser_context_id| {
         if (std.mem.eql(u8, browser_context_id, bc.id) == false) {
@@ -57,9 +57,7 @@ fn clearCookies(cmd: anytype) !void {
 
 fn getCookies(cmd: anytype) !void {
     const bc = cmd.browser_context orelse return error.BrowserContextNotLoaded;
-    const params = (try cmd.params(struct {
-        browserContextId: ?[]const u8 = null,
-    })) orelse return error.InvalidParams;
+    const params = (try cmd.params(BrowserContextParam)) orelse BrowserContextParam{};
 
     if (params.browserContextId) |browser_context_id| {
         if (std.mem.eql(u8, browser_context_id, bc.id) == false) {