It used to be that we had PodEqual and it required both arguments to have the same type, so I added EqualChars for use when this is not true (typically when comparing Latin1Char and char16_t strings). However, we now use ArrayEqual instead of PodEqual, and EqualChars also just calls ArrayEqual.

In other words, using both functions depending on argument types like we do here is no longer necessary.

Waldo, do you feel like we should just use mozilla::ArrayEqual directly? I don't care much, I just don't like using both in the same code.