[content] Rename MainMessageLoopStart to CreateMainMessageLoop

@qa: This CL is a no-op. The meaning of this phase has diverged and become increasingly confusing over the years. This is not the timing at which the main message loop starts but rather this is where it is created. The previous term is easily confused with things like VariationsService::PerformPreMainMessageLoopStartup startup_metric_utils::RecordBrowserMainMessageLoopStart that actually represent when the message loop starts. This confusion has led to new calls being associated with the wrong phase, e.g. https://chromium-review.googlesource.com/c/chromium/src/+/2685139/6#message-d1f863178a7982698d1ff5a032b79e33e9e1d5ce Bug: 1175074 Change-Id: I0535042abe69431501c9b3da3cfb2eae445fde0a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2892278 Commit-Queue: Gabriel Charette <gab@chromium.org> Reviewed-by: John Abd-El-Malek <jam@chromium.org> Owners-Override: John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#883468}
FairyWorld · May 17, 2021 · 09c6a96 · 09c6a96
1 parent 53710cf
commit 09c6a96
Show file tree

Hide file tree

Showing 45 changed files with 103 additions and 103 deletions.
diff --git a/chrome/browser/chrome_browser_main.cc b/chrome/browser/chrome_browser_main.cc
@@ -715,15 +715,15 @@ void ChromeBrowserMainParts::ToolkitInitialized() {
     chrome_extra_parts_[i]->ToolkitInitialized();
 }
 
