On Wed Jul 17 14:20:47 2024 +0000, Jacek Caban wrote:
There are more places that would need a similar change.
While looking for them I found this construct in some places (such as in `getPropertyValue`):
```c style_entry = lookup_style_tbl(This, name); nsAString_InitDepend(&name_str, style_entry ? style_entry->name : name); ``` What's the point of the whole `style_entry` thing? `lookup_style_tbl` uses case sensitive lookup, so it either always matches `name` or returns NULL. Even if it returns NULL due to `ATTR_COMPAT_IE10`, it ends up using name anyway which is the same thing?
Am I missing something?