mash: Use null AcceleratedWidget for widgets in tests

mash: Use null AcceleratedWidget for widgets in tests

When running in tests, the ui::ContextFactory creates test
contexts that do not support real GL operations. Some of the
code (e.g. InProcessContextProvider) expects a null widget
for ui::Compositor::widget() in this situation. So update
WindowTreeHostMus to follow this.

Also, re-enable the tests, and make sure the test context
factory are destroyed correctly during test tear-down.

BUG=652871
Committed: https://crrev.com/c2df26e737f98f72f03dfe4ac1fe1ea91545bab2
Cr-Commit-Position: refs/heads/master@{#424515}

[sadrul, 2016-10-11 03:25:51]

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

[commit-bot: I haz the power, 2016-10-11 03:26:04]

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, 2016-10-11 03:36:31]

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

[commit-bot: I haz the power, 2016-10-11 03:36:32]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sadrul, 2016-10-11 06:12:50]

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

[commit-bot: I haz the power, 2016-10-11 06:13:00]

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, 2016-10-11 06:48:24]

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

[commit-bot: I haz the power, 2016-10-11 06:48:25]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[sadrul, 2016-10-11 07:04:02]

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

[commit-bot: I haz the power, 2016-10-11 07:04:14]

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

[sadrul, 2016-10-11 07:08:59]

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

[commit-bot: I haz the power, 2016-10-11 07:09:12]

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, 2016-10-11 08:07:57]

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

[commit-bot: I haz the power, 2016-10-11 08:07:57]

Dry run: This issue passed the CQ dry run.

[sadrul, 2016-10-11 15:16:18]

Description was changed from

==========
mash: Reenable WorkspaceLayoutManagerBackdropTest.BackdropCrashTest

BUG=652871
==========

to

==========
mash: Use null AcceleratedWidget for widgets in tests

When running in tests, the ui::ContextFactory creates test
contexts that do not support real GL operations. Some of the
code (e.g. InProcessContextProvider) expects a null widget
for ui::Compositor::widget() in this situation. So update
WindowTreeHostMus to follow this.

Also, re-enable the tests, and make sure the test context
factory are destroyed correctly during test tear-down.

BUG=652871
==========

[sadrul, 2016-10-11 15:16:37]

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

[sky, 2016-10-11 19:19:03]

LGTM and thanks for looking at this so quickly.

[sadrul, 2016-10-11 19:40:35]

The CQ bit was checked by sadrul@chromium.org

[commit-bot: I haz the power, 2016-10-11 19:41:28]

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, 2016-10-11 19:46:16]

Description was changed from

==========
mash: Use null AcceleratedWidget for widgets in tests

When running in tests, the ui::ContextFactory creates test
contexts that do not support real GL operations. Some of the
code (e.g. InProcessContextProvider) expects a null widget
for ui::Compositor::widget() in this situation. So update
WindowTreeHostMus to follow this.

Also, re-enable the tests, and make sure the test context
factory are destroyed correctly during test tear-down.

BUG=652871
==========

to

==========
mash: Use null AcceleratedWidget for widgets in tests

When running in tests, the ui::ContextFactory creates test
contexts that do not support real GL operations. Some of the
code (e.g. InProcessContextProvider) expects a null widget
for ui::Compositor::widget() in this situation. So update
WindowTreeHostMus to follow this.

Also, re-enable the tests, and make sure the test context
factory are destroyed correctly during test tear-down.

BUG=652871
==========

[commit-bot: I haz the power, 2016-10-11 19:46:17]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-10-11 19:48:02]

Description was changed from

==========
mash: Use null AcceleratedWidget for widgets in tests

When running in tests, the ui::ContextFactory creates test
contexts that do not support real GL operations. Some of the
code (e.g. InProcessContextProvider) expects a null widget
for ui::Compositor::widget() in this situation. So update
WindowTreeHostMus to follow this.

Also, re-enable the tests, and make sure the test context
factory are destroyed correctly during test tear-down.

BUG=652871
==========

to

==========
mash: Use null AcceleratedWidget for widgets in tests

When running in tests, the ui::ContextFactory creates test
contexts that do not support real GL operations. Some of the
code (e.g. InProcessContextProvider) expects a null widget
for ui::Compositor::widget() in this situation. So update
WindowTreeHostMus to follow this.

Also, re-enable the tests, and make sure the test context
factory are destroyed correctly during test tear-down.

BUG=652871

Committed: https://crrev.com/c2df26e737f98f72f03dfe4ac1fe1ea91545bab2
Cr-Commit-Position: refs/heads/master@{#424515}
==========

[commit-bot: I haz the power, 2016-10-11 19:48:03]

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