-void ChromeBrowserMainParts::PreMainMessageLoopStart() {
-  TRACE_EVENT0("startup", "ChromeBrowserMainParts::PreMainMessageLoopStart");
+void ChromeBrowserMainParts::PreCreateMainMessageLoop() {
+  TRACE_EVENT0("startup", "ChromeBrowserMainParts::PreCreateMainMessageLoop");
 
   for (size_t i = 0; i < chrome_extra_parts_.size(); ++i)
-    chrome_extra_parts_[i]->PreMainMessageLoopStart();
+    chrome_extra_parts_[i]->PreCreateMainMessageLoop();
 }
 
-void ChromeBrowserMainParts::PostMainMessageLoopStart() {
-  TRACE_EVENT0("startup", "ChromeBrowserMainParts::PostMainMessageLoopStart");
+void ChromeBrowserMainParts::PostCreateMainMessageLoop() {
+  TRACE_EVENT0("startup", "ChromeBrowserMainParts::PostCreateMainMessageLoop");
 
 #if !defined(OS_ANDROID)
   // Initialize the upgrade detector here after ChromeBrowserMainPartsChromeos
@@ -745,7 +745,7 @@ void ChromeBrowserMainParts::PostMainMessageLoopStart() {
     device_event_log::Initialize(0 /* default max entries */);
 
   for (size_t i = 0; i < chrome_extra_parts_.size(); ++i)
-    chrome_extra_parts_[i]->PostMainMessageLoopStart();
+    chrome_extra_parts_[i]->PostCreateMainMessageLoop();
 }
 
 int ChromeBrowserMainParts::PreCreateThreads() {

diff --git a/chrome/browser/chrome_browser_main.h b/chrome/browser/chrome_browser_main.h
@@ -66,8 +66,8 @@ class ChromeBrowserMainParts : public content::BrowserMainParts {
   int PreEarlyInitialization() override;
   void PostEarlyInitialization() override;
   void ToolkitInitialized() override;
-  void PreMainMessageLoopStart() override;
-  void PostMainMessageLoopStart() override;
+  void PreCreateMainMessageLoop() override;
+  void PostCreateMainMessageLoop() override;
   int PreCreateThreads() override;
   void PostCreateThreads() override;
   int PreMainMessageLoopRun() override;

diff --git a/chrome/browser/chrome_browser_main_extra_parts.h b/chrome/browser/chrome_browser_main_extra_parts.h
@@ -28,9 +28,9 @@ class ChromeBrowserMainExtraParts {
   // ToolkitInitialized methods.
   virtual void ToolkitInitialized() {}
 
-  // MainMessageLoopStart methods.
-  virtual void PreMainMessageLoopStart() {}
-  virtual void PostMainMessageLoopStart() {}
+  // CreateMainMessageLoop methods.
+  virtual void PreCreateMainMessageLoop() {}
+  virtual void PostCreateMainMessageLoop() {}
 
   // MainMessageLoopRun methods.
   virtual void PreCreateThreads() {}

diff --git a/chrome/browser/chrome_browser_main_extra_parts_ozone.cc b/chrome/browser/chrome_browser_main_extra_parts_ozone.cc
@@ -31,7 +31,7 @@ void ChromeBrowserMainExtraPartsOzone::PreEarlyInitialization() {
 #endif
 }
 
-void ChromeBrowserMainExtraPartsOzone::PostMainMessageLoopStart() {
+void ChromeBrowserMainExtraPartsOzone::PostCreateMainMessageLoop() {
   auto shutdown_cb = base::BindOnce([] {
 #if BUILDFLAG(IS_CHROMEOS_LACROS)
     // Force a crash so that a crash report is generated.
@@ -42,7 +42,7 @@ void ChromeBrowserMainExtraPartsOzone::PostMainMessageLoopStart() {
   });
 #if defined(USE_OZONE)
   if (features::IsUsingOzonePlatform()) {
-    ui::OzonePlatform::GetInstance()->PostMainMessageLoopStart(
+    ui::OzonePlatform::GetInstance()->PostCreateMainMessageLoop(
         std::move(shutdown_cb));
     return;
   }

diff --git a/chrome/browser/chrome_browser_main_extra_parts_ozone.h b/chrome/browser/chrome_browser_main_extra_parts_ozone.h
@@ -21,7 +21,7 @@ class ChromeBrowserMainExtraPartsOzone : public ChromeBrowserMainExtraParts {
  private:
   // ChromeBrowserMainExtraParts overrides.
   void PreEarlyInitialization() override;
-  void PostMainMessageLoopStart() override;
+  void PostCreateMainMessageLoop() override;
   void PostMainMessageLoopRun() override;
 };
 

diff --git a/chrome/browser/chrome_browser_main_linux.cc b/chrome/browser/chrome_browser_main_linux.cc
@@ -77,13 +77,12 @@ void ChromeBrowserMainPartsLinux::PreProfileInit() {
   ChromeBrowserMainPartsPosix::PreProfileInit();
 }
 
-
-void ChromeBrowserMainPartsLinux::PostMainMessageLoopStart() {
+void ChromeBrowserMainPartsLinux::PostCreateMainMessageLoop() {
 #if !BUILDFLAG(IS_CHROMEOS_ASH)
   bluez::BluezDBusManager::Initialize(nullptr /* system_bus */);
 #endif
 
-  ChromeBrowserMainPartsPosix::PostMainMessageLoopStart();
+  ChromeBrowserMainPartsPosix::PostCreateMainMessageLoop();
 }
 
 void ChromeBrowserMainPartsLinux::PostDestroyThreads() {

diff --git a/chrome/browser/chrome_browser_main_linux.h b/chrome/browser/chrome_browser_main_linux.h
@@ -19,7 +19,7 @@ class ChromeBrowserMainPartsLinux : public ChromeBrowserMainPartsPosix {
 
   // ChromeBrowserMainParts overrides.
   void PreProfileInit() override;
-  void PostMainMessageLoopStart() override;
+  void PostCreateMainMessageLoop() override;
   void PostDestroyThreads() override;
 
  private:

diff --git a/chrome/browser/chrome_browser_main_mac.h b/chrome/browser/chrome_browser_main_mac.h
@@ -16,8 +16,8 @@ class ChromeBrowserMainPartsMac : public ChromeBrowserMainPartsPosix {
 
   // BrowserParts overrides.
   int PreEarlyInitialization() override;
-  void PreMainMessageLoopStart() override;
-  void PostMainMessageLoopStart() override;
+  void PreCreateMainMessageLoop() override;
+  void PostCreateMainMessageLoop() override;
   void PreProfileInit() override;
   void PostProfileInit() override;
 

diff --git a/chrome/browser/chrome_browser_main_mac.mm b/chrome/browser/chrome_browser_main_mac.mm
@@ -73,10 +73,10 @@
   return ChromeBrowserMainPartsPosix::PreEarlyInitialization();
 }
 
-void ChromeBrowserMainPartsMac::PreMainMessageLoopStart() {
+void ChromeBrowserMainPartsMac::PreCreateMainMessageLoop() {
   MacStartupProfiler::GetInstance()->Profile(
       MacStartupProfiler::PRE_MAIN_MESSAGE_LOOP_START);
-  ChromeBrowserMainPartsPosix::PreMainMessageLoopStart();
+  ChromeBrowserMainPartsPosix::PreCreateMainMessageLoop();
 
   // ChromeBrowserMainParts should have loaded the resource bundle by this
   // point (needed to load the nib).
@@ -133,10 +133,10 @@
   }
 }
 
-void ChromeBrowserMainPartsMac::PostMainMessageLoopStart() {
+void ChromeBrowserMainPartsMac::PostCreateMainMessageLoop() {
   MacStartupProfiler::GetInstance()->Profile(
       MacStartupProfiler::POST_MAIN_MESSAGE_LOOP_START);
-  ChromeBrowserMainPartsPosix::PostMainMessageLoopStart();
+  ChromeBrowserMainPartsPosix::PostCreateMainMessageLoop();
 
   if (base::FeatureList::IsEnabled(
           net::features::kCertVerifierBuiltinFeature)) {

diff --git a/chrome/browser/chrome_browser_main_posix.cc b/chrome/browser/chrome_browser_main_posix.cc
@@ -155,8 +155,8 @@ int ChromeBrowserMainPartsPosix::PreEarlyInitialization() {
   return content::RESULT_CODE_NORMAL_EXIT;
 }
 
-void ChromeBrowserMainPartsPosix::PostMainMessageLoopStart() {
-  ChromeBrowserMainParts::PostMainMessageLoopStart();
+void ChromeBrowserMainPartsPosix::PostCreateMainMessageLoop() {
+  ChromeBrowserMainParts::PostCreateMainMessageLoop();
 
   // Exit in response to SIGINT, SIGTERM, etc.
   InstallShutdownSignalHandlers(

diff --git a/chrome/browser/chrome_browser_main_posix.h b/chrome/browser/chrome_browser_main_posix.h
@@ -15,7 +15,7 @@ class ChromeBrowserMainPartsPosix : public ChromeBrowserMainParts {
 
   // content::BrowserMainParts overrides.
   int PreEarlyInitialization() override;
-  void PostMainMessageLoopStart() override;
+  void PostCreateMainMessageLoop() override;
 
   // ChromeBrowserMainParts overrides.
   void ShowMissingLocaleMessageBox() override;

diff --git a/chrome/browser/chrome_browser_main_win.cc b/chrome/browser/chrome_browser_main_win.cc
@@ -582,7 +582,7 @@ void ChromeBrowserMainPartsWin::ToolkitInitialized() {
   gfx::win::SetGetMinimumFontSizeCallback(&GetMinimumFontSize);
 }
 
-void ChromeBrowserMainPartsWin::PreMainMessageLoopStart() {
+void ChromeBrowserMainPartsWin::PreCreateMainMessageLoop() {
   // installer_util references strings that are normally compiled into
   // setup.exe.  In Chrome, these strings are in the locale files.
   SetupInstallerUtilStrings();
@@ -594,7 +594,7 @@ void ChromeBrowserMainPartsWin::PreMainMessageLoopStart() {
   bool os_crypt_init = OSCrypt::Init(local_state);
   DCHECK(os_crypt_init);
 
-  ChromeBrowserMainParts::PreMainMessageLoopStart();
+  ChromeBrowserMainParts::PreCreateMainMessageLoop();
   if (!parameters().ui_task) {
     // Make sure that we know how to handle exceptions from the message loop.
     InitializeWindowProcExceptions();

diff --git a/chrome/browser/chrome_browser_main_win.h b/chrome/browser/chrome_browser_main_win.h
@@ -33,7 +33,7 @@ class ChromeBrowserMainPartsWin : public ChromeBrowserMainParts {
 
   // BrowserParts overrides.
   void ToolkitInitialized() override;
-  void PreMainMessageLoopStart() override;
+  void PreCreateMainMessageLoop() override;
   int PreCreateThreads() override;
   void PostMainMessageLoopRun() override;
 

diff --git a/chrome/browser/chromeos/chrome_browser_main_chromeos.cc b/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
@@ -561,15 +561,15 @@ int ChromeBrowserMainPartsChromeos::PreEarlyInitialization() {
   return ChromeBrowserMainPartsLinux::PreEarlyInitialization();
 }
 
-void ChromeBrowserMainPartsChromeos::PreMainMessageLoopStart() {
+void ChromeBrowserMainPartsChromeos::PreCreateMainMessageLoop() {
   // Initialize session manager in early stage in case others want to listen
   // to session state change right after browser is started.
   g_browser_process->platform_part()->InitializeSessionManager();
 
-  ChromeBrowserMainPartsLinux::PreMainMessageLoopStart();
+  ChromeBrowserMainPartsLinux::PreCreateMainMessageLoop();
 }
 
-void ChromeBrowserMainPartsChromeos::PostMainMessageLoopStart() {
+void ChromeBrowserMainPartsChromeos::PostCreateMainMessageLoop() {
   // Used by ChromeOS components to retrieve the system token certificate
   // database.
   SystemTokenCertDbStorage::Initialize();
@@ -586,10 +586,10 @@ void ChromeBrowserMainPartsChromeos::PostMainMessageLoopStart() {
   // Need to be done after LoginState has been initialized in DBusServices().
   ::memory::MemoryKillsMonitor::Initialize();
 
-  ChromeBrowserMainPartsLinux::PostMainMessageLoopStart();
+  ChromeBrowserMainPartsLinux::PostCreateMainMessageLoop();
 }
 
-// Threads are initialized between MainMessageLoopStart and MainMessageLoopRun.
+// Threads are initialized between CreateMainMessageLoop and MainMessageLoopRun.
 // about_flags settings are applied in ChromeBrowserMainParts::PreCreateThreads.
 int ChromeBrowserMainPartsChromeos::PreMainMessageLoopRun() {
   network_change_manager_client_ = std::make_unique<NetworkChangeManagerClient>(

diff --git a/chrome/browser/chromeos/chrome_browser_main_chromeos.h b/chrome/browser/chromeos/chrome_browser_main_chromeos.h
@@ -128,8 +128,8 @@ class ChromeBrowserMainPartsChromeos : public ChromeBrowserMainPartsLinux {
 
   // ChromeBrowserMainParts overrides.
   int PreEarlyInitialization() override;
-  void PreMainMessageLoopStart() override;
-  void PostMainMessageLoopStart() override;
+  void PreCreateMainMessageLoop() override;
+  void PostCreateMainMessageLoop() override;
   int PreMainMessageLoopRun() override;
 
   // Stages called from PreMainMessageLoopRun.

diff --git a/chrome/browser/enterprise/reporting/report_scheduler_browsertest.cc b/chrome/browser/enterprise/reporting/report_scheduler_browsertest.cc
@@ -30,7 +30,7 @@ class ChromeBrowserExtraSetUp : public ChromeBrowserMainExtraParts {
   explicit ChromeBrowserExtraSetUp(
       policy::ChromeBrowserCloudManagementController::Observer* observer)
       : observer_(observer) {}
-  void PreMainMessageLoopStart() override {
+  void PreCreateMainMessageLoop() override {
     g_browser_process->browser_policy_connector()
         ->chrome_browser_cloud_management_controller()
         ->AddObserver(observer_);

diff --git a/chrome/browser/performance_monitor/chrome_browser_main_extra_parts_performance_monitor.cc b/chrome/browser/performance_monitor/chrome_browser_main_extra_parts_performance_monitor.cc
@@ -14,7 +14,8 @@ ChromeBrowserMainExtraPartsPerformanceMonitor::
 ChromeBrowserMainExtraPartsPerformanceMonitor::
     ~ChromeBrowserMainExtraPartsPerformanceMonitor() = default;
 
-void ChromeBrowserMainExtraPartsPerformanceMonitor::PostMainMessageLoopStart() {
+void ChromeBrowserMainExtraPartsPerformanceMonitor::
+    PostCreateMainMessageLoop() {
   process_monitor_ = performance_monitor::ProcessMonitor::Create();
   system_monitor_ = performance_monitor::SystemMonitor::Create();
 }

diff --git a/chrome/browser/performance_monitor/chrome_browser_main_extra_parts_performance_monitor.h b/chrome/browser/performance_monitor/chrome_browser_main_extra_parts_performance_monitor.h
@@ -27,7 +27,7 @@ class ChromeBrowserMainExtraPartsPerformanceMonitor
       const ChromeBrowserMainExtraPartsPerformanceMonitor&) = delete;
 
   // ChromeBrowserMainExtraParts:
-  void PostMainMessageLoopStart() override;
+  void PostCreateMainMessageLoop() override;
   void PreMainMessageLoopRun() override;
 
  private:

diff --git a/chrome/browser/policy/cloud/chrome_browser_cloud_management_browsertest.cc b/chrome/browser/policy/cloud/chrome_browser_cloud_management_browsertest.cc
@@ -145,7 +145,7 @@ class ChromeBrowserExtraSetUp : public ChromeBrowserMainExtraParts {
   explicit ChromeBrowserExtraSetUp(
       ChromeBrowserCloudManagementControllerObserver* observer)
       : observer_(observer) {}
-  void PreMainMessageLoopStart() override {
+  void PreCreateMainMessageLoop() override {
     g_browser_process->browser_policy_connector()
         ->chrome_browser_cloud_management_controller()
         ->AddObserver(observer_);

diff --git a/chrome/browser/ui/ash/chrome_browser_main_extra_parts_ash.cc b/chrome/browser/ui/ash/chrome_browser_main_extra_parts_ash.cc
@@ -114,7 +114,7 @@ ChromeBrowserMainExtraPartsAsh::ChromeBrowserMainExtraPartsAsh() = default;
 
 ChromeBrowserMainExtraPartsAsh::~ChromeBrowserMainExtraPartsAsh() = default;
 
-void ChromeBrowserMainExtraPartsAsh::PreMainMessageLoopStart() {
+void ChromeBrowserMainExtraPartsAsh::PreCreateMainMessageLoop() {
   user_profile_loaded_observer_ = std::make_unique<UserProfileLoadedObserver>();
 }
 

diff --git a/chrome/browser/ui/ash/chrome_browser_main_extra_parts_ash.h b/chrome/browser/ui/ash/chrome_browser_main_extra_parts_ash.h
@@ -62,7 +62,7 @@ class ChromeBrowserMainExtraPartsAsh : public ChromeBrowserMainExtraParts {
   ~ChromeBrowserMainExtraPartsAsh() override;
 
   // Overridden from ChromeBrowserMainExtraParts:
-  void PreMainMessageLoopStart() override;
+  void PreCreateMainMessageLoop() override;
   void PreProfileInit() override;
   void PostProfileInit() override;
   void PostBrowserStart() override;

diff --git a/chrome/test/base/in_process_browser_test.cc b/chrome/test/base/in_process_browser_test.cc
@@ -178,7 +178,7 @@ class ChromeBrowserMainExtraPartsBrowserProcessInjection
   ChromeBrowserMainExtraPartsBrowserProcessInjection() = default;
 
   // ChromeBrowserMainExtraParts implementation
-  void PreMainMessageLoopStart() override {
+  void PreCreateMainMessageLoop() override {
     // The real GeolocationManager initializes a CLLocationManager. It has
     // been observed that when thousands of instances of this object are
     // created, as happens when running browser tests, the CoreLocationAgent

diff --git a/chromecast/browser/cast_browser_main_parts.cc b/chromecast/browser/cast_browser_main_parts.cc
@@ -454,7 +454,7 @@ content::BrowserContext* CastBrowserMainParts::browser_context() {
   return cast_browser_process_->browser_context();
 }
 
-void CastBrowserMainParts::PreMainMessageLoopStart() {
+void CastBrowserMainParts::PreCreateMainMessageLoop() {
   // GroupedHistograms needs to be initialized before any threads are created
   // to prevent race conditions between calls to Preregister and those threads
   // attempting to collect metrics.
@@ -474,7 +474,7 @@ void CastBrowserMainParts::PreMainMessageLoopStart() {
 #endif  // !(defined(OS_ANDROID) || defined(OS_FUCHSIA))
 }
 
-void CastBrowserMainParts::PostMainMessageLoopStart() {
+void CastBrowserMainParts::PostCreateMainMessageLoop() {
   // Ensure CastMetricsHelper initialized on UI thread.
   metrics::CastMetricsHelper::GetInstance();
 }

diff --git a/chromecast/browser/cast_browser_main_parts.h b/chromecast/browser/cast_browser_main_parts.h
@@ -80,8 +80,8 @@ class CastBrowserMainParts : public content::BrowserMainParts {
   content::BrowserContext* browser_context();
 
   // content::BrowserMainParts implementation:
-  void PreMainMessageLoopStart() override;
-  void PostMainMessageLoopStart() override;
+  void PreCreateMainMessageLoop() override;
+  void PostCreateMainMessageLoop() override;
   void ToolkitInitialized() override;
   int PreCreateThreads() override;
   int PreMainMessageLoopRun() override;

diff --git a/components/session_manager/README.md b/components/session_manager/README.md
@@ -19,5 +19,5 @@ SessionManager::Get() method gets the singleton like instance. The instance
 in Chrome browser is ChromeSessionManager derived from SessionManager. This
 is needed because SessionManager logic depends on policy code which is still
 part of the browser. ChromeSessionManager is in browser/ and can access that.
-The ChromeSessionManager instance is created in the PreMainMessageLoopStart
+The ChromeSessionManager instance is created in the PreCreateMainMessageLoop
 stage and destroyed in the PostMainMessageLoopRun.
diff --git a/content/app/content_main_runner_impl.cc b/content/app/content_main_runner_impl.cc
@@ -995,7 +995,7 @@ int ContentMainRunnerImpl::RunBrowser(MainFunctionParams& main_params,
     // Register the TaskExecutor for posting task to the BrowserThreads. It is
     // incorrect to post to a BrowserThread before this point. This instantiates
     // and binds the MessageLoopForUI on the main thread (but it's only labeled
-    // as BrowserThread::UI in BrowserMainLoop::MainMessageLoopStart).
+    // as BrowserThread::UI in BrowserMainLoop::CreateMainMessageLoop).
     BrowserTaskExecutor::Create();
 
     delegate_->PostEarlyInitialization(main_params.ui_task != nullptr);

diff --git a/content/browser/browser_main_loop.cc b/content/browser/browser_main_loop.cc
@@ -629,19 +629,20 @@ int BrowserMainLoop::EarlyInitialization() {
   return RESULT_CODE_NORMAL_EXIT;
 }
 
-void BrowserMainLoop::PreMainMessageLoopStart() {
-  TRACE_EVENT0("startup",
-               "BrowserMainLoop::MainMessageLoopStart:PreMainMessageLoopStart");
+void BrowserMainLoop::PreCreateMainMessageLoop() {
+  TRACE_EVENT0(
+      "startup",
+      "BrowserMainLoop::CreateMainMessageLoop:PreCreateMainMessageLoop");
   if (parts_) {
-    parts_->PreMainMessageLoopStart();
+    parts_->PreCreateMainMessageLoop();
   }
 }
 
-void BrowserMainLoop::MainMessageLoopStart() {
-  // DO NOT add more code here. Use PreMainMessageLoopStart() above or
-  // PostMainMessageLoopStart() below.
+void BrowserMainLoop::CreateMainMessageLoop() {
+  // DO NOT add more code here. Use PreCreateMainMessageLoop() above or
+  // PostCreateMainMessageLoop() below.
 
-  TRACE_EVENT0("startup", "BrowserMainLoop::MainMessageLoopStart");
+  TRACE_EVENT0("startup", "BrowserMainLoop::CreateMainMessageLoop");
 
   base::PlatformThread::SetName("CrBrowserMain");
 
@@ -653,8 +654,8 @@ void BrowserMainLoop::MainMessageLoopStart() {
       BrowserThread::UI, base::ThreadTaskRunnerHandle::Get()));
 }
 
-void BrowserMainLoop::PostMainMessageLoopStart() {
-  TRACE_EVENT0("startup", "BrowserMainLoop::PostMainMessageLoopStart");
+void BrowserMainLoop::PostCreateMainMessageLoop() {
+  TRACE_EVENT0("startup", "BrowserMainLoop::PostCreateMainMessageLoop");
   {
     TRACE_EVENT0("startup", "BrowserMainLoop::Subsystem:SystemMonitor");
     system_monitor_ = std::make_unique<base::SystemMonitor>();
@@ -706,7 +707,7 @@ void BrowserMainLoop::PostMainMessageLoopStart() {
   }
 
   if (parts_)
-    parts_->PostMainMessageLoopStart();
+    parts_->PostCreateMainMessageLoop();
 
 #if defined(OS_ANDROID)
   {