Fix BridgedNativeWidgetTest test crashes after r379525

Fix Mac views_unittests regressions after r379525

All the BridgedNativeWidgetTest tests, and
NativeWidgetMacTest.NonWidgetParent crash after r379525. The problem is
an inability to obtain the Widget's focus manager when a native focus
change occurs because the test harness doesn't have one at that point.

For BridgedNativeWidgetTest, Widget::Close(..) normally informs the
BridgedNativeWidget that the views::RootView is about to be destroyed,
but this unit test harness doesn't use views::Widgets, so that part
needs to be mocked out.

NativeWidgetMacTest.NonWidgetParent was creating a `child` Widget, which
relies on the FocusManager provided by a parent Widget which wasn't
present.

BUG=591016
TBR=kirr@yandex-team.ru

Committed: https://crrev.com/3436bb5299b3c8becec4c7033c25f3f8c70c5692
Cr-Commit-Position: refs/heads/master@{#379773}

[tapted, 2016-03-08 00:49:20]

Description was changed from

==========
Fix BridgedNativeWidgetTest test crashes after r379525

All the BridgedNativeWidgetTest tests crash after r379525.

Normally Widget::Close(..) informs the BridgedNativeWidget that the
views::RootView is about to be destroyed, but this unit test doesn't use
views::Widgets, so that part is mocked out.

BUG=591016
TBR=kirr@yandex-team.ru
==========

to

==========
Fix BridgedNativeWidgetTest test crashes after r379525

All the BridgedNativeWidgetTest tests crash after r379525.

Normally Widget::Close(..) informs the BridgedNativeWidget that the
views::RootView is about to be destroyed, but this unit test harness
doesn't use views::Widgets, so that part needs to be mocked out.

BUG=591016
TBR=kirr@yandex-team.ru
==========

[tapted, 2016-03-08 00:49:44]

The CQ bit was checked by tapted@chromium.org

[commit-bot: I haz the power, 2016-03-08 00:50:00]

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

[commit-bot: I haz the power, 2016-03-08 01:34:35]

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

[commit-bot: I haz the power, 2016-03-08 01:34:36]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[tapted, 2016-03-08 02:29:49]

Description was changed from

==========
Fix BridgedNativeWidgetTest test crashes after r379525

All the BridgedNativeWidgetTest tests crash after r379525.

Normally Widget::Close(..) informs the BridgedNativeWidget that the
views::RootView is about to be destroyed, but this unit test harness
doesn't use views::Widgets, so that part needs to be mocked out.

BUG=591016
TBR=kirr@yandex-team.ru
==========

to

==========
Fix Mac views_unittests regressions after r379525

All the BridgedNativeWidgetTest tests, and
NativeWidgetMacTest.NonWidgetParent crash after r379525. The problem is
an inability to obtain the Widget's focus manager when a native focus
change occurs because the test harness doesn't have one at that point.

For BridgedNativeWidgetTest, Widget::Close(..) normally informs the
BridgedNativeWidget that the views::RootView is about to be destroyed,
but this unit test harness doesn't use views::Widgets, so that part
needs to be mocked out.

NativeWidgetMacTest.NonWidgetParent was creating a `child` Widget, which
relies on the FocusManager provided by a parent Widget which wasn't
present.

BUG=591016
TBR=kirr@yandex-team.ru
==========

[tapted, 2016-03-08 02:31:10]

NativeWidgetMacTest.NonWidgetParent is failing for a similar reason, so I've
rolled in a fix fo that too

[tapted, 2016-03-08 07:09:53]

The CQ bit was checked by tapted@chromium.org

[commit-bot: I haz the power, 2016-03-08 07:10:09]

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

[commit-bot: I haz the power, 2016-03-08 07:14:48]

Description was changed from

==========
Fix Mac views_unittests regressions after r379525

All the BridgedNativeWidgetTest tests, and
NativeWidgetMacTest.NonWidgetParent crash after r379525. The problem is
an inability to obtain the Widget's focus manager when a native focus
change occurs because the test harness doesn't have one at that point.

For BridgedNativeWidgetTest, Widget::Close(..) normally informs the
BridgedNativeWidget that the views::RootView is about to be destroyed,
but this unit test harness doesn't use views::Widgets, so that part
needs to be mocked out.

NativeWidgetMacTest.NonWidgetParent was creating a `child` Widget, which
relies on the FocusManager provided by a parent Widget which wasn't
present.

BUG=591016
TBR=kirr@yandex-team.ru
==========

to

==========
Fix Mac views_unittests regressions after r379525

All the BridgedNativeWidgetTest tests, and
NativeWidgetMacTest.NonWidgetParent crash after r379525. The problem is
an inability to obtain the Widget's focus manager when a native focus
change occurs because the test harness doesn't have one at that point.

For BridgedNativeWidgetTest, Widget::Close(..) normally informs the
BridgedNativeWidget that the views::RootView is about to be destroyed,
but this unit test harness doesn't use views::Widgets, so that part
needs to be mocked out.

NativeWidgetMacTest.NonWidgetParent was creating a `child` Widget, which
relies on the FocusManager provided by a parent Widget which wasn't
present.

BUG=591016
TBR=kirr@yandex-team.ru
==========

[commit-bot: I haz the power, 2016-03-08 07:14:49]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-08 07:16:06]

