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

ui/views/mus/mus_client.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 "ui/views/mus/mus_client.h"
 
 #include "base/bind.h"
 #include "base/memory/ptr_util.h"
 #include "base/threading/thread.h"
 #include "components/discardable_memory/client/client_discardable_shared_memory_manager.h"
@@ skipping 133 matching lines @@
     const Widget::InitParams& init_params) {
   using PrimitiveType = aura::PropertyConverter::PrimitiveType;
   std::map<std::string, std::vector<uint8_t>> properties =
       init_params.mus_properties;
 
   // Widget::InitParams::Type matches ui::mojom::WindowType.
   properties[ui::mojom::WindowManager::kWindowType_InitProperty] =
       mojo::ConvertTo<std::vector<uint8_t>>(
           static_cast<int32_t>(init_params.type));
 
+  properties[ui::mojom::WindowManager::kActivatable_InitProperty] =
+      mojo::ConvertTo<std::vector<uint8_t>>(init_params.CanActivate());

[sadrul, 2017/01/24 17:34:49]

Instead of introducing a property for this, can mus-client just use the
WindowTree::SetCanFocus() API?

[Hadi, 2017/01/24 18:42:33]

WindowTree::SetCanFocus() needs window_id, but I think we don't have it here
yet. Am I missing something?

[sadrul, 2017/01/25 05:31:40]

The window-id is set pretty early on in the client-side during initialization
[1]. If we notify the server SetCanFocus() right after calling
NewTopLevelWindow() [2], would that work?

1:
https://cs.chromium.org/chromium/src/ui/aura/mus/window_tree_client.cc?type=c...

2:
https://cs.chromium.org/chromium/src/ui/aura/mus/window_tree_client.cc?type=c...

+
   if (!init_params.bounds.IsEmpty()) {
     properties[ui::mojom::WindowManager::kBounds_InitProperty] =
         mojo::ConvertTo<std::vector<uint8_t>>(init_params.bounds);
   }
 
   if (!init_params.name.empty()) {
     properties[ui::mojom::WindowManager::kName_Property] =
         mojo::ConvertTo<std::vector<uint8_t>>(init_params.name);
   }
 
@@ skipping 105 matching lines @@
       return root->GetTopWindowContainingPoint(relative_point);
   }
   return nullptr;
 }
 
 std::unique_ptr<OSExchangeData::Provider> MusClient::BuildProvider() {
   return base::MakeUnique<aura::OSExchangeDataProviderMus>();
 }
 
 }  // namespace views