Hi Gabriel, On 11/29/21 5:31 PM, Gabriel Ivăncescu wrote:
+static inline WCHAR *translate_attr_name(WCHAR *attr_name, compat_mode_t compat_mode) +{ + WCHAR *ret = attr_name; + + if(compat_mode >= COMPAT_MODE_IE8 && !wcsicmp(attr_name, L"class")) + ret = (WCHAR*)L"className"; + return ret; +}
Please avoid this cast.
+ if(get_dispid_type(dispid) == DISPEXPROP_BUILTIN) { + VariantClear(AttributeValue); + V_VT(AttributeValue) = VT_NULL; + return S_OK; + }
It's not right in general, see the attached test. I'm not expecting that you implement all those IE8-specific things (in fact, I don't think we need to care too much about it), but the patch in its current shape can break things. Could you just remove this if() and live todo_wines? Thanks, Jacek