LGTM.
http://codereview.chromium.org/5720003/diff/1/src/runtime-profiler.cc File src/runtime-profiler.cc (right): http://codereview.chromium.org/5720003/diff/1/src/runtime-profiler.cc#newcode429 src/runtime-profiler.cc:429: if (top_is_in_js_state) { Do you need to check for RuntimeProfiler::IsEnabled() here too? http://codereview.chromium.org/5720003/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
