MacViews: Implement event monitoring for a specific window (Reland)

MacViews: Implement event monitoring for a specific window (Reland)

ManagePasswordBubbleView wants to monitor events on a specific window,
currently using aura specific code that won't work for MacViews.
views::EventMonitor is enhanced to support monitoring events on a window,
with both Mac and Aura implementations.

Also includes minor compile fixes in ZoomBubbleView and StatusBubbleView.

Originally reviewed in https://crrev.com/730833006/

BUG=425229
Committed: https://crrev.com/5fa7a281e925c9320d86a80e463998ffaf6e51eb
Cr-Commit-Position: refs/heads/master@{#308112}

[Andre, 2014-12-12 02:15:17]

andresantoso@chromium.org changed reviewers:
+ sky@chromium.org, tapted@chromium.org

[Andre, 2014-12-12 02:15:18]

Trent, Scott, PTAL.
There was a bug in the test code, the target window need to outlive the event
monitor.
win8_chromium_ng fails with patch #1 but passes with patch #2.

[tapted, 2014-12-12 02:34:30]

lgtm

I would just add

"Originally reviewed in http://crrev.com/730833006/"

to the CL description.

(Aaand.. opinions differ.. but for a minor 1-line update like this I think it's
OK to TBR OWNERS that have already reviewed it, so long as at least one reviewer
has looked over it.   re-opening a CL is another way of doing this if your
OWNERS are OK with that. Again, opinions differ, but someone will tell you off
if they disagree :).

[sky, 2014-12-12 15:51:44]

LGTM

[Andre, 2014-12-12 16:36:11]

On 2014/12/12 02:34:30, tapted wrote:
> lgtm
> 
> I would just add
> 
> "Originally reviewed in http://crrev.com/730833006/%22
> 
> to the CL description.
> 
> (Aaand.. opinions differ.. but for a minor 1-line update like this I think
it's
> OK to TBR OWNERS that have already reviewed it, so long as at least one
reviewer
> has looked over it.   re-opening a CL is another way of doing this if your
> OWNERS are OK with that. Again, opinions differ, but someone will tell you off
> if they disagree :).

Done.

[Andre, 2014-12-12 16:36:15]

The CQ bit was checked by andresantoso@chromium.org

[commit-bot: I haz the power, 2014-12-12 16:36:57]

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

[commit-bot: I haz the power, 2014-12-12 17:36:21]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-12-12 17:38:10]

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