msxml3: Implement IXMLDOMDocumentFragment Interface.

dlls/msxml3/Makefile.in

@@ -12,6 +12,7 @@ C_SRCS = \
 	attribute.c \
 	cdata.c \
 	comment.c \
+	docfrag.c \
 	domdoc.c \
 	domimpl.c \
 	element.c \

dlls/msxml3/docfrag.c

+/*
+ *    DOM Document Fragment implementation
+ *
+ * Copyright 2007 Alistair Leslie-Hughes
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
+ */
+
+#define COBJMACROS
+
+#include "config.h"
+
+#include <stdarg.h>
+#include "windef.h"
+#include "winbase.h"
+#include "winuser.h"
+#include "ole2.h"
+#include "msxml2.h"
+
+#include "msxml_private.h"
+
+#include "wine/debug.h"
+
+WINE_DEFAULT_DEBUG_CHANNEL(msxml);
+
+#ifdef HAVE_LIBXML2
+
+typedef struct _domfrag
+{
+    const struct IXMLDOMDocumentFragmentVtbl *lpVtbl;
+    LONG ref;
+    IUnknown *node_unk;
+    IXMLDOMNode *node;
+} domfrag;
+
+static inline domfrag *impl_from_IXMLDOMDocumentFragment( IXMLDOMDocumentFragment *iface )
+{
+    return (domfrag *)((char*)iface - FIELD_OFFSET(domfrag, lpVtbl));
+}
+
+static HRESULT WINAPI domfrag_QueryInterface(
+    IXMLDOMDocumentFragment *iface,
+    REFIID riid,
+    void** ppvObject )
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    TRACE("%p %s %p\n", This, debugstr_guid(riid), ppvObject);
+
+    if ( IsEqualGUID( riid, &IID_IXMLDOMDocumentFragment ) ||
+         IsEqualGUID( riid, &IID_IUnknown ) )
+    {
+        *ppvObject = iface;
+    }
+    else if ( IsEqualGUID( riid, &IID_IDispatch ) ||
+              IsEqualGUID( riid, &IID_IXMLDOMNode ) )
+    {
+        return IUnknown_QueryInterface(This->node_unk, riid, ppvObject);
+    }
+    else
+    {
+        FIXME("Unsupported inteferace %s\n", debugstr_guid(riid));
+        return E_NOINTERFACE;
+    }
+
+    IXMLDOMDocumentFragment_AddRef( iface );
+
+    return S_OK;
+}
+
+static ULONG WINAPI domfrag_AddRef(
+    IXMLDOMDocumentFragment *iface )
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return InterlockedIncrement( &This->ref );
+}
+
+static ULONG WINAPI domfrag_Release(
+    IXMLDOMDocumentFragment *iface )
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    ULONG ref;
+
+    ref = InterlockedDecrement( &This->ref );
+    if ( ref == 0 )
+    {
+        IUnknown_Release( This->node_unk );
+        HeapFree( GetProcessHeap(), 0, This );
+    }
+
+    return ref;
+}
+
+static HRESULT WINAPI domfrag_GetTypeInfoCount(
+    IXMLDOMDocumentFragment *iface,
+    UINT* pctinfo )
+{
+    FIXME("\n");
+    return E_NOTIMPL;
+}
+
+static HRESULT WINAPI domfrag_GetTypeInfo(
+    IXMLDOMDocumentFragment *iface,
+    UINT iTInfo, LCID lcid,
+    ITypeInfo** ppTInfo )
+{
+    FIXME("\n");
+    return E_NOTIMPL;
+}
+
+static HRESULT WINAPI domfrag_GetIDsOfNames(
+    IXMLDOMDocumentFragment *iface,
+    REFIID riid, LPOLESTR* rgszNames,
+    UINT cNames, LCID lcid, DISPID* rgDispId )
+{
+    FIXME("\n");
+    return E_NOTIMPL;
+}
+
+static HRESULT WINAPI domfrag_Invoke(
+    IXMLDOMDocumentFragment *iface,
+    DISPID dispIdMember, REFIID riid, LCID lcid,
+    WORD wFlags, DISPPARAMS* pDispParams, VARIANT* pVarResult,
+    EXCEPINFO* pExcepInfo, UINT* puArgErr )
+{
+    FIXME("\n");
+    return E_NOTIMPL;
+}
+
+static HRESULT WINAPI domfrag_get_nodeName(
+    IXMLDOMDocumentFragment *iface,
+    BSTR* p )
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_nodeName( This->node, p );
+}
+
+static HRESULT WINAPI domfrag_get_nodeValue(
+    IXMLDOMDocumentFragment *iface,
+    VARIANT* var1 )
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_nodeValue( This->node, var1 );
+}
+
+static HRESULT WINAPI domfrag_put_nodeValue(
+    IXMLDOMDocumentFragment *iface,
+    VARIANT var1 )
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_put_nodeValue( This->node, var1 );
+}
+
+static HRESULT WINAPI domfrag_get_nodeType(
+    IXMLDOMDocumentFragment *iface,
+    DOMNodeType* domNodeType )
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_nodeType( This->node, domNodeType );
+}
+
+static HRESULT WINAPI domfrag_get_parentNode(
+    IXMLDOMDocumentFragment *iface,
+    IXMLDOMNode** parent )
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_parentNode( This->node, parent );
+}
+
+static HRESULT WINAPI domfrag_get_childNodes(
+    IXMLDOMDocumentFragment *iface,
+    IXMLDOMNodeList** outList)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_childNodes( This->node, outList );
+}
+
+static HRESULT WINAPI domfrag_get_firstChild(
+    IXMLDOMDocumentFragment *iface,
+    IXMLDOMNode** domNode)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_firstChild( This->node, domNode );
+}
+
+static HRESULT WINAPI domfrag_get_lastChild(
+    IXMLDOMDocumentFragment *iface,
+    IXMLDOMNode** domNode)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_lastChild( This->node, domNode );
+}
+
+static HRESULT WINAPI domfrag_get_previousSibling(
+    IXMLDOMDocumentFragment *iface,
+    IXMLDOMNode** domNode)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_previousSibling( This->node, domNode );
+}
+
+static HRESULT WINAPI domfrag_get_nextSibling(
+    IXMLDOMDocumentFragment *iface,
+    IXMLDOMNode** domNode)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_nextSibling( This->node, domNode );
+}
+
+static HRESULT WINAPI domfrag_get_attributes(
+    IXMLDOMDocumentFragment *iface,
+    IXMLDOMNamedNodeMap** attributeMap)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_attributes( This->node, attributeMap );
+}
+
+static HRESULT WINAPI domfrag_insertBefore(
+    IXMLDOMDocumentFragment *iface,
+    IXMLDOMNode* newNode, VARIANT var1,
+    IXMLDOMNode** outOldNode)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_insertBefore( This->node, newNode, var1, outOldNode );
+}
+
+static HRESULT WINAPI domfrag_replaceChild(
+    IXMLDOMDocumentFragment *iface,
+    IXMLDOMNode* newNode,
+    IXMLDOMNode* oldNode,
+    IXMLDOMNode** outOldNode)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_replaceChild( This->node, newNode, oldNode, outOldNode );
+}
+
+static HRESULT WINAPI domfrag_removeChild(
+    IXMLDOMDocumentFragment *iface,
+    IXMLDOMNode* domNode, IXMLDOMNode** oldNode)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_removeChild( This->node, domNode, oldNode );
+}
+
+static HRESULT WINAPI domfrag_appendChild(
+    IXMLDOMDocumentFragment *iface,
+    IXMLDOMNode* newNode, IXMLDOMNode** outNewNode)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_appendChild( This->node, newNode, outNewNode );
+}
+
+static HRESULT WINAPI domfrag_hasChildNodes(
+    IXMLDOMDocumentFragment *iface,
+    VARIANT_BOOL* pbool)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_hasChildNodes( This->node, pbool );
+}
+
+static HRESULT WINAPI domfrag_get_ownerDocument(
+    IXMLDOMDocumentFragment *iface,
+    IXMLDOMDocument** domDocument)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_ownerDocument( This->node, domDocument );
+}
+
+static HRESULT WINAPI domfrag_cloneNode(
+    IXMLDOMDocumentFragment *iface,
+    VARIANT_BOOL pbool, IXMLDOMNode** outNode)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_cloneNode( This->node, pbool, outNode );
+}
+
+static HRESULT WINAPI domfrag_get_nodeTypeString(
+    IXMLDOMDocumentFragment *iface,
+    BSTR* p)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_nodeTypeString( This->node, p );
+}
+
+static HRESULT WINAPI domfrag_get_text(
+    IXMLDOMDocumentFragment *iface,
+    BSTR* p)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_text( This->node, p );
+}
+
+static HRESULT WINAPI domfrag_put_text(
+    IXMLDOMDocumentFragment *iface,
+    BSTR p)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_put_text( This->node, p );
+}
+
+static HRESULT WINAPI domfrag_get_specified(
+    IXMLDOMDocumentFragment *iface,
+    VARIANT_BOOL* pbool)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_specified( This->node, pbool );
+}
+
+static HRESULT WINAPI domfrag_get_definition(
+    IXMLDOMDocumentFragment *iface,
+    IXMLDOMNode** domNode)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_definition( This->node, domNode );
+}
+
+static HRESULT WINAPI domfrag_get_nodeTypedValue(
+    IXMLDOMDocumentFragment *iface,
+    VARIANT* var1)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_nodeTypedValue( This->node, var1 );
+}
+
+static HRESULT WINAPI domfrag_put_nodeTypedValue(
+    IXMLDOMDocumentFragment *iface,
+    VARIANT var1)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_put_nodeTypedValue( This->node, var1 );
+}
+
+static HRESULT WINAPI domfrag_get_dataType(
+    IXMLDOMDocumentFragment *iface,
+    VARIANT* var1)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_dataType( This->node, var1 );
+}
+
+static HRESULT WINAPI domfrag_put_dataType(
+    IXMLDOMDocumentFragment *iface,
+    BSTR p)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_put_dataType( This->node, p );
+}
+
+static HRESULT WINAPI domfrag_get_xml(
+    IXMLDOMDocumentFragment *iface,
+    BSTR* p)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_xml( This->node, p );
+}
+
+static HRESULT WINAPI domfrag_transformNode(
+    IXMLDOMDocumentFragment *iface,
+    IXMLDOMNode* domNode, BSTR* p)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_transformNode( This->node, domNode, p );
+}
+
+static HRESULT WINAPI domfrag_selectNodes(
+    IXMLDOMDocumentFragment *iface,
+    BSTR p, IXMLDOMNodeList** outList)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_selectNodes( This->node, p, outList );
+}
+
+static HRESULT WINAPI domfrag_selectSingleNode(
+    IXMLDOMDocumentFragment *iface,
+    BSTR p, IXMLDOMNode** outNode)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_selectSingleNode( This->node, p, outNode );
+}
+
+static HRESULT WINAPI domfrag_get_parsed(
+    IXMLDOMDocumentFragment *iface,
+    VARIANT_BOOL* pbool)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_parsed( This->node, pbool );
+}
+
+static HRESULT WINAPI domfrag_get_namespaceURI(
+    IXMLDOMDocumentFragment *iface,
+    BSTR* p)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_namespaceURI( This->node, p );
+}
+
+static HRESULT WINAPI domfrag_get_prefix(
+    IXMLDOMDocumentFragment *iface,
+    BSTR* p)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_prefix( This->node, p );
+}
+
+static HRESULT WINAPI domfrag_get_baseName(
+    IXMLDOMDocumentFragment *iface,
+    BSTR* p)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_get_baseName( This->node, p );
+}
+
+static HRESULT WINAPI domfrag_transformNodeToObject(
+    IXMLDOMDocumentFragment *iface,
+    IXMLDOMNode* domNode, VARIANT var1)
+{
+    domfrag *This = impl_from_IXMLDOMDocumentFragment( iface );
+    return IXMLDOMNode_transformNodeToObject( This->node, domNode, var1 );
+}
+
+static const struct IXMLDOMDocumentFragmentVtbl domfrag_vtbl =
+{
+    domfrag_QueryInterface,
+    domfrag_AddRef,
+    domfrag_Release,
+    domfrag_GetTypeInfoCount,
+    domfrag_GetTypeInfo,
+    domfrag_GetIDsOfNames,
+    domfrag_Invoke,
+    domfrag_get_nodeName,
+    domfrag_get_nodeValue,
+    domfrag_put_nodeValue,
+    domfrag_get_nodeType,
+    domfrag_get_parentNode,
+    domfrag_get_childNodes,
+    domfrag_get_firstChild,
+    domfrag_get_lastChild,
+    domfrag_get_previousSibling,
+    domfrag_get_nextSibling,
+    domfrag_get_attributes,
+    domfrag_insertBefore,
+    domfrag_replaceChild,
+    domfrag_removeChild,
+    domfrag_appendChild,
+    domfrag_hasChildNodes,
+    domfrag_get_ownerDocument,
+    domfrag_cloneNode,
+    domfrag_get_nodeTypeString,
+    domfrag_get_text,
+    domfrag_put_text,
+    domfrag_get_specified,
+    domfrag_get_definition,
+    domfrag_get_nodeTypedValue,
+    domfrag_put_nodeTypedValue,
+    domfrag_get_dataType,
+    domfrag_put_dataType,
+    domfrag_get_xml,
+    domfrag_transformNode,
+    domfrag_selectNodes,
+    domfrag_selectSingleNode,
+    domfrag_get_parsed,
+    domfrag_get_namespaceURI,
+    domfrag_get_prefix,
+    domfrag_get_baseName,
+    domfrag_transformNodeToObject
+};
+
+IUnknown* create_doc_fragment( xmlNodePtr fragment )
+{
+    domfrag *This;
+    HRESULT hr;
+
+    This = HeapAlloc( GetProcessHeap(), 0, sizeof *This );
+    if ( !This )
+        return NULL;
+
+    This->lpVtbl = &domfrag_vtbl;
+    This->ref = 1;
+
+    This->node_unk = create_basic_node( fragment, (IUnknown*)&This->lpVtbl );
+    if(!This->node_unk)
+    {
+        HeapFree(GetProcessHeap(), 0, This);
+        return NULL;
+    }
+
+    hr = IUnknown_QueryInterface(This->node_unk, &IID_IXMLDOMNode, (LPVOID*)&This->node);
+    if(FAILED(hr))
+    {
+        IUnknown_Release(This->node_unk);
+        HeapFree( GetProcessHeap(), 0, This );
+        return NULL;
+    }
+    /* The ref on This->node is actually looped back into this object, so release it */
+    IXMLDOMNode_Release(This->node);
+
+    return (IUnknown*) &This->lpVtbl;
+}
+
+#endif

