Re: [PATCH 0/3] MR8503: mshtml: document.open and link color fixes.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/mshtml/htmldoc.c:
> - nsAString nsstr;
> - nsresult nsres;
> - HRESULT hres;
> -
> - TRACE("(%p)->(%p)\n", This, p);
> -
> - if(!This->html_document) {
> -
>    FIXME("Not implemented for XML document\n");
> -
>    return E_NOTIMPL;
> - }
> -
> - nsAString_Init(&nsstr, NULL);
> - nsres = nsIDOMHTMLDocument_GetLinkColor(This->html_document, &nsstr);
> - hres = return_nsstr_variant(nsres, &nsstr, NSSTR_COLOR, p);
> - if(hres == S_OK && V_VT(p) == VT_BSTR && !V_BSTR(p) && (!(V_BSTR(p) = SysAllocString(L"#0000ff"))))
> -
>    hres = E_OUTOFMEMORY;

Do you have anything depending on this fallback? My guess is that Gecko returns it only based on body element attributes, while IE fallbacks to computed value. If that's the case, then should CSS likely impact the fallback, so it's not necessarily '#0000ff`.

Such differences are rarely impactful, the usual practice is to just follow Gecko unless we have a reason not to.