Re: [PATCH v2 0/7] MR8251: mshtml: getAttributeNode and removal fixes.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/mshtml/htmlelem.c:
>  return S_OK;
>
> }
>
> +static inline BOOL is_valid_attr_dispid(HTMLAttributeCollection *col, DISPID id)
> +{
> - /* Builtin props (non-methods) that are not attributes; MUST be sorted by underlying dispid value. */

Following DISPIDs for manual sorting feels awkward. More generally, what’s being implemented here is essentially an additional “noattr” property flag for dispex properties. We can’t express that directly in the IDL, but we already have a mechanism for supplementing missing type library info via dispex hooks (though perhaps we should consider renaming them). We could extend `dispex_hook_t` to include an additional field for such flags.