[PATCH] msxml3/tests: Add mxwriter DOMDocument test.

[Jefferson Carpenter]

Let me know what you think about this test.

I could add a double for loop to check over the whole Cartesian product 
of CLSID_MXXMLWriter* and CLSID_DOMDocument* structs.

P.S. We'll see what Marvin has to say about this.  (If Marvin rejects 
it, which he probably will, I'll check support data).

[Nikolay Sivov]

On 1/20/21 2:15 AM, Jefferson Carpenter wrote:
> Let me know what you think about this test.
>
> I could add a double for loop to check over the whole Cartesian
> product of CLSID_MXXMLWriter* and CLSID_DOMDocument* structs.
>
> P.S. We'll see what Marvin has to say about this.  (If Marvin rejects
> it, which he probably will, I'll check support data).

I think the question is whether MXWriter explicitly supports document
for output, or if it's simply a case of document not exposing IStream,
see [1].

[1] https://bugs.winehq.org/show_bug.cgi?id=4811
https://bugs.winehq.org/show_bug.cgi?id=4811

[Jefferson Carpenter]

On 1/20/2021 7:14 AM, Nikolay Sivov wrote:
> I think the question is whether MXWriter explicitly supports document
> for output, or if it's simply a case of document not exposing IStream,
> see [1].
>
> [1] https://bugs.winehq.org/show_bug.cgi?id=4811
> https://bugs.winehq.org/show_bug.cgi?id=4811

That could be the case.  I could write a class that implements IStream 
and IDispatch and pass it in the V_DISPATCH slot to put_output and see 
how it's used.

I was just going by the msdn page that says the IMXWriter output 
property can be set to either an IStream, a string, or a DOMDocument 
object [1]

Strangely - and I don't understand this - if I cast the IXMLDOMDocument 
to an IDispatch (patch) or query for IDispatch, the test succeeds on 
Windows, but if I query for IStream and cast to IDispatch it does not. 
I should probably query for IDispatch in the patch so that it doesn't 
perform unnecessary casting.

[1] 
https://docs.microsoft.com/en-us/previous-versions/windows/desktop/ms755493(...)

[Jefferson Carpenter]

On 1/20/2021 6:07 PM, Jefferson Carpenter wrote:
> I was just going by the msdn page that says the IMXWriter output 
> property can be set to either an IStream, a string, or a DOMDocument 
> object [1]

Well, that and some FIXME output that says "unhandled destination type 
0x3181c0 {VT_DISPATCH: 0x2253010}" when I run the game Rise of Legends.