Chromium Code Reviews| 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..bd1baa197b83215019e57da0d6ecfa25f8644c8a 100644 |
| --- a/extensions/renderer/scripts_run_info.cc |
| +++ b/extensions/renderer/scripts_run_info.cc |
| @@ -34,17 +34,29 @@ 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); |
| + } |
| + if ((num_css || num_js) && !num_blocking_js) |
|
Devlin
2015/03/04 18:09:31
nit: make this
else if (num_css || num_js)
(Same
kozy
2015/03/05 09:45:51
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); |
| + } |
| + if (num_js && !num_blocking_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); |
| + } |
| + if (num_js && !num_blocking_js) |
| UMA_HISTOGRAM_TIMES("Extensions.InjectIdle_Time", timer.Elapsed()); |
| break; |
| case UserScript::RUN_DEFERRED: |