Fix Node.isConnected

The previous implementation just checked if a node had a parent. But it should
check the node has a document ancestor:

```
const d1 = document.createElement('div');
document.createElement('div').appendChild(d1);
d1.isConnected
```

should be `false`.

In addition to this fix, also added support for DocumentFragments which are
part of the ShadowRoot. This, like events, is one of those apis that DOES break
the ShadowRoot isolation.

Author: karlseguin

src/browser/dom/node.zig

@@ -180,11 +180,35 @@ pub const Node = struct {
     }
 
     pub fn get_isConnected(self: *parser.Node) !bool {
-        // TODO: handle Shadow DOM
-        if (try parser.nodeType(self) == .document) {
-            return true;
+        var node = self;
+        while (true) {
+            const node_type = try parser.nodeType(node);
+            if (node_type == .document) {
+                return true;
+            }
+
+            if (try parser.nodeParentNode(node)) |parent| {
+                // didn't find a document, but node has a parent, let's see
+                // if it's connected;
+                node = parent;
+                continue;
+            }
+
+            if (node_type != .document_fragment) {
+                // doesn't have a parent and isn't a document_fragment
+                // can't be connected
+                return false;
+            }
+
+            if (parser.documentFragmentGetHost(@ptrCast(node))) |host| {
+                // node doesn't have a parent, but it's a document fragment
+                // with a host. The host is like the parent, but we only want to
+                // traverse up (or down) to it in specific cases, like isConnected.
+                node = host;
+                continue;
+            }
+            return false;
         }
-        return try Node.get_parentNode(self) != null;
     }
 
     // Read/Write attributes
@@ -652,7 +676,13 @@ test "Browser.DOM.node" {
     try runner.testCases(&.{
         .{ "content.isConnected", "true" },
         .{ "document.isConnected", "true" },
-        .{ "document.createElement('div').isConnected", "false" },
+        .{ "const connDiv = document.createElement('div')", null },
+        .{ "connDiv.isConnected", "false" },
+        .{ "const connParentDiv = document.createElement('div')", null },
+        .{ "connParentDiv.appendChild(connDiv)", null },
+        .{ "connDiv.isConnected", "false" },
+        .{ "content.appendChild(connParentDiv)", null },
+        .{ "connDiv.isConnected", "true" },
     }, .{});
 
     try runner.testCases(&.{

src/browser/dom/shadow_root.zig

@@ -62,7 +62,7 @@ const testing = @import("../../testing.zig");
 test "Browser.DOM.ShadowRoot" {
     defer testing.reset();
 
-    var runner = try testing.jsRunner(testing.tracking_allocator, .{ .html =
+    var runner = try testing.jsRunner(testing.tracking_allocator, .{ .html = 
         \\ <div id=conflict>nope</div>
     });
     defer runner.deinit();
@@ -94,8 +94,8 @@ test "Browser.DOM.ShadowRoot" {
     try runner.testCases(&.{
         .{ "sr2.getElementById('conflict')", "null" },
         .{ "const n1 = document.createElement('div')", null },
-        .{ "n1.id = 'conflict'", null},
-        .{ "sr2.append(n1)", null},
+        .{ "n1.id = 'conflict'", null },
+        .{ "sr2.append(n1)", null },
         .{ "sr2.getElementById('conflict') == n1", "true" },
     }, .{});