mus: Disregard windows that explicitly set can_accept_events to be false when sending events.

services/ui/public/cpp/window.h

 // 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.
 
 #ifndef SERVICES_UI_PUBLIC_CPP_WINDOW_H_
 #define SERVICES_UI_PUBLIC_CPP_WINDOW_H_
 
 #include <stdint.h>
 #include <vector>
 
@@ skipping 191 matching lines @@
 
   bool HasCapture() const;
   void SetCapture();
   void ReleaseCapture();
 
   // Focus. See WindowTreeClient::ClearFocus() to reset focus.
   void SetFocus();
   bool HasFocus() const;
   void SetCanFocus(bool can_focus);
 
+  // Sets whether this window accepts events.
+  void SetCanAcceptEvents(bool can_accept_events);
+
   // Embedding. See window_tree.mojom for details.
   void Embed(ui::mojom::WindowTreeClientPtr client, uint32_t flags = 0);
 
   // NOTE: callback is run synchronously if Embed() is not allowed on this
   // Window.
   void Embed(ui::mojom::WindowTreeClientPtr client,
              const EmbedCallback& callback,
              uint32_t flags = 0);
 
   // TODO(sky): this API is only applicable to the WindowManager. Move it
@@ skipping 110 matching lines @@
 
   gfx::Rect bounds_;
   gfx::Insets client_area_;
   std::vector<gfx::Rect> additional_client_areas_;
   std::unique_ptr<gfx::Rect> hit_test_mask_;
 
   bool visible_;
   float opacity_;
   int64_t display_id_;
 
+  // Whether this window can accept events.
+  // Only call SetCanAcceptEvents if the new setting is different from

[msw, 2016/07/12 22:07:23]

nit: move this sentence up to |SetCanAcceptEvents|.

[riajiang, 2016/07/13 00:01:45]

Done.

+  // the current one. Initialize it to be true since server window will

[msw, 2016/07/12 22:07:23]

nit: "Initialized to true to match ServerWindow".

[riajiang, 2016/07/13 00:01:45]

Done.

+  // also initialize can_accept_events_ to be true.

[msw, 2016/07/12 22:07:23]

nit: |can_accept_events_| (if you keep this)

+  bool can_accept_events_ = true;
+
   mojom::Cursor cursor_id_;
 
   SharedProperties properties_;
 
   // Drawn state of our parent. This is only meaningful for root Windows, in
   // which the parent Window isn't exposed to the client.
   bool parent_drawn_;
 
   // Value struct to keep the name and deallocator for this property.
   // Key cannot be used for this purpose because it can be char* or
   // WindowProperty<>.
   struct Value {
     const char* name;
     int64_t value;
     PropertyDeallocator deallocator;
   };
 
   std::map<const void*, Value> prop_map_;
 
   DISALLOW_COPY_AND_ASSIGN(Window);
 };
 
 }  // namespace ui
 
 #endif  // SERVICES_UI_PUBLIC_CPP_WINDOW_H_