Bug 51667 – JSON.stringify must exist as a function taking 3 parameters

• Component: JavaScriptCore
• Status: RESOLVED FIXED

[Helder Correia]

The reported function length is 1 instead.

[Helder Correia]

Created attachment 77537 [details]
Patch

Diff is applied to patch 77536 from bug 51666.

[Eric Seidel (no email)]

Comment on attachment 77537 [details]
Patch

It appears it does actually take 3 values:

// ECMA-262 v5 15.12.3
EncodedJSValue JSC_HOST_CALL JSONProtoFuncStringify(ExecState* exec)
{
    if (!exec->argumentCount())
        return throwVMError(exec, createError(exec, "No input to stringify"));
    JSValue value = exec->argument(0);
    JSValue replacer = exec->argument(1);
    JSValue space = exec->argument(2);
    return JSValue::encode(Stringifier(exec, replacer, space).stringify(value));
}

We need to find a way to catch this class of bugs somehow.

[WebKit Commit Bot]

The commit-queue encountered the following flaky tests while processing attachment 77537 [details]:

java/java-and-plugins.html bug 51690 (authors: [e-mail] and [e-mail])
The commit-queue is continuing to process your patch.

[WebKit Commit Bot]

Comment on attachment 77537 [details]
Patch

Clearing flags on attachment: 77537

Committed r74727: <http://trac.webkit.org/changeset/74727>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.