Bug 1925128 Comment 7 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
For the JS shell micro-benchmark below:
```
--setpref=experimental.json_parse_with_source=false: 450 ms
--setpref=experimental.json_parse_with_source=true : 13442 ms
```
Bryan, as a first step I think we should flip the pref on beta because it will give us more time to fix this regression on Nightly.

```js
function f() {
    var o = [];
    for (var i = 0; i < 100000; i++) {
        o.push({x: "foo".repeat(10) + i, y: "bar".repeat(10) + i});
    }
    var json = JSON.stringify(o);
    var res;
    var t = new Date;
    for (var i = 0; i < 5; i++) {
        res = JSON.parse(json, (key, value) => value);
    }
    print(new Date - t);
    return res;
}
f();
```
Revision 1 by
on 
For the JS shell micro-benchmark below:
```
--setpref=experimental.json_parse_with_source=false: 450 ms
--setpref=experimental.json_parse_with_source=true : 13442 ms
```
Bryan, as a first step I think we should flip the pref, definitely on beta, because it will give us more time to fix this regression on Nightly.

```js
function f() {
    var o = [];
    for (var i = 0; i < 100000; i++) {
        o.push({x: "foo".repeat(10) + i, y: "bar".repeat(10) + i});
    }
    var json = JSON.stringify(o);
    var res;
    var t = new Date;
    for (var i = 0; i < 5; i++) {
        res = JSON.parse(json, (key, value) => value);
    }
    print(new Date - t);
    return res;
}
f();
```

Back to Bug 1925128 Comment 7