Call XSetInputFocus after XRaiseWindow when activating.

ui/views/widget/desktop_aura/x11_desktop_handler.cc

Index: ui/views/widget/desktop_aura/x11_desktop_handler.cc
diff --git a/ui/views/widget/desktop_aura/x11_desktop_handler.cc b/ui/views/widget/desktop_aura/x11_desktop_handler.cc
index 3a14e04c076cebcbace6271db290e940ef144ef7..d99a501b2840eb18197113e7bfa56ce8144da908 100644
--- a/ui/views/widget/desktop_aura/x11_desktop_handler.cc
+++ b/ui/views/widget/desktop_aura/x11_desktop_handler.cc
@@ -94,6 +94,15 @@ void X11DesktopHandler::ActivateWindow(::Window window) {
                &xclient);
   } else {
     XRaiseWindow(xdisplay_, window);
+
+    // XRaiseWindow will not give input focus to the window. We now need to ask
+    // the X server to do that. However, if the window is not viewable this will
+    // produce an error so we have to check that before making the call.
+    XWindowAttributes attributes;
+    XGetWindowAttributes(xdisplay_, window, &attributes);
+    if (attributes.map_state == IsViewable)

[sadrul, 2014/01/15 19:23:50]

Is this only ever called from DesktopWindowTreeHostX11::Activate()? Can we make
sure the window is mapped in there? i.e.:

  if (!window_mapped_)
    Show();
  ...

Would that fix the issue? If it does, then I think this would be better.

[mlamouri (slow - plz ping), 2014/01/15 23:15:18]

The assertion goes away but the test fail with that.

However, we can't do that because it is assumed that ::Activate() should not
show the widget if the widget isn't shown yet.

[sadrul, 2014/01/15 23:18:53]

Shouldn't we return early from DesktopWindowTreeHostX11::Activate() if
!window_mapped_, in that case? What does XRaiseWindow do to a hidden window?

[mlamouri (slow - plz ping), 2014/01/15 23:27:50]

I think that would be reasonable but it would break
DesktopScreenX11Test.GetWindowAtScreenPoint. I believe the reason is that
DesktopScreenX11::GetWindowAtScreenPoint() go trough the windows and check their
bounds. Activating the window must change its position in the list.

If you are fine with me disabling the test, I can put an early return in
::Activate().

[sadrul, 2014/01/15 23:31:35]

The test can be fixed by making sure the widgets created are all Show()n before
trying to activate. The test does expect the windows to be visible, otherwise
trying to get the window at a specific point in the screen doesn't make sense.

+      XSetInputFocus(xdisplay_, window, RevertToParent, CurrentTime);
+
     OnActiveWindowChanged(window);
   }
 }