LGTM
Thanks, Vitaly http://codereview.chromium.org/5720003/diff/12001/src/runtime-profiler.cc File src/runtime-profiler.cc (right): http://codereview.chromium.org/5720003/diff/12001/src/runtime-profiler.cc#newcode353 src/runtime-profiler.cc:353: static void AddStateSample(SamplerState current_state) { Rename to "UpdateStateRatio"? http://codereview.chromium.org/5720003/diff/12001/src/runtime-profiler.cc#newcode374 src/runtime-profiler.cc:374: AddStateSample(IN_JS_STATE); Nice place to use ?: operator. Up to you though. http://codereview.chromium.org/5720003/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
