LGTM with a nit
http://codereview.chromium.org/11368142/diff/1/src/messages.cc File src/messages.cc (right): http://codereview.chromium.org/11368142/diff/1/src/messages.cc#newcode158 src/messages.cc:158: ASSERT(data->IsJSMessageObject()); No need for this ASSERT, all cast methods do this implicitly. http://codereview.chromium.org/11368142/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
