Closed Bug 1860889 Opened 8 months ago Closed 3 months ago

Implement JSON.rawJSON and JSON.isRawJSON functions

Categories

(Core :: JavaScript Engine, task, P3)

task

Tracking

()

RESOLVED FIXED
126 Branch
Tracking Status
firefox126 --- fixed

People

(Reporter: bthrall, Assigned: bthrall)

References

(Blocks 1 open bug)

Details

Attachments

(4 files)

Implement the JSON.rawJSON() and JSON.isRawJSON() functions, protected behind the shell option that enables the JSON.parse with source proposal.

Severity: -- → N/A
Priority: -- → P3

The function IsJSONWhitespace() also appears in JSONParser.cpp, but at this
point it doesn't seem worth the effort of deduplicating it.

Assignee: nobody → bthrall
Attachment #9377727 - Attachment description: WIP: Bug 1860889 - Support JSON.rawJSON() and JSON.isRawJSON() → Bug 1860889 - Support JSON.rawJSON() and JSON.isRawJSON() r=jandem
Status: NEW → ASSIGNED
Attachment #9377728 - Attachment description: WIP: Bug 1860889 - Report SyntaxErrors from JSON.rawJSON() → Bug 1860889 - Report SyntaxErrors from JSON.rawJSON() r=jandem
Attachment #9377729 - Attachment description: WIP: Bug 1860889 - Enable test262 JSON.parse with source tests → Bug 1860889 - Enable test262 JSON.parse with source tests r=jandem

It should only be available (via a pref) in Nightly, now.

Skip the non262 test in browser reftests because that's what other tests that
require setting a preference do. We should enable it when we remove the
preference.

Attachment #9377729 - Attachment description: Bug 1860889 - Enable test262 JSON.parse with source tests r=jandem → WIP: Bug 1860889 - Enable test262 tests for JSON.parse with source
Attachment #9377727 - Attachment description: Bug 1860889 - Support JSON.rawJSON() and JSON.isRawJSON() r=jandem → Bug 1860889 - Support JSON.rawJSON() and JSON.isRawJSON() r=jandem,dminor
Attachment #9377728 - Attachment description: Bug 1860889 - Report SyntaxErrors from JSON.rawJSON() r=jandem → Bug 1860889 - Report SyntaxErrors from JSON.rawJSON() r=jandem,dminor
Attachment #9389136 - Attachment description: WIP: Bug 1860889 - Remove build config for JSON.parse with source → Bug 1860889 - Remove build config for JSON.parse with source r=jandem,dminor
Attachment #9377729 - Attachment description: WIP: Bug 1860889 - Enable test262 tests for JSON.parse with source → Bug 1860889 - Enable test262 tests for JSON.parse with source r=jandem,dminor
Pushed by bthrall@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3614bce991ac
Support JSON.rawJSON() and JSON.isRawJSON() r=jandem
https://hg.mozilla.org/integration/autoland/rev/0f5d6ecc7a9f
Report SyntaxErrors from JSON.rawJSON() r=jandem
https://hg.mozilla.org/integration/autoland/rev/2267e7765e87
Remove build config for JSON.parse with source r=jandem
https://hg.mozilla.org/integration/autoland/rev/5129e0aba202
Enable test262 tests for JSON.parse with source r=mgaudet
Pushed by bthrall@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/50d4cdecac19
Support JSON.rawJSON() and JSON.isRawJSON() r=jandem
https://hg.mozilla.org/integration/autoland/rev/b0f251a4bb8f
Report SyntaxErrors from JSON.rawJSON() r=jandem
https://hg.mozilla.org/integration/autoland/rev/07a8dbfb23ae
Remove build config for JSON.parse with source r=jandem
https://hg.mozilla.org/integration/autoland/rev/82c125858029
Enable test262 tests for JSON.parse with source r=mgaudet
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: