Re: [PATCH v2 0/3] MR9131: iertutil: Add an initial URI parser.

[Jacek Caban (＠jacek)]

> +    hr = IUri_GetAbsoluteUri(pBaseUri, &base_url);

This is not always the same as `base->canon_uri`, for example (but not only) when URI_DISPLAY_NO_ABSOLUTE_URI is set. Did you check that GetAbsoluteUri is what we need?

Also, it would be great if you could split that massive patch. You could first factor out those private functions while keeping everything in urlmon (so it is clear what changes there), and then move them to iertutil in a separate patch. That way, the patch moving the code would not need to touch those `CoInternet*` functions.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/9131#note_133769