[counters] adding more counters and log-events

src/deoptimizer.cc

Index: src/deoptimizer.cc
diff --git a/src/deoptimizer.cc b/src/deoptimizer.cc
index f7791d210ec3c6dde23c3f060775a5b976412b47..b46631988ae58d13ed9a92b5334ed78a6ac88e0d 100644
--- a/src/deoptimizer.cc
+++ b/src/deoptimizer.cc
@@ -426,6 +426,7 @@ void Deoptimizer::DeoptimizeMarkedCodeForContext(Context* context) {
 
 
 void Deoptimizer::DeoptimizeAll(Isolate* isolate) {
+  TimerEventScope<TimerEventDeoptimizeCode> timer(isolate);
   if (FLAG_trace_deopt) {
     CodeTracer::Scope scope(isolate->GetCodeTracer());
     PrintF(scope.file(), "[deoptimize all code in all contexts]\n");
@@ -443,6 +444,7 @@ void Deoptimizer::DeoptimizeAll(Isolate* isolate) {
 
 
 void Deoptimizer::DeoptimizeMarkedCode(Isolate* isolate) {
+  TimerEventScope<TimerEventDeoptimizeCode> timer(isolate);
   if (FLAG_trace_deopt) {
     CodeTracer::Scope scope(isolate->GetCodeTracer());
     PrintF(scope.file(), "[deoptimize marked code in all contexts]\n");
@@ -472,6 +474,7 @@ void Deoptimizer::MarkAllCodeForContext(Context* context) {
 void Deoptimizer::DeoptimizeFunction(JSFunction* function) {
   Code* code = function->code();
   if (code->kind() == Code::OPTIMIZED_FUNCTION) {
+    TimerEventScope<TimerEventDeoptimizeCode> timer(isolate);

[Michael Starzinger, 2016/02/09 09:42:43]

nit: Let's keep this "as simple as possible" by moving the scope to the very top
of the function. I am already not trusting that we covered all entry points with
these scopes, so lets at least stick to either keeping the entire entry point in
scope or not at all. Because the next person coming along refactoring this
method will not give a damn about the timer scope and happily add costly
functionality outside the scope.

     // Mark the code for deoptimization and unlink any functions that also
     // refer to that code. The code cannot be shared across native contexts,
     // so we only need to search one.