Only call OnActiveWindowChanged() for NotifyNormal FocusIn/FocusOut events.

Only call OnActiveWindowChanged() for NotifyNormal FocusIn/FocusOut events.

This fixes flickering when switching the X11 input focus between non-chrome
windows, but having the pointer be above the chrome window (you need to use
keyboard shortcuts to switch focus to trigger this).

The reason for the flickering is that OnActiveWindowChanged() is called with
state ACTIVE, even though the window was not activated. Before my change, the
code called OnActiveWindowChanged() even for FocusIn/FocusOut events with
mode==NotifyGrab, which is not intended. For details on the events, see:
http://tronche.com/gui/x/xlib/events/input-focus/normal-and-grabbed.html

Committed: https://crrev.com/a5f6b107185e48a96f1317ca01cec501a32e1f04
Cr-Commit-Position: refs/heads/master@{#321821}

[stapelberg, 2015-03-22 14:39:43]

stapelberg@google.com changed reviewers:
+ erg@google.com

[Elliot Glaysher, 2015-03-23 17:48:11]

erg@chromium.org changed reviewers:
+ erg@chromium.org

[Elliot Glaysher, 2015-03-23 17:48:12]

lgtm

[stapelberg, 2015-03-23 18:05:00]

The CQ bit was checked by stapelberg@google.com

[commit-bot: I haz the power, 2015-03-23 18:05:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1027883003/1

[commit-bot: I haz the power, 2015-03-23 18:48:53]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-03-23 18:49:37]

Patchset 1 (id:??) landed as
https://crrev.com/a5f6b107185e48a96f1317ca01cec501a32e1f04
Cr-Commit-Position: refs/heads/master@{#321821}