Profiler-instrumentation of the startup time.

chrome/app/chrome_main_delegate.h

 // Copyright (c) 2011 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.
 
 #ifndef CHROME_APP_CHROME_MAIN_DELEGATE_H_
 #define CHROME_APP_CHROME_MAIN_DELEGATE_H_
 
 #include "base/memory/scoped_ptr.h"
 #include "chrome/common/chrome_content_client.h"
 #include "content/public/app/content_main_delegate.h"
@@ skipping 24 matching lines @@
   bool ProcessRegistersWithSystemProcess(
       const std::string& process_type) override;
   bool ShouldSendMachPort(const std::string& process_type) override;
   bool DelaySandboxInitialization(const std::string& process_type) override;
 #elif defined(OS_POSIX) && !defined(OS_ANDROID)
   void ZygoteStarting(
       ScopedVector<content::ZygoteForkDelegate>* delegates) override;
   void ZygoteForked() override;
 #endif
 
+  bool ShouldEnableProfilerRecording() override;

[cpu_(ooo_6.6-7.5), 2015/03/16 22:16:25]

this function looks like it is in the middle of a bunch of override impl of a
delegate.

[yao, 2015/03/17 14:32:01]

Sorry I don't quite understand your suggestion here. Are you saying this
function should not be placed here? But it's in the same order with the base
class ContentMainDelegate. 

[vadimt, 2015/03/17 14:37:26]

yao@, it looks like there are 2 sections of overrides, separated by an empty
line.

The first group is "just" overrides, and the second one is "Create*" overrides.
Perhaps, you should move your method into the first group.

[yao, 2015/03/17 14:44:46]

Like this? 

(Also, The first half is public in the base class, and the second half is
protected.)

   content::ContentBrowserClient* CreateContentBrowserClient() override;
   content::ContentPluginClient* CreateContentPluginClient() override;
   content::ContentRendererClient* CreateContentRendererClient() override;
   content::ContentUtilityClient* CreateContentUtilityClient() override;
 
 #if defined(OS_MACOSX)
   void InitMacCrashReporter(const base::CommandLine& command_line,
                             const std::string& process_type);
 #endif  // defined(OS_MACOSX)
 
   ChromeContentClient chrome_content_client_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeMainDelegate);
 };
 
 #endif  // CHROME_APP_CHROME_MAIN_DELEGATE_H_