switch to using native JSON object

BUG=12192
R=aa

Committed: http://src.chromium.org/viewvc/17269

[rafaelw, 2009-05-26 23:20:31]

adding Pavel as a reviewer. Pavel, just review that is it safe to remove json.js
from the devtools.html. Thanks.

[Aaron Boodman, 2009-05-27 01:28:29]

lgtm

hooray for deleting code.

http://codereview.chromium.org/115681/diff/12/20
File chrome/renderer/resources/event_bindings.js (left):

http://codereview.chromium.org/115681/diff/12/20#oldcode38
Line 38: args = goog.json.parse(args);
Did you do a search for this string? I worry that not all usages are covered by
unit tests.

[rafaelw, 2009-05-27 01:41:16]

http://codereview.chromium.org/115681/diff/12/20
File chrome/renderer/resources/event_bindings.js (left):

http://codereview.chromium.org/115681/diff/12/20#oldcode38
Line 38: args = goog.json.parse(args);
On 2009/05/27 01:28:29, Aaron Boodman wrote:
> Did you do a search for this string? I worry that not all usages are covered
by
> unit tests.

Yup. I used gsearch ('goog\.json') and tracked down all uses. I believe I've got
them all. Feel free to verify. There is still a json.js checked in in
test/reference_build/chrome_linux/resources/jsons.js, but that appears to still
be in use.

[pfeldman, 2009-05-27 04:38:21]

LGTM