Static strings [ i.e. UString() and UString("") ] are used as Identifiers, but 'isIdentifier()' returns false. This is a little screwy, and also a little suboptimal – Identifier::add(UString("")) will unnecessarily hit addSlowCase, rather than being handled in the inline function in the header. The performance is probably not a big issue here, but the code is a little clearer if we make all UStringImpls that are used as Identifers return true from isIdentifier(). Add missing asserts to JSC::UStringImpl re static strings, and add matching asserts to WebCore::StringImpl.
Created attachment 50638 [details] The patch No performance impact
Comment on attachment 50638 [details] The patch r=me
fixed in r55943.