Allow parallel creation of windows

services/ui/demo/mus_demo.h

 // 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.
 
 #ifndef SERVICES_UI_DEMO_MUS_DEMO_H_
 #define SERVICES_UI_DEMO_MUS_DEMO_H_
 
 #include <memory>
 
 #include "base/callback.h"
@@ skipping 28 matching lines @@
 class MusDemo : public service_manager::Service,
                 public aura::WindowTreeClientDelegate {
  public:
   MusDemo();
   ~MusDemo() override;
 
  protected:
   void AddPrimaryDisplay(const display::Display& display);
 
   // These functions help to manage the list of WindowTreeData structures.
-  // AppendWindowTreeData is used to add an uninitialized structure at the end
+  // AppendWindowTreeData is used to add uninitialized structures at the end
   // of the list. When a new WindowTreeHostMus is created and is sent to
-  // MusDemo (via OnWmNewDisplay or OnEmbed), the WindowTreeData is initialized
-  // by a call to InitWindowTreeData and the demo starts. When the destruction
-  // of the WindowTreeHostMus is announced to MusDemo (via OnWmDisplayRemoved
-  // or OnEmbedRootDestroyed), the corresponding WindowTreeData is removed by
-  // a call to RemoveWindowTreeData.
+  // MusDemo via OnWmNewDisplay, the WindowTreeData is initialized
+  // by a call to InitWindowTreeData and the demo starts. In case if
+  // OnEmbedRootReady is called, corresponding WindowTreeData is found by
+  // looking associated WindowTreeHostMus with it. When the destruction of the
+  // WindowTreeHostMus is announced to MusDemo (via OnWmDisplayRemoved or
+  // OnEmbedRootDestroyed), the corresponding WindowTreeData is removed by a
+  // call to RemoveWindowTreeData.
   void AppendWindowTreeData(std::unique_ptr<WindowTreeData> window_tree_data);
   void InitWindowTreeData(
       std::unique_ptr<aura::WindowTreeHostMus> window_tree_host);

[fwang, 2017/03/16 10:54:55]

Shoudn't this be removed?

[msisov(use msisov igalia.com), 2017/03/16 11:06:33]

Done.

   void RemoveWindowTreeData(aura::WindowTreeHostMus* window_tree_host);
+  std::vector<std::unique_ptr<WindowTreeData>>::iterator FindWindowTreeData(
+      aura::WindowTreeHostMus* window_tree_host);
 
   aura::WindowTreeClient* window_tree_client() {
     return window_tree_client_.get();
   }
 
  private:
   virtual void OnStartImpl() = 0;
   virtual std::unique_ptr<aura::WindowTreeClient> CreateWindowTreeClient() = 0;
-  bool HasPendingWindowTreeData() const;
 
   // service_manager::Service:
   void OnStart() override;
   bool OnConnect(const service_manager::ServiceInfo& remote_info,
                  service_manager::InterfaceRegistry* registry) override;
 
   // aura::WindowTreeClientDelegate:
   void OnEmbed(
       std::unique_ptr<aura::WindowTreeHostMus> window_tree_host) override;
   void OnUnembed(aura::Window* root) override;
@@ skipping 16 matching lines @@
   // next call to InitWindowTreeData.
   std::vector<std::unique_ptr<WindowTreeData>> window_tree_data_list_;
 
   DISALLOW_COPY_AND_ASSIGN(MusDemo);
 };
 
 }  // namespace demo
 }  // namespace ui
 
 #endif  // SERVICES_UI_DEMO_MUS_DEMO_H_