mus: Implement interwindow drag and drop

services/ui/ws/default_access_policy.cc

 // Copyright 2014 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 "services/ui/ws/default_access_policy.h"
 
 #include "services/ui/ws/access_policy_delegate.h"
 #include "services/ui/ws/server_window.h"
 
 namespace ui {
@@ skipping 131 matching lines @@
 bool DefaultAccessPolicy::CanSetClientArea(const ServerWindow* window) const {
   return WasCreatedByThisClient(window) ||
          delegate_->HasRootForAccessPolicy(window);
 }
 
 bool DefaultAccessPolicy::CanSetHitTestMask(const ServerWindow* window) const {
   return WasCreatedByThisClient(window) ||
          delegate_->HasRootForAccessPolicy(window);
 }
 
+bool DefaultAccessPolicy::CanSetAcceptDrops(const ServerWindow* window) const {
+  return delegate_->HasRootForAccessPolicy(window);

[sky, 2016/09/08 23:37:20]

This only accepts drops on the root. You also want
WasCreatedByThisClient(window) here, otherwise windows created by this client
can't accept drops. I think you'll also want to forbid the drop if there is
another client embedded in the window. In other words I think you want the same
thing as is in DefaultAccessPolicy::CanDescendIntoWindowForWindowTree(). I think
you want the same in the WindowManagerAccessPolicy.

+}
+
 bool DefaultAccessPolicy::CanSetAcceptEvents(const ServerWindow* window) const {
   return WasCreatedByThisClient(window) ||
          delegate_->HasRootForAccessPolicy(window);
 }
 
 bool DefaultAccessPolicy::CanSetCursorProperties(
     const ServerWindow* window) const {
   return WasCreatedByThisClient(window) ||
          delegate_->HasRootForAccessPolicy(window);
 }
@@ skipping 42 matching lines @@
 bool DefaultAccessPolicy::IsValidIdForNewWindow(
     const ClientWindowId& id) const {
   // Clients using DefaultAccessPolicy only see windows they have created (for
   // the embed point they choose the id), so it's ok for clients to use whatever
   // id they want.
   return true;
 }
 
 }  // namespace ws
 }  // namespace ui