On Fri Nov 22 15:40:33 2024 +0000, Gabriel Ivăncescu wrote:
> On second thought maybe just forwarding to `IHTMLCommentElement2` would
> be enough, but we still need the `IHTMLDOMTextNode2` so the prototype
> works right?
Yeah, it shows that those interfaces do not reflect prototypes. In a sense, `IHTMLDOMTextNode2` is as bad for `CharacterData` as `IHTMLCommentElement2`. Do we have a lot of cases like that? I'm wondering if we need to change the mechanism or could we just introduce a private `IHTMLCharacterData` interface.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/6887#note_88529
On Fri Nov 22 15:39:38 2024 +0000, Gabriel Ivăncescu wrote:
> Interesting, but I'm not sure. The issue here is that these are in the
> `CharacterDataPrototype`, not in the comment element itself. So, won't
> they fail if we don't handle `IHTMLDOMTextNode2` when called on the
> comment element? Since it's looked up by IID/DISPID pair, not name.
> `IHTMLCommentElement2` might be used for legacy modes though, I guess I
> will need to test it from C code to know.
On second thought maybe just forwarding to `IHTMLCommentElement2` would be enough, but we still need the `IHTMLDOMTextNode2` so the prototype works right?
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/6887#note_88528
Probably caused by some winehq.org HTTP server update, it now supports HTTP/2 and sends "Upgrade, Keep-Alive" connection strings (was "Keep-Alive" before).
--
v3: wininet: Parse multi-token Connection strings for Keep-Alive.
https://gitlab.winehq.org/wine/wine/-/merge_requests/6882
On Fri Nov 22 15:33:09 2024 +0000, Jacek Caban wrote:
> Shouldn't it be `IHTMLCommentElement2` instead?
Interesting, but I'm not sure. The issue here is that these are in the `CharacterDataPrototype`, not in the comment element itself. So, won't they fail if we don't handle `IHTMLDOMTextNode2` when called on the comment element? Since it's looked up by IID/DISPID pair, not name.
`IHTMLCommentElement2` might be used for legacy modes though, I guess I will need to test it from C code to know.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/6887#note_88525
Probably caused by some winehq.org HTTP server update, it now supports HTTP/2 and sends "Upgrade, Keep-Alive" connection strings (was "Keep-Alive" before).
--
v2: wininet: Parse multi-token Connection strings for Keep-Alive.
https://gitlab.winehq.org/wine/wine/-/merge_requests/6882