Simplify the logic in DesktopWindowTreeHostX11 for FocusOut

Simplify the logic in DesktopWindowTreeHostX11 for FocusOut

We should have the same handling regardless of whether the 'mode' member is
NotifyGrab or NotifyUngrab. According to
http://tronche.com/gui/x/xlib/events/input-focus/grab.html, whether the mode
field is NotifyGrab or NotifyUngrab depends on whether there is a new
capturer for the keyboard. The X window has lost keyboard capture in both cases.

BUG=392480
TEST=None

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=283823

[pkotwicz, 2014-07-09 17:35:17]

Elliot, can you please take a look?

FocusOut in x11_whole_screen_move_loop.cc should be fixed as well but I am
slightly hesitant to change that file independant of making other changes. In
particular, x11_whole_screen_move_loop.cc does not check the target window of
the FocusOut event.

[pkotwicz, 2014-07-09 17:43:31]

I have also checked and this CL does not bring back the bug in
https://chromiumcodereview.appspot.com/11189148

[erg, 2014-07-10 20:25:08]

lgtm

[pkotwicz, 2014-07-10 21:20:52]

The CQ bit was checked by pkotwicz@chromium.org

[commit-bot: I haz the power, 2014-07-10 21:22:10]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/379773003/60001

[commit-bot: I haz the power, 2014-07-10 23:51:07]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[commit-bot: I haz the power, 2014-07-10 23:53:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-07-10 23:53:21]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[pkotwicz, 2014-07-11 00:21:10]

Sadrul for OWNERS

[pkotwicz, 2014-07-11 00:21:32]

Sadrul for OWNERS for real

[sadrul, 2014-07-15 22:45:49]

cool. LGTM

[pkotwicz, 2014-07-16 14:56:49]

The CQ bit was checked by pkotwicz@chromium.org

[commit-bot: I haz the power, 2014-07-16 14:58:19]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/379773003/60001

[commit-bot: I haz the power, 2014-07-16 15:02:18]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[commit-bot: I haz the power, 2014-07-16 15:04:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-07-16 15:04:54]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[pkotwicz, 2014-07-17 14:40:15]

The CQ bit was checked by pkotwicz@chromium.org

[commit-bot: I haz the power, 2014-07-17 14:40:59]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/379773003/80001

[commit-bot: I haz the power, 2014-07-17 18:37:36]

Change committed as 283823

[pkotwicz, 2014-07-23 14:04:50]

A revert of this CL has been created in
https://codereview.chromium.org/409223003/ by pkotwicz@chromium.org.

The reason for reverting is: I am reverting this CL because it causes
regressions like crbug.com/395985. This CL is in the right direction, but I need
to make the capture code for tab dragging less crazy first. See comment #3 in
crbug.com/395985 for more details.