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

Revert of Service Manager: Remove ServiceContext* arg from Service::OnStart() (patchset #7 id:120001 of https://codereview.chromium.org/2487573002/ )

Reason for revert:
Compile failure, conflicting with another CL

Original issue's description:
> Service Manager: Remove ServiceContext* arg from Service::OnStart()
>
> This restores the context() accessor to Service, but now
> guarantees that ServiceContext always sets the context
> before invoking OnStart().
>
> This means that all Service implementations are free to safely
> access context() any time after OnStart() regardless of what
> service-running mechanism was used to create their
> ServiceContext.
>
> BUG=654986
>
> Committed: https://crrev.com/6d9b6cd9869d3f8385a42b3272edd7f5b3a6ac5e
> Cr-Commit-Position: refs/heads/master@{#431752}

TBR=ben@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=654986
Committed: https://crrev.com/13d2c57f10e29de32f916e28ba2eda880f29775c
Cr-Commit-Position: refs/heads/master@{#431754}

[Ken Rockot(use gerrit already), 2016-11-12 02:56:52]

The CQ bit was checked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2016-11-12 02:56:53]

Created Revert of Service Manager: Remove ServiceContext* arg from
Service::OnStart()

[commit-bot: I haz the power, 2016-11-12 02:57:05]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-12 02:58:47]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-12 03:00:17]

Description was changed from

==========
Revert of Service Manager: Remove ServiceContext* arg from Service::OnStart()
(patchset #7 id:120001 of https://codereview.chromium.org/2487573002/ )

Reason for revert:
Compile failure, conflicting with another CL

Original issue's description:
> Service Manager: Remove ServiceContext* arg from Service::OnStart()
> 
> This restores the context() accessor to Service, but now
> guarantees that ServiceContext always sets the context
> before invoking OnStart().
> 
> This means that all Service implementations are free to safely
> access context() any time after OnStart() regardless of what
> service-running mechanism was used to create their
> ServiceContext.
> 
> BUG=654986
> 
> Committed: https://crrev.com/6d9b6cd9869d3f8385a42b3272edd7f5b3a6ac5e
> Cr-Commit-Position: refs/heads/master@{#431752}

TBR=ben@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=654986
==========

to

==========
Revert of Service Manager: Remove ServiceContext* arg from Service::OnStart()
(patchset #7 id:120001 of https://codereview.chromium.org/2487573002/ )

Reason for revert:
Compile failure, conflicting with another CL

Original issue's description:
> Service Manager: Remove ServiceContext* arg from Service::OnStart()
>
> This restores the context() accessor to Service, but now
> guarantees that ServiceContext always sets the context
> before invoking OnStart().
>
> This means that all Service implementations are free to safely
> access context() any time after OnStart() regardless of what
> service-running mechanism was used to create their
> ServiceContext.
>
> BUG=654986
>
> Committed: https://crrev.com/6d9b6cd9869d3f8385a42b3272edd7f5b3a6ac5e
> Cr-Commit-Position: refs/heads/master@{#431752}

TBR=ben@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=654986

Committed: https://crrev.com/13d2c57f10e29de32f916e28ba2eda880f29775c
Cr-Commit-Position: refs/heads/master@{#431754}
==========

[commit-bot: I haz the power, 2016-11-12 03:00:18]

Patchset 1 (id:??) landed as
https://crrev.com/13d2c57f10e29de32f916e28ba2eda880f29775c
Cr-Commit-Position: refs/heads/master@{#431754}