dlls/msxml3/domdoc.c

@@ -885,8 +885,26 @@ static HRESULT WINAPI domdoc_createDocumentFragment(
     IXMLDOMDocument2 *iface,
     IXMLDOMDocumentFragment** docFrag )
 {
-    FIXME("\n");
-    return E_NOTIMPL;
+    domdoc *This = impl_from_IXMLDOMDocument2( iface );
+    xmlNodePtr xmlnode;
+
+    TRACE("%p\n", iface);
+
+    if(!docFrag)
+        return E_INVALIDARG;
+
+    *docFrag = NULL;
+
+    xmlnode = xmlNewDocFragment(get_doc( This ) );
+
+    if(!xmlnode)
+        return E_FAIL;
+
+    xmlnode->doc = get_doc( This );
+
+    *docFrag = (IXMLDOMDocumentFragment*)create_doc_fragment(xmlnode);
+
+    return S_OK;
 }
 
 

dlls/msxml3/msxml_private.h

@@ -45,6 +45,7 @@ extern IUnknown         *create_cdata( xmlNodePtr text );
 extern IXMLDOMNodeList  *create_children_nodelist( xmlNodePtr );
 extern IXMLDOMNamedNodeMap *create_nodemap( IXMLDOMNode *node );
 extern IUnknown         *create_doc_Implementation();
