Changes to get mash browser_tests shutdown working correctly

Changes to get mash browser_tests shutdown working correctly

. Adds ability to set callback on MojoShellConnection when shell is
  lost, and wires through in chrome side to trigger exiting.
. Adds ability for TestLauncherDelegate to be notified after tests
  have run. Needed so I can shutdown state while AtExitManager is
  valid.
. Made context not attempt to shutdown edk if context did not start
  edk.
. Made MashBrowserTestsMain shutdown MojoTestConnector at a time when
  it can be shutdown.

BUG=581733
TEST=covered by tests
R=jam@chromium.org

Committed: https://crrev.com/220db613792b3fa65361e478ddd00da753e7aa7c
Cr-Commit-Position: refs/heads/master@{#382864}

[jam, 2016-03-23 15:12:16]

lgtm

https://codereview.chromium.org/1822213002/diff/20001/content/public/test/tes...
File content/public/test/test_launcher.h (right):

https://codereview.chromium.org/1822213002/diff/20001/content/public/test/tes...
content/public/test/test_launcher.h:56: virtual void OnDoneRunningTests();
nit: move this to the end (after line 70) so that the function order matches the
order they're called in?

[sky, 2016-03-23 15:36:23]

The CQ bit was checked by sky@chromium.org

[sky, 2016-03-23 15:36:24]

The patchset sent to the CQ was uploaded after l-g-t-m from jam@chromium.org
Link to the patchset: https://codereview.chromium.org/1822213002/#ps40001
(title: "order")

[commit-bot: I haz the power, 2016-03-23 15:36:42]

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

[commit-bot: I haz the power, 2016-03-23 16:39:03]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-23 16:40:19]

Description was changed from

==========
Changes to get mash browser_tests shutdown working correctly

. Adds ability to set callback on MojoShellConnection when shell is
  lost, and wires through in chrome side to trigger exiting.
. Adds ability for TestLauncherDelegate to be notified after tests
  have run. Needed so I can shutdown state while AtExitManager is
  valid.
. Made context not attempt to shutdown edk if context did not start
  edk.
. Made MashBrowserTestsMain shutdown MojoTestConnector at a time when
  it can be shutdown.

BUG=581733
TEST=covered by tests
R=jam@chromium.org
==========

to

==========
Changes to get mash browser_tests shutdown working correctly

. Adds ability to set callback on MojoShellConnection when shell is
  lost, and wires through in chrome side to trigger exiting.
. Adds ability for TestLauncherDelegate to be notified after tests
  have run. Needed so I can shutdown state while AtExitManager is
  valid.
. Made context not attempt to shutdown edk if context did not start
  edk.
. Made MashBrowserTestsMain shutdown MojoTestConnector at a time when
  it can be shutdown.

BUG=581733
TEST=covered by tests
R=jam@chromium.org

Committed: https://crrev.com/220db613792b3fa65361e478ddd00da753e7aa7c
Cr-Commit-Position: refs/heads/master@{#382864}
==========

[commit-bot: I haz the power, 2016-03-23 16:40:20]

Patchset 3 (id:??) landed as
https://crrev.com/220db613792b3fa65361e478ddd00da753e7aa7c
Cr-Commit-Position: refs/heads/master@{#382864}