Changeset 24873 in webkit for trunk/JavaScriptCore/kjs/string_object.h

Timestamp:

Aug 4, 2007, 10:20:35 PM (18 years ago)

Author:

mjs

Message:

Reviewed by Darin and Adam.

<rdar://problem/5368990> REGRESSION: newsgator.com sign-on 6x slower than Safari 3 beta due to GC changes (14808)

• kjs/string_object.cpp: (KJS::replace): if the string didn't change (very common in some cases) reuse the original string value. (KJS::StringProtoFunc::callAsFunction): Pass in the StringImp* when replacing, not just the UString.
• kjs/string_object.h: (KJS::StringInstance::internalValue): covariant override to return StringImp for convenience

File:

• trunk/JavaScriptCore/kjs/string_object.h (modified) (1 diff)

trunk/JavaScriptCore/kjs/string_object.h

@@ -42 +42 @@
     virtual const ClassInfo *classInfo() const { return &info; }
     static const ClassInfo info;
+
+    StringImp* internalValue() const { return static_cast<StringImp*>(JSWrapperObject::internalValue());}
+
   private:
     static JSValue *lengthGetter(ExecState *exec, JSObject *, const Identifier&, const PropertySlot &slot);