NewRunnable* conversion to base::Bind for service process utils.

chrome/common/service_process_util_posix.cc

 // Copyright (c) 2011 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 "chrome/common/service_process_util_posix.h"
 
 #include "base/basictypes.h"
+#include "base/bind.h"
 #include "base/eintr_wrapper.h"
 #include "base/message_loop_proxy.h"
 #include "base/synchronization/waitable_event.h"
 
 namespace {
 int g_signal_socket = -1;
 }
 
 ServiceProcessTerminateMonitor::ServiceProcessTerminateMonitor(
-    Task* terminate_task)
+    const base::Closure& terminate_task)
     : terminate_task_(terminate_task) {
 }
 
 ServiceProcessTerminateMonitor::~ServiceProcessTerminateMonitor() {
 }
 
 void ServiceProcessTerminateMonitor::OnFileCanReadWithoutBlocking(int fd) {
-  if (terminate_task_.get()) {
+  if (!terminate_task_.is_null()) {
     int buffer;
     int length = read(fd, &buffer, sizeof(buffer));
     if ((length == sizeof(buffer)) && (buffer == kTerminateMessage)) {
-      terminate_task_->Run();
-      terminate_task_.reset();
+      terminate_task_.Run();
+      terminate_task_.Reset();
     } else if (length > 0) {
       DLOG(ERROR) << "Unexpected read: " << buffer;
     } else if (length == 0) {
       DLOG(ERROR) << "Unexpected fd close";
     } else if (length < 0) {
       DPLOG(ERROR) << "read";
     }
   }
 }
 
@@ skipping 88 matching lines @@
   state_ = new StateData;
 
   // Explicitly adding a reference here (and removing it in TearDownState)
   // because StateData is refcounted on Mac and Linux so that methods can
   // be called on other threads.
   // It is not refcounted on Windows at this time.
   state_->AddRef();
 }
 
 bool ServiceProcessState::SignalReady(
-    base::MessageLoopProxy* message_loop_proxy, Task* terminate_task) {
+    base::MessageLoopProxy* message_loop_proxy,
+    const base::Closure& terminate_task) {
   DCHECK(state_);
 
-  scoped_ptr<Task> scoped_terminate_task(terminate_task);
 #if defined(OS_POSIX) && !defined(OS_MACOSX)
   state_->running_lock_.reset(TakeServiceRunningLock(true));
   if (state_->running_lock_.get() == NULL) {
     return false;
   }
 #endif
   state_->terminate_monitor_.reset(
-      new ServiceProcessTerminateMonitor(scoped_terminate_task.release()));
+      new ServiceProcessTerminateMonitor(terminate_task));
   if (pipe(state_->sockets_) < 0) {
     DPLOG(ERROR) << "pipe";
     return false;
   }
   base::WaitableEvent signal_ready(true, false);
   bool success = false;
 
   message_loop_proxy->PostTask(FROM_HERE,
-      NewRunnableMethod(state_, &ServiceProcessState::StateData::SignalReady,
-                        &signal_ready,
-                        &success));
+      base::Bind(&ServiceProcessState::StateData::SignalReady,
+                 state_,
+                 &signal_ready,
+                 &success));
   signal_ready.Wait();
   return success;
 }
 
 void ServiceProcessState::TearDownState() {
   if (state_) {
     state_->Release();
     state_ = NULL;
   }
 }