Fix Crostini browser test failures with network service.

Change-Id: I7b39e8d244b150ab4093c2dcee21ca9873934661 Reviewed-on: https://chromium-review.googlesource.com/c/1478471 Reviewed-by: Scott Violet <[email protected]> Commit-Queue: Robbie McElrath <[email protected]> Cr-Commit-Position: refs/heads/master@{#633865}
feiyue1206 · Feb 20, 2019 · baa6303 · baa6303
1 parent dca111a
commit baa6303
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/chrome/browser/ui/views/crostini/crostini_browser_test_util.cc b/chrome/browser/ui/views/crostini/crostini_browser_test_util.cc
@@ -59,8 +59,10 @@ class CrostiniBrowserTestChromeBrowserMainExtraParts
     browser_process_platform_part_test_api_->InitializeCrosComponentManager(
         std::move(cros_component_manager));
   }
-  void ServiceManagerConnectionStarted(
-      content::ServiceManagerConnection* connection) override {
+  // Ideally we'd call SetConnectionType in ServiceManagerConnectionStarted,
+  // but currently we have to wait for PreProfileInit to complete, since that
+  // creates the ash::Shell that AshService needs in order to start.
+  void PostProfileInit() override {
     connection_change_simulator_.SetConnectionType(
         network::mojom::ConnectionType::CONNECTION_WIFI);
   }