Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2128)

Unified Diff: chrome/browser/profiles/profile_impl_io_data.cc

Issue 790853007: Instrumenting InitializeInternal to find jank (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: mmenke comment Created 6 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/profiles/off_the_record_profile_io_data.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/profiles/profile_impl_io_data.cc
diff --git a/chrome/browser/profiles/profile_impl_io_data.cc b/chrome/browser/profiles/profile_impl_io_data.cc
index 8b3943958d0f4b30f2a011e7f4998dee43722a5d..5349f1a4f5d59d805dd664534e65a5744d469c58 100644
--- a/chrome/browser/profiles/profile_impl_io_data.cc
+++ b/chrome/browser/profiles/profile_impl_io_data.cc
@@ -10,6 +10,7 @@
#include "base/metrics/field_trial.h"
#include "base/prefs/pref_member.h"
#include "base/prefs/pref_service.h"
+#include "base/profiler/scoped_tracker.h"
#include "base/sequenced_task_runner.h"
#include "base/stl_util.h"
#include "base/strings/string_util.h"
@@ -426,6 +427,11 @@ void ProfileImplIOData::InitializeInternal(
ProfileParams* profile_params,
content::ProtocolHandlerMap* protocol_handlers,
content::URLRequestInterceptorScopedVector request_interceptors) const {
+ // TODO(vadimt): Remove ScopedTracker below once crbug.com/436671 is fixed.
+ tracked_objects::ScopedTracker tracking_profile(
+ FROM_HERE_WITH_EXPLICIT_FUNCTION(
+ "436671 ProfileImplIOData::InitializeInternal"));
+
net::URLRequestContext* main_context = main_request_context();
IOThread* const io_thread = profile_params->io_thread;
@@ -434,6 +440,11 @@ void ProfileImplIOData::InitializeInternal(
chrome_network_delegate->set_predictor(predictor_.get());
if (domain_reliability_monitor_) {
+ // TODO(vadimt): Remove ScopedTracker below once crbug.com/436671 is fixed.
+ tracked_objects::ScopedTracker tracking_profile1(
+ FROM_HERE_WITH_EXPLICIT_FUNCTION(
+ "436671 ProfileImplIOData::InitializeInternal1"));
+
domain_reliability::DomainReliabilityMonitor* monitor =
domain_reliability_monitor_.get();
monitor->InitURLRequestContext(main_context);
@@ -442,6 +453,11 @@ void ProfileImplIOData::InitializeInternal(
chrome_network_delegate->set_domain_reliability_monitor(monitor);
}
+ // TODO(vadimt): Remove ScopedTracker below once crbug.com/436671 is fixed.
+ tracked_objects::ScopedTracker tracking_profile2(
+ FROM_HERE_WITH_EXPLICIT_FUNCTION(
+ "436671 ProfileImplIOData::InitializeInternal2"));
+
set_data_reduction_proxy_auth_request_handler(
scoped_ptr<data_reduction_proxy::DataReductionProxyAuthRequestHandler>
(new data_reduction_proxy::DataReductionProxyAuthRequestHandler(
@@ -457,6 +473,11 @@ void ProfileImplIOData::InitializeInternal(
data_reduction_proxy_usage_stats()->set_unavailable_callback(
data_reduction_proxy_unavailable_callback());
+ // TODO(vadimt): Remove ScopedTracker below once crbug.com/436671 is fixed.
+ tracked_objects::ScopedTracker tracking_profile3(
+ FROM_HERE_WITH_EXPLICIT_FUNCTION(
+ "436671 ProfileImplIOData::InitializeInternal3"));
+
scoped_ptr<data_reduction_proxy::DataReductionProxyNetworkDelegate>
data_reduction_proxy_network_delegate(
new data_reduction_proxy::DataReductionProxyNetworkDelegate(
@@ -477,6 +498,11 @@ void ProfileImplIOData::InitializeInternal(
// Initialize context members.
+ // TODO(vadimt): Remove ScopedTracker below once crbug.com/436671 is fixed.
+ tracked_objects::ScopedTracker tracking_profile4(
+ FROM_HERE_WITH_EXPLICIT_FUNCTION(
+ "436671 ProfileImplIOData::InitializeInternal4"));
+
ApplyProfileParamsToContext(main_context);
if (http_server_properties_manager_)
@@ -521,6 +547,10 @@ void ProfileImplIOData::InitializeInternal(
base::WorkerPool::GetTaskRunner(true));
}
+ // TODO(vadimt): Remove ScopedTracker below once crbug.com/436671 is fixed.
+ tracked_objects::ScopedTracker tracking_profile5(
+ FROM_HERE_WITH_EXPLICIT_FUNCTION(
+ "436671 ProfileImplIOData::InitializeInternal5"));
// setup cookie store
if (!cookie_store.get()) {
@@ -538,6 +568,11 @@ void ProfileImplIOData::InitializeInternal(
main_context->set_cookie_store(cookie_store.get());
+ // TODO(vadimt): Remove ScopedTracker below once crbug.com/436671 is fixed.
+ tracked_objects::ScopedTracker tracking_profile6(
+ FROM_HERE_WITH_EXPLICIT_FUNCTION(
+ "436671 ProfileImplIOData::InitializeInternal6"));
+
// Setup server bound cert service.
if (!channel_id_service) {
DCHECK(!lazy_params_->channel_id_path.empty());
@@ -556,6 +591,11 @@ void ProfileImplIOData::InitializeInternal(
set_channel_id_service(channel_id_service);
main_context->set_channel_id_service(channel_id_service);
+ // TODO(vadimt): Remove ScopedTracker below once crbug.com/436671 is fixed.
+ tracked_objects::ScopedTracker tracking_profile7(
+ FROM_HERE_WITH_EXPLICIT_FUNCTION(
+ "436671 ProfileImplIOData::InitializeInternal7"));
+
net::HttpCache::DefaultBackend* main_backend =
new net::HttpCache::DefaultBackend(
net::DISK_CACHE,
@@ -565,6 +605,12 @@ void ProfileImplIOData::InitializeInternal(
BrowserThread::GetMessageLoopProxyForThread(BrowserThread::CACHE));
scoped_ptr<net::HttpCache> main_cache = CreateMainHttpFactory(
profile_params, main_backend);
+
+ // TODO(vadimt): Remove ScopedTracker below once crbug.com/436671 is fixed.
+ tracked_objects::ScopedTracker tracking_profile71(
+ FROM_HERE_WITH_EXPLICIT_FUNCTION(
+ "436671 ProfileImplIOData::InitializeInternal71"));
+
main_cache->InitializeInfiniteCache(lazy_params_->infinite_cache_path);
if (chrome_browser_net::ShouldUseInMemoryCookiesAndCache()) {
@@ -581,6 +627,11 @@ void ProfileImplIOData::InitializeInternal(
new net::FtpNetworkLayer(io_thread_globals->host_resolver.get()));
#endif // !defined(DISABLE_FTP_SUPPORT)
+ // TODO(vadimt): Remove ScopedTracker below once crbug.com/436671 is fixed.
+ tracked_objects::ScopedTracker tracking_profile8(
+ FROM_HERE_WITH_EXPLICIT_FUNCTION(
+ "436671 ProfileImplIOData::InitializeInternal8"));
+
scoped_ptr<net::URLRequestJobFactoryImpl> main_job_factory(
new net::URLRequestJobFactoryImpl());
InstallProtocolHandlers(main_job_factory.get(), protocol_handlers);
@@ -600,6 +651,11 @@ void ProfileImplIOData::InitializeInternal(
ftp_factory_.get());
main_context->set_job_factory(main_job_factory_.get());
+ // TODO(vadimt): Remove ScopedTracker below once crbug.com/436671 is fixed.
+ tracked_objects::ScopedTracker tracking_profile9(
+ FROM_HERE_WITH_EXPLICIT_FUNCTION(
+ "436671 ProfileImplIOData::InitializeInternal9"));
+
#if defined(ENABLE_EXTENSIONS)
InitializeExtensionsRequestContext(profile_params);
#endif
« no previous file with comments | « chrome/browser/profiles/off_the_record_profile_io_data.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698