JavaScriptCore-1097.13.tar.gz

[apple/javascriptcore.git] / runtime / JSONObject.cpp

diff --git a/runtime/JSONObject.cpp b/runtime/JSONObject.cpp
index ce0dcffe0569565fe78a3d01d4113dda68627a5d..436e78353b170a07506157053c704b56e3f23b39 100644 (file)
--- a/runtime/JSONObject.cpp
+++ b/runtime/JSONObject.cpp
@@ -30,17 +30,23 @@
 #include "Error.h"
 #include "ExceptionHelpers.h"
 #include "JSArray.h"
 #include "Error.h"
 #include "ExceptionHelpers.h"
 #include "JSArray.h"

+#include "JSGlobalObject.h"

 #include "LiteralParser.h"
 #include "LiteralParser.h"

+#include "Local.h"
+#include "LocalScope.h"
+#include "Lookup.h"

 #include "PropertyNameArray.h"
 #include "PropertyNameArray.h"

-#include "StringBuilder.h"
+#include "UStringBuilder.h"
+#include "UStringConcatenate.h"

 #include <wtf/MathExtras.h>
 
 namespace JSC {
 
 ASSERT_CLASS_FITS_IN_CELL(JSONObject);
 #include <wtf/MathExtras.h>
 
 namespace JSC {
 
 ASSERT_CLASS_FITS_IN_CELL(JSONObject);

+ASSERT_HAS_TRIVIAL_DESTRUCTOR(JSONObject);

 
 

-static JSValue JSC_HOST_CALL JSONProtoFuncParse(ExecState*, JSObject*, JSValue, const ArgList&);
-static JSValue JSC_HOST_CALL JSONProtoFuncStringify(ExecState*, JSObject*, JSValue, const ArgList&);
+static EncodedJSValue JSC_HOST_CALL JSONProtoFuncParse(ExecState*);
+static EncodedJSValue JSC_HOST_CALL JSONProtoFuncStringify(ExecState*);

 
 }
 
 
 }
 


@@ -48,6 +54,17 @@ static JSValue JSC_HOST_CALL JSONProtoFuncStringify(ExecState*, JSObject*, JSVal
 
 namespace JSC {
 
 
 namespace JSC {
 

+JSONObject::JSONObject(JSGlobalObject* globalObject, Structure* structure)
+    : JSNonFinalObject(globalObject->globalData(), structure)
+{
+}
+
+void JSONObject::finishCreation(JSGlobalObject* globalObject)
+{
+    Base::finishCreation(globalObject->globalData());
+    ASSERT(inherits(&s_info));
+}
+

 // PropertyNameForFunctionCall objects must be on the stack, since the JSValue that they create is not marked.
 class PropertyNameForFunctionCall {
 public:
 // PropertyNameForFunctionCall objects must be on the stack, since the JSValue that they create is not marked.
 class PropertyNameForFunctionCall {
 public:


@@ -62,25 +79,25 @@ private:
     mutable JSValue m_value;
 };
 
     mutable JSValue m_value;
 };
 

-class Stringifier : public Noncopyable {
+class Stringifier {
+    WTF_MAKE_NONCOPYABLE(Stringifier);

 public:
 public:

-    Stringifier(ExecState*, JSValue replacer, JSValue space);
-    ~Stringifier();
-    JSValue stringify(JSValue);
+    Stringifier(ExecState*, const Local<Unknown>& replacer, const Local<Unknown>& space);
+    Local<Unknown> stringify(Handle<Unknown>);

 
 

-    void markAggregate(MarkStack&);
+    void visitAggregate(SlotVisitor&);

 
 private:
     class Holder {
     public:
 
 private:
     class Holder {
     public:

-        Holder(JSObject*);
+        Holder(JSGlobalData&, JSObject*);

 
 

-        JSObject* object() const { return m_object; }
+        JSObject* object() const { return m_object.get(); }

 
 

-        bool appendNextProperty(Stringifier&, StringBuilder&);
+        bool appendNextProperty(Stringifier&, UStringBuilder&);

 
     private:
 
     private:

-        JSObject* const m_object;
+        Local<JSObject> m_object;

         const bool m_isArray;
         bool m_isJSArray;
         unsigned m_index;
         const bool m_isArray;
         bool m_isJSArray;
         unsigned m_index;


@@ -90,28 +107,26 @@ private:
 
     friend class Holder;
 
 
     friend class Holder;
 

-    static void appendQuotedString(StringBuilder&, const UString&);
+    static void appendQuotedString(UStringBuilder&, const UString&);

 
     JSValue toJSON(JSValue, const PropertyNameForFunctionCall&);
 
     enum StringifyResult { StringifyFailed, StringifySucceeded, StringifyFailedDueToUndefinedValue };
 
     JSValue toJSON(JSValue, const PropertyNameForFunctionCall&);
 
     enum StringifyResult { StringifyFailed, StringifySucceeded, StringifyFailedDueToUndefinedValue };

-    StringifyResult appendStringifiedValue(StringBuilder&, JSValue, JSObject* holder, const PropertyNameForFunctionCall&);
+    StringifyResult appendStringifiedValue(UStringBuilder&, JSValue, JSObject* holder, const PropertyNameForFunctionCall&);

 
     bool willIndent() const;
     void indent();
     void unindent();
 
     bool willIndent() const;
     void indent();
     void unindent();

-    void startNewLine(StringBuilder&) const;
+    void startNewLine(UStringBuilder&) const;

 
 

-    Stringifier* const m_nextStringifierToMark;

     ExecState* const m_exec;
     ExecState* const m_exec;

-    const JSValue m_replacer;
+    const Local<Unknown> m_replacer;

     bool m_usingArrayReplacer;
     PropertyNameArray m_arrayReplacerPropertyNames;
     CallType m_replacerCallType;
     CallData m_replacerCallData;
     const UString m_gap;
 
     bool m_usingArrayReplacer;
     PropertyNameArray m_arrayReplacerPropertyNames;
     CallType m_replacerCallType;
     CallData m_replacerCallData;
     const UString m_gap;
 

-    HashSet<JSObject*> m_holderCycleDetector;

     Vector<Holder, 16> m_holderStack;
     UString m_repeatedGap;
     UString m_indent;
     Vector<Holder, 16> m_holderStack;
     UString m_repeatedGap;
     UString m_indent;


@@ -124,23 +139,23 @@ static inline JSValue unwrapBoxedPrimitive(ExecState* exec, JSValue value)
     if (!value.isObject())
         return value;
     JSObject* object = asObject(value);
     if (!value.isObject())
         return value;
     JSObject* object = asObject(value);

-    if (object->inherits(&NumberObject::info))
-        return jsNumber(exec, object->toNumber(exec));
-    if (object->inherits(&StringObject::info))
-        return jsString(exec, object->toString(exec));
-    if (object->inherits(&BooleanObject::info))
+    if (object->inherits(&NumberObject::s_info))
+        return jsNumber(object->toNumber(exec));
+    if (object->inherits(&StringObject::s_info))
+        return object->toString(exec);
+    if (object->inherits(&BooleanObject::s_info))

         return object->toPrimitive(exec);
     return value;
 }
 
 static inline UString gap(ExecState* exec, JSValue space)
 {
         return object->toPrimitive(exec);
     return value;
 }
 
 static inline UString gap(ExecState* exec, JSValue space)
 {

-    const int maxGapLength = 10;
+    const unsigned maxGapLength = 10;

     space = unwrapBoxedPrimitive(exec, space);
 
     // If the space value is a number, create a gap string with that number of spaces.
     space = unwrapBoxedPrimitive(exec, space);
 
     // If the space value is a number, create a gap string with that number of spaces.

-    double spaceCount;
-    if (space.getNumber(spaceCount)) {
+    if (space.isNumber()) {
+        double spaceCount = space.asNumber();

         int count;
         if (spaceCount > maxGapLength)
             count = maxGapLength;
         int count;
         if (spaceCount > maxGapLength)
             count = maxGapLength;


@@ -156,8 +171,8 @@ static inline UString gap(ExecState* exec, JSValue space)
 
     // If the space value is a string, use it as the gap string, otherwise use no gap string.
     UString spaces = space.getString(exec);
 
     // If the space value is a string, use it as the gap string, otherwise use no gap string.
     UString spaces = space.getString(exec);

-    if (spaces.size() > maxGapLength) {
-        spaces = spaces.substr(0, maxGapLength);
+    if (spaces.length() > maxGapLength) {
+        spaces = spaces.substringSharingImpl(0, maxGapLength);

     }
     return spaces;
 }
     }
     return spaces;
 }


@@ -181,106 +196,67 @@ JSValue PropertyNameForFunctionCall::value(ExecState* exec) const
         if (m_identifier)
             m_value = jsString(exec, m_identifier->ustring());
         else
         if (m_identifier)
             m_value = jsString(exec, m_identifier->ustring());
         else

-            m_value = jsNumber(exec, m_number);
+            m_value = jsNumber(m_number);

     }
     return m_value;
 }
 
 // ------------------------------ Stringifier --------------------------------
 
     }
     return m_value;
 }
 
 // ------------------------------ Stringifier --------------------------------
 

-Stringifier::Stringifier(ExecState* exec, JSValue replacer, JSValue space)
-    : m_nextStringifierToMark(exec->globalData().firstStringifierToMark)
-    , m_exec(exec)
+Stringifier::Stringifier(ExecState* exec, const Local<Unknown>& replacer, const Local<Unknown>& space)
+    : m_exec(exec)

     , m_replacer(replacer)
     , m_usingArrayReplacer(false)
     , m_arrayReplacerPropertyNames(exec)
     , m_replacerCallType(CallTypeNone)
     , m_replacer(replacer)
     , m_usingArrayReplacer(false)
     , m_arrayReplacerPropertyNames(exec)
     , m_replacerCallType(CallTypeNone)

-    , m_gap(gap(exec, space))
+    , m_gap(gap(exec, space.get()))

 {
 {

-    exec->globalData().firstStringifierToMark = this;
-

     if (!m_replacer.isObject())
         return;
 
     if (!m_replacer.isObject())
         return;
 

-    if (asObject(m_replacer)->inherits(&JSArray::info)) {
+    if (m_replacer.asObject()->inherits(&JSArray::s_info)) {

         m_usingArrayReplacer = true;
         m_usingArrayReplacer = true;

-        JSObject* array = asObject(m_replacer);
+        Handle<JSObject> array = m_replacer.asObject();

         unsigned length = array->get(exec, exec->globalData().propertyNames->length).toUInt32(exec);
         for (unsigned i = 0; i < length; ++i) {
             JSValue name = array->get(exec, i);
             if (exec->hadException())
                 break;
 
         unsigned length = array->get(exec, exec->globalData().propertyNames->length).toUInt32(exec);
         for (unsigned i = 0; i < length; ++i) {
             JSValue name = array->get(exec, i);
             if (exec->hadException())
                 break;
 

-            UString propertyName;
-            if (name.getString(exec, propertyName)) {
-                m_arrayReplacerPropertyNames.add(Identifier(exec, propertyName));
-                continue;
-            }
-
-            double value = 0;
-            if (name.getNumber(value)) {
-                m_arrayReplacerPropertyNames.add(Identifier::from(exec, value));
-                continue;
-            }
-

             if (name.isObject()) {
             if (name.isObject()) {

-                if (!asObject(name)->inherits(&NumberObject::info) && !asObject(name)->inherits(&StringObject::info))
+                if (!asObject(name)->inherits(&NumberObject::s_info) && !asObject(name)->inherits(&StringObject::s_info))

                     continue;
                     continue;

-                propertyName = name.toString(exec);
-                if (exec->hadException())
-                    break;
-                m_arrayReplacerPropertyNames.add(Identifier(exec, propertyName));

             }
             }

+
+            m_arrayReplacerPropertyNames.add(Identifier(exec, name.toString(exec)->value(exec)));

         }
         return;
     }
 
         }
         return;
     }
 

-    m_replacerCallType = asObject(m_replacer)->getCallData(m_replacerCallData);
-}
-
-Stringifier::~Stringifier()
-{
-    ASSERT(m_exec->globalData().firstStringifierToMark == this);
-    m_exec->globalData().firstStringifierToMark = m_nextStringifierToMark;
-}
-
-void Stringifier::markAggregate(MarkStack& markStack)
-{
-    for (Stringifier* stringifier = this; stringifier; stringifier = stringifier->m_nextStringifierToMark) {
-        size_t size = m_holderStack.size();
-        for (size_t i = 0; i < size; ++i)
-            markStack.append(m_holderStack[i].object());
-    }
+    m_replacerCallType = m_replacer.asObject()->methodTable()->getCallData(m_replacer.asObject().get(), m_replacerCallData);

 }
 
 }
 

-JSValue Stringifier::stringify(JSValue value)
+Local<Unknown> Stringifier::stringify(Handle<Unknown> value)

 {
     JSObject* object = constructEmptyObject(m_exec);
     if (m_exec->hadException())
 {
     JSObject* object = constructEmptyObject(m_exec);
     if (m_exec->hadException())

-        return jsNull();
+        return Local<Unknown>(m_exec->globalData(), jsNull());

 
     PropertyNameForFunctionCall emptyPropertyName(m_exec->globalData().propertyNames->emptyIdentifier);
 
     PropertyNameForFunctionCall emptyPropertyName(m_exec->globalData().propertyNames->emptyIdentifier);

-    object->putDirect(m_exec->globalData().propertyNames->emptyIdentifier, value);
+    object->putDirect(m_exec->globalData(), m_exec->globalData().propertyNames->emptyIdentifier, value.get());

 
 

-    StringBuilder result;
-    if (appendStringifiedValue(result, value, object, emptyPropertyName) != StringifySucceeded)
-        return jsUndefined();
+    UStringBuilder result;
+    if (appendStringifiedValue(result, value.get(), object, emptyPropertyName) != StringifySucceeded)
+        return Local<Unknown>(m_exec->globalData(), jsUndefined());

     if (m_exec->hadException())
     if (m_exec->hadException())

-        return jsNull();
+        return Local<Unknown>(m_exec->globalData(), jsNull());

 
 

-    return jsString(m_exec, result.release());
+    return Local<Unknown>(m_exec->globalData(), jsString(m_exec, result.toUString()));

 }
 
 }
 

-void Stringifier::appendQuotedString(StringBuilder& builder, const UString& value)
+template <typename CharType>
+static void appendStringToUStringBuilder(UStringBuilder& builder, const CharType* data, int length)

 {
 {

-    int length = value.size();
-
-    // String length plus 2 for quote marks plus 8 so we can accomodate a few escaped characters.
-    builder.reserveCapacity(builder.size() + length + 2 + 8);
-
-    builder.append('"');
-
-    const UChar* data = value.data();

     for (int i = 0; i < length; ++i) {
         int start = i;
         while (i < length && (data[i] > 0x1F && data[i] != '"' && data[i] != '\\'))
     for (int i = 0; i < length; ++i) {
         int start = i;
         while (i < length && (data[i] > 0x1F && data[i] != '"' && data[i] != '\\'))


@@ -289,42 +265,54 @@ void Stringifier::appendQuotedString(StringBuilder& builder, const UString& valu
         if (i >= length)
             break;
         switch (data[i]) {
         if (i >= length)
             break;
         switch (data[i]) {

-            case '\t':
-                builder.append('\\');
-                builder.append('t');
-                break;
-            case '\r':
-                builder.append('\\');
-                builder.append('r');
-                break;
-            case '\n':
-                builder.append('\\');
-                builder.append('n');
-                break;
-            case '\f':
-                builder.append('\\');
-                builder.append('f');
-                break;
-            case '\b':
-                builder.append('\\');
-                builder.append('b');
-                break;
-            case '"':
-                builder.append('\\');
-                builder.append('"');
-                break;
-            case '\\':
-                builder.append('\\');
-                builder.append('\\');
-                break;
-            default:
-                static const char hexDigits[] = "0123456789abcdef";
-                UChar ch = data[i];
-                UChar hex[] = { '\\', 'u', hexDigits[(ch >> 12) & 0xF], hexDigits[(ch >> 8) & 0xF], hexDigits[(ch >> 4) & 0xF], hexDigits[ch & 0xF] };
-                builder.append(hex, sizeof(hex) / sizeof(UChar));
-                break;
+        case '\t':
+            builder.append('\\');
+            builder.append('t');
+            break;
+        case '\r':
+            builder.append('\\');
+            builder.append('r');
+            break;
+        case '\n':
+            builder.append('\\');
+            builder.append('n');
+            break;
+        case '\f':
+            builder.append('\\');
+            builder.append('f');
+            break;
+        case '\b':
+            builder.append('\\');
+            builder.append('b');
+            break;
+        case '"':
+            builder.append('\\');
+            builder.append('"');
+            break;
+        case '\\':
+            builder.append('\\');
+            builder.append('\\');
+            break;
+        default:
+            static const char hexDigits[] = "0123456789abcdef";
+            UChar ch = data[i];
+            LChar hex[] = { '\\', 'u', hexDigits[(ch >> 12) & 0xF], hexDigits[(ch >> 8) & 0xF], hexDigits[(ch >> 4) & 0xF], hexDigits[ch & 0xF] };
+            builder.append(hex, WTF_ARRAY_LENGTH(hex));
+            break;

         }
     }
         }
     }

+}
+    
+void Stringifier::appendQuotedString(UStringBuilder& builder, const UString& value)
+{
+    int length = value.length();
+
+    builder.append('"');
+
+    if (value.is8Bit())
+        appendStringToUStringBuilder<LChar>(builder, value.characters8(), length);
+    else
+        appendStringToUStringBuilder<UChar>(builder, value.characters16(), length);

 
     builder.append('"');
 }
 
     builder.append('"');
 }


@@ -344,16 +332,16 @@ inline JSValue Stringifier::toJSON(JSValue value, const PropertyNameForFunctionC
 
     JSObject* object = asObject(toJSONFunction);
     CallData callData;
 
     JSObject* object = asObject(toJSONFunction);
     CallData callData;

-    CallType callType = object->getCallData(callData);
+    CallType callType = object->methodTable()->getCallData(object, callData);

     if (callType == CallTypeNone)
         return value;
 
     if (callType == CallTypeNone)
         return value;
 

-    JSValue list[] = { propertyName.value(m_exec) };
-    ArgList args(list, sizeof(list) / sizeof(JSValue));
+    MarkedArgumentBuffer args;
+    args.append(propertyName.value(m_exec));

     return call(m_exec, object, callType, callData, value, args);
 }
 
     return call(m_exec, object, callType, callData, value, args);
 }
 

-Stringifier::StringifyResult Stringifier::appendStringifiedValue(StringBuilder& builder, JSValue value, JSObject* holder, const PropertyNameForFunctionCall& propertyName)
+Stringifier::StringifyResult Stringifier::appendStringifiedValue(UStringBuilder& builder, JSValue value, JSObject* holder, const PropertyNameForFunctionCall& propertyName)

 {
     // Call the toJSON function.
     value = toJSON(value, propertyName);
 {
     // Call the toJSON function.
     value = toJSON(value, propertyName);


@@ -362,14 +350,15 @@ Stringifier::StringifyResult Stringifier::appendStringifiedValue(StringBuilder&
 
     // Call the replacer function.
     if (m_replacerCallType != CallTypeNone) {
 
     // Call the replacer function.
     if (m_replacerCallType != CallTypeNone) {

-        JSValue list[] = { propertyName.value(m_exec), value };
-        ArgList args(list, sizeof(list) / sizeof(JSValue));
-        value = call(m_exec, m_replacer, m_replacerCallType, m_replacerCallData, holder, args);
+        MarkedArgumentBuffer args;
+        args.append(propertyName.value(m_exec));
+        args.append(value);
+        value = call(m_exec, m_replacer.get(), m_replacerCallType, m_replacerCallData, holder, args);

         if (m_exec->hadException())
             return StringifyFailed;
     }
 
         if (m_exec->hadException())
             return StringifyFailed;
     }
 

-    if (value.isUndefined() && !holder->inherits(&JSArray::info))
+    if (value.isUndefined() && !holder->inherits(&JSArray::s_info))

         return StringifyFailedDueToUndefinedValue;
 
     if (value.isNull()) {
         return StringifyFailedDueToUndefinedValue;
 
     if (value.isNull()) {


@@ -383,7 +372,7 @@ Stringifier::StringifyResult Stringifier::appendStringifiedValue(StringBuilder&
         return StringifyFailed;
 
     if (value.isBoolean()) {
         return StringifyFailed;
 
     if (value.isBoolean()) {

-        builder.append(value.getBoolean() ? "true" : "false");
+        builder.append(value.isTrue() ? "true" : "false");

         return StringifySucceeded;
     }
 
         return StringifySucceeded;
     }
 


@@ -393,12 +382,12 @@ Stringifier::StringifyResult Stringifier::appendStringifiedValue(StringBuilder&
         return StringifySucceeded;
     }
 
         return StringifySucceeded;
     }
 

-    double numericValue;
-    if (value.getNumber(numericValue)) {
-        if (!isfinite(numericValue))
+    if (value.isNumber()) {
+        double number = value.asNumber();
+        if (!isfinite(number))

             builder.append("null");
         else
             builder.append("null");
         else

-            builder.append(UString::from(numericValue));
+            builder.append(UString::number(number));

         return StringifySucceeded;
     }
 
         return StringifySucceeded;
     }
 


@@ -408,8 +397,8 @@ Stringifier::StringifyResult Stringifier::appendStringifiedValue(StringBuilder&
     JSObject* object = asObject(value);
 
     CallData callData;
     JSObject* object = asObject(value);
 
     CallData callData;

-    if (object->getCallData(callData) != CallTypeNone) {
-        if (holder->inherits(&JSArray::info)) {
+    if (object->methodTable()->getCallData(object, callData) != CallTypeNone) {
+        if (holder->inherits(&JSArray::s_info)) {

             builder.append("null");
             return StringifySucceeded;
         }
             builder.append("null");
             return StringifySucceeded;
         }


@@ -417,12 +406,14 @@ Stringifier::StringifyResult Stringifier::appendStringifiedValue(StringBuilder&
     }
 
     // Handle cycle detection, and put the holder on the stack.
     }
 
     // Handle cycle detection, and put the holder on the stack.

-    if (!m_holderCycleDetector.add(object).second) {
-        throwError(m_exec, TypeError, "JSON.stringify cannot serialize cyclic structures.");
-        return StringifyFailed;
+    for (unsigned i = 0; i < m_holderStack.size(); i++) {
+        if (m_holderStack[i].object() == object) {
+            throwError(m_exec, createTypeError(m_exec, "JSON.stringify cannot serialize cyclic structures."));
+            return StringifyFailed;
+        }

     }
     bool holderStackWasEmpty = m_holderStack.isEmpty();
     }
     bool holderStackWasEmpty = m_holderStack.isEmpty();

-    m_holderStack.append(object);
+    m_holderStack.append(Holder(m_exec->globalData(), object));

     if (!holderStackWasEmpty)
         return StringifySucceeded;
 
     if (!holderStackWasEmpty)
         return StringifySucceeded;
 


@@ -436,13 +427,12 @@ Stringifier::StringifyResult Stringifier::appendStringifiedValue(StringBuilder&
                 return StringifyFailed;
             if (!--tickCount) {
                 if (localTimeoutChecker.didTimeOut(m_exec)) {
                 return StringifyFailed;
             if (!--tickCount) {
                 if (localTimeoutChecker.didTimeOut(m_exec)) {

-                    m_exec->setException(createInterruptedExecutionException(&m_exec->globalData()));
+                    throwError(m_exec, createInterruptedExecutionException(&m_exec->globalData()));

                     return StringifyFailed;
                 }
                 tickCount = localTimeoutChecker.ticksUntilNextCheck();
             }
         }
                     return StringifyFailed;
                 }
                 tickCount = localTimeoutChecker.ticksUntilNextCheck();
             }
         }

-        m_holderCycleDetector.remove(m_holderStack.last().object());

         m_holderStack.removeLast();
     } while (!m_holderStack.isEmpty());
     return StringifySucceeded;
         m_holderStack.removeLast();
     } while (!m_holderStack.isEmpty());
     return StringifySucceeded;


@@ -456,20 +446,20 @@ inline bool Stringifier::willIndent() const
 inline void Stringifier::indent()
 {
     // Use a single shared string, m_repeatedGap, so we don't keep allocating new ones as we indent and unindent.
 inline void Stringifier::indent()
 {
     // Use a single shared string, m_repeatedGap, so we don't keep allocating new ones as we indent and unindent.

-    int newSize = m_indent.size() + m_gap.size();
-    if (newSize > m_repeatedGap.size())
-        m_repeatedGap = makeString(m_repeatedGap, m_gap);
-    ASSERT(newSize <= m_repeatedGap.size());
-    m_indent = m_repeatedGap.substr(0, newSize);
+    unsigned newSize = m_indent.length() + m_gap.length();
+    if (newSize > m_repeatedGap.length())
+        m_repeatedGap = makeUString(m_repeatedGap, m_gap);
+    ASSERT(newSize <= m_repeatedGap.length());
+    m_indent = m_repeatedGap.substringSharingImpl(0, newSize);

 }
 
 inline void Stringifier::unindent()
 {
 }
 
 inline void Stringifier::unindent()
 {

-    ASSERT(m_indent.size() >= m_gap.size());
-    m_indent = m_repeatedGap.substr(0, m_indent.size() - m_gap.size());
+    ASSERT(m_indent.length() >= m_gap.length());
+    m_indent = m_repeatedGap.substringSharingImpl(0, m_indent.length() - m_gap.length());

 }
 
 }
 

-inline void Stringifier::startNewLine(StringBuilder& builder) const
+inline void Stringifier::startNewLine(UStringBuilder& builder) const

 {
     if (m_gap.isEmpty())
         return;
 {
     if (m_gap.isEmpty())
         return;


@@ -477,14 +467,14 @@ inline void Stringifier::startNewLine(StringBuilder& builder) const
     builder.append(m_indent);
 }
 
     builder.append(m_indent);
 }
 

-inline Stringifier::Holder::Holder(JSObject* object)
-    : m_object(object)
-    , m_isArray(object->inherits(&JSArray::info))
+inline Stringifier::Holder::Holder(JSGlobalData& globalData, JSObject* object)
+    : m_object(globalData, object)
+    , m_isArray(object->inherits(&JSArray::s_info))

     , m_index(0)
 {
 }
 
     , m_index(0)
 {
 }
 

-bool Stringifier::Holder::appendNextProperty(Stringifier& stringifier, StringBuilder& builder)
+bool Stringifier::Holder::appendNextProperty(Stringifier& stringifier, UStringBuilder& builder)

 {
     ASSERT(m_index <= m_size);
 
 {
     ASSERT(m_index <= m_size);
 


@@ -493,7 +483,7 @@ bool Stringifier::Holder::appendNextProperty(Stringifier& stringifier, StringBui
     // First time through, initialize.
     if (!m_index) {
         if (m_isArray) {
     // First time through, initialize.
     if (!m_index) {
         if (m_isArray) {

-            m_isJSArray = isJSArray(&exec->globalData(), m_object);
+            m_isJSArray = isJSArray(m_object.get());

             m_size = m_object->get(exec, exec->globalData().propertyNames->length).toUInt32(exec);
             builder.append('[');
         } else {
             m_size = m_object->get(exec, exec->globalData().propertyNames->length).toUInt32(exec);
             builder.append('[');
         } else {


@@ -501,7 +491,7 @@ bool Stringifier::Holder::appendNextProperty(Stringifier& stringifier, StringBui
                 m_propertyNames = stringifier.m_arrayReplacerPropertyNames.data();
             else {
                 PropertyNameArray objectPropertyNames(exec);
                 m_propertyNames = stringifier.m_arrayReplacerPropertyNames.data();
             else {
                 PropertyNameArray objectPropertyNames(exec);

-                m_object->getOwnPropertyNames(exec, objectPropertyNames);
+                m_object->methodTable()->getOwnPropertyNames(m_object.get(), exec, objectPropertyNames, ExcludeDontEnumProperties);

                 m_propertyNames = objectPropertyNames.releaseData();
             }
             m_size = m_propertyNames->propertyNameVector().size();
                 m_propertyNames = objectPropertyNames.releaseData();
             }
             m_size = m_propertyNames->propertyNameVector().size();


@@ -513,7 +503,7 @@ bool Stringifier::Holder::appendNextProperty(Stringifier& stringifier, StringBui
     // Last time through, finish up and return false.
     if (m_index == m_size) {
         stringifier.unindent();
     // Last time through, finish up and return false.
     if (m_index == m_size) {
         stringifier.unindent();

-        if (m_size && builder[builder.size() - 1] != '{')
+        if (m_size && builder[builder.length() - 1] != '{')

             stringifier.startNewLine(builder);
         builder.append(m_isArray ? ']' : '}');
         return false;
             stringifier.startNewLine(builder);
         builder.append(m_isArray ? ']' : '}');
         return false;


@@ -526,11 +516,11 @@ bool Stringifier::Holder::appendNextProperty(Stringifier& stringifier, StringBui
     if (m_isArray) {
         // Get the value.
         JSValue value;
     if (m_isArray) {
         // Get the value.
         JSValue value;

-        if (m_isJSArray && asArray(m_object)->canGetIndex(index))
-            value = asArray(m_object)->getIndex(index);
+        if (m_isJSArray && asArray(m_object.get())->canGetIndex(index))
+            value = asArray(m_object.get())->getIndex(index);

         else {
         else {

-            PropertySlot slot(m_object);
-            if (!m_object->getOwnPropertySlot(exec, index, slot))
+            PropertySlot slot(m_object.get());
+            if (!m_object->methodTable()->getOwnPropertySlotByIndex(m_object.get(), exec, index, slot))

                 slot.setUndefined();
             if (exec->hadException())
                 return false;
                 slot.setUndefined();
             if (exec->hadException())
                 return false;


@@ -543,18 +533,18 @@ bool Stringifier::Holder::appendNextProperty(Stringifier& stringifier, StringBui
         stringifier.startNewLine(builder);
 
         // Append the stringified value.
         stringifier.startNewLine(builder);
 
         // Append the stringified value.

-        stringifyResult = stringifier.appendStringifiedValue(builder, value, m_object, index);
+        stringifyResult = stringifier.appendStringifiedValue(builder, value, m_object.get(), index);

     } else {
         // Get the value.
     } else {
         // Get the value.

-        PropertySlot slot(m_object);
+        PropertySlot slot(m_object.get());

         Identifier& propertyName = m_propertyNames->propertyNameVector()[index];
         Identifier& propertyName = m_propertyNames->propertyNameVector()[index];

-        if (!m_object->getOwnPropertySlot(exec, propertyName, slot))
+        if (!m_object->methodTable()->getOwnPropertySlot(m_object.get(), exec, propertyName, slot))

             return true;
         JSValue value = slot.getValue(exec, propertyName);
         if (exec->hadException())
             return false;
 
             return true;
         JSValue value = slot.getValue(exec, propertyName);
         if (exec->hadException())
             return false;
 

-        rollBackPoint = builder.size();
+        rollBackPoint = builder.length();

 
         // Append the separator string.
         if (builder[rollBackPoint - 1] != '{')
 
         // Append the separator string.
         if (builder[rollBackPoint - 1] != '{')


@@ -568,7 +558,7 @@ bool Stringifier::Holder::appendNextProperty(Stringifier& stringifier, StringBui
             builder.append(' ');
 
         // Append the stringified value.
             builder.append(' ');
 
         // Append the stringified value.

-        stringifyResult = stringifier.appendStringifiedValue(builder, value, m_object, propertyName);
+        stringifyResult = stringifier.appendStringifiedValue(builder, value, m_object.get(), propertyName);

     }
 
     // From this point on, no access to the this pointer or to any members, because the
     }
 
     // From this point on, no access to the this pointer or to any members, because the


@@ -594,37 +584,32 @@ bool Stringifier::Holder::appendNextProperty(Stringifier& stringifier, StringBui
 
 // ------------------------------ JSONObject --------------------------------
 
 
 // ------------------------------ JSONObject --------------------------------
 

-const ClassInfo JSONObject::info = { "JSON", 0, 0, ExecState::jsonTable };
+const ClassInfo JSONObject::s_info = { "JSON", &JSNonFinalObject::s_info, 0, ExecState::jsonTable, CREATE_METHOD_TABLE(JSONObject) };

 
 /* Source for JSONObject.lut.h
 @begin jsonTable
 
 /* Source for JSONObject.lut.h
 @begin jsonTable

-  parse         JSONProtoFuncParse             DontEnum|Function 1
-  stringify     JSONProtoFuncStringify         DontEnum|Function 1
+  parse         JSONProtoFuncParse             DontEnum|Function 2
+  stringify     JSONProtoFuncStringify         DontEnum|Function 3

 @end
 */
 
 // ECMA 15.8
 
 @end
 */
 
 // ECMA 15.8
 

-bool JSONObject::getOwnPropertySlot(ExecState* exec, const Identifier& propertyName, PropertySlot& slot)
+bool JSONObject::getOwnPropertySlot(JSCell* cell, ExecState* exec, const Identifier& propertyName, PropertySlot& slot)

 {
 {

-    return getStaticFunctionSlot<JSObject>(exec, ExecState::jsonTable(exec), this, propertyName, slot);
+    return getStaticFunctionSlot<JSObject>(exec, ExecState::jsonTable(exec), jsCast<JSONObject*>(cell), propertyName, slot);

 }
 
 }
 

-bool JSONObject::getOwnPropertyDescriptor(ExecState* exec, const Identifier& propertyName, PropertyDescriptor& descriptor)
+bool JSONObject::getOwnPropertyDescriptor(JSObject* object, ExecState* exec, const Identifier& propertyName, PropertyDescriptor& descriptor)

 {
 {

-    return getStaticFunctionDescriptor<JSObject>(exec, ExecState::jsonTable(exec), this, propertyName, descriptor);
-}
-
-void JSONObject::markStringifiers(MarkStack& markStack, Stringifier* stringifier)
-{
-    stringifier->markAggregate(markStack);
+    return getStaticFunctionDescriptor<JSObject>(exec, ExecState::jsonTable(exec), jsCast<JSONObject*>(object), propertyName, descriptor);

 }
 
 class Walker {
 public:
 }
 
 class Walker {
 public:

-    Walker(ExecState* exec, JSObject* function, CallType callType, CallData callData)
+    Walker(ExecState* exec, Handle<JSObject> function, CallType callType, CallData callData)

         : m_exec(exec)
         : m_exec(exec)

-        , m_function(function)
+        , m_function(exec->globalData(), function)

         , m_callType(callType)
         , m_callData(callData)
     {
         , m_callType(callType)
         , m_callData(callData)
     {


@@ -633,15 +618,16 @@ public:
 private:
     JSValue callReviver(JSObject* thisObj, JSValue property, JSValue unfiltered)
     {
 private:
     JSValue callReviver(JSObject* thisObj, JSValue property, JSValue unfiltered)
     {

-        JSValue args[] = { property, unfiltered };
-        ArgList argList(args, 2);
-        return call(m_exec, m_function, m_callType, m_callData, thisObj, argList);
+        MarkedArgumentBuffer args;
+        args.append(property);
+        args.append(unfiltered);
+        return call(m_exec, m_function.get(), m_callType, m_callData, thisObj, args);

     }
 
     friend class Holder;
 
     ExecState* m_exec;
     }
 
     friend class Holder;
 
     ExecState* m_exec;

-    JSObject* m_function;
+    Local<JSObject> m_function;

     CallType m_callType;
     CallData m_callData;
 };
     CallType m_callType;
     CallData m_callData;
 };


@@ -655,8 +641,8 @@ NEVER_INLINE JSValue Walker::walk(JSValue unfiltered)
 {
     Vector<PropertyNameArray, 16> propertyStack;
     Vector<uint32_t, 16> indexStack;
 {
     Vector<PropertyNameArray, 16> propertyStack;
     Vector<uint32_t, 16> indexStack;

-    Vector<JSObject*, 16> objectStack;
-    Vector<JSArray*, 16> arrayStack;
+    LocalStack<JSObject, 16> objectStack(m_exec->globalData());
+    LocalStack<JSArray, 16> arrayStack(m_exec->globalData());

     
     Vector<WalkerState, 16> stateStack;
     WalkerState state = StateUnknown;
     
     Vector<WalkerState, 16> stateStack;
     WalkerState state = StateUnknown;


@@ -671,40 +657,36 @@ NEVER_INLINE JSValue Walker::walk(JSValue unfiltered)
             arrayStartState:
             case ArrayStartState: {
                 ASSERT(inValue.isObject());
             arrayStartState:
             case ArrayStartState: {
                 ASSERT(inValue.isObject());

-                ASSERT(isJSArray(&m_exec->globalData(), asObject(inValue)) || asObject(inValue)->inherits(&JSArray::info));
-                if (objectStack.size() + arrayStack.size() > maximumFilterRecursion) {
-                    m_exec->setException(createStackOverflowError(m_exec));
-                    return jsUndefined();
-                }
+                ASSERT(isJSArray(asObject(inValue)) || asObject(inValue)->inherits(&JSArray::s_info));
+                if (objectStack.size() + arrayStack.size() > maximumFilterRecursion)
+                    return throwError(m_exec, createStackOverflowError(m_exec));

 
                 JSArray* array = asArray(inValue);
 
                 JSArray* array = asArray(inValue);

-                arrayStack.append(array);
+                arrayStack.push(array);

                 indexStack.append(0);
                 // fallthrough
             }
             arrayStartVisitMember:
             case ArrayStartVisitMember: {
                 if (!--tickCount) {
                 indexStack.append(0);
                 // fallthrough
             }
             arrayStartVisitMember:
             case ArrayStartVisitMember: {
                 if (!--tickCount) {

-                    if (localTimeoutChecker.didTimeOut(m_exec)) {
-                        m_exec->setException(createInterruptedExecutionException(&m_exec->globalData()));
-                        return jsUndefined();
-                    }
+                    if (localTimeoutChecker.didTimeOut(m_exec))
+                        return throwError(m_exec, createInterruptedExecutionException(&m_exec->globalData()));

                     tickCount = localTimeoutChecker.ticksUntilNextCheck();
                 }
 
                     tickCount = localTimeoutChecker.ticksUntilNextCheck();
                 }
 

-                JSArray* array = arrayStack.last();
+                JSArray* array = arrayStack.peek();

                 uint32_t index = indexStack.last();
                 if (index == array->length()) {
                     outValue = array;
                 uint32_t index = indexStack.last();
                 if (index == array->length()) {
                     outValue = array;

-                    arrayStack.removeLast();
+                    arrayStack.pop();

                     indexStack.removeLast();
                     break;
                 }
                     indexStack.removeLast();
                     break;
                 }

-                if (isJSArray(&m_exec->globalData(), array) && array->canGetIndex(index))
+                if (isJSArray(array) && array->canGetIndex(index))

                     inValue = array->getIndex(index);
                 else {
                     PropertySlot slot;
                     inValue = array->getIndex(index);
                 else {
                     PropertySlot slot;

-                    if (array->getOwnPropertySlot(m_exec, index, slot))
+                    if (array->methodTable()->getOwnPropertySlotByIndex(array, m_exec, index, slot))

                         inValue = slot.getValue(m_exec, index);
                     else
                         inValue = jsUndefined();
                         inValue = slot.getValue(m_exec, index);
                     else
                         inValue = jsUndefined();


@@ -718,16 +700,12 @@ NEVER_INLINE JSValue Walker::walk(JSValue unfiltered)
                 // fallthrough
             }
             case ArrayEndVisitMember: {
                 // fallthrough
             }
             case ArrayEndVisitMember: {

-                JSArray* array = arrayStack.last();
-                JSValue filteredValue = callReviver(array, jsString(m_exec, UString::from(indexStack.last())), outValue);
+                JSArray* array = arrayStack.peek();
+                JSValue filteredValue = callReviver(array, jsString(m_exec, UString::number(indexStack.last())), outValue);

                 if (filteredValue.isUndefined())
                 if (filteredValue.isUndefined())

-                    array->deleteProperty(m_exec, indexStack.last());
-                else {
-                    if (isJSArray(&m_exec->globalData(), array) && array->canSetIndex(indexStack.last()))
-                        array->setIndex(indexStack.last(), filteredValue);
-                    else
-                        array->put(m_exec, indexStack.last(), filteredValue);
-                }
+                    array->methodTable()->deletePropertyByIndex(array, m_exec, indexStack.last());
+                else
+                    array->putDirectIndex(m_exec, indexStack.last(), filteredValue, false);

                 if (m_exec->hadException())
                     return jsNull();
                 indexStack.last()++;
                 if (m_exec->hadException())
                     return jsNull();
                 indexStack.last()++;


@@ -736,41 +714,37 @@ NEVER_INLINE JSValue Walker::walk(JSValue unfiltered)
             objectStartState:
             case ObjectStartState: {
                 ASSERT(inValue.isObject());
             objectStartState:
             case ObjectStartState: {
                 ASSERT(inValue.isObject());

-                ASSERT(!isJSArray(&m_exec->globalData(), asObject(inValue)) && !asObject(inValue)->inherits(&JSArray::info));
-                if (objectStack.size() + arrayStack.size() > maximumFilterRecursion) {
-                    m_exec->setException(createStackOverflowError(m_exec));
-                    return jsUndefined();
-                }
+                ASSERT(!isJSArray(asObject(inValue)) && !asObject(inValue)->inherits(&JSArray::s_info));
+                if (objectStack.size() + arrayStack.size() > maximumFilterRecursion)
+                    return throwError(m_exec, createStackOverflowError(m_exec));

 
                 JSObject* object = asObject(inValue);
 
                 JSObject* object = asObject(inValue);

-                objectStack.append(object);
+                objectStack.push(object);

                 indexStack.append(0);
                 propertyStack.append(PropertyNameArray(m_exec));
                 indexStack.append(0);
                 propertyStack.append(PropertyNameArray(m_exec));

-                object->getOwnPropertyNames(m_exec, propertyStack.last());
+                object->methodTable()->getOwnPropertyNames(object, m_exec, propertyStack.last(), ExcludeDontEnumProperties);

                 // fallthrough
             }
             objectStartVisitMember:
             case ObjectStartVisitMember: {
                 if (!--tickCount) {
                 // fallthrough
             }
             objectStartVisitMember:
             case ObjectStartVisitMember: {
                 if (!--tickCount) {

-                    if (localTimeoutChecker.didTimeOut(m_exec)) {
-                        m_exec->setException(createInterruptedExecutionException(&m_exec->globalData()));
-                        return jsUndefined();
-                    }
+                    if (localTimeoutChecker.didTimeOut(m_exec))
+                        return throwError(m_exec, createInterruptedExecutionException(&m_exec->globalData()));

                     tickCount = localTimeoutChecker.ticksUntilNextCheck();
                 }
 
                     tickCount = localTimeoutChecker.ticksUntilNextCheck();
                 }
 

-                JSObject* object = objectStack.last();
+                JSObject* object = objectStack.peek();

                 uint32_t index = indexStack.last();
                 PropertyNameArray& properties = propertyStack.last();
                 if (index == properties.size()) {
                     outValue = object;
                 uint32_t index = indexStack.last();
                 PropertyNameArray& properties = propertyStack.last();
                 if (index == properties.size()) {
                     outValue = object;

-                    objectStack.removeLast();
+                    objectStack.pop();

                     indexStack.removeLast();
                     propertyStack.removeLast();
                     break;
                 }
                 PropertySlot slot;
                     indexStack.removeLast();
                     propertyStack.removeLast();
                     break;
                 }
                 PropertySlot slot;

-                if (object->getOwnPropertySlot(m_exec, properties[index], slot))
+                if (object->methodTable()->getOwnPropertySlot(object, m_exec, properties[index], slot))

                     inValue = slot.getValue(m_exec, properties[index]);
                 else
                     inValue = jsUndefined();
                     inValue = slot.getValue(m_exec, properties[index]);
                 else
                     inValue = jsUndefined();


@@ -787,14 +761,14 @@ NEVER_INLINE JSValue Walker::walk(JSValue unfiltered)
                 // fallthrough
             }
             case ObjectEndVisitMember: {
                 // fallthrough
             }
             case ObjectEndVisitMember: {

-                JSObject* object = objectStack.last();
+                JSObject* object = objectStack.peek();

                 Identifier prop = propertyStack.last()[indexStack.last()];
                 PutPropertySlot slot;
                 JSValue filteredValue = callReviver(object, jsString(m_exec, prop.ustring()), outValue);
                 if (filteredValue.isUndefined())
                 Identifier prop = propertyStack.last()[indexStack.last()];
                 PutPropertySlot slot;
                 JSValue filteredValue = callReviver(object, jsString(m_exec, prop.ustring()), outValue);
                 if (filteredValue.isUndefined())

-                    object->deleteProperty(m_exec, prop);
+                    object->methodTable()->deleteProperty(object, m_exec, prop);

                 else
                 else

-                    object->put(m_exec, prop, filteredValue, slot);
+                    object->methodTable()->put(object, m_exec, prop, filteredValue, slot);

                 if (m_exec->hadException())
                     return jsNull();
                 indexStack.last()++;
                 if (m_exec->hadException())
                     return jsNull();
                 indexStack.last()++;


@@ -807,7 +781,7 @@ NEVER_INLINE JSValue Walker::walk(JSValue unfiltered)
                     break;
                 }
                 JSObject* object = asObject(inValue);
                     break;
                 }
                 JSObject* object = asObject(inValue);

-                if (isJSArray(&m_exec->globalData(), object) || object->inherits(&JSArray::info))
+                if (isJSArray(object) || object->inherits(&JSArray::s_info))

                     goto arrayStartState;
                 goto objectStartState;
         }
                     goto arrayStartState;
                 goto objectStartState;
         }


@@ -818,54 +792,70 @@ NEVER_INLINE JSValue Walker::walk(JSValue unfiltered)
         stateStack.removeLast();
 
         if (!--tickCount) {
         stateStack.removeLast();
 
         if (!--tickCount) {

-            if (localTimeoutChecker.didTimeOut(m_exec)) {
-                m_exec->setException(createInterruptedExecutionException(&m_exec->globalData()));
-                return jsUndefined();
-            }
+            if (localTimeoutChecker.didTimeOut(m_exec))
+                return throwError(m_exec, createInterruptedExecutionException(&m_exec->globalData()));

             tickCount = localTimeoutChecker.ticksUntilNextCheck();
         }
     }
     JSObject* finalHolder = constructEmptyObject(m_exec);
     PutPropertySlot slot;
             tickCount = localTimeoutChecker.ticksUntilNextCheck();
         }
     }
     JSObject* finalHolder = constructEmptyObject(m_exec);
     PutPropertySlot slot;

-    finalHolder->put(m_exec, m_exec->globalData().propertyNames->emptyIdentifier, outValue, slot);
+    finalHolder->methodTable()->put(finalHolder, m_exec, m_exec->globalData().propertyNames->emptyIdentifier, outValue, slot);

     return callReviver(finalHolder, jsEmptyString(m_exec), outValue);
 }
 
 // ECMA-262 v5 15.12.2
     return callReviver(finalHolder, jsEmptyString(m_exec), outValue);
 }
 
 // ECMA-262 v5 15.12.2

-JSValue JSC_HOST_CALL JSONProtoFuncParse(ExecState* exec, JSObject*, JSValue, const ArgList& args)
+EncodedJSValue JSC_HOST_CALL JSONProtoFuncParse(ExecState* exec)

 {
 {

-    if (args.isEmpty())
-        return throwError(exec, GeneralError, "JSON.parse requires at least one parameter");
-    JSValue value = args.at(0);
-    UString source = value.toString(exec);
+    if (!exec->argumentCount())
+        return throwVMError(exec, createError(exec, "JSON.parse requires at least one parameter"));
+    UString source = exec->argument(0).toString(exec)->value(exec);

     if (exec->hadException())
     if (exec->hadException())

-        return jsNull();
-    
-    LiteralParser jsonParser(exec, source, LiteralParser::StrictJSON);
-    JSValue unfiltered = jsonParser.tryLiteralParse();
-    if (!unfiltered)
-        return throwError(exec, SyntaxError, "Unable to parse JSON string");
+        return JSValue::encode(jsNull());
+
+    JSValue unfiltered;
+    LocalScope scope(exec->globalData());
+    if (source.is8Bit()) {
+        LiteralParser<LChar> jsonParser(exec, source.characters8(), source.length(), StrictJSON);
+        unfiltered = jsonParser.tryLiteralParse();
+        if (!unfiltered)
+            return throwVMError(exec, createSyntaxError(exec, jsonParser.getErrorMessage()));
+    } else {
+        LiteralParser<UChar> jsonParser(exec, source.characters16(), source.length(), StrictJSON);
+        unfiltered = jsonParser.tryLiteralParse();
+        if (!unfiltered)
+            return throwVMError(exec, createSyntaxError(exec, jsonParser.getErrorMessage()));        
+    }

     
     

-    if (args.size() < 2)
-        return unfiltered;
+    if (exec->argumentCount() < 2)
+        return JSValue::encode(unfiltered);

     
     

-    JSValue function = args.at(1);
+    JSValue function = exec->argument(1);

     CallData callData;
     CallData callData;

-    CallType callType = function.getCallData(callData);
+    CallType callType = getCallData(function, callData);

     if (callType == CallTypeNone)
     if (callType == CallTypeNone)

-        return unfiltered;
-    return Walker(exec, asObject(function), callType, callData).walk(unfiltered);
+        return JSValue::encode(unfiltered);
+    return JSValue::encode(Walker(exec, Local<JSObject>(exec->globalData(), asObject(function)), callType, callData).walk(unfiltered));

 }
 
 // ECMA-262 v5 15.12.3
 }
 
 // ECMA-262 v5 15.12.3

-JSValue JSC_HOST_CALL JSONProtoFuncStringify(ExecState* exec, JSObject*, JSValue, const ArgList& args)
+EncodedJSValue JSC_HOST_CALL JSONProtoFuncStringify(ExecState* exec)
+{
+    if (!exec->argumentCount())
+        return throwVMError(exec, createError(exec, "No input to stringify"));
+    LocalScope scope(exec->globalData());
+    Local<Unknown> value(exec->globalData(), exec->argument(0));
+    Local<Unknown> replacer(exec->globalData(), exec->argument(1));
+    Local<Unknown> space(exec->globalData(), exec->argument(2));
+    return JSValue::encode(Stringifier(exec, replacer, space).stringify(value).get());
+}
+
+UString JSONStringify(ExecState* exec, JSValue value, unsigned indent)

 {
 {

-    if (args.isEmpty())
-        return throwError(exec, GeneralError, "No input to stringify");
-    JSValue value = args.at(0);
-    JSValue replacer = args.at(1);
-    JSValue space = args.at(2);
-    return Stringifier(exec, replacer, space).stringify(value);
+    LocalScope scope(exec->globalData());
+    Local<Unknown> result = Stringifier(exec, Local<Unknown>(exec->globalData(), jsNull()), Local<Unknown>(exec->globalData(), jsNumber(indent))).stringify(Local<Unknown>(exec->globalData(), value));
+    if (result.isUndefinedOrNull())
+        return UString();
+    return result.getString(exec);

 }
 
 } // namespace JSC
 }
 
 } // namespace JSC