mus: Changes SetDisplayRoot() to create actual display

services/ui/ws/window_manager_window_tree_factory.cc

 // Copyright 2016 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.
 
 #include "services/ui/ws/window_manager_window_tree_factory.h"
 
 #include "base/bind.h"
 #include "services/ui/ws/window_manager_window_tree_factory_set.h"
 #include "services/ui/ws/window_server.h"
+#include "services/ui/ws/window_server_delegate.h"
 #include "services/ui/ws/window_tree.h"
 
 namespace ui {
 namespace ws {
 
 WindowManagerWindowTreeFactory::WindowManagerWindowTreeFactory(
     WindowManagerWindowTreeFactorySet* window_manager_window_tree_factory_set,
     const UserId& user_id,
     mojo::InterfaceRequest<mojom::WindowManagerWindowTreeFactory> request)
     : WindowManagerWindowTreeFactory(window_manager_window_tree_factory_set,
@@ skipping 11 matching lines @@
   if (window_tree_) {
     DVLOG(1) << "CreateWindowTree() called more than once.";
     return;
   }
 
   // CreateWindowTree() can only be called once, so there is no reason to keep
   // the binding around.
   if (binding_.is_bound())
     binding_.Close();
 
+  window_manager_window_tree_factory_set_->window_server()
+      ->delegate()
+      ->OnWillCreateTreeForWindowManager(automatically_create_display_roots);

[kylechar, 2017/04/19 18:06:08]

Will this work with multiple connected WMs? I know we don't actually do that,
but I think some of the tests might pretend there are multiple WMs.

[sky, 2017/04/19 19:30:03]

You are correct it would be problematic to have multiple wms connect with
different values. Service::OnWillCreateTreeForWindowManager() ignores the second
call. Will have to think what the right long term solution will be for multiple
wms. One easy solution is do away the boolean and only support the new config.

[kylechar, 2017/04/19 19:47:09]

Ah, right it ignores it. Gotcha.

+
   SetWindowTree(GetWindowServer()->CreateTreeForWindowManager(
       user_id_, std::move(window_tree_request), std::move(window_tree_client),
       automatically_create_display_roots));
 }
 
 WindowManagerWindowTreeFactory::WindowManagerWindowTreeFactory(
     WindowManagerWindowTreeFactorySet* window_manager_window_tree_factory_set,
     const UserId& user_id)
     : window_manager_window_tree_factory_set_(
           window_manager_window_tree_factory_set),
       user_id_(user_id),
       binding_(this) {}
 
 WindowServer* WindowManagerWindowTreeFactory::GetWindowServer() {
   return window_manager_window_tree_factory_set_->window_server();
 }
 
 void WindowManagerWindowTreeFactory::SetWindowTree(WindowTree* window_tree) {
   DCHECK(!window_tree_);
   window_tree_ = window_tree;
 
   window_manager_window_tree_factory_set_
       ->OnWindowManagerWindowTreeFactoryReady(this);
 }
 
 }  // namespace ws
 }  // namespace ui