Extensions: suspend extension's scripts when V8 is paused

extensions/renderer/scripts_run_info.cc

Index: extensions/renderer/scripts_run_info.cc
diff --git a/extensions/renderer/scripts_run_info.cc b/extensions/renderer/scripts_run_info.cc
index a4073d1a5ec78fce9f2ae808f6256745a8749861..99c1c548ee4e3a065ff6feb99f1dce7252538323 100644
--- a/extensions/renderer/scripts_run_info.cc
+++ b/extensions/renderer/scripts_run_info.cc
@@ -34,17 +34,26 @@ void ScriptsRunInfo::LogRun(blink::WebFrame* frame,
     case UserScript::DOCUMENT_START:
       UMA_HISTOGRAM_COUNTS_100("Extensions.InjectStart_CssCount", num_css);
       UMA_HISTOGRAM_COUNTS_100("Extensions.InjectStart_ScriptCount", num_js);
-      if (num_css || num_js)
+      if (num_blocking_js) {
+        UMA_HISTOGRAM_COUNTS_100("Extensions.InjectStart_BlockingScriptCount",
+                                 num_blocking_js);
+      } else if (num_css || num_js)

[Devlin, 2015/03/05 16:32:13]

if an "else" has brackets on one side, it should on the other, too.  So this
should be:
if (num_blocking_js) {
...
} else if (num_css || num_js) {
...
}

(Same for below.)

[kozy, 2015/03/05 21:41:37]

Done.

         UMA_HISTOGRAM_TIMES("Extensions.InjectStart_Time", timer.Elapsed());
       break;
     case UserScript::DOCUMENT_END:
       UMA_HISTOGRAM_COUNTS_100("Extensions.InjectEnd_ScriptCount", num_js);
-      if (num_js)
+      if (num_blocking_js) {
+        UMA_HISTOGRAM_COUNTS_100("Extensions.InjectEnd_BlockingScriptCount",
+                                 num_blocking_js);
+      } else if (num_js)
         UMA_HISTOGRAM_TIMES("Extensions.InjectEnd_Time", timer.Elapsed());
       break;
     case UserScript::DOCUMENT_IDLE:
       UMA_HISTOGRAM_COUNTS_100("Extensions.InjectIdle_ScriptCount", num_js);
-      if (num_js)
+      if (num_blocking_js) {
+        UMA_HISTOGRAM_COUNTS_100("Extensions.InjectIdle_BlockingScriptCount",
+                                 num_blocking_js);
+      } else if (num_js)
         UMA_HISTOGRAM_TIMES("Extensions.InjectIdle_Time", timer.Elapsed());
       break;
     case UserScript::RUN_DEFERRED: