Delete abandoned ShouldEnableProfilerRecording().

It is marked for deletion and has no callers.

Change-Id: Iceb00f843a300c7dcb1d6925423227e9c17d49a3
Reviewed-on: https://chromium-review.googlesource.com/c/1337010Reviewed-by: Gabriel Charette <gab@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#608428}

chrome/app/chrome_main_delegate.cc

@@ -1118,20 +1118,6 @@ ChromeMainDelegate::CreateContentUtilityClient() {
 #endif
 }
 
-bool ChromeMainDelegate::ShouldEnableProfilerRecording() {
-  switch (chrome::GetChannel()) {
-    case version_info::Channel::UNKNOWN:
-    case version_info::Channel::CANARY:
-      return true;
-    case version_info::Channel::DEV:
-    case version_info::Channel::BETA:
-    case version_info::Channel::STABLE:
-    default:
-      // Don't enable instrumentation.
-      return false;
-  }
-}
-
 service_manager::ProcessType ChromeMainDelegate::OverrideProcessType() {
   const auto& command_line = *base::CommandLine::ForCurrentProcess();
   if (command_line.GetSwitchValueASCII(switches::kProcessType) ==

chrome/app/chrome_main_delegate.h

@@ -61,7 +61,6 @@ class ChromeMainDelegate : public content::ContentMainDelegate {
           delegates) override;
   void ZygoteForked() override;
 #endif
-  bool ShouldEnableProfilerRecording() override;
   service_manager::ProcessType OverrideProcessType() override;
   void PreCreateMainMessageLoop() override;
 #if !defined(CHROME_MULTIPLE_DLL_CHILD)

content/public/app/content_main_delegate.cc

@@ -55,10 +55,6 @@ int ContentMainDelegate::TerminateForFatalInitializationError() {
   return 0;
 }
 
-bool ContentMainDelegate::ShouldEnableProfilerRecording() {
-  return false;
-}
-
 service_manager::ProcessType ContentMainDelegate::OverrideProcessType() {
   return service_manager::ProcessType::kDefault;
 }

content/public/app/content_main_delegate.h

@@ -88,11 +88,6 @@ class CONTENT_EXPORT ContentMainDelegate {
   virtual void ZygoteForked() {}
 #endif  // defined(OS_LINUX)
 
-  // TODO(vadimt, yiyaoliu): Remove this function once crbug.com/453640 is
-  // fixed.
-  // Returns whether or not profiler recording should be enabled.
-  virtual bool ShouldEnableProfilerRecording();
-
   // Fatal errors during initialization are reported by this function, so that
   // the embedder can implement graceful exit by displaying some message and
   // returning initialization error code. Default behavior is CHECK(false).