public/browser
1
0
Fork 0
mirror of https://github.com/lightpanda-io/browser.git synced 2026-02-04 14:33:47 +00:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

Dynamically added scripts default to async

Browse Source
This commit is contained in:
Karl Seguin
2025-12-21 16:51:39 +08:00
parent 32c83d166d
commit 8fbd64955f
4 changed files with 24 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/browser/Page.zig
View File

@@ -895,8 +895,8 @@ pub fn tick(self: *Page) void {
self.js.runMicrotasks();
}
pub fn scriptAddedCallback(self: *Page, script: *Element.Html.Script) !void {
self._script_manager.addFromElement(script, "parsing") catch |err| {
pub fn scriptAddedCallback(self: *Page, comptime from_parser: bool, script: *Element.Html.Script) !void {
self._script_manager.addFromElement(from_parser, script, "parsing") catch |err| {
log.err(.page, "page.scriptAddedCallback", .{
.err = err,
.src = script.asElement().getAttributeSafe("src"),
@@ -2180,7 +2180,7 @@ fn nodeIsReady(self: *Page, comptime from_parser: bool, node: *Node) !void {
return;
}
self.scriptAddedCallback(script) catch |err| {
self.scriptAddedCallback(from_parser, script) catch |err| {
log.err(.page, "page.nodeIsReady", .{ .err = err });
return err;
};

15
src/browser/ScriptManager.zig
View File

@@ -143,7 +143,7 @@ fn clearList(list: *std.DoublyLinkedList) void {
}
}
pub fn addFromElement(self: *ScriptManager, script_element: *Element.Html.Script, comptime ctx: []const u8) !void {
pub fn addFromElement(self: *ScriptManager, comptime from_parser: bool, script_element: *Element.Html.Script, comptime ctx: []const u8) !void {
if (script_element._executed) {
// If a script tag gets dynamically created and added to the dom:
// document.getElementsByTagName('head')[0].appendChild(script)
@@ -220,12 +220,25 @@ pub fn addFromElement(self: *ScriptManager, script_element: *Element.Html.Script
if (source == .@"inline") {
break :blk if (kind == .module) .@"defer" else .normal;
}
if (element.getAttributeSafe("async") != null) {
break :blk .async;
}
// Check for defer or module (before checking dynamic script default)
if (kind == .module or element.getAttributeSafe("defer") != null) {
break :blk .@"defer";
}
// For dynamically-inserted scripts (not from parser), default to async
// unless async was explicitly set to false (which removes the attribute)
// and defer was set to true (checked above)
if (comptime !from_parser) {
// Script has src and no explicit async/defer attributes
// Per HTML spec, dynamically created scripts default to async
break :blk .async;
}
break :blk .normal;
},
};

7
src/browser/tests/element/html/script/dynamic.html
View File

@@ -2,12 +2,15 @@
<head></head>
<script src="../../../testing.js"></script>
<script id=append_before_src>
<script id=append_before_src1>
loaded1 = 0;
const script1 = document.createElement('script');
script1.async = false;
script1.src = "dynamic1.js";
document.getElementsByTagName('head')[0].appendChild(script1);
testing.eventually(() => {
testing.expectEqual(1, loaded1);
});
</script>
<script id=no_double_execute>
@@ -17,7 +20,7 @@
});
</script>
<script id=append_before_src>
<script id=append_before_src2>
loaded2 = 0;
const script2a = document.createElement('script');
script2a.src = "dynamic2.js";

2
src/browser/webapi/element/html/Script.zig
View File

@@ -54,7 +54,7 @@ pub fn setSrc(self: *Script, src: []const u8, page: *Page) !void {
try element.setAttributeSafe("src", src, page);
self._src = element.getAttributeSafe("src") orelse unreachable;
if (element.asNode().isConnected()) {
try page.scriptAddedCallback(self);
try page.scriptAddedCallback(false, self);
}
}