Break chrome_initial's dependence on //components/startup_metric_utils/browser:lib

chrome/app/chrome_main_delegate.cc

Index: chrome/app/chrome_main_delegate.cc
diff --git a/chrome/app/chrome_main_delegate.cc b/chrome/app/chrome_main_delegate.cc
index aed0427230ba5a2cc4661c62f0c0597b37188047..19bbcd562a86d9533de0c4452b7b92835acb558c 100644
--- a/chrome/app/chrome_main_delegate.cc
+++ b/chrome/app/chrome_main_delegate.cc
@@ -434,8 +434,10 @@ void InitLogging(const std::string& process_type) {
 #endif
 
 #if !defined(CHROME_MULTIPLE_DLL_CHILD)
-void RecordMainStartupMetrics() {
+void RecordMainStartupMetrics(base::TimeTicks exe_entry_point_ticks) {
 #if defined(OS_MACOSX) || defined(OS_WIN) || defined(OS_LINUX)
+  if (!exe_entry_point_ticks.is_null())
+    startup_metric_utils::RecordExeMainEntryPointTicks(exe_entry_point_ticks);

[fdoray, 2016/09/20 12:25:21]

The #if is there because base::CurrentProcessInfo::CreationTime() isn't defined
on Android. There is no reason to put RecordExeMainEntryPointTicks() inside the
#if.

|exe_entry_point_ticks| will be null on all platforms except Windows so you
could surround RecordExeMainEntryPointTicks() with #if defined(OS_WIN). However,
I would not do that because it would force us to modify this file when we add
exe entry point time on Mac (we plan to do it soon).

[grt (UTC plus 2), 2016/09/20 19:20:26]

Done.

   // Record the startup process creation time on supported platforms.
   startup_metric_utils::RecordStartupProcessCreationTime(
       base::CurrentProcessInfo::CreationTime());
@@ -454,13 +456,16 @@ void RecordMainStartupMetrics() {
 
 }  // namespace
 
-ChromeMainDelegate::ChromeMainDelegate() {
+ChromeMainDelegate::ChromeMainDelegate() : ChromeMainDelegate(0) {}
+
+ChromeMainDelegate::ChromeMainDelegate(int64_t exe_entry_point_ticks) {
 #if !defined(CHROME_MULTIPLE_DLL_CHILD)
   // Record startup metrics in the browser process. For component builds, there
   // is no way to know the type of process (process command line is not yet
   // initialized), so the function below will also be called in renderers.
   // This doesn't matter as it simply sets global variables.
-  RecordMainStartupMetrics();
+  RecordMainStartupMetrics(
+      base::TimeTicks::FromInternalValue(exe_entry_point_ticks));
 #endif  // !defined(CHROME_MULTIPLE_DLL_CHILD)
 }