+extern IUnknown         *create_doc_fragment( xmlNodePtr fragment );
 
 extern HRESULT queryresult_create( xmlNodePtr, LPWSTR, IXMLDOMNodeList ** );
 

dlls/msxml3/tests/domdoc.c

@@ -154,6 +154,7 @@ static WCHAR szCDataXML[] = {'<','!','[','C','D','A','T','A','[','[','1',']','*'
                              'g','e','e',' ','t','h','a','t','s',' ','n','o','t',' ','r','i','g','h','t',
                              '!',']',']','>',0};
 static WCHAR szCDataNodeText[] = {'#','c','d','a','t','a','-','s','e','c','t','i','o','n',0 };
+static WCHAR szDocFragmentText[] = {'#','d','o','c','u','m','e','n','t','-','f','r','a','g','m','e','n','t',0 };
 
 #define expect_bstr_eq_and_free(bstr, expect) { \
     BSTR bstrExp = alloc_str_from_narrow(expect); \
@@ -1873,6 +1874,7 @@ static void test_xmlTypes(void)
     IXMLDOMNamedNodeMap *pAttribs;
     IXMLDOMCDATASection *pCDataSec;
     IXMLDOMImplementation *pIXMLDOMImplementation = NULL;
+    IXMLDOMDocumentFragment *pDocFrag = NULL;
     BSTR str;
     IXMLDOMNode *pNextChild = (IXMLDOMNode *)0x1;   /* Used for testing Siblings */
     VARIANT v;
@@ -2194,6 +2196,70 @@ static void test_xmlTypes(void)
                 IXMLDOMCDATASection_Release(pCDataSec);
             }
 
+            /* Document Fragments */
+            hr = IXMLDOMDocument_createDocumentFragment(doc, NULL);
+            ok(hr == E_INVALIDARG, "ret %08x\n", hr );
+
+            hr = IXMLDOMDocument_createDocumentFragment(doc, &pDocFrag);
+            ok(hr == S_OK, "ret %08x\n", hr );
+            if(hr == S_OK)
+            {
+                IXMLDOMNode *pNextChild = (IXMLDOMNode *)0x1;
+
+                hr = IXMLDOMElement_appendChild(pRoot, (IXMLDOMNode*)pDocFrag, NULL);
+                ok(hr == S_OK, "ret %08x\n", hr );
+
+                /* get Attribute Tests */
+                hr = IXMLDOMDocumentFragment_get_attributes(pDocFrag, NULL);
+                ok(hr == E_INVALIDARG, "ret %08x\n", hr );
+
+                pAttribs = (IXMLDOMNamedNodeMap*)0x1;
+                hr = IXMLDOMDocumentFragment_get_attributes(pDocFrag, &pAttribs);
+                ok(hr == S_FALSE, "ret %08x\n", hr );
+                ok(pAttribs == NULL, "pAttribs != NULL\n");
+
+                hr = IXMLDOMDocumentFragment_get_nodeName(pDocFrag, &str);
+                ok(hr == S_OK, "ret %08x\n", hr );
+                ok( !lstrcmpW( str, szDocFragmentText ), "incorrect docfragment node Name\n");
+                SysFreeString(str);
+
+                /* test next Sibling*/
+                hr = IXMLDOMDocumentFragment_get_nextSibling(pDocFrag, NULL);
+                ok(hr == E_INVALIDARG, "ret %08x\n", hr );
+
+                pNextChild = (IXMLDOMNode *)0x1;
+                hr = IXMLDOMDocumentFragment_get_nextSibling(pDocFrag, &pNextChild);
+                ok(hr == S_FALSE, "ret %08x\n", hr );
+                ok(pNextChild == NULL, "pNextChild not NULL\n");
+
+                /* test Previous Sibling*/
+                hr = IXMLDOMDocumentFragment_get_previousSibling(pDocFrag, NULL);
+                ok(hr == E_INVALIDARG, "ret %08x\n", hr );
+
+                pNextChild = (IXMLDOMNode *)0x1;
+                hr = IXMLDOMDocumentFragment_get_previousSibling(pDocFrag, &pNextChild);
+                ok(hr == S_FALSE, "ret %08x\n", hr );
+                ok(pNextChild == NULL, "pNextChild not NULL\n");
+
+                /* test get_dataType */
+                hr = IXMLDOMDocumentFragment_get_dataType(pDocFrag, NULL);
+                ok(hr == E_INVALIDARG, "ret %08x\n", hr );
+
+                hr = IXMLDOMDocumentFragment_get_dataType(pDocFrag, &v);
+                ok(hr == S_FALSE, "ret %08x\n", hr );
+                ok( V_VT(&v) == VT_NULL, "incorrect dataType type\n");
+                VariantClear(&v);
+
+                /* test nodeTypeString */
+                hr = IXMLDOMDocumentFragment_get_nodeTypeString(pDocFrag, &str);
+                ok(hr == S_OK, "ret %08x\n", hr );
+                ok( !lstrcmpW( str, _bstr_("documentfragment") ), "incorrect nodeTypeString string\n");
+                SysFreeString(str);
+
+                IXMLDOMDocumentFragment_Release(pCDataSec);
+            }
+
+
 
 
             IXMLDOMElement_Release( pRoot );