Profiler-instrumentation of the startup time.

content/app/content_main_runner.cc

Index: content/app/content_main_runner.cc
diff --git a/content/app/content_main_runner.cc b/content/app/content_main_runner.cc
index 863f02be27411b4490f047a8b4d8d944d6216518..d844d6db84315808964a62326b3b0adb002eb859 100644
--- a/content/app/content_main_runner.cc
+++ b/content/app/content_main_runner.cc
@@ -568,6 +568,9 @@ class ContentMainRunnerImpl : public ContentMainRunner {
 
     base::EnableTerminationOnHeapCorruption();
 
+    // Enable profiler recording right after command line is initialized.
+    delegate_->EnableProfilerRecording();

[vadimt, 2015/03/12 18:23:03]

Please move this call to BrowserMainRunnerImpl::Initialize.

[yao, 2015/03/12 18:32:19]

This place runs earlier than that. we may want to intrument code that's earlier
than BrowserMainRunnerImpl::Initialize? do we? 

[vadimt, 2015/03/12 18:37:26]

Unlikely. But we'll benefit from not spreading this logic across multiple
places. Putting everything together will make it easier to understand and clean
up code.

(This is a strong opinion :) )

[yao, 2015/03/13 14:28:31]

I see. 

But this function takes in ContentMainParams, which has delegate as one of the
params. However, the function you are suggesting takes in MainFunctionParams and
it doesn't not have that delegate any more. 

[vadimt, 2015/03/13 14:32:58]

Then it's OK to leave it here. Please mark ALL added code with the comment to
remove it after crbug.com/453640 is fixed.

[yao, 2015/03/13 15:24:49]

Done.

+
 #if !defined(OS_IOS)
     SetProcessTitleFromCommandLine(argv);
 #endif