Revert "Add InitParams for views::AuraInit"

This reverts commit dfd53f11.

Reason for revert: GPU FYI Windows bots compile failure
crbug.com/854442

Compile error:
[40824/45748] LINK task_viewer.service.exe task_viewer.service.exe.pdb
FAILED: task_viewer.service.exe task_viewer.service.exe.pdb 
C:/b/swarming/w/ir/cache/vpython/80ee9f/Scripts/python.exe ../../build/toolchain/win/tool_wrapper.py link-wrapper environment.x64 False ../../third_party/llvm-build/Release+Asserts/bin/lld-link.exe /nologo /OUT:./task_viewer.service.exe /PDB:./task_viewer.service.exe.pdb @./task_viewer.service.exe.rsp
C:\b\swarming\w\ir\cache\builder\src\third_party\llvm-build\Release+Asserts\bin\lld-link.exe: error: undefined symbol: ??0InitParams@AuraInit@views@@QEAA@XZ
>>> referenced by obj/mash/task_viewer/lib/task_viewer.obj:(?OnStart@TaskViewer@task_viewer@mash@@EEAAXXZ)
C:\b\swarming\w\ir\cache\builder\src\third_party\llvm-build\Release+Asserts\bin\lld-link.exe: error: undefined symbol: ??1InitParams@AuraInit@views@@QEAA@XZ
>>> referenced by obj/mash/task_viewer/lib/task_viewer.obj:(?OnStart@TaskViewer@task_viewer@mash@@EEAAXXZ)

Original change's description:
> Add InitParams for views::AuraInit
> 
> It's up to 7 parameters and I'm about to add an 8th.
> 
> Bug: 851578
> Test: existing tests
> Change-Id: I5dbfb98c90861a334ef9046cec4d69fb756879a1
> Reviewed-on: https://chromium-review.googlesource.com/1106431
> Reviewed-by: Scott Violet <sky@chromium.org>
> Commit-Queue: James Cook <jamescook@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#568594}

TBR=jamescook@chromium.org,sky@chromium.org

Change-Id: Iea7d59178394a95cbc6f419cd7079d00d66a8ac6
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 851578, 854442
Reviewed-on: https://chromium-review.googlesource.com/1107139Reviewed-by: weiliangc <weiliangc@chromium.org>
Commit-Queue: weiliangc <weiliangc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#568678}

ash/components/autoclick/autoclick_application.cc

