mojo: shell::Context should outlive the shell MessageLoop

mojo/shell/child_process_host_unittest.cc

Index: mojo/shell/child_process_host_unittest.cc
diff --git a/mojo/shell/child_process_host_unittest.cc b/mojo/shell/child_process_host_unittest.cc
index 97bc96dca08d25d873a7ec287644c404a7de3634..8104ec309bad42d6302d3783eedcac523490122b 100644
--- a/mojo/shell/child_process_host_unittest.cc
+++ b/mojo/shell/child_process_host_unittest.cc
@@ -38,6 +38,7 @@ TEST_F(ChildProcessHostTest, Basic) {
       scoped_ptr<base::MessagePump>(new common::MessagePumpMojo()));
 
   Context context;

[jamesr, 2014/07/28 20:11:27]

any reason not to move this context c'tor up before the MessageLoop allocation
like the other ones?

[tim (not reviewing), 2014/07/28 22:16:37]

Oops, done.

+  context.Init();
   TestChildProcessHostDelegate child_process_host_delegate;
   ChildProcessHost child_process_host(&context,
                                       &child_process_host_delegate,