String: refactor isAllWhitespace into String

src/SemanticTree.zig

@@ -20,6 +20,7 @@ const std = @import("std");
 
 const lp = @import("lightpanda");
 const log = @import("log.zig");
+const isAllWhitespace = @import("string.zig").isAllWhitespace;
 const Page = lp.Page;
 
 const CData = @import("browser/webapi/CData.zig");
@@ -28,7 +29,7 @@ const Node = @import("browser/webapi/Node.zig");
 const AXNode = @import("cdp/AXNode.zig");
 const CDPNode = @import("cdp/Node.zig");
 
-const SemanticTree = @This();
+const Self = @This();
 
 dom_node: *Node,
 registry: *CDPNode.Registry,
@@ -42,13 +43,6 @@ pub fn jsonStringify(self: @This(), jw: *std.json.Stringify) error{WriteFailed}!
     };
 }
 
-fn isAllWhitespace(_: @This(), text: []const u8) bool {
-    for (text) |c| {
-        if (!std.ascii.isWhitespace(c)) return false;
-    }
-    return true;
-}
-
 fn getXPathSegment(self: @This(), node: *Node) ![]const u8 {
     if (node.is(Element)) |el| {
         const tag = el.getTagNameLower();
@@ -82,7 +76,7 @@ fn getXPathSegment(self: @This(), node: *Node) ![]const u8 {
     return "";
 }
 
-fn dumpNode(self: @This(), node: *Node, jw: *std.json.Stringify, parent_xpath: []const u8) !void {
+fn dumpNode(self: Self, node: *Node, jw: *std.json.Stringify, parent_xpath: []const u8) !void {
     // 1. Skip non-content nodes
     if (node.is(Element)) |el| {
         const tag = el.getTagNameLower();
@@ -113,7 +107,7 @@ fn dumpNode(self: @This(), node: *Node, jw: *std.json.Stringify, parent_xpath: [
     } else if (node.is(CData.Text) != null) {
         const text_node = node.is(CData.Text).?;
         const text = text_node.getWholeText();
-        if (self.isAllWhitespace(text)) {
+        if (isAllWhitespace(text)) {
             return;
         }
     } else if (node._type != .document and node._type != .document_fragment) {

src/browser/markdown.zig

@@ -24,6 +24,7 @@ const TreeWalker = @import("webapi/TreeWalker.zig");
 const CData = @import("webapi/CData.zig");
 const Element = @import("webapi/Element.zig");
 const Node = @import("webapi/Node.zig");
+const isAllWhitespace = @import("../string.zig").isAllWhitespace;
 
 pub const Opts = struct {
     // Options for future customization (e.g., dialect)
@@ -109,12 +110,6 @@ fn getAnchorLabel(el: *Element) ?[]const u8 {
     return el.getAttributeSafe(comptime .wrap("aria-label")) orelse el.getAttributeSafe(comptime .wrap("title"));
 }
 
-fn isAllWhitespace(text: []const u8) bool {
-    return for (text) |c| {
-        if (!std.ascii.isWhitespace(c)) break false;
-    } else true;
-}
-
 fn hasBlockDescendant(root: *Node) bool {
     var tw = TreeWalker.FullExcludeSelf.Elements.init(root, .{});
     while (tw.next()) |el| {

src/string.zig

@@ -305,6 +305,12 @@ pub const String = packed struct {
     }
 };
 
+pub fn isAllWhitespace(text: []const u8) bool {
+    return for (text) |c| {
+        if (!std.ascii.isWhitespace(c)) break false;
+    } else true;
+}
+
 // Discriminatory type that signals the bridge to use arena instead of call_arena
 // Use this for strings that need to persist beyond the current call
 // The caller can unwrap and store just the underlying .str field