Introduce new compile histogram that includes parsing/caching.

src/counters.h

Index: src/counters.h
diff --git a/src/counters.h b/src/counters.h
index 41107cfc33b8c944496e37d269c4ba647ad260fc..cbd8093970dbac9e2a55f642684fb3fd5bd5390a 100644
--- a/src/counters.h
+++ b/src/counters.h
@@ -309,12 +309,14 @@ class HistogramTimerScope BASE_EMBEDDED {
   HT(parse, V8.Parse)                                        \
   HT(parse_lazy, V8.ParseLazy)                               \
   HT(pre_parse, V8.PreParse)                                 \
-  /* Total compilation times. */                             \
+  /* Compilation times. */                                   \
   HT(compile, V8.Compile)                                    \
   HT(compile_eval, V8.CompileEval)                           \
   /* Serialization as part of compilation (code caching) */  \
   HT(compile_serialize, V8.CompileSerialize)                 \
-  HT(compile_deserialize, V8.CompileDeserialize)
+  HT(compile_deserialize, V8.CompileDeserialize)             \
+  /* Total compilation time incl. caching/parsing */         \

[vogelheim, 2015/01/14 14:37:38]

I was initially surprised at the comment: V8.Compile includes caching/parsing,
too, while the point of this counter is to only show the API-initiated compiles?

Also, super nitpick: CS shows this is will also count the compilation of the
built-ins. That shouldn't happen in any standard Chrome build, though, so I
guess that's not much of a point.

Maybe we can move the counter into api.cc, ScriptCompiler::CompileUnbound. That
shouldn't be used by any internal compiles.

[Yang, 2015/01/14 14:55:20]

As discussed offline, V8.Compile does not include parsing/caching. I'll just
leave the new histogram where they currently are in this CL.

+  HT(compile_script, V8.CompileScript)
 
 
 #define HISTOGRAM_PERCENTAGE_LIST(HP)                                 \