Revert of service_manager: More consistent Service lifecycle API

services/service_manager/public/cpp/lib/service.cc

Index: services/service_manager/public/cpp/lib/service.cc
diff --git a/services/service_manager/public/cpp/lib/service.cc b/services/service_manager/public/cpp/lib/service.cc
index 1044226e65607b4d089fdcaf0451e6d5c8343e15..72222efac5352fd48fb510ba9ee9a67fd2418cfe 100644
--- a/services/service_manager/public/cpp/lib/service.cc
+++ b/services/service_manager/public/cpp/lib/service.cc
@@ -44,10 +44,6 @@
 
 bool Service::OnStop() { return true; }
 
-void Service::OnStartFailed() {
-  context()->QuitNow();
-}
-
 ServiceContext* Service::context() const {
   DCHECK(service_context_)
       << "Service::context() may only be called during or after OnStart().";
@@ -68,19 +64,6 @@
   return target_->OnConnect(remote_info, registry);
 }
 
-void ForwardingService::OnBindInterface(
-    const ServiceInfo& remote_info,
-    const std::string& interface_name,
-    mojo::ScopedMessagePipeHandle interface_pipe) {
-  target_->OnBindInterface(remote_info, interface_name,
-                           std::move(interface_pipe));
-}
-
 bool ForwardingService::OnStop() { return target_->OnStop(); }
 
-void ForwardingService::OnStartFailed() {
-  target_->set_context(context());
-  return target_->OnStartFailed();
-}
-
 }  // namespace service_manager