Fixing Node#parse for HTML document nodes.

See related commit 89db311 which broke this functionality.

CHANGELOG.ja.rdoc

@@ -5,6 +5,7 @@
 * [JRuby] Fix Ruby memory exhaustion vulnerability. #1087 (Thanks, @ocher)
 * [MRI] Fix segfault during GC when using `libxml-ruby` and `nokogiri` together in multi-threaded environment. #895 (Thanks, @ender672!)
 * Building on OSX 10.9 stock ruby 2.0.0 now works. #1101 (Thanks, @zenspider!)
+* Node#parse now works again for HTML document nodes (broken in 1.6.2+).
 
 
 === 1.6.2.1 / 2014年5月13日

CHANGELOG.rdoc

@@ -5,6 +5,7 @@
 * [JRuby] Fix Ruby memory exhaustion vulnerability. #1087 (Thanks, @ocher)
 * [MRI] Fix segfault during GC when using `libxml-ruby` and `nokogiri` together in multi-threaded environment. #895 (Thanks, @ender672!)
 * Building on OSX 10.9 stock ruby 2.0.0 now works. #1101 (Thanks, @zenspider!)
+* Node#parse now works again for HTML document nodes (broken in 1.6.2+).
 
 
 === 1.6.2.1 / 2014-05-13

lib/nokogiri/xml/node.rb

@@ -513,7 +513,7 @@ def parse string_or_io, options = nil
         # document as the parsing context instead. Otherwise, the in-context
         # parser cannot find an element or a document node.
         # Document Fragments are also not usable by the in-context parser.
-        if !element? && !xml? && (!parent || parent.fragment?)
+        if !element? && !is_a?(Nokogiri::XML::Document) && (!parent || parent.fragment?)
           return document.parse(string_or_io, options)
         end
 

test/xml/test_node.rb

@@ -136,7 +136,29 @@ def test_parse_error_on_fragment_with_empty_document
       def test_parse_with_unparented_text_context_node
         doc = XML::Document.new
         elem = XML::Text.new("foo", doc)
-        elem.parse("<bar/>")
+        x = elem.parse("<bar/>") # should not raise an exception
+        assert_equal x.first.name, "bar"
+      end
+
+      def test_parse_with_unparented_html_text_context_node
+        doc = HTML::Document.new
+        elem = XML::Text.new("div", doc)
+        x = elem.parse("<div/>") # should not raise an exception
+        assert_equal x.first.name, "div"
+      end
+
+      def test_parse_with_unparented_fragment_text_context_node
+        doc = XML::DocumentFragment.parse "<div><span>foo</span></div>"
+        elem = doc.at_css "span"
+        x = elem.parse("<span/>") # should not raise an exception
+        assert_equal x.first.name, "span"
+      end
+
+      def test_parse_with_unparented_html_fragment_text_context_node
+        doc = HTML::DocumentFragment.parse "<div><span>foo</span></div>"
+        elem = doc.at_css "span"
+        x = elem.parse("<span/>") # should not raise an exception
+        assert_equal x.first.name, "span"
       end
 
       def test_subclass_dup