Make some runtime arguments checks be RUNTIME_ASSERT, not ASSERT.

Make some runtime arguments checks be RUNTIME_ASSERT, not ASSERT.

[Rico, 2010-09-17 08:17:28]

LGTM

http://codereview.chromium.org/3411013/diff/1/2
File src/runtime.cc (right):

http://codereview.chromium.org/3411013/diff/1/2#newcode8955
src/runtime.cc:8955: 
CONVERT_BOOLEAN_CHECKED(enable, args[1]) and CONVERT_NUMBER_CHECKED for args[0]?

[Lasse Reichstein, 2010-09-17 08:31:04]

http://codereview.chromium.org/3411013/diff/1/2
File src/runtime.cc (right):

http://codereview.chromium.org/3411013/diff/1/2#newcode8948
src/runtime.cc:8948: // args[0]: boolean indicating uncaught exceptions
Comment is wrong too: It's not a boolean.
Reworded.

http://codereview.chromium.org/3411013/diff/1/2#newcode8950
src/runtime.cc:8950: static Object* Runtime_ChangeBreakOnException(Arguments
args) {
Contrary to what I said offline, the exception in fuzz-natives.js can be removed
(which means that the comment in fuzz-natives.js didn't apply to this function -
it does no error checking in recursive calls).

http://codereview.chromium.org/3411013/diff/1/2#newcode8955
src/runtime.cc:8955: 
The boolean is ok, the number conversion introduces an unnecessary intermediate
variable.

[Rico, 2010-09-17 08:33:25]

LGTM

[jokosuwito, 2012-11-17 20:17:35]

re. Publish my gmail: <jokoswt@gmail.com>