Prevent crash in NativeWidgetAura::IsMouseEventsEnabled() when no root window

Prevent crash in NativeWidgetAura::IsMouseEventsEnabled() when no root window

This patch adds a root window check in NativeWidgetAura::IsMouseEventsEnabled().

BUG=712091
TEST=none

[yoshiki, 2017-04-14 15:18:30]

The CQ bit was checked by yoshiki@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-14 15:18:38]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-14 16:03:31]

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

[commit-bot: I haz the power, 2017-04-14 16:03:32]

Dry run: This issue passed the CQ dry run.

[yoshiki, 2017-04-17 04:22:53]

Description was changed from

==========
wip


BUG=
==========

to

==========
wip


BUG=712091
==========

[yoshiki, 2017-04-17 04:29:14]

Description was changed from

==========
wip


BUG=712091
==========

to

==========
Prevent crash in NativeWidgetAura::IsMouseEventsEnabled() when no root window

This patch adds a root window check in NativeWidgetAura::IsMouseEventsEnabled().

BUG=712091
TEST=none
==========

[yoshiki, 2017-04-17 04:29:15]

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

[yoshiki, 2017-04-17 04:29:30]

Sadrul, PTAL. Thanks.

[yoshiki, 2017-04-18 06:20:20]

yoshiki@chromium.org changed reviewers:
+ sky@chromium.org

[yoshiki, 2017-04-18 06:20:30]

Scott, could you take a look?

[sky, 2017-04-18 16:14:25]

On 2017/04/18 06:20:30, yoshiki wrote:
> Scott, could you take a look?

I think this will fix the crash, but I'm worried there is something more wrong
here. In order for this crash to occur the widget would not be parented, which
is likely to cause other problems. Why is ArcCustomNotificationView trying to
Show() a Widget that isn't attached to a root window?

[yoshiki, 2017-04-19 15:32:44]

On 2017/04/18 16:14:25, sky wrote:
> On 2017/04/18 06:20:30, yoshiki wrote:
> > Scott, could you take a look?
> 
> I think this will fix the crash, but I'm worried there is something more wrong
> here. In order for this crash to occur the widget would not be parented, which
> is likely to cause other problems. Why is ArcCustomNotificationView trying to
> Show() a Widget that isn't attached to a root window?

Ok thanks, let me investigate the root cause.

[yoshiki, 2017-05-08 04:44:56]

yoshiki@chromium.org changed reviewers:
- sadrul@chromium.org, sky@chromium.org