Avoid blocking while mapping an X11 window

ui/views/widget/desktop_aura/desktop_window_tree_host_x11.h

 // Copyright (c) 2012 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 UI_VIEWS_WIDGET_DESKTOP_AURA_DESKTOP_WINDOW_TREE_HOST_X11_H_
 #define UI_VIEWS_WIDGET_DESKTOP_AURA_DESKTOP_WINDOW_TREE_HOST_X11_H_
 
 #include <stddef.h>
 #include <stdint.h>
 #include <X11/extensions/shape.h>
@@ skipping 276 matching lines @@
   ::Window xwindow_;
 
   // Events selected on |xwindow_|.
   std::unique_ptr<ui::XScopedEventSelector> xwindow_events_;
 
   // The native root window.
   ::Window x_root_window_;
 
   ui::X11AtomCache atom_cache_;
 
-  // Is the window mapped to the screen?
+  // Whether the window is mapped with respect to the X server.
   bool window_mapped_;
 
-  // Should we wait for an UnmapNotify before trying to remap the window?
-  // If |wait_for_unmap_| is true, we have sent an XUnmapWindow request to the
-  // server and have yet to receive an UnmapNotify.
-  bool wait_for_unmap_;
+  // Whether the window is visible with respect to Aura.
+  bool is_visible_;

[danakj, 2017/03/09 20:45:34]

I would name these more literally as the state they represent.

window_mapped_in_client_
window_mapped_in_server_

or something like that.

[Tom (Use chromium acct), 2017/03/10 01:29:57]

Done.

 
   // The bounds of |xwindow_|.
   gfx::Rect bounds_in_pixels_;
 
   // Whenever the bounds are set, we keep the previous set of bounds around so
   // we can have a better chance of getting the real
   // |restored_bounds_in_pixels_|. Window managers tend to send a Configure
   // message with the maximized bounds, and then set the window maximized
   // property. (We don't rely on this for when we request that the window be
   // maximized, only when we detect that some other process has requested that
@@ skipping 119 matching lines @@
 
   base::WeakPtrFactory<DesktopWindowTreeHostX11> close_widget_factory_;
   base::WeakPtrFactory<DesktopWindowTreeHostX11> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(DesktopWindowTreeHostX11);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_WIDGET_DESKTOP_AURA_DESKTOP_WINDOW_TREE_HOST_X11_H_