Add WindowPinType property on arua::Window

chrome/browser/chrome_browser_main_extra_parts_exo.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 "chrome/browser/chrome_browser_main_extra_parts_exo.h"
 
 #include "base/memory/ptr_util.h"
 
 #if defined(USE_GLIB)
 #include <glib.h>
 #endif
 
+#include "ash/public/cpp/window_properties.h"
+#include "ash/public/interfaces/window_pin_type.mojom.h"
 #include "base/command_line.h"
 #include "base/macros.h"
 #include "base/message_loop/message_loop.h"
 #include "chrome/browser/ui/ash/ash_util.h"
 #include "chrome/common/chrome_switches.h"
 #include "components/exo/display.h"
 #include "components/exo/wayland/server.h"
 #include "components/exo/wm_helper_ash.h"
 #include "components/exo/wm_helper_mus.h"
 #include "content/public/browser/browser_thread.h"
 #include "ui/arc/notification/arc_notification_surface_manager.h"
+#include "ui/aura/mus/property_converter.h"
+#include "ui/views/mus/mus_client.h"
 
 #if defined(USE_GLIB)
 namespace {
 
 struct GLibWaylandSource : public GSource {
   // Note: The GLibWaylandSource is created and destroyed by GLib. So its
   // constructor/destructor may or may not get called.
   exo::wayland::Server* server;
   GPollFD* poll_fd;
 };
@@ skipping 76 matching lines @@
   exo::wayland::Server* const server_;
 
   DISALLOW_COPY_AND_ASSIGN(WaylandWatcher);
 };
 #endif
 
 ChromeBrowserMainExtraPartsExo::ChromeBrowserMainExtraPartsExo() {}
 
 ChromeBrowserMainExtraPartsExo::~ChromeBrowserMainExtraPartsExo() {}
 
+void ChromeBrowserMainExtraPartsExo::ServiceManagerConnectionStarted(

[sky, 2017/04/03 16:38:56]

This property is really ash specific. I think you should put this code in
ChromeBrowserMainExtraPartsAsh.

[Peng, 2017/04/03 18:03:48]

Done.

+    content::ServiceManagerConnection* connection) {
+  if (ash_util::IsRunningInMash()) {
+    // Register exo-specific window properties with Chrome's property converter.
+    // This propagates ash properties set on exo windows to ash, via mojo.
+    DCHECK(views::MusClient::Exists());
+    views::MusClient* mus_client = views::MusClient::Get();
+    aura::WindowTreeClientDelegate* delegate = mus_client;
+    aura::PropertyConverter* converter = delegate->GetPropertyConverter();
+
+    converter->RegisterProperty(ash::kWindowPinTypeKey,
+                                ash::mojom::kWindowPinType_Property,
+                                base::Bind(&ash::IsValidWindowPinType));
+  }
+}
+
 void ChromeBrowserMainExtraPartsExo::PreProfileInit() {
   if (!base::CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kEnableWaylandServer))
     return;
 
   arc_notification_surface_manager_ =
       base::MakeUnique<arc::ArcNotificationSurfaceManager>();
   if (ash_util::IsRunningInMash())
     wm_helper_ = base::MakeUnique<exo::WMHelperMus>();
   else
     wm_helper_ = base::MakeUnique<exo::WMHelperAsh>();
   exo::WMHelper::SetInstance(wm_helper_.get());
   display_ =
       base::MakeUnique<exo::Display>(arc_notification_surface_manager_.get());
   wayland_server_ = exo::wayland::Server::Create(display_.get());
   // Wayland server creation can fail if XDG_RUNTIME_DIR is not set correctly.
   if (wayland_server_)
     wayland_watcher_ = base::MakeUnique<WaylandWatcher>(wayland_server_.get());
 }
 
 void ChromeBrowserMainExtraPartsExo::PostMainMessageLoopRun() {
   wayland_watcher_.reset();
   wayland_server_.reset();
   if (wm_helper_) {
     exo::WMHelper::SetInstance(nullptr);
     wm_helper_.reset();
   }
 }