Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright 2014 The Chromium Authors. All rights reserved. | 1 // Copyright 2014 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "extensions/renderer/scripts_run_info.h" | 5 #include "extensions/renderer/scripts_run_info.h" |
| 6 | 6 |
| 7 #include "base/metrics/histogram.h" | 7 #include "base/metrics/histogram.h" |
| 8 #include "content/public/renderer/render_view.h" | 8 #include "content/public/renderer/render_view.h" |
| 9 #include "extensions/common/extension_messages.h" | 9 #include "extensions/common/extension_messages.h" |
| 10 #include "extensions/renderer/script_context.h" | 10 #include "extensions/renderer/script_context.h" |
| (...skipping 16 matching lines...) Expand all Loading... | |
| 27 render_view->Send(new ExtensionHostMsg_ContentScriptsExecuting( | 27 render_view->Send(new ExtensionHostMsg_ContentScriptsExecuting( |
| 28 render_view->GetRoutingID(), | 28 render_view->GetRoutingID(), |
| 29 executing_scripts, | 29 executing_scripts, |
| 30 ScriptContext::GetDataSourceURLForFrame(frame))); | 30 ScriptContext::GetDataSourceURLForFrame(frame))); |
| 31 } | 31 } |
| 32 | 32 |
| 33 switch (location) { | 33 switch (location) { |
| 34 case UserScript::DOCUMENT_START: | 34 case UserScript::DOCUMENT_START: |
| 35 UMA_HISTOGRAM_COUNTS_100("Extensions.InjectStart_CssCount", num_css); | 35 UMA_HISTOGRAM_COUNTS_100("Extensions.InjectStart_CssCount", num_css); |
| 36 UMA_HISTOGRAM_COUNTS_100("Extensions.InjectStart_ScriptCount", num_js); | 36 UMA_HISTOGRAM_COUNTS_100("Extensions.InjectStart_ScriptCount", num_js); |
| 37 if (num_css || num_js) | 37 if (num_blocking_js) { |
| 38 UMA_HISTOGRAM_COUNTS_100("Extensions.InjectStart_BlockingScriptCount", | |
| 39 num_blocking_js); | |
| 40 } | |
| 41 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.
| |
| 38 UMA_HISTOGRAM_TIMES("Extensions.InjectStart_Time", timer.Elapsed()); | 42 UMA_HISTOGRAM_TIMES("Extensions.InjectStart_Time", timer.Elapsed()); |
| 39 break; | 43 break; |
| 40 case UserScript::DOCUMENT_END: | 44 case UserScript::DOCUMENT_END: |
| 41 UMA_HISTOGRAM_COUNTS_100("Extensions.InjectEnd_ScriptCount", num_js); | 45 UMA_HISTOGRAM_COUNTS_100("Extensions.InjectEnd_ScriptCount", num_js); |
| 42 if (num_js) | 46 if (num_blocking_js) { |
| 47 UMA_HISTOGRAM_COUNTS_100("Extensions.InjectEnd_BlockingScriptCount", | |
| 48 num_blocking_js); | |
| 49 } | |
| 50 if (num_js && !num_blocking_js) | |
| 43 UMA_HISTOGRAM_TIMES("Extensions.InjectEnd_Time", timer.Elapsed()); | 51 UMA_HISTOGRAM_TIMES("Extensions.InjectEnd_Time", timer.Elapsed()); |
| 44 break; | 52 break; |
| 45 case UserScript::DOCUMENT_IDLE: | 53 case UserScript::DOCUMENT_IDLE: |
| 46 UMA_HISTOGRAM_COUNTS_100("Extensions.InjectIdle_ScriptCount", num_js); | 54 UMA_HISTOGRAM_COUNTS_100("Extensions.InjectIdle_ScriptCount", num_js); |
| 47 if (num_js) | 55 if (num_blocking_js) { |
| 56 UMA_HISTOGRAM_COUNTS_100("Extensions.InjectIdle_BlockingScriptCount", | |
| 57 num_blocking_js); | |
| 58 } | |
| 59 if (num_js && !num_blocking_js) | |
| 48 UMA_HISTOGRAM_TIMES("Extensions.InjectIdle_Time", timer.Elapsed()); | 60 UMA_HISTOGRAM_TIMES("Extensions.InjectIdle_Time", timer.Elapsed()); |
| 49 break; | 61 break; |
| 50 case UserScript::RUN_DEFERRED: | 62 case UserScript::RUN_DEFERRED: |
| 51 case UserScript::BROWSER_DRIVEN: | 63 case UserScript::BROWSER_DRIVEN: |
| 52 // TODO(rdevlin.cronin): Add histograms. | 64 // TODO(rdevlin.cronin): Add histograms. |
| 53 break; | 65 break; |
| 54 case UserScript::UNDEFINED: | 66 case UserScript::UNDEFINED: |
| 55 case UserScript::RUN_LOCATION_LAST: | 67 case UserScript::RUN_LOCATION_LAST: |
| 56 NOTREACHED(); | 68 NOTREACHED(); |
| 57 } | 69 } |
| 58 } | 70 } |
| 59 | 71 |
| 60 } // namespace extensions | 72 } // namespace extensions |
| OLD | NEW |