@@ -93,12 +93,11 @@ AutoclickApplication::AutoclickApplication()
 AutoclickApplication::~AutoclickApplication() = default;
 
 void AutoclickApplication::OnStart() {
-  views::AuraInit::InitParams params;
-  params.connector = context()->connector();
-  params.identity = context()->identity();
-  params.mode = views::AuraInit::Mode::AURA_MUS;
-  params.register_path_provider = running_standalone_;
-  aura_init_ = views::AuraInit::Create(params);
+  const bool register_path_provider = running_standalone_;
+  aura_init_ = views::AuraInit::Create(
+      context()->connector(), context()->identity(), "views_mus_resources.pak",
+      std::string(), nullptr, views::AuraInit::Mode::AURA_MUS,
+      register_path_provider);
   if (!aura_init_) {
     context()->QuitNow();
     return;

ash/components/quick_launch/quick_launch_application.cc

@@ -174,12 +174,11 @@ void QuickLaunchApplication::OnStart() {
   // If AuraInit was unable to initialize there is no longer a peer connection.
   // The ServiceManager is in the process of shutting down, however we haven't
   // been notified yet. Close our ServiceContext and shutdown.
-  views::AuraInit::InitParams params;
-  params.connector = context()->connector();
-  params.identity = context()->identity();
-  params.mode = views::AuraInit::Mode::AURA_MUS2;
-  params.register_path_provider = running_standalone_;
-  aura_init_ = views::AuraInit::Create(params);
+  const bool register_path_provider = running_standalone_;
+  aura_init_ = views::AuraInit::Create(
+      context()->connector(), context()->identity(), "views_mus_resources.pak",
+      std::string(), nullptr, views::AuraInit::Mode::AURA_MUS2,
+      register_path_provider);
   if (!aura_init_) {
     context()->QuitNow();
     return;

ash/components/shortcut_viewer/shortcut_viewer_application.cc

@@ -33,12 +33,10 @@ void ShortcutViewerApplication::RegisterForTraceEvents() {
 }
 
 void ShortcutViewerApplication::OnStart() {
-  views::AuraInit::InitParams params;
-  params.connector = context()->connector();
-  params.identity = context()->identity();
-  params.mode = views::AuraInit::Mode::AURA_MUS2;
-  params.register_path_provider = false;
-  aura_init_ = views::AuraInit::Create(params);
+  aura_init_ = views::AuraInit::Create(
+      context()->connector(), context()->identity(), "views_mus_resources.pak",
+      std::string(), nullptr, views::AuraInit::Mode::AURA_MUS2,
+      false /*register_path_provider*/);
   if (!aura_init_) {
     context()->QuitNow();
     return;

ash/components/tap_visualizer/tap_visualizer_app.cc

@@ -45,12 +45,11 @@ void TapVisualizerApp::Start() {
 }
 
 void TapVisualizerApp::OnStart() {
-  views::AuraInit::InitParams params;
-  params.connector = context()->connector();
-  params.identity = context()->identity();
-  params.mode = views::AuraInit::Mode::AURA_MUS2;
-  params.register_path_provider = false;
-  aura_init_ = views::AuraInit::Create(params);
+  const bool register_path_provider = false;
+  aura_init_ = views::AuraInit::Create(
+      context()->connector(), context()->identity(), "views_mus_resources.pak",
+      std::string(), nullptr, views::AuraInit::Mode::AURA_MUS2,
+      register_path_provider);
   if (!aura_init_) {
     context()->QuitNow();
     return;

ash/window_manager_service.cc

@@ -179,14 +179,11 @@ void WindowManagerService::OnStart() {
   registry_.AddInterface(base::BindRepeating(
       &WindowManagerService::BindServiceFactory, base::Unretained(this)));
 
-  views::AuraInit::InitParams params;
-  params.connector = context()->connector();
-  params.identity = context()->identity();
-  params.resource_file = "ash_service_resources.pak";
-  params.resource_file_200 = "ash_service_resources_200.pak";
-  params.mode = views::AuraInit::Mode::AURA_MUS_WINDOW_MANAGER;
-  params.register_path_provider = running_standalone_;
-  aura_init_ = views::AuraInit::Create(params);
+  const bool register_path_provider = running_standalone_;
+  aura_init_ = views::AuraInit::Create(
+      context()->connector(), context()->identity(),
+      "ash_service_resources.pak", "ash_service_resources_200.pak", nullptr,
+      views::AuraInit::Mode::AURA_MUS_WINDOW_MANAGER, register_path_provider);
   if (!aura_init_) {
     context()->QuitNow();
     return;

mash/catalog_viewer/catalog_viewer.cc

@@ -222,11 +222,9 @@ void CatalogViewer::RemoveWindow(views::Widget* window) {
 }
 
 void CatalogViewer::OnStart() {
-  views::AuraInit::InitParams params;
-  params.connector = context()->connector();
-  params.identity = context()->identity();
-  params.mode = views::AuraInit::Mode::AURA_MUS;
-  aura_init_ = views::AuraInit::Create(params);
+  aura_init_ = views::AuraInit::Create(
+      context()->connector(), context()->identity(), "views_mus_resources.pak",
+      std::string(), nullptr, views::AuraInit::Mode::AURA_MUS);
   if (!aura_init_)
     context()->QuitNow();
 }

mash/example/views_examples/views_examples.cc

@@ -31,11 +31,10 @@ class ViewsExamples : public service_manager::Service,
  private:
   // service_manager::Service:
   void OnStart() override {
-    views::AuraInit::InitParams params;
-    params.connector = context()->connector();
-    params.identity = context()->identity();
-    params.mode = views::AuraInit::Mode::AURA_MUS;
-    aura_init_ = views::AuraInit::Create(params);
+    aura_init_ =
+        views::AuraInit::Create(context()->connector(), context()->identity(),
+                                "views_mus_resources.pak", std::string(),
+                                nullptr, views::AuraInit::Mode::AURA_MUS);
     if (!aura_init_)
       context()->QuitNow();
   }

mash/example/window_type_launcher/window_type_launcher.cc

@@ -453,11 +453,9 @@ void WindowTypeLauncher::RemoveWindow(views::Widget* window) {
 }
 
 void WindowTypeLauncher::OnStart() {
-  views::AuraInit::InitParams params;
-  params.connector = context()->connector();
-  params.identity = context()->identity();
-  params.mode = views::AuraInit::Mode::AURA_MUS;
-  aura_init_ = views::AuraInit::Create(params);
+  aura_init_ = views::AuraInit::Create(
+      context()->connector(), context()->identity(), "views_mus_resources.pak",
+      std::string(), nullptr, views::AuraInit::Mode::AURA_MUS);
   if (!aura_init_)
     context()->QuitNow();
 }

mash/simple_wm/simple_wm.cc

@@ -366,11 +366,9 @@ void SimpleWM::OnStart() {
   started_ = true;
   screen_ = std::make_unique<display::ScreenBase>();
   display::Screen::SetScreenInstance(screen_.get());
-  views::AuraInit::InitParams params;
-  params.connector = context()->connector();
-  params.identity = context()->identity();
-  params.mode = views::AuraInit::Mode::AURA_MUS_WINDOW_MANAGER;
-  aura_init_ = views::AuraInit::Create(params);
+  aura_init_ = views::AuraInit::Create(
+      context()->connector(), context()->identity(), "views_mus_resources.pak",
+      std::string(), nullptr, views::AuraInit::Mode::AURA_MUS_WINDOW_MANAGER);
   if (!aura_init_) {
     context()->QuitNow();
     return;

mash/task_viewer/task_viewer.cc

@@ -297,11 +297,9 @@ void TaskViewer::RemoveWindow(views::Widget* widget) {
 }
 
 void TaskViewer::OnStart() {
-  views::AuraInit::InitParams params;
-  params.connector = context()->connector();
-  params.identity = context()->identity();
-  params.mode = views::AuraInit::Mode::AURA_MUS;
-  aura_init_ = views::AuraInit::Create(params);
+  aura_init_ = views::AuraInit::Create(
+      context()->connector(), context()->identity(), "views_mus_resources.pak",
+      std::string(), nullptr, views::AuraInit::Mode::AURA_MUS);
   if (!aura_init_)
     context()->QuitNow();
 }

ui/views/mus/aura_init.cc

@@ -46,10 +46,6 @@ class MusViewsDelegate : public ViewsDelegate {
 
 }  // namespace
 
-AuraInit::InitParams::InitParams() : resource_file("views_mus_resources.pak") {}
-
-AuraInit::InitParams::~InitParams() = default;
-
 AuraInit::AuraInit() {
   if (!ViewsDelegate::GetInstance())
     views_delegate_ = std::make_unique<MusViewsDelegate>();
@@ -57,53 +53,71 @@ AuraInit::AuraInit() {
 
 AuraInit::~AuraInit() = default;
 
-// static
-std::unique_ptr<AuraInit> AuraInit::Create(const InitParams& params) {
-  // Using 'new' to access a non-public constructor. go/totw/134
+std::unique_ptr<AuraInit> AuraInit::Create(
+    service_manager::Connector* connector,
+    const service_manager::Identity& identity,
+    const std::string& resource_file,
+    const std::string& resource_file_200,
+    scoped_refptr<base::SingleThreadTaskRunner> io_task_runner,
+    Mode mode,
+    bool register_path_provider) {
   std::unique_ptr<AuraInit> aura_init = base::WrapUnique(new AuraInit());
-  if (!aura_init->Init(params))
+  if (!aura_init->Init(connector, identity, resource_file, resource_file_200,
+                       io_task_runner, mode, register_path_provider)) {
     aura_init.reset();
+  }
   return aura_init;
 }
 
-bool AuraInit::Init(const InitParams& params) {
+bool AuraInit::Init(service_manager::Connector* connector,
+                    const service_manager::Identity& identity,
+                    const std::string& resource_file,
+                    const std::string& resource_file_200,
+                    scoped_refptr<base::SingleThreadTaskRunner> io_task_runner,
+                    Mode mode,
+                    bool register_path_provider) {
   env_ = aura::Env::CreateInstance(aura::Env::Mode::MUS);
 
-  if (params.mode == Mode::AURA_MUS || params.mode == Mode::AURA_MUS2) {
-    MusClient::InitParams mus_params;
-    mus_params.connector = params.connector;
-    mus_params.identity = params.identity;
-    mus_params.io_task_runner = params.io_task_runner;
-    mus_params.wtc_config = params.mode == Mode::AURA_MUS2
-                                ? aura::WindowTreeClient::Config::kMus2
-                                : aura::WindowTreeClient::Config::kMash;
-    mus_params.create_wm_state = true;
-    mus_client_ = std::make_unique<MusClient>(mus_params);
+  if (mode == Mode::AURA_MUS || mode == Mode::AURA_MUS2) {
+    MusClient::InitParams params;
+    params.connector = connector;
+    params.identity = identity;
+    params.io_task_runner = io_task_runner;
+    params.wtc_config = mode == Mode::AURA_MUS2
+                            ? aura::WindowTreeClient::Config::kMus2
+                            : aura::WindowTreeClient::Config::kMash;
+    params.create_wm_state = true;
+    mus_client_ = std::make_unique<MusClient>(params);
   }
   // MaterialDesignController may have initialized already (such as happens
   // in the utility process).
   if (!ui::MaterialDesignController::is_mode_initialized())
     ui::MaterialDesignController::Initialize();
-  if (!InitializeResources(params))
+  if (!InitializeResources(connector, resource_file, resource_file_200,
+                           register_path_provider)) {
     return false;
+  }
 
   ui::InitializeInputMethodForTesting();
   return true;
 }
 
-bool AuraInit::InitializeResources(const InitParams& params) {
+bool AuraInit::InitializeResources(service_manager::Connector* connector,
+                                   const std::string& resource_file,
+                                   const std::string& resource_file_200,
+                                   bool register_path_provider) {
   // Resources may have already been initialized (e.g. when chrome with mash is
   // used to launch the current app).
   if (ui::ResourceBundle::HasSharedInstance())
     return true;
 
-  std::set<std::string> resource_paths({params.resource_file});
-  if (!params.resource_file_200.empty())
-    resource_paths.insert(params.resource_file_200);
+  std::set<std::string> resource_paths({resource_file});
+  if (!resource_file_200.empty())
+    resource_paths.insert(resource_file_200);
 
   catalog::ResourceLoader loader;
   filesystem::mojom::DirectoryPtr directory;
-  params.connector->BindInterface(catalog::mojom::kServiceName, &directory);
+  connector->BindInterface(catalog::mojom::kServiceName, &directory);
   // TODO(jonross): if this proves useful in resolving the crash of
   // mash_unittests then switch AuraInit to have an Init method, returning a
   // bool for success. Then update all callsites to use this to determine the
@@ -113,17 +127,17 @@ bool AuraInit::InitializeResources(const InitParams& params) {
   // Calling services will shutdown ServiceContext as appropriate.
   if (!loader.OpenFiles(std::move(directory), resource_paths))
     return false;
-  if (params.register_path_provider)
+  if (register_path_provider)
     ui::RegisterPathProvider();
-  base::File pak_file = loader.TakeFile(params.resource_file);
+  base::File pak_file = loader.TakeFile(resource_file);
   base::File pak_file_2 = pak_file.Duplicate();
   ui::ResourceBundle::InitSharedInstanceWithPakFileRegion(
       std::move(pak_file), base::MemoryMappedFile::Region::kWholeFile);
   ui::ResourceBundle::GetSharedInstance().AddDataPackFromFile(
       std::move(pak_file_2), ui::SCALE_FACTOR_100P);
-  if (!params.resource_file_200.empty())
+  if (!resource_file_200.empty())
     ui::ResourceBundle::GetSharedInstance().AddDataPackFromFile(
-        loader.TakeFile(params.resource_file_200), ui::SCALE_FACTOR_200P);
+        loader.TakeFile(resource_file_200), ui::SCALE_FACTOR_200P);
   return true;
 }
 

ui/views/mus/aura_init.h

@@ -9,7 +9,6 @@
 #include <string>
 
 #include "base/macros.h"
-#include "services/service_manager/public/cpp/identity.h"
 #include "ui/aura/env.h"
 #include "ui/views/mus/mus_export.h"
 
@@ -23,6 +22,7 @@ class SingleThreadTaskRunner;
 
 namespace service_manager {
 class Connector;
+class Identity;
 }
 
 namespace views {
@@ -49,24 +49,20 @@ class VIEWS_MUS_EXPORT AuraInit {
 
   ~AuraInit();
 
-  struct InitParams {
-    InitParams();
-    ~InitParams();
-    service_manager::Connector* connector = nullptr;
-    service_manager::Identity identity;
-    // File for strings and 1x icons. Defaults to views_mus_resources.pak.
-    std::string resource_file;
-    // File for 2x icons. Can be empty.
-    std::string resource_file_200;
-    scoped_refptr<base::SingleThreadTaskRunner> io_task_runner = nullptr;
-    Mode mode = Mode::AURA_MUS;
-    bool register_path_provider = true;
-  };
-
   // Returns an AuraInit if initialization can be completed successfully,
   // otherwise a nullptr is returned. If initialization fails then Aura is in an
   // unusable state, and calling services should shutdown.
-  static std::unique_ptr<AuraInit> Create(const InitParams& params);
+  // |resource_file| is the file to load strings and 1x icons from.
+  // |resource_file_200| can be an empty string, otherwise it is the file to
+  // load 2x icons from.
+  static std::unique_ptr<AuraInit> Create(
+      service_manager::Connector* connector,
+      const service_manager::Identity& identity,
+      const std::string& resource_file,
+      const std::string& resource_file_200 = std::string(),
+      scoped_refptr<base::SingleThreadTaskRunner> io_task_runner = nullptr,
+      Mode mode = Mode::AURA_MUS,
+      bool register_path_provider = true);
 
   // Only valid if Mode::AURA_MUS was passed to constructor.
   MusClient* mus_client() { return mus_client_.get(); }
@@ -77,10 +73,18 @@ class VIEWS_MUS_EXPORT AuraInit {
   // Returns true if AuraInit was able to successfully complete initialization.
   // If this returns false, then Aura is in an unusable state, and calling
   // services should shutdown.
-  bool Init(const InitParams& params);
-
-  // Returns true on success.
-  bool InitializeResources(const InitParams& params);
+  bool Init(service_manager::Connector* connector,
+            const service_manager::Identity& identity,
+            const std::string& resource_file,
+            const std::string& resource_file_200,
+            scoped_refptr<base::SingleThreadTaskRunner> io_task_runner,
+            Mode mode,
+            bool register_path_provider);
+
+  bool InitializeResources(service_manager::Connector* connector,
+                           const std::string& resource_file,
+                           const std::string& resource_file_200,
+                           bool register_path_provider);
 
   std::unique_ptr<aura::Env> env_;
   std::unique_ptr<MusClient> mus_client_;