Disable XI2 events on mirror window

Disable XI2 events on mirror window

BUG=406064
Committed: https://crrev.com/8417c8e514f88b0f431ed8c4d7e64f27faaffdab
Cr-Commit-Position: refs/heads/master@{#294477}

[kpschoedel, 2014-09-11 17:19:47]

kpschoedel@chromium.org changed reviewers:
+ oshima@chromium.org

[oshima, 2014-09-11 17:30:28]

oshima@chromium.org changed reviewers:
+ sadrul@chromium.org

[oshima, 2014-09-11 17:30:29]

+sadrul

lgtm if sadrul approved.

[oshima, 2014-09-11 17:33:55]

https://codereview.chromium.org/558503005/diff/1/ash/display/mirror_window_co...
File ash/display/mirror_window_controller.cc (right):

https://codereview.chromium.org/558503005/diff/1/ash/display/mirror_window_co...
ash/display/mirror_window_controller.cc:50: memset(mask, 0, sizeof(mask));
nit:

unsigned char mask[..] = {0}

[kpschoedel, 2014-09-11 17:49:19]

https://codereview.chromium.org/558503005/diff/1/ash/display/mirror_window_co...
File ash/display/mirror_window_controller.cc (right):

https://codereview.chromium.org/558503005/diff/1/ash/display/mirror_window_co...
ash/display/mirror_window_controller.cc:50: memset(mask, 0, sizeof(mask));
On 2014/09/11 17:33:55, oshima wrote:
> nit:
> 
> unsigned char mask[..] = {0}
> 

Done.

[sadrul, 2014-09-11 19:33:14]

LGTM

What would be the fix for ozone (i.e. non-x11)? Can you file a bug to make sure
it is tracked?

[kpschoedel, 2014-09-11 19:54:36]

The CQ bit was checked by kpschoedel@chromium.org

[commit-bot: I haz the power, 2014-09-11 19:56:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/558503005/20001

[kpschoedel, 2014-09-11 20:10:58]

On 2014/09/11 19:33:14, sadrul wrote:
> What would be the fix for ozone (i.e. non-x11)?

Good question. I suspect it doesn't arise because we don't have X11
(mis-)directing events to windows.

> Can you file a bug to make sure
> it is tracked?

Issue 413376

[commit-bot: I haz the power, 2014-09-11 22:20:50]

Committed patchset #2 (id:20001) as d61231318166745ac78c1df55359de9481971945

[commit-bot: I haz the power, 2014-09-11 22:31:05]

Patchset 2 (id:??) landed as
https://crrev.com/8417c8e514f88b0f431ed8c4d7e64f27faaffdab
Cr-Commit-Position: refs/heads/master@{#294477}