Description was changed from

==========
Fix Mac views_unittests regressions after r379525

All the BridgedNativeWidgetTest tests, and
NativeWidgetMacTest.NonWidgetParent crash after r379525. The problem is
an inability to obtain the Widget's focus manager when a native focus
change occurs because the test harness doesn't have one at that point.

For BridgedNativeWidgetTest, Widget::Close(..) normally informs the
BridgedNativeWidget that the views::RootView is about to be destroyed,
but this unit test harness doesn't use views::Widgets, so that part
needs to be mocked out.

NativeWidgetMacTest.NonWidgetParent was creating a `child` Widget, which
relies on the FocusManager provided by a parent Widget which wasn't
present.

BUG=591016
TBR=kirr@yandex-team.ru
==========

to

==========
Fix Mac views_unittests regressions after r379525

All the BridgedNativeWidgetTest tests, and
NativeWidgetMacTest.NonWidgetParent crash after r379525. The problem is
an inability to obtain the Widget's focus manager when a native focus
change occurs because the test harness doesn't have one at that point.

For BridgedNativeWidgetTest, Widget::Close(..) normally informs the
BridgedNativeWidget that the views::RootView is about to be destroyed,
but this unit test harness doesn't use views::Widgets, so that part
needs to be mocked out.

NativeWidgetMacTest.NonWidgetParent was creating a `child` Widget, which
relies on the FocusManager provided by a parent Widget which wasn't
present.

BUG=591016
TBR=kirr@yandex-team.ru

Committed: https://crrev.com/3436bb5299b3c8becec4c7033c25f3f8c70c5692
Cr-Commit-Position: refs/heads/master@{#379773}
==========

[commit-bot: I haz the power, 2016-03-08 07:16:07]

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

[kirr, 2016-03-09 07:15:49]

On 2016/03/08 02:31:10, tapted wrote:
> NativeWidgetMacTest.NonWidgetParent is failing for a similar reason, so I've
> rolled in a fix fo that too

Thanks. I should have run all views_unittests.
Have you any plans to use trybot for MacViews compilation and tests soon?

[tapted, 2016-03-09 07:48:42]

On 2016/03/09 07:15:49, kirr wrote:
> On 2016/03/08 02:31:10, tapted wrote:
> > NativeWidgetMacTest.NonWidgetParent is failing for a similar reason, so I've
> > rolled in a fix fo that too
> 
> Thanks. I should have run all views_unittests.
> Have you any plans to use trybot for MacViews compilation and tests soon?

Yup - we just need to get all the tests passing (and not flaky) first :). Keep
an eye on http://crbug.com/500806