Commit 45628974 authored by Devlin Cronin's avatar Devlin Cronin Committed by Commit Bot

[Extensions] Add UMA for extensions client initialization time

It seems that ChromeExtensionsClient::Init() is taking a non-trivial
amount of time (see linked bug). Add UMA so that we can track it,
and measure any future improvements.

Bug: 847237
Change-Id: I1b86dd9a8f2a7500841f253580a9e6d64c9f4d45
Reviewed-on: https://chromium-review.googlesource.com/1142617Reviewed-by: default avatarIlya Sherman <isherman@chromium.org>
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#577700}
parent 1f52b3be
......@@ -105,6 +105,8 @@ ChromeExtensionsClient::~ChromeExtensionsClient() {
}
void ChromeExtensionsClient::Initialize() {
SCOPED_UMA_HISTOGRAM_TIMER("Extensions.ChromeExtensionsClientInitTime");
// Registration could already be finalized in unit tests, where the utility
// thread runs in-process.
if (!ManifestHandler::IsRegistrationFinalized()) {
......
......@@ -27200,6 +27200,17 @@ uploading your change for review.
</summary>
</histogram>
<histogram name="Extensions.ChromeExtensionsClientInitTime" units="ms"
expires_after="2019-12-01">
<owner>rdevlin.cronin@chromium.org</owner>
<owner>extensions-core@chromium.org</owner>
<summary>
The amount of elapsed time taken to initialize the ChromeExtensionsClient.
Recorded once per client initialization, which happens once per instance of
Chrome.
</summary>
</histogram>
<histogram name="Extensions.ContentVerification.ComputedHashesInitTime"
units="ms" expires_after="2019-11-30">
<owner>lazyboy@chromium.org</owner>
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment