From 9acb1890b4a7c4350b962770e38ee13f8827fde0 Mon Sep 17 00:00:00 2001
From: Pierre Tachoire <pierre@tch.re>
Date: Fri, 27 Oct 2023 18:11:08 +0200
Subject: [PATCH 1/3] netsurf: add elementGetAttribute func

---
 src/netsurf.zig | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/src/netsurf.zig b/src/netsurf.zig
index 5e368d98..5f246d98 100644
--- a/src/netsurf.zig
+++ b/src/netsurf.zig
@@ -609,6 +609,15 @@ pub fn elementLocalName(elem: *Element) []const u8 {
     return nodeLocalName(node);
 }
 
+pub fn elementGetAttribute(elem: *Element, name: []const u8) ?[]const u8 {
+    var s: ?*String = undefined;
+    _ = elementVtable(elem).dom_element_get_attribute.?(elem, stringFromData(name), &s);
+    if (s == null) {
+        return null;
+    }
+    return stringToData(s.?);
+}
+
 // ElementHTML
 pub const ElementHTML = c.dom_html_element;
 

From 89431971ac99d3ba19c2833d7e6a5beb775298fd Mon Sep 17 00:00:00 2001
From: Pierre Tachoire <pierre@tch.re>
Date: Fri, 27 Oct 2023 19:00:26 +0200
Subject: [PATCH 2/3] wpt: load script soruce file dynamically

---
 src/main_wpt.zig | 31 +++++++++++++++++++------------
 1 file changed, 19 insertions(+), 12 deletions(-)

diff --git a/src/main_wpt.zig b/src/main_wpt.zig
index fe75630a..c389a077 100644
--- a/src/main_wpt.zig
+++ b/src/main_wpt.zig
@@ -82,7 +82,7 @@ pub fn main() !void {
     defer vm.deinit();
 
     // prepare libraries to load on each test case.
-    var loader = FileLoader.init(alloc, "tests/wpt");
+    var loader = FileLoader.init(alloc, wpt_dir);
     defer loader.deinit();
 
     // browse the dir to get the tests dynamically.
@@ -160,6 +160,8 @@ fn runWPT(arena: *std.heap.ArenaAllocator, comptime apis: []jsruntime.API, f: []
     const html_doc = try parser.documentHTMLParseFromFileAlloc(alloc, f);
     const doc = parser.documentHTMLToDocument(html_doc);
 
+    const dirname = std.fs.path.dirname(f[wpt_dir.len..]) orelse unreachable;
+
     // create JS env
     var loop = try Loop.init(alloc);
     defer loop.deinit();
@@ -207,23 +209,28 @@ fn runWPT(arena: *std.heap.ArenaAllocator, comptime apis: []jsruntime.API, f: []
         return res;
     }
 
-    // TODO load <script src> attributes instead of the static list.
-    res = try evalJS(js_env, alloc, try loader.get("/resources/testharness.js"), "testharness.js");
-    if (!res.success) {
-        return res;
-    }
-    res = try evalJS(js_env, alloc, try loader.get("/resources/testharnessreport.js"), "testharnessreport.js");
-    if (!res.success) {
-        return res;
-    }
-
     // loop hover the scripts.
     const scripts = parser.documentGetElementsByTagName(doc, "script");
     const slen = parser.nodeListLength(scripts);
     for (0..slen) |i| {
         const s = parser.nodeListItem(scripts, @intCast(i)).?;
 
-        const src = parser.nodeTextContent(s).?;
+        // If the script contains an src attribute, load it.
+        if (parser.elementGetAttribute(@as(*parser.Element, @ptrCast(s)), "src")) |src| {
+            var path = src;
+            if (!std.mem.startsWith(u8, src, "/")) {
+                // no need to free path, thanks to the arena.
+                path = try std.fs.path.join(alloc, &.{ "/", dirname, path });
+            }
+
+            res = try evalJS(js_env, alloc, try loader.get(path), src);
+            if (!res.success) {
+                return res;
+            }
+        }
+
+        // If the script as a source text, execute it.
+        const src = parser.nodeTextContent(s) orelse continue;
         res = try evalJS(js_env, alloc, src, "");
 
         // return the first failure.

From bd0ade29d24ca95f55c0dff967a4784ffb7b79b3 Mon Sep 17 00:00:00 2001
From: Pierre Tachoire <pierre@tch.re>
Date: Fri, 27 Oct 2023 19:04:31 +0200
Subject: [PATCH 3/3] wpt: use fs.path.join

---
 src/main_wpt.zig | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/main_wpt.zig b/src/main_wpt.zig
index c389a077..7f567ea2 100644
--- a/src/main_wpt.zig
+++ b/src/main_wpt.zig
@@ -10,6 +10,8 @@ const Loop = jsruntime.Loop;
 const DOM = @import("dom.zig");
 const HTMLElem = @import("html/elements.zig");
 
+const fspath = std.fs.path;
+
 const wpt_dir = "tests/wpt";
 
 // FileLoader loads files content from the filesystem.
@@ -36,7 +38,7 @@ const FileLoader = struct {
         return self.files.get(name).?;
     }
     fn load(self: *FileLoader, name: []const u8) !void {
-        const filename = try std.mem.concat(self.alloc, u8, &.{ self.path, name });
+        const filename = try fspath.join(self.alloc, &.{ self.path, name });
         defer self.alloc.free(filename);
         var file = try std.fs.cwd().openFile(filename, .{});
         defer file.close();
@@ -160,7 +162,7 @@ fn runWPT(arena: *std.heap.ArenaAllocator, comptime apis: []jsruntime.API, f: []
     const html_doc = try parser.documentHTMLParseFromFileAlloc(alloc, f);
     const doc = parser.documentHTMLToDocument(html_doc);
 
-    const dirname = std.fs.path.dirname(f[wpt_dir.len..]) orelse unreachable;
+    const dirname = fspath.dirname(f[wpt_dir.len..]) orelse unreachable;
 
     // create JS env
     var loop = try Loop.init(alloc);
@@ -220,7 +222,7 @@ fn runWPT(arena: *std.heap.ArenaAllocator, comptime apis: []jsruntime.API, f: []
             var path = src;
             if (!std.mem.startsWith(u8, src, "/")) {
                 // no need to free path, thanks to the arena.
-                path = try std.fs.path.join(alloc, &.{ "/", dirname, path });
+                path = try fspath.join(alloc, &.{ "/", dirname, path });
             }
 
             res = try evalJS(js_env, alloc, try loader.get(path), src);
@@ -282,6 +284,6 @@ fn findWPTTests(allocator: std.mem.Allocator, path: []const u8, list: *std.Array
             continue;
         }
 
-        try list.append(try std.fs.path.join(allocator, &.{ path, entry.path }));
+        try list.append(try fspath.join(allocator, &.{ path, entry.path }));
     }
 }