Just a few fly-by comments.
http://codereview.chromium.org/39256/diff/1/6 File src/counters.cc (right): http://codereview.chromium.org/39256/diff/1/6#newcode60 Line 60: GetHistogram(); How about calling GetHistogram EnsureHistogram and make its return value void? http://codereview.chromium.org/39256/diff/1/6#newcode69 Line 69: GetHistogram(); Is this call needed in Stop as well as in Start? http://codereview.chromium.org/39256 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
