nokogiri 1.5.0.beta.4-java → 1.5.0-java

data/ext/java/nokogiri/HtmlDocument.java

@@ -32,9 +32,7 @@
 
 package nokogiri;
 
-import static nokogiri.internals.NokogiriHelpers.getNokogiriClass;
 import nokogiri.internals.HtmlDomParserContext;
-import nokogiri.internals.SaveContext;
 
 import org.jruby.Ruby;
 import org.jruby.RubyClass;
@@ -45,7 +43,6 @@ import org.jruby.runtime.Arity;
 import org.jruby.runtime.ThreadContext;
 import org.jruby.runtime.builtin.IRubyObject;
 import org.w3c.dom.Document;
-import org.w3c.dom.DocumentType;
 
 /**
  * Class for Nokogiri::HTML::Document.
@@ -88,7 +85,7 @@ public class HtmlDocument extends XmlDocument {
         Arity.checkArgumentCount(ruby, args, 4, 4);
         HtmlDomParserContext ctx =
             new HtmlDomParserContext(ruby, args[2], args[3]);
-        ctx.setInputSource(context, args[0]);
+        ctx.setInputSource(context, args[0], args[1]);
         return ctx.parse(context, klass, args[1]);
     }
 
@@ -119,31 +116,4 @@ public class HtmlDocument extends XmlDocument {
                                           IRubyObject[] args) {
         return do_parse(context, cls, args);
     }
-
-    @Override
-    public void saveContent(ThreadContext context, SaveContext ctx) {
-        Document doc = getDocument();
-        DocumentType dtd = doc.getDoctype();
-
-        if(dtd != null) {
-            ctx.append("<!DOCTYPE ");
-            ctx.append(dtd.getName());
-            if(dtd.getPublicId() != null) {
-                ctx.append(" PUBLIC ");
-                ctx.appendQuoted(dtd.getPublicId());
-                if(dtd.getSystemId() != null) {
-                    ctx.append(" ");
-                    ctx.appendQuoted(dtd.getSystemId());
-                }
-            } else if(dtd.getSystemId() != null) {
-                ctx.append(" SYSTEM ");
-                ctx.appendQuoted(dtd.getSystemId());
-            }
-            ctx.append(">\n");
-        }
-
-        this.saveNodeListContent(context,
-                (XmlNodeSet) this.children(context), ctx);
-        ctx.append("\n");
-    }
 }

data/ext/java/nokogiri/HtmlSaxParserContext.java

@@ -215,7 +215,7 @@ public class HtmlSaxParserContext extends XmlSaxParserContext {
                                        IRubyObject encoding) {
         HtmlSaxParserContext ctx = (HtmlSaxParserContext) NokogiriService.HTML_SAXPARSER_CONTEXT_ALLOCATOR.allocate(context.getRuntime(), (RubyClass)klazz);
         ctx.initialize(context.getRuntime());
-        ctx.setInputSource(context, data);
+        ctx.setInputSource(context, data, context.getRuntime().getNil());
         String javaEncoding = findEncoding(context, encoding);
         if (javaEncoding != null) {
             ctx.getInputSource().setEncoding(javaEncoding);

data/ext/java/nokogiri/NokogiriService.java

@@ -77,13 +77,17 @@ public class NokogiriService implements BasicLibraryService {
         nokogiriClassCache.put("Nokogiri::XML::ElementContent", (RubyClass)ruby.getClassFromPath("Nokogiri::XML::ElementContent"));
         nokogiriClassCache.put("Nokogiri::XML::ElementDecl", (RubyClass)ruby.getClassFromPath("Nokogiri::XML::ElementDecl"));
         nokogiriClassCache.put("Nokogiri::XML::EntityDecl", (RubyClass)ruby.getClassFromPath("Nokogiri::XML::EntityDecl"));
+        nokogiriClassCache.put("Nokogiri::XML::EntityReference", (RubyClass)ruby.getClassFromPath("Nokogiri::XML::EntityReference"));
+        nokogiriClassCache.put("Nokogiri::XML::ProcessingInstruction", (RubyClass)ruby.getClassFromPath("Nokogiri::XML::ProcessingInstruction"));
         nokogiriClassCache.put("Nokogiri::XML::CDATA", (RubyClass)ruby.getClassFromPath("Nokogiri::XML::CDATA"));
         nokogiriClassCache.put("Nokogiri::XML::Node", (RubyClass)ruby.getClassFromPath("Nokogiri::XML::Node"));
         nokogiriClassCache.put("Nokogiri::XML::NodeSet", (RubyClass)ruby.getClassFromPath("Nokogiri::XML::NodeSet"));
         nokogiriClassCache.put("Nokogiri::XML::Namespace", (RubyClass)ruby.getClassFromPath("Nokogiri::XML::Namespace"));
         nokogiriClassCache.put("Nokogiri::XML::SyntaxError", (RubyClass)ruby.getClassFromPath("Nokogiri::XML::SyntaxError"));
+        nokogiriClassCache.put("Nokogiri::XML::Reader", (RubyClass)ruby.getClassFromPath("Nokogiri::XML::Reader"));
         nokogiriClassCache.put("Nokogiri::XML::RelaxNG", (RubyClass)ruby.getClassFromPath("Nokogiri::XML::RelaxNG"));
         nokogiriClassCache.put("Nokogiri::XML::Schema", (RubyClass)ruby.getClassFromPath("Nokogiri::XML::Schema"));
+        nokogiriClassCache.put("Nokogiri::XML::XPathContext", (RubyClass)ruby.getClassFromPath("Nokogiri::XML::XPathContext"));
         nokogiriClassCache.put("Nokogiri::XML::AttributeDecl", (RubyClass)ruby.getClassFromPath("Nokogiri::XML::AttributeDecl"));
         nokogiriClassCache.put("Nokogiri::XML::SAX::ParserContext", (RubyClass)ruby.getClassFromPath("Nokogiri::XML::SAX::ParserContext"));
     }
@@ -154,6 +158,7 @@ public class NokogiriService implements BasicLibraryService {
         
         RubyClass entityDecl = xmlModule.defineClassUnder("EntityDecl", node, XML_ENTITY_DECL_ALLOCATOR);
         entityDecl.defineAnnotatedMethods(XmlEntityDecl.class);
+        
         entityDecl.defineConstant("INTERNAL_GENERAL", RubyFixnum.newFixnum(ruby, XmlEntityDecl.INTERNAL_GENERAL));
         entityDecl.defineConstant("EXTERNAL_GENERAL_PARSED", RubyFixnum.newFixnum(ruby, XmlEntityDecl.EXTERNAL_GENERAL_PARSED));
         entityDecl.defineConstant("EXTERNAL_GENERAL_UNPARSED", RubyFixnum.newFixnum(ruby, XmlEntityDecl.EXTERNAL_GENERAL_UNPARSED));
@@ -312,13 +317,6 @@ public class NokogiriService implements BasicLibraryService {
         }
     };
 
-    private static ObjectAllocator XML_PROCESSING_INSTRUCTION_ALLOCATOR =
-        new ObjectAllocator() {
-        public IRubyObject allocate(Ruby runtime, RubyClass klazz) {
-            throw runtime.newNotImplementedError("not implemented");
-        }
-    };
-
     public static final ObjectAllocator XML_DOCUMENT_ALLOCATOR = new ObjectAllocator() {
         private XmlDocument xmlDocument = null;
         public IRubyObject allocate(Ruby runtime, RubyClass klazz) {
@@ -347,9 +345,17 @@ public class NokogiriService implements BasicLibraryService {
         }
     };
 
-    private static ObjectAllocator XML_DTD_ALLOCATOR = new ObjectAllocator() {
+    public static final ObjectAllocator XML_DTD_ALLOCATOR = new ObjectAllocator() {
+        private XmlDtd xmlDtd = null;
         public IRubyObject allocate(Ruby runtime, RubyClass klazz) {
-            return new XmlDtd(runtime, klazz);
+            if (xmlDtd == null) xmlDtd = new XmlDtd(runtime, klazz);
+            try {
+                XmlDtd clone = (XmlDtd)xmlDtd.clone();
+                clone.setMetaClass(klazz);
+                return clone;
+            } catch (CloneNotSupportedException e) {
+                return new XmlDtd(runtime, klazz);
+            }
         }
     };
 
@@ -366,13 +372,35 @@ public class NokogiriService implements BasicLibraryService {
             }
         }
     };
-
-    private static ObjectAllocator XML_ENTITY_REFERENCE_ALLOCATOR = new ObjectAllocator() {
+    
+    public static ObjectAllocator XML_ELEMENT_DECL_ALLOCATOR = new ObjectAllocator() {
+        private XmlElementDecl xmlElementDecl = null;
         public IRubyObject allocate(Ruby runtime, RubyClass klazz) {
-            return new XmlEntityReference(runtime, klazz);
+            if (xmlElementDecl == null) xmlElementDecl = new XmlElementDecl(runtime, klazz);
+            try {
+                XmlElementDecl clone = (XmlElementDecl)xmlElementDecl.clone();
+                clone.setMetaClass(klazz);
+                return clone;
+            } catch (CloneNotSupportedException e) {
+                return new XmlElementDecl(runtime, klazz);
+            }
         }
     };
 
+    public static ObjectAllocator XML_ENTITY_REFERENCE_ALLOCATOR = new ObjectAllocator() {
+        private XmlEntityReference xmlEntityRef = null;
+        public IRubyObject allocate(Ruby runtime, RubyClass klazz) {
+            if (xmlEntityRef == null) xmlEntityRef = new XmlEntityReference(runtime, klazz);
+            try {
+                XmlEntityReference clone = (XmlEntityReference)xmlEntityRef.clone();
+                clone.setMetaClass(klazz);
+                return clone;
+            } catch (CloneNotSupportedException e) {
+                return new XmlEntityReference(runtime, klazz);
+            }
+        }
+    };
+    
     public static final ObjectAllocator XML_NAMESPACE_ALLOCATOR = new ObjectAllocator() {
         private XmlNamespace xmlNamespace = null;
         public IRubyObject allocate(Ruby runtime, RubyClass klazz) {
@@ -410,26 +438,45 @@ public class NokogiriService implements BasicLibraryService {
                 clone.setMetaClass(klazz);
                 return clone;
             } catch (CloneNotSupportedException e) {
-                return new XmlNodeSet(runtime, klazz, RubyArray.newEmptyArray(runtime));
+                xmlNodeSet = new XmlNodeSet(runtime, klazz);
+                xmlNodeSet.setNodes(RubyArray.newEmptyArray(runtime));
+                return xmlNodeSet;
             }
         }
     };
-
-    private static ObjectAllocator XML_READER_ALLOCATOR = new ObjectAllocator() {
+    
+    public static ObjectAllocator XML_PROCESSING_INSTRUCTION_ALLOCATOR = new ObjectAllocator() {
+        private XmlProcessingInstruction xmlProcessingInstruction = null;
         public IRubyObject allocate(Ruby runtime, RubyClass klazz) {
-            return new XmlReader(runtime, klazz);
+            if (xmlProcessingInstruction == null) xmlProcessingInstruction = new XmlProcessingInstruction(runtime, klazz);
+            try {
+                XmlProcessingInstruction clone = (XmlProcessingInstruction)xmlProcessingInstruction.clone();
+                clone.setMetaClass(klazz);
+                return clone;
+            } catch (CloneNotSupportedException e) {
+                return new XmlProcessingInstruction(runtime, klazz);
+            }
         }
     };
 
-    private static ObjectAllocator XML_ATTRIBUTE_DECL_ALLOCATOR = new ObjectAllocator() {
+    public static ObjectAllocator XML_READER_ALLOCATOR = new ObjectAllocator() {
+        private XmlReader xmlReader = null;
         public IRubyObject allocate(Ruby runtime, RubyClass klazz) {
-            return new XmlAttributeDecl(runtime, klazz);
+            if (xmlReader == null) xmlReader = new XmlReader(runtime, klazz);
+            try {
+                XmlReader clone  = (XmlReader) xmlReader.clone();
+                clone.setMetaClass(klazz);
+                return clone;
+            } catch (CloneNotSupportedException e) {
+                xmlReader = new XmlReader(runtime, klazz);
+                return xmlReader;
+            }
         }
     };
 
-    private static ObjectAllocator XML_ELEMENT_DECL_ALLOCATOR = new ObjectAllocator() {
+    private static ObjectAllocator XML_ATTRIBUTE_DECL_ALLOCATOR = new ObjectAllocator() {
         public IRubyObject allocate(Ruby runtime, RubyClass klazz) {
-            return new XmlElementDecl(runtime, klazz);
+            return new XmlAttributeDecl(runtime, klazz);
         }
     };
 
@@ -521,9 +568,17 @@ public class NokogiriService implements BasicLibraryService {
         }
     };
 
-    private static ObjectAllocator XML_XPATHCONTEXT_ALLOCATOR = new ObjectAllocator() {
+    public static ObjectAllocator XML_XPATHCONTEXT_ALLOCATOR = new ObjectAllocator() {
+        private XmlXpathContext xmlXpathContext = null;
         public IRubyObject allocate(Ruby runtime, RubyClass klazz) {
-            throw runtime.newNotImplementedError("not implemented");
+            if (xmlXpathContext == null) xmlXpathContext = new XmlXpathContext(runtime, klazz);
+            try {
+                XmlXpathContext clone  = (XmlXpathContext) xmlXpathContext.clone();
+                clone.setMetaClass(klazz);
+                return clone;
+            } catch (CloneNotSupportedException e) {
+                return new XmlXpathContext(runtime, klazz);
+            }
         }
     };
 

data/ext/java/nokogiri/XmlAttr.java

@@ -35,10 +35,9 @@ package nokogiri;
 import static nokogiri.internals.NokogiriHelpers.getNokogiriClass;
 import static nokogiri.internals.NokogiriHelpers.rubyStringToString;
 import nokogiri.internals.NokogiriHelpers;
-import nokogiri.internals.SaveContext;
+import nokogiri.internals.SaveContextVisitor;
 
 import org.jruby.Ruby;
-import org.jruby.RubyBoolean;
 import org.jruby.RubyClass;
 import org.jruby.RubyString;
 import org.jruby.anno.JRubyClass;
@@ -98,7 +97,7 @@ public class XmlAttr extends XmlNode{
         setNode(context, attr);
     }
 
-    public boolean isHtmlBooleanAttr() {
+    private boolean isHtmlBooleanAttr() {
         String name = node.getNodeName().toLowerCase();
 
         for(String s : HTML_BOOLEAN_ATTRS) {
@@ -160,19 +159,9 @@ public class XmlAttr extends XmlNode{
     }
 
     @Override
-    public void saveContent(ThreadContext context, SaveContext ctx) {
-        boolean docType = isHtml(context);
-        Attr attr = (Attr) node;
-
-        ctx.maybeSpace();
-        ctx.append(attr.getName());
-
-        if (!ctx.asHtml() || !isHtmlBooleanAttr()) {
-            ctx.append("=");
-            ctx.append("\"");
-            ctx.append(serializeAttrTextContent(attr.getValue(), docType));
-            ctx.append("\"");
-        }
+    public void accept(ThreadContext context, SaveContextVisitor visitor) {
+        visitor.enter((Attr)node);
+        visitor.leave((Attr)node);
     }
     
     private boolean isHtml(ThreadContext context) {

data/ext/java/nokogiri/XmlCdata.java

@@ -34,7 +34,7 @@ package nokogiri;
 
 import static nokogiri.internals.NokogiriHelpers.rubyStringToString;
 
-import nokogiri.internals.SaveContext;
+import nokogiri.internals.SaveContextVisitor;
 
 import org.jruby.Ruby;
 import org.jruby.RubyClass;
@@ -77,15 +77,8 @@ public class XmlCdata extends XmlText {
     }
 
     @Override
-    public void saveContent(ThreadContext context, SaveContext ctx) {
-        CDATASection cdata = (CDATASection) node;
-
-        if(cdata.getData().length() == 0) {
-            ctx.append("<![CDATA[]]>");
-        } else {
-            ctx.append("<![CDATA[");
-            ctx.append(cdata.getData());
-            ctx.append("]]>");
-        }
+    public void accept(ThreadContext context, SaveContextVisitor visitor) {
+        visitor.enter((CDATASection)node);
+        visitor.leave((CDATASection)node);
     }
 }

data/ext/java/nokogiri/XmlComment.java

@@ -33,13 +33,14 @@
 package nokogiri;
 
 import static nokogiri.internals.NokogiriHelpers.rubyStringToString;
-import nokogiri.internals.SaveContext;
+import nokogiri.internals.SaveContextVisitor;
 
 import org.jruby.Ruby;
 import org.jruby.RubyClass;
 import org.jruby.anno.JRubyClass;
 import org.jruby.runtime.ThreadContext;
 import org.jruby.runtime.builtin.IRubyObject;
+import org.w3c.dom.Comment;
 import org.w3c.dom.Document;
 import org.w3c.dom.Node;
 
@@ -78,9 +79,8 @@ public class XmlComment extends XmlNode {
     public boolean isComment() { return true; }
 
     @Override
-    public void saveContent(ThreadContext context, SaveContext ctx) {
-        ctx.append("<!--");
-        ctx.append(content(context).convertToString().asJavaString());
-        ctx.append("-->");
+    public void accept(ThreadContext context, SaveContextVisitor visitor) {
+        visitor.enter((Comment)node);
+        visitor.leave((Comment)node);
     }
 }

data/ext/java/nokogiri/XmlDocument.java

@@ -44,7 +44,7 @@ import javax.xml.parsers.ParserConfigurationException;
 
 import nokogiri.internals.NokogiriHelpers;
 import nokogiri.internals.NokogiriNamespaceCache;
-import nokogiri.internals.SaveContext;
+import nokogiri.internals.SaveContextVisitor;
 import nokogiri.internals.XmlDomParserContext;
 
 import org.jruby.Ruby;
@@ -53,6 +53,7 @@ import org.jruby.RubyFixnum;
 import org.jruby.RubyNil;
 import org.jruby.anno.JRubyClass;
 import org.jruby.anno.JRubyMethod;
+import org.jruby.javasupport.JavaUtil;
 import org.jruby.javasupport.util.RuntimeHelpers;
 import org.jruby.runtime.Arity;
 import org.jruby.runtime.ThreadContext;
@@ -76,8 +77,8 @@ public class XmlDocument extends XmlNode {
     
     /* UserData keys for storing extra info in the document node. */
     public final static String DTD_RAW_DOCUMENT = "DTD_RAW_DOCUMENT";
-    protected final static String DTD_INTERNAL_SUBSET = "DTD_INTERNAL_SUBSET";
-    protected final static String DTD_EXTERNAL_SUBSET = "DTD_EXTERNAL_SUBSET";
+    public final static String DTD_INTERNAL_SUBSET = "DTD_INTERNAL_SUBSET";
+    public final static String DTD_EXTERNAL_SUBSET = "DTD_EXTERNAL_SUBSET";
     
     /* DocumentBuilderFactory implementation class name. This needs to set a classloader into it.
      * Setting an appropriate classloader resolves issue 380.
@@ -142,11 +143,10 @@ public class XmlDocument extends XmlNode {
                     String attrName = attr.getName();
                     // not sure, but need to get value always before document is referred.
                     // or lose attribute value
-                    String attrValue = attr.getValue();
+                    String attrValue = attr.getValue(); // don't delete this line
                     if (isNamespace(attrName)) {
-                        String prefix = getLocalNameForNamespace(attrName);
-                        prefix = prefix != null ? prefix : "";
-                        nsCache.put(ruby, prefix, attrValue, node, this);
+                        // create and cache
+                        XmlNamespace.createFromAttr(ruby, attr);
                     }
                 }
             }
@@ -309,7 +309,7 @@ public class XmlDocument extends XmlNode {
         Arity.checkArgumentCount(ruby, args, 4, 4);
         XmlDomParserContext ctx =
             new XmlDomParserContext(ruby, args[2], args[3]);
-        ctx.setInputSource(context, args[0]);
+        ctx.setInputSource(context, args[0], args[1]);
         return ctx.parse(context, klass, args[1]);
     }
 
@@ -420,16 +420,11 @@ public class XmlDocument extends XmlNode {
     }
 
     public IRubyObject getInternalSubset(ThreadContext context) {
-        IRubyObject dtd =
-            (IRubyObject) node.getUserData(DTD_INTERNAL_SUBSET);
+        IRubyObject dtd = (IRubyObject) node.getUserData(DTD_INTERNAL_SUBSET);
 
         if (dtd == null) {
-            if (getDocument().getDoctype() == null)
-                dtd = context.getRuntime().getNil();
-            else
-                dtd = XmlDtd.newFromInternalSubset(context.getRuntime(),
-                                                   getDocument());
-
+            if (getDocument().getDoctype() == null) dtd = context.getRuntime().getNil();
+            else dtd = XmlDtd.newFromInternalSubset(context.getRuntime(), getDocument());
             setInternalSubset(dtd);
         }
 
@@ -456,15 +451,9 @@ public class XmlDocument extends XmlNode {
     }
 
     public IRubyObject getExternalSubset(ThreadContext context) {
-        IRubyObject dtd = (IRubyObject)
-            node.getUserData(DTD_EXTERNAL_SUBSET);
-
-        if (dtd == null) {
-            dtd = XmlDtd.newFromExternalSubset(context.getRuntime(),
-                                               getDocument());
-            setExternalSubset(dtd);
-        }
+        IRubyObject dtd = (IRubyObject) node.getUserData(DTD_EXTERNAL_SUBSET);
 
+        if (dtd == null) return context.getRuntime().getNil();
         return dtd;
     }
 
@@ -487,43 +476,44 @@ public class XmlDocument extends XmlNode {
         node.setUserData(DTD_EXTERNAL_SUBSET, data, null);
     }
 
-    //public IRubyObject createE
-
     @Override
-    public void saveContent(ThreadContext context, SaveContext ctx) {
-        if(!ctx.noDecl()) {
-            ctx.append("<?xml version=\"");
-            ctx.append(getDocument().getXmlVersion());
-            ctx.append("\"");
-//            if(!cur.encoding(context).isNil()) {
-//                ctx.append(" encoding=");
-//                ctx.append(cur.encoding(context).asJavaString());
-//            }
-
-            String encoding = ctx.getEncoding();
-
-            if(encoding == null &&
-                    !encoding(context).isNil()) {
-                encoding = encoding(context).convertToString().asJavaString();
-            }
-
-            if(encoding != null) {
-                ctx.append(" encoding=\"");
-                ctx.append(encoding);
-                ctx.append("\"");
+    public void accept(ThreadContext context, SaveContextVisitor visitor) {
+        Document document = getDocument();
+        visitor.enter(document);
+        NodeList children = document.getChildNodes();
+        for (int i=0; i<children.getLength(); i++) {
+            Node child = children.item(i);
+            short type = child.getNodeType();
+            if (type == Node.COMMENT_NODE) {
+                XmlComment xmlComment = (XmlComment) getCachedNodeOrCreate(context.getRuntime(), child);
+                xmlComment.accept(context, visitor);
+            } else if (type == Node.DOCUMENT_TYPE_NODE) {
+                XmlDtd xmlDtd = (XmlDtd) getCachedNodeOrCreate(context.getRuntime(), child);
+                xmlDtd.accept(context, visitor);
+            } else if (type == Node.PROCESSING_INSTRUCTION_NODE) {
+                XmlProcessingInstruction xmlProcessingInstruction = (XmlProcessingInstruction) getCachedNodeOrCreate(context.getRuntime(), child);
+                xmlProcessingInstruction.accept(context, visitor);
+            } else if (type == Node.TEXT_NODE) {
+                XmlText xmlText = (XmlText) getCachedNodeOrCreate(context.getRuntime(), child);
+                xmlText.accept(context, visitor);
+            } else if (type == Node.ELEMENT_NODE) {
+                XmlElement xmlElement = (XmlElement) getCachedNodeOrCreate(context.getRuntime(), child);
+                xmlElement.accept(context, visitor);
             }
-
-            //ctx.append(" standalone=\"");
-            //ctx.append(getDocument().getXmlStandalone() ? "yes" : "no");
-            ctx.append("?>\n");
         }
-
-        IRubyObject maybeRoot = root(context);
-        if (maybeRoot.isNil())
-            throw context.getRuntime().newRuntimeError("no root document");
-
-        XmlNode root = (XmlNode) maybeRoot;
-        root.saveContent(context, ctx);
-        ctx.append("\n");
+        visitor.leave(document);
+    }
+    
+    @JRubyMethod(meta=true)
+    public static IRubyObject wrapJavaDocument(ThreadContext context, IRubyObject klazz, IRubyObject arg) {
+        XmlDocument xmlDocument = (XmlDocument) NokogiriService.XML_DOCUMENT_ALLOCATOR.allocate(context.getRuntime(), getNokogiriClass(context.getRuntime(), "Nokogiri::XML::Document"));
+        Document document = (Document)arg.toJava(Document.class);
+        xmlDocument.setNode(context, document);
+        return xmlDocument;
+    }
+    
+    @JRubyMethod
+    public IRubyObject toJavaDocument(ThreadContext context) {
+        return JavaUtil.convertJavaToUsableRubyObject(context.getRuntime(), (org.w3c.dom.Document)node);
     }
 }