mojo: fix little bug in DynamicServiceLoader/Runner

If the Runner wasn't actually told to Start(), the DelegateSimpleThread would
DCHECK for not having been Started. This patch makes it so we don't allocate
a DST unless we need to.

Also adds basic unittest files for DynamicServiceLoader and Runner.

BUG=

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=285650

Review URL: https://codereview.chromium.org/414173004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285849 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 20 insertions, 0 deletions

diff --git a/mojo/shell/in_process_dynamic_service_runner_unittest.cc b/mojo/shell/in_process_dynamic_service_runner_unittest.cc
new file mode 100644
index 0000000..6ebb1a6
--- /dev/null
+++ b/mojo/shell/in_process_dynamic_service_runner_unittest.cc
@@ -0,0 +1,20 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "mojo/shell/context.h"
+#include "mojo/shell/in_process_dynamic_service_runner.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace mojo {
+namespace shell {
+
+TEST(InProcessDynamicServiceRunnerTest, NotStarted) {
+  base::MessageLoop loop;
+  Context context;
+  InProcessDynamicServiceRunner runner(&context);
+  // Shouldn't crash or DCHECK on destruction.
+}
+
+}  // namespace shell
+}  // namespace mojo