Set focusibility correctly when initializing a window in mus+ash.

ash/mus/top_level_window_factory.cc

 // Copyright 2017 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 "ash/mus/top_level_window_factory.h"
 
 #include "ash/common/wm/container_finder.h"
 #include "ash/common/wm/window_state.h"
 #include "ash/common/wm_shell.h"
 #include "ash/common/wm_window.h"
 #include "ash/mus/disconnected_app_handler.h"
 #include "ash/mus/non_client_frame_controller.h"
 #include "ash/mus/property_util.h"
 #include "ash/mus/window_manager.h"
 #include "ash/public/cpp/shell_window_ids.h"
 #include "ash/root_window_controller.h"
 #include "ash/root_window_settings.h"
 #include "mojo/public/cpp/bindings/type_converter.h"
 #include "services/ui/public/cpp/property_type_converters.h"
 #include "services/ui/public/interfaces/window_manager.mojom.h"
 #include "services/ui/public/interfaces/window_manager_constants.mojom.h"
 #include "ui/aura/mus/property_converter.h"
 #include "ui/aura/mus/property_utils.h"
+#include "ui/aura/mus/window_tree_client.h"
 #include "ui/aura/window.h"
 #include "ui/display/display.h"
 
 namespace ash {
 namespace mus {
 namespace {
 
 // Returns true if a fullscreen window was requested.
 bool IsFullscreen(aura::PropertyConverter* property_converter,
                   const std::vector<uint8_t>& transport_data) {
@@ skipping 142 matching lines @@
       GetRootWindowControllerForNewTopLevelWindow(properties);
   aura::Window* window = CreateAndParentTopLevelWindowInRoot(
       window_manager, root_window_controller, window_type, properties);
   DisconnectedAppHandler::Create(window);
   if (properties->count(
           ui::mojom::WindowManager::kWindowIgnoredByShelf_Property)) {
     wm::WindowState* window_state = WmWindow::Get(window)->GetWindowState();
     window_state->set_ignored_by_shelf(mojo::ConvertTo<bool>(
         (*properties)
             [ui::mojom::WindowManager::kWindowIgnoredByShelf_Property]));
     // No need to persist this value.

[sky, 2017/01/26 18:59:04]

Any properties left in properties are mirrored to the window-server. As
focusable is only useful at construction time you can remove it from properties
similar to what happens on this line.

[Hadi, 2017/01/26 20:23:17]

Done.

     properties->erase(ui::mojom::WindowManager::kWindowIgnoredByShelf_Property);
   }
+  if (properties->count(ui::mojom::WindowManager::kFocusable_InitProperty)) {
+    window_manager->window_tree_client()->SetCanFocus(
+        window,
+        mojo::ConvertTo<bool>(
+            (*properties)[ui::mojom::WindowManager::kFocusable_InitProperty]));
+  }
   return window;
 }
 
 }  // namespace mus
 }  // namespace ash