Use JSON.parse instead of eval for the debugger JSON protocol.

test/mjsunit/debug-scripts-request.js

 // Copyright 2008 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 53 matching lines @@
 
     // Test illegal scripts requests.
     testArguments(dcp, '{"types":"xx"}', false);
 
     // Test legal scripts requests.
     testArguments(dcp, '{}', true);
     testArguments(dcp, '{"types":1}', true);
     testArguments(dcp, '{"types":2}', true);
     testArguments(dcp, '{"types":4}', true);
     testArguments(dcp, '{"types":7}', true);
-    testArguments(dcp, '{"types":0xFF}', true);

[Søren Thygesen Gjesse, 2011/05/02 14:07:09]

255 instead of deleting this line?

[Mads Ager (chromium), 2011/05/02 14:19:15]

Done.

 
     // Test request for all scripts.
     var request = '{' + base_request + '}'
     var response = safeEval(dcp.processDebugJSONRequest(request));
     assertTrue(response.success);
 
     // Test filtering by id.
     assertEquals(2, response.body.length);
     var script = response.body[0];
     var request = '{' + base_request + ',"arguments":{"ids":[' +
@@ skipping 14 matching lines @@
 // Add the debug event listener.
 Debug.setListener(listener);
 
 // Call debugger to invoke the debug event listener.
 debugger;
 
 // Make sure that the debug event listener vas invoked with no exceptions.
 assertTrue(listenerComplete,
            "listener did not run to completion, exception: " + exception);
 assertFalse(exception, "exception in listener")