dommy 0.7.0 → 0.8.0

data/lib/dommy/media_query_list.rb

@@ -67,11 +67,17 @@ module Dommy
         remove_event_listener("change", @onchange) if @onchange
         @onchange = value
         add_event_listener("change", value) if value
+      else
+        return Bridge::UNHANDLED
       end
 
       nil
     end
 
+    include Bridge::Methods
+    js_methods %w[
+      matches addListener removeListener addEventListener removeEventListener dispatchEvent
+    ]
     def __js_call__(method, args)
       case method
       when "matches"
@@ -83,7 +89,7 @@ module Dommy
       when "addEventListener"
         add_event_listener(args[0], args[1], args[2])
       when "removeEventListener"
-        remove_event_listener(args[0], args[1])
+        remove_event_listener(args[0], args[1], args[2])
       when "dispatchEvent"
         dispatch_event(args[0])
       end

data/lib/dommy/message_channel.rb

@@ -100,11 +100,15 @@ module Dommy
         @inline_message_handler = value
         add_event_listener("message", value) if value
         start if value
+      else
+        return Bridge::UNHANDLED
       end
 
       nil
     end
 
+    include Bridge::Methods
+    js_methods %w[postMessage start close addEventListener removeEventListener dispatchEvent]
     def __js_call__(method, args)
       case method
       when "postMessage"
@@ -116,7 +120,7 @@ module Dommy
       when "addEventListener"
         add_event_listener(args[0], args[1], args[2])
       when "removeEventListener"
-        remove_event_listener(args[0], args[1])
+        remove_event_listener(args[0], args[1], args[2])
       when "dispatchEvent"
         dispatch_event(args[0])
       end
@@ -157,6 +161,28 @@ module Dommy
         super
       end
     end
+
+    js_methods %w[initMessageEvent]
+    def __js_call__(method, args)
+      case method
+      when "initMessageEvent"
+        # Deprecated initMessageEvent(type, bubbles, cancelable, data, origin,
+        # lastEventId, source, ports); a no-op while the event is dispatching.
+        raise Bridge::TypeError, "initMessageEvent requires a type argument" if args.empty?
+
+        unless @dispatch_flag
+          init_event(args[0], args[1], args[2])
+          @data = args[3]
+          @origin = (args[4] || "").to_s
+          @last_event_id = (args[5] || "").to_s
+          @source = args[6]
+          @ports = args[7] || []
+        end
+        nil
+      else
+        super
+      end
+    end
   end
 
   # `BroadcastChannel` — same-origin pub/sub. Dommy keeps a per-window
@@ -222,11 +248,15 @@ module Dommy
         remove_event_listener("message", @onmessage) if @onmessage
         @onmessage = value
         add_event_listener("message", value) if value
+      else
+        return Bridge::UNHANDLED
       end
 
       nil
     end
 
+    include Bridge::Methods
+    js_methods %w[postMessage close addEventListener removeEventListener dispatchEvent]
     def __js_call__(method, args)
       case method
       when "postMessage"
@@ -236,7 +266,7 @@ module Dommy
       when "addEventListener"
         add_event_listener(args[0], args[1], args[2])
       when "removeEventListener"
-        remove_event_listener(args[0], args[1])
+        remove_event_listener(args[0], args[1], args[2])
       when "dispatchEvent"
         dispatch_event(args[0])
       end

data/lib/dommy/mutation_observer.rb

@@ -75,8 +75,11 @@ module Dommy
       @observed = []
       @records = []
       @scheduled = false
+      @registered_docs = []
     end
 
+    include Bridge::Methods
+    js_methods %w[observe disconnect takeRecords]
     def __js_call__(method, args)
       case method
       when "observe"
@@ -136,16 +139,29 @@ module Dommy
       # is supplied; `characterData: true` is implied if
       # characterDataOldValue is supplied. Matches the spec's option
       # normalization in MutationObserverInit.
-      attrs_implied = !attribute_filter.nil? || truthy_option(opts, "attributeOldValue")
-      char_implied = truthy_option(opts, "characterDataOldValue")
-      attributes_on = truthy_option(opts, "attributes") || attrs_implied
+      # `attributes`/`characterData` are *implied* true only when their
+      # old-value/filter companion is present AND the member itself is omitted.
+      # If the member is present but false, that companion is a TypeError.
+      attr_present = opts.key?("attributes") || opts.key?(:attributes)
+      char_present = opts.key?("characterData") || opts.key?(:characterData)
+      attrs_extras = !attribute_filter.nil? || truthy_option(opts, "attributeOldValue")
+      char_extras = truthy_option(opts, "characterDataOldValue")
+
+      if attrs_extras && attr_present && !truthy_option(opts, "attributes")
+        raise Bridge::TypeError, "attributeOldValue/attributeFilter requires attributes to be true"
+      end
+      if char_extras && char_present && !truthy_option(opts, "characterData")
+        raise Bridge::TypeError, "characterDataOldValue requires characterData to be true"
+      end
+
+      attributes_on = truthy_option(opts, "attributes") || (attrs_extras && !attr_present)
       child_list_on = truthy_option(opts, "childList")
-      character_data_on = truthy_option(opts, "characterData") || char_implied
+      character_data_on = truthy_option(opts, "characterData") || (char_extras && !char_present)
 
       # Per spec, observe() must request at least one of childList,
       # attributes, or characterData; otherwise TypeError.
       unless child_list_on || attributes_on || character_data_on
-        raise TypeError, "MutationObserver.observe: at least one of childList, attributes, characterData must be true"
+        raise Bridge::TypeError, "MutationObserver.observe: at least one of childList, attributes, characterData must be true"
       end
 
       entry = {
@@ -169,7 +185,14 @@ module Dommy
         @observed << entry
       end
 
-      @document.register_observer(self)
+      # Register on the TARGET's node-document — not just the observer's own
+      # document — so an observer watching a node in another document (e.g. a
+      # DOMParser-created XML document) is reached by that document's mutations.
+      doc = document_for_target(target)
+      unless @registered_docs.include?(doc)
+        doc.register_observer(self)
+        @registered_docs << doc
+      end
       nil
     end
 
@@ -177,10 +200,19 @@ module Dommy
       @records.clear
       @scheduled = false
       @observed.clear
-      @document.unregister_observer(self)
+      @registered_docs.each { |doc| doc.unregister_observer(self) }
+      @registered_docs.clear
       nil
     end
 
+    # The node-document a target belongs to (the target itself when it is a
+    # Document).
+    def document_for_target(target)
+      return target if target.is_a?(Dommy::Document)
+
+      target.instance_variable_get(:@document) || @document
+    end
+
     def take_records
       out = @records.dup
       @records.clear
@@ -194,16 +226,27 @@ module Dommy
 
       records = @records.dup
       @records.clear
-      if @callback.respond_to?(:__js_call__)
-        @callback.__js_call__("call", [records])
+      # Per spec the callback receives (mutationRecords, observer) and is invoked
+      # with `this` set to the observer.
+      if @callback.respond_to?(:__js_call_with_this__)
+        @callback.__js_call_with_this__([records, self], self)
+      elsif @callback.respond_to?(:__js_call__)
+        @callback.__js_call__("call", [records, self])
       elsif @callback.respond_to?(:call)
-        @callback.call(records)
+        @callback.call(records, self)
       end
     end
 
+    # A MutationObserverInit member is a WebIDL `boolean`, so its value is
+    # converted with JS ToBoolean — any object (e.g. `attributes: ["abc"]`) is
+    # truthy; only false / 0 / "" / null / undefined / NaN are falsy.
     def truthy_option(hash, key)
-      value = hash[key] || hash[key.to_sym]
-      value == true || value.to_s == "true"
+      value = hash.key?(key) ? hash[key] : hash[key.to_sym]
+      return false if value.nil? || value == false || value == 0 || value == ""
+      return false if defined?(Bridge::UNDEFINED) && value.equal?(Bridge::UNDEFINED)
+      return false if value.is_a?(Float) && value.nan?
+
+      true
     end
   end
 end

data/lib/dommy/navigator.rb

@@ -106,6 +106,8 @@ module Dommy
       end
     end
 
+    include Bridge::Methods
+    js_methods %w[share canShare vibrate getBattery]
     def __js_call__(method, args)
       case method
       when "share"
@@ -120,7 +122,7 @@ module Dommy
     end
 
     def __js_set__(_key, _value)
-      nil
+      Bridge::UNHANDLED
     end
   end
 
@@ -171,9 +173,11 @@ module Dommy
     end
 
     def __js_set__(_key, _value)
-      nil
+      Bridge::UNHANDLED
     end
 
+    include Bridge::Methods
+    js_methods %w[readText writeText read write addEventListener removeEventListener dispatchEvent]
     def __js_call__(method, args)
       case method
       when "readText"
@@ -187,7 +191,7 @@ module Dommy
       when "addEventListener"
         add_event_listener(args[0], args[1], args[2])
       when "removeEventListener"
-        remove_event_listener(args[0], args[1])
+        remove_event_listener(args[0], args[1], args[2])
       when "dispatchEvent"
         dispatch_event(args[0])
       end
@@ -259,9 +263,11 @@ module Dommy
     end
 
     def __js_set__(_key, _value)
-      nil
+      Bridge::UNHANDLED
     end
 
+    include Bridge::Methods
+    js_methods %w[query]
     def __js_call__(method, args)
       case method
       when "query"
@@ -311,17 +317,21 @@ module Dommy
         remove_event_listener("change", @onchange) if @onchange
         @onchange = value
         add_event_listener("change", value) if value
+      else
+        return Bridge::UNHANDLED
       end
 
       nil
     end
 
+    include Bridge::Methods
+    js_methods %w[addEventListener removeEventListener dispatchEvent]
     def __js_call__(method, args)
       case method
       when "addEventListener"
         add_event_listener(args[0], args[1], args[2])
       when "removeEventListener"
-        remove_event_listener(args[0], args[1])
+        remove_event_listener(args[0], args[1], args[2])
       when "dispatchEvent"
         dispatch_event(args[0])
       end
@@ -393,6 +403,8 @@ module Dommy
 
     alias clearWatch clear_watch
 
+    include Bridge::Methods
+    js_methods %w[getCurrentPosition watchPosition clearWatch]
     def __js_call__(method, args)
       case method
       when "getCurrentPosition"
@@ -415,13 +427,7 @@ module Dommy
     end
 
     def invoke(callback, payload)
-      return if callback.nil?
-
-      if callback.respond_to?(:__js_call__)
-        callback.__js_call__("call", [payload])
-      elsif callback.respond_to?(:call)
-        callback.call(payload)
-      end
+      CallableInvoker.invoke(callback, payload)
     end
   end
 
@@ -439,6 +445,8 @@ module Dommy
       PromiseValue.resolve(@window, WakeLockSentinel.new(@window, type.to_s))
     end
 
+    include Bridge::Methods
+    js_methods %w[request]
     def __js_call__(method, args)
       case method
       when "request"
@@ -479,6 +487,8 @@ module Dommy
       end
     end
 
+    include Bridge::Methods
+    js_methods %w[release]
     def __js_call__(method, _args)
       case method
       when "release"
@@ -561,6 +571,8 @@ module Dommy
       PromiseValue.resolve(@window, {"held" => held, "pending" => []})
     end
 
+    include Bridge::Methods
+    js_methods %w[request query]
     def __js_call__(method, args)
       case method
       when "request"
@@ -617,6 +629,8 @@ module Dommy
       PromiseValue.resolve(@window, @persisted)
     end
 
+    include Bridge::Methods
+    js_methods %w[estimate persist persisted]
     def __js_call__(method, _args)
       case method
       when "estimate"

data/lib/dommy/node.rb

@@ -10,12 +10,6 @@ module Dommy
   # snapshots tree state at the time of the query, matching what
   # most happy-dom test patterns expect.
   class NodeList < Array
-    # Methods routed through __js_call__ (keep in sync with its when-arms).
-    JS_METHOD_NAMES = %w[item forEach entries keys values].freeze
-    def __js_method_names__
-      JS_METHOD_NAMES
-    end
-
     # Spec-compliant: out-of-range returns nil, not raise (Array#[] is
     # close but we make negative indices fail too — DOM `item(-1)` is
     # nil, not Array#[-1]'s last element).
@@ -60,24 +54,24 @@ module Dommy
       when "length"
         length
       else
-        if key.is_a?(Integer) || key.to_s.match?(/\A\d+\z/)
-          item(key.to_i)
+        # Indexed getter: out-of-range yields JS `undefined` (item() returns null).
+        if key.is_a?(Integer) || key.to_s.match?(/\A-?\d+\z/)
+          token = item(key.to_i)
+          token.nil? ? Bridge::UNDEFINED : token
         end
       end
     end
 
+    include Bridge::Methods
+    # keys/values/entries/Symbol.iterator are provided JS-side (the array-like
+    # prototype) so they return real iterators, not arrays — see host_runtime.js.
+    js_methods %w[item forEach]
     def __js_call__(method, args)
       case method
       when "item"
         item(args[0])
       when "forEach"
         for_each(&args[0])
-      when "entries"
-        entries
-      when "keys"
-        keys
-      when "values"
-        values
       end
     end
   end
@@ -94,12 +88,6 @@ module Dommy
   class LiveNodeList
     include Enumerable
 
-    # Methods routed through __js_call__ (keep in sync with its when-arms).
-    JS_METHOD_NAMES = %w[item forEach entries keys values].freeze
-    def __js_method_names__
-      JS_METHOD_NAMES
-    end
-
     def initialize(&block)
       @compute = block
     end
@@ -175,24 +163,24 @@ module Dommy
       when "length"
         length
       else
-        if key.is_a?(Integer) || key.to_s.match?(/\A\d+\z/)
-          item(key.to_i)
+        # Indexed getter: out-of-range yields JS `undefined` (item() returns null).
+        if key.is_a?(Integer) || key.to_s.match?(/\A-?\d+\z/)
+          token = item(key.to_i)
+          token.nil? ? Bridge::UNDEFINED : token
         end
       end
     end
 
+    include Bridge::Methods
+    # keys/values/entries/Symbol.iterator are provided JS-side (the array-like
+    # prototype) so they return real iterators, not arrays — see host_runtime.js.
+    js_methods %w[item forEach]
     def __js_call__(method, args)
       case method
       when "item"
         item(args[0])
       when "forEach"
         for_each(&args[0])
-      when "entries"
-        entries
-      when "keys"
-        keys
-      when "values"
-        values
       end
     end
   end
@@ -226,5 +214,147 @@ module Dommy
     DOCUMENT_POSITION_CONTAINS = 0x08
     DOCUMENT_POSITION_CONTAINED_BY = 0x10
     DOCUMENT_POSITION_IMPLEMENTATION_SPECIFIC = 0x20
+
+    # WHATWG Node.isEqualNode — deep structural equality (type-specific data
+    # plus equal, in-order, recursively-equal children). Available on every node
+    # class that includes Node; the bridge routes "isEqualNode" here.
+    def is_equal_node(other)
+      Internal::NodeEquality.equal?(self, other)
+    end
+
+    # Node.isSameNode — strict reference identity (deprecated alias for `===`).
+    def is_same_node(other)
+      equal?(other)
+    end
+
+    # Node.compareDocumentPosition(other) — a bitmask describing where `other`
+    # sits relative to this node: 0 for the same node, CONTAINS/CONTAINED_BY for
+    # ancestor/descendant, PRECEDING/FOLLOWING for tree order, or DISCONNECTED
+    # (with a stable IMPLEMENTATION_SPECIFIC|PRECEDING) for unrelated nodes.
+    # Generic over any node with a backing Nokogiri node.
+    def compare_document_position(other)
+      return 0 if equal?(other)
+      unless respond_to?(:__dommy_backend_node__) && other.respond_to?(:__dommy_backend_node__)
+        return DOCUMENT_POSITION_DISCONNECTED | DOCUMENT_POSITION_IMPLEMENTATION_SPECIFIC | DOCUMENT_POSITION_PRECEDING
+      end
+
+      self_node = __dommy_backend_node__
+      other_node = other.__dommy_backend_node__
+      self_ancestors = node_ancestor_chain(self_node)
+      other_ancestors = node_ancestor_chain(other_node)
+
+      common = self_ancestors.find { |a| other_ancestors.include?(a) }
+      unless common
+        return DOCUMENT_POSITION_DISCONNECTED | DOCUMENT_POSITION_IMPLEMENTATION_SPECIFIC | DOCUMENT_POSITION_PRECEDING
+      end
+      return DOCUMENT_POSITION_CONTAINED_BY | DOCUMENT_POSITION_FOLLOWING if common == self_node
+      return DOCUMENT_POSITION_CONTAINS | DOCUMENT_POSITION_PRECEDING if common == other_node
+
+      self_branch = node_branch_under(common, self_ancestors)
+      other_branch = node_branch_under(common, other_ancestors)
+      common.children.each do |child|
+        return DOCUMENT_POSITION_FOLLOWING if child == self_branch
+        return DOCUMENT_POSITION_PRECEDING if child == other_branch
+      end
+      DOCUMENT_POSITION_DISCONNECTED
+    end
+
+    # Node.getRootNode — the topmost ancestor of this node (the document, a
+    # ShadowRoot, a detached subtree root, or the node itself). Generic default
+    # for any node backed by a Nokogiri node; classes with special roots
+    # (Element's shadow handling) override it.
+    def get_root_node(_options = nil)
+      return self unless respond_to?(:__dommy_backend_node__) && instance_variable_defined?(:@document)
+
+      node = __dommy_backend_node__
+      node = node.parent while node.respond_to?(:parent) && node.parent
+      # The topmost node of an attached subtree is the Nokogiri document, which
+      # has no element wrapper — map it to the Document. A detached node's root is
+      # itself.
+      return @document if @document && node.equal?(@document.nokogiri_doc)
+
+      (@document && @document.wrap_node(node)) || self
+    end
+
+    HTML_NAMESPACE = "http://www.w3.org/1999/xhtml"
+
+    # Node.lookupNamespaceURI(prefix) — the namespace bound to `prefix` (or the
+    # default namespace for a null/empty prefix) in this node's scope, walking up
+    # the element ancestors' namespace declarations. HTML elements default to the
+    # XHTML namespace.
+    def lookup_namespace_uri(prefix)
+      wanted = namespace_prefix_arg(prefix)
+      nk = nearest_namespaceable_node
+      while nk.respond_to?(:element?) && nk.element?
+        nk.namespace_definitions.each do |d|
+          return d.href if normalize_ns_prefix(d.prefix) == wanted
+        end
+        return nk.namespace ? nk.namespace.href : HTML_NAMESPACE if wanted.nil?
+
+        nk = nk.parent
+      end
+      nil
+    end
+
+    # Node.lookupPrefix(namespace) — a prefix bound to `namespace`, or null for
+    # the default namespace.
+    def lookup_prefix(namespace)
+      ns = namespace.to_s
+      return nil if ns.empty?
+
+      nk = nearest_namespaceable_node
+      while nk.respond_to?(:element?) && nk.element?
+        nk.namespace_definitions.each do |d|
+          return d.prefix if d.href == ns && d.prefix
+        end
+        nk = nk.parent
+      end
+      nil
+    end
+
+    # Node.isDefaultNamespace(namespace) — true if `namespace` (null/"" → null) is
+    # the default namespace in this node's scope.
+    def is_default_namespace(namespace)
+      ns = namespace.nil? ? nil : namespace.to_s
+      ns = nil if ns == ""
+      lookup_namespace_uri(nil) == ns
+    end
+
+    private
+
+    def node_ancestor_chain(node)
+      chain = [node]
+      Internal::NodeTraversal.each_ancestor(node) { |n| chain << n }
+      chain
+    end
+
+    def node_branch_under(common, chain)
+      chain.each_with_index do |node, i|
+        return node if i.zero? && node == common
+        return node if node.respond_to?(:parent) && node.parent == common
+      end
+      nil
+    end
+
+    def namespace_prefix_arg(prefix)
+      return nil if prefix.nil? || prefix.to_s.empty?
+      return nil if defined?(Bridge::UNDEFINED) && prefix.equal?(Bridge::UNDEFINED)
+
+      prefix.to_s
+    end
+
+    def normalize_ns_prefix(prefix)
+      prefix.nil? || prefix.to_s.empty? ? nil : prefix.to_s
+    end
+
+    def nearest_namespaceable_node
+      return nil unless respond_to?(:__dommy_backend_node__)
+
+      nk = __dommy_backend_node__
+      nk = nk.parent while nk.respond_to?(:element?) && !nk.element? && nk.respond_to?(:parent) && nk.parent
+      nk
+    end
   end
 end
+
+require_relative "internal/node_equality"

data/lib/dommy/notification.rb

@@ -69,6 +69,8 @@ module Dommy
       end
     end
 
+    include Bridge::Methods
+    js_methods %w[close addEventListener removeEventListener dispatchEvent]
     def __js_call__(method, args)
       case method
       when "close"
@@ -76,7 +78,7 @@ module Dommy
       when "addEventListener"
         add_event_listener(args[0], args[1], args[2])
       when "removeEventListener"
-        remove_event_listener(args[0], args[1])
+        remove_event_listener(args[0], args[1], args[2])
       when "dispatchEvent"
         dispatch_event(args[0])
       end

data/lib/dommy/performance.rb

@@ -93,6 +93,10 @@ module Dommy
       end
     end
 
+    include Bridge::Methods
+    js_methods %w[
+      now mark measure clearMarks clearMeasures getEntries getEntriesByName getEntriesByType
+    ]
     def __js_call__(method, args)
       case method
       when "now"

data/lib/dommy/performance_observer.rb

@@ -41,6 +41,8 @@ module Dommy
       invoke_callback(entries)
     end
 
+    include Bridge::Methods
+    js_methods %w[observe disconnect takeRecords]
     def __js_call__(method, args)
       case method
       when "observe"

data/lib/dommy/promise.rb

@@ -51,6 +51,8 @@ module Dommy
       @handlers = []
     end
 
+    include Bridge::Methods
+    js_methods %w[then catch]
     def __js_call__(method, args)
       case method
       when "then"
@@ -133,8 +135,12 @@ module Dommy
       other.__js_call__(
         "then",
         [
-          proc { |resolved| fulfill(resolved) },
-          proc { |reason| reject(reason) }
+          # Return nil, not the result of fulfill/reject: those return `self`
+          # (a PromiseValue), and run_handler would treat that as a thenable to
+          # chain onto — re-adopting forever. These are adoption sinks; their
+          # return value must be ignored.
+          proc { |resolved| fulfill(resolved); nil },
+          proc { |reason| reject(reason); nil }
         ]
       )
       self
@@ -167,13 +173,15 @@ module Dommy
         return
       end
 
-      result = invoke_callback(callback, @value)
-      if result.is_a?(PromiseValue)
+      result = CallableInvoker.invoke(callback, @value)
+      if result.is_a?(PromiseValue) && !result.equal?(handler.child)
+        # Adopt the returned thenable. The continuation procs return nil so their
+        # `self`-returning fulfill/reject don't get re-adopted (infinite chain).
         result.__js_call__(
           "then",
           [
-            proc { |resolved| handler.child.fulfill(resolved) },
-            proc { |reason| handler.child.reject(reason) }
+            proc { |resolved| handler.child.fulfill(resolved); nil },
+            proc { |reason| handler.child.reject(reason); nil }
           ]
         )
       else
@@ -187,13 +195,5 @@ module Dommy
     def propagate(child)
       @state == :fulfilled ? child.fulfill(@value) : child.reject(@value)
     end
-
-    def invoke_callback(callback, value)
-      if callback.respond_to?(:__js_call__)
-        callback.__js_call__("call", [value])
-      else
-        callback.call(value)
-      end
-    end
   end
 end