Makes NewWindow() take set of properties for window

mash/wm/window_manager_impl.cc

 // Copyright 2015 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 "mash/wm/window_manager_impl.h"
 
 #include "components/mus/common/types.h"
 #include "components/mus/public/cpp/property_type_converters.h"
 #include "components/mus/public/cpp/window.h"
 #include "components/mus/public/cpp/window_property.h"
@@ skipping 46 matching lines @@
   return gfx::Rect(40 + (state_->window_count() % 4) * 40,
                    40 + (state_->window_count() % 4) * 40, width, height);
 }
 
 gfx::Rect WindowManagerImpl::GetMaximizedWindowBounds() const {
   return gfx::Rect(state_->root()->bounds().size());
 }
 
 void WindowManagerImpl::OpenWindow(
     mus::mojom::WindowTreeClientPtr client,
-    mojo::Map<mojo::String, mojo::Array<uint8_t>> properties) {
+    mojo::Map<mojo::String, mojo::Array<uint8_t>> transport_properties) {
   mus::Window* root = state_->root();
   DCHECK(root);
 
-  mus::Window* child_window = root->connection()->NewWindow();
-  // TODO(beng): mus::Window should have a "SetSharedProperties" method that
-  //             joins the supplied map onto the internal one.
-  for (auto prop : properties)
-    child_window->SetSharedProperty(prop.GetKey(), prop.GetValue());
-
+  mus::Window::SharedProperties properties =
+      transport_properties.To<mus::Window::SharedProperties>();
+  // TODO(sky): constrain to valid properties here.
+  mus::Window* child_window = root->connection()->NewWindow(&properties);
   child_window->SetBounds(CalculateDefaultBounds(child_window));
   GetContainerForChild(child_window)->AddChild(child_window);
   child_window->Embed(client.Pass());
 
   // NonClientFrameController deletes itself when the window is destroyed.
   new NonClientFrameController(state_->app()->shell(), child_window);
 
   state_->IncrementWindowCount();
 }
 
@@ skipping 20 matching lines @@
 
   config->maximized_client_area_insets = mojo::Insets::From(client_area_insets);
 
   callback.Run(config.Pass());
 }
 
 mus::Window* WindowManagerImpl::GetContainerForChild(mus::Window* child) {
   mash::wm::mojom::Container container = GetRequestedContainer(child);
   return state_->GetWindowForContainer(container);
 }