Reviewers: Vitaly Repeshko,
Description:
Minor cosmetic changes.
Please review this at http://codereview.chromium.org/6875003/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files:
M src/messages.cc
M test/cctest/test-api.cc
Index: src/messages.cc
diff --git a/src/messages.cc b/src/messages.cc
index
abc25377851b5c4a76a404b5519edf75adc70226..0fdbbbb79ed9ef5c960dfb5c49c5f1de21c4008b
100644
--- a/src/messages.cc
+++ b/src/messages.cc
@@ -131,7 +131,7 @@ void MessageHandler::ReportMessage(Isolate* isolate,
Handle<Object> callback_data(listener.get(1));
{
// Do not allow exceptions to propagate.
- v8::TryCatch tryCatch;
+ v8::TryCatch try_catch;
callback(api_message_obj, v8::Utils::ToLocal(callback_data));
}
if (isolate->has_scheduled_exception()) {
Index: test/cctest/test-api.cc
diff --git a/test/cctest/test-api.cc b/test/cctest/test-api.cc
index
d7621d12cf2ff1af6c02c36abd4c626b418eb612..28a9be159937b9f1399efa6f7de5e9734751ba55
100644
--- a/test/cctest/test-api.cc
+++ b/test/cctest/test-api.cc
@@ -8711,6 +8711,8 @@ THREADED_TEST(ExceptionsDoNotPropagatePastTryCatch) {
if (callback != NULL) {
V8::AddMessageListener(callback);
}
+ // Some small number to control number of times message handler should
+ // throw an exception.
call_depth = 5;
ExpectFalse(
"var thrown = false;\n"
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev