Re: [PATCH 0/3] MR1461: ieframe: Remove function heap_strdupW.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/ieframe/navigate.c:
>  WCHAR *new_url;
>
>  if(url) {
> -
>    new_url = heap_strdupW(url);
> -
>    new_url = wcsdup(url);
>    if(!new_url)
>        return E_OUTOFMEMORY;
> }else {
>        new_url = NULL;
>    }
> - heap_free(This->url);
> - free(This->url);

This does not match `DocHost_Release`. It's not the first such case, the way you selectively convert allocations is both more error-prone and harder to review than needed.