Small fix in out_of_process_dynamic_service_runner.cc

mojo/shell/out_of_process_dynamic_service_runner.cc

Index: mojo/shell/out_of_process_dynamic_service_runner.cc
diff --git a/mojo/shell/out_of_process_dynamic_service_runner.cc b/mojo/shell/out_of_process_dynamic_service_runner.cc
index c1243a3f2a9baf0f146eae48d122a139ffd818e5..cec762cdefde5e311f618f1da906c3b0f030c442 100644
--- a/mojo/shell/out_of_process_dynamic_service_runner.cc
+++ b/mojo/shell/out_of_process_dynamic_service_runner.cc
@@ -34,9 +34,6 @@ void OutOfProcessDynamicServiceRunner::Start(
     const base::Closure& app_completed_callback) {
   app_path_ = app_path;
 
-  DCHECK(!service_handle_.is_valid());

[qsr, 2014/12/03 14:23:05]

The private field is never used, outside of this method.

Right now, the handle is passed to the private field, and then the same scoped
handle is passed to StartApp -> so an invalid handle is sent, and the handle is
just kept in this class forever without being used.

-  service_handle_ = service_handle.Pass();
-
   DCHECK(app_completed_callback_.is_null());
   app_completed_callback_ = app_completed_callback;