Fix Restore after Minimize on X11.

ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc

Index: ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc
diff --git a/ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc b/ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc
index fa50d9ea26d0b9156135672c19be67c5e53bfe65..f603cc6633a5e7acc0028ab80ac21b1512625fa2 100644
--- a/ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc
+++ b/ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc
@@ -568,7 +568,7 @@ void DesktopWindowTreeHostX11::ShowWindowWithState(
     ui::WindowShowState show_state) {
   if (compositor())
     compositor()->SetVisible(true);
-  if (!IsVisible())
+  if (!IsVisible() || !window_mapped_in_server_)

[msw, 2017/05/18 18:37:44]

I hope Elliot understands this aspect of the CL, I don't!

[alshabalin, 2017/05/23 06:48:18]

The problem was: during minimization some WMs (e.g. openbox) unmap the window
(which sets window_mapped_in_server_ to false). Now, when we call
Widget::Restore, we need to map the window back but at that moment
Widget::IsVisible() is true (because window_mapped_in_client_ is true).

I decided to go for this solution instead of modifying window_mapped_in_client_
during minimization, because asking to map window seems to makes sense when
either of window_mapped_in_* is false.

     MapWindow(show_state);
 
   switch (show_state) {