Service Manager: Remove ServiceContext* arg from Service::OnStart()

mash/session/session.cc

 // Copyright 2015 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 "mash/session/session.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/message_loop/message_loop.h"
 #include "mash/login/public/interfaces/login.mojom.h"
@@ skipping 11 matching lines @@
 }
 
 }  // namespace
 
 namespace mash {
 namespace session {
 
 Session::Session() : screen_locked_(false) {}
 Session::~Session() {}
 
-void Session::OnStart(service_manager::ServiceContext* context) {
-  context_ = context;
-
+void Session::OnStart() {
   StartWindowManager();
   StartQuickLaunch();
 
   // Launch a chrome window for dev convience; don't do this in the long term.
-  context->connector()->Connect("service:content_browser");
+  context()->connector()->Connect("service:content_browser");
 }
 
 bool Session::OnConnect(const service_manager::ServiceInfo& remote_info,
                         service_manager::InterfaceRegistry* registry) {
   registry->AddInterface<mojom::Session>(this);
   return true;
 }
 
 void Session::Logout() {
   // TODO(beng): Notify connected listeners that login is happening, potentially
   // give them the option to stop it.
   mash::login::mojom::LoginPtr login;
-  context_->connector()->ConnectToInterface("service:login", &login);
+  context()->connector()->ConnectToInterface("service:login", &login);
   login->ShowLoginUI();
   // This kills the user environment.
   base::MessageLoop::current()->QuitWhenIdle();
 }
 
 void Session::SwitchUser() {
   mash::login::mojom::LoginPtr login;
-  context_->connector()->ConnectToInterface("service:login", &login);
+  context()->connector()->ConnectToInterface("service:login", &login);
   login->SwitchUser();
 }
 
 void Session::AddScreenlockStateListener(
     mojom::ScreenlockStateListenerPtr listener) {
   listener->ScreenlockStateChanged(screen_locked_);
   screenlock_listeners_.AddPtr(std::move(listener));
 }
 
 void Session::LockScreen() {
@@ skipping 48 matching lines @@
   DCHECK(connections_.end() != connection);
   connections_.erase(connection);
 }
 
 void Session::StartRestartableService(
     const std::string& url,
     const base::Closure& restart_callback) {
   // TODO(beng): This would be the place to insert logic that counted restarts
   //             to avoid infinite crash-restart loops.
   std::unique_ptr<service_manager::Connection> connection =
-      context_->connector()->Connect(url);
+      context()->connector()->Connect(url);
   // Note: |connection| may be null if we've lost our connection to the service
   // manager.
   if (connection) {
     connection->SetConnectionLostClosure(
         base::Bind(&LogAndCallServiceRestartCallback, url, restart_callback));
     connections_[url] = std::move(connection);
   }
 }
 
 }  // namespace session
 }  // namespace main