Refactor Extension initialization logic. Add tracing and additional histogram.

chrome/browser/extensions/component_loader.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/extensions/component_loader.h"
 
 #include <string>
 
 #include "base/command_line.h"
 #include "base/files/file_util.h"
@@ skipping 115 matching lines @@
       browser_context_(browser_context),
       extension_service_(extension_service),
       weak_factory_(this) {}
 
 ComponentLoader::~ComponentLoader() {
   ClearAllRegistered();
 }
 
 void ComponentLoader::LoadAll() {
   TRACE_EVENT0("browser,startup", "ComponentLoader::LoadAll");
-  const base::TimeTicks start_time = base::TimeTicks::Now();
+  SCOPED_UMA_HISTOGRAM_TIMER("Extensions.LoadAllComponentTime");
+
   for (RegisteredComponentExtensions::iterator it =
           component_extensions_.begin();
       it != component_extensions_.end(); ++it) {
     Load(*it);
   }
-  UMA_HISTOGRAM_TIMES("Extensions.LoadAllComponentTime",
-                      base::TimeTicks::Now() - start_time);
 }
 
 base::DictionaryValue* ComponentLoader::ParseManifest(
     const std::string& manifest_contents) const {
   JSONStringValueSerializer serializer(manifest_contents);
   scoped_ptr<base::Value> manifest(serializer.Deserialize(NULL, NULL));
 
   if (!manifest.get() || !manifest->IsType(base::Value::TYPE_DICTIONARY)) {
     LOG(ERROR) << "Failed to parse extension manifest.";
     return NULL;
@@ skipping 519 matching lines @@
         off_the_record_context, Extension::GetBaseURLFromExtensionId(id));
     storage::FileSystemContext* file_system_context =
         content::BrowserContext::GetStoragePartitionForSite(
             off_the_record_context, site)->GetFileSystemContext();
     file_system_context->EnableTemporaryFileSystemInIncognito();
   }
 #endif
 }
 
 }  // namespace extensions