Revert of service_manager: More consistent Service lifecycle API

Revert of service_manager: More consistent Service lifecycle API (patchset #3 id:40001 of https://codereview.chromium.org/2700693004/ )

Reason for revert:
CL is the suspected culprit of breaking ContentBrowserTest.BrowserCrashCallStack

Original issue's description:
> service_manager: More consistent Service lifecycle API
> 
> This introduces OnStartFailed() to be invoked in lieu of OnStart()
> if the Service pipe breaks before OnStart() can be invoked.
> 
> This disambiguates the meaning of OnStop(), ensuring that it will
> now only be called in the event that OnStart() happened first.
> 
> BUG=672614
> R=sky@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2700693004
> Cr-Commit-Position: refs/heads/master@{#451252}
> Committed: https://chromium.googlesource.com/chromium/src/+/6cadd4324682783aa817250cdf229c022086666a

TBR=sky@chromium.org,rockot@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=672614
Review-Url: https://codereview.chromium.org/2700123002
Cr-Commit-Position: refs/heads/master@{#451274}
Committed: https://chromium.googlesource.com/chromium/src/+/da1a75010d17285002bab1fda9ca510119c69814

[cfroussios, 2017-02-17 10:05:57]

The CQ bit was checked by cfroussios@chromium.org

[cfroussios, 2017-02-17 10:05:58]

Created Revert of service_manager: More consistent Service lifecycle API

[commit-bot: I haz the power, 2017-02-17 10:06:23]

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, 2017-02-17 10:07:03]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487325957861390, "parent_rev":
"864f2986aa206bae957944685e1bbb0d7b4fdcc4", "commit_rev":
"da1a75010d17285002bab1fda9ca510119c69814"}

[commit-bot: I haz the power, 2017-02-17 10:07:39]

Description was changed from

==========
Revert of service_manager: More consistent Service lifecycle API (patchset #3
id:40001 of https://codereview.chromium.org/2700693004/ )

Reason for revert:
CL is the suspected culprit of breaking ContentBrowserTest.BrowserCrashCallStack

Original issue's description:
> service_manager: More consistent Service lifecycle API
> 
> This introduces OnStartFailed() to be invoked in lieu of OnStart()
> if the Service pipe breaks before OnStart() can be invoked.
> 
> This disambiguates the meaning of OnStop(), ensuring that it will
> now only be called in the event that OnStart() happened first.
> 
> BUG=672614
> R=sky@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2700693004
> Cr-Commit-Position: refs/heads/master@{#451252}
> Committed:
https://chromium.googlesource.com/chromium/src/+/6cadd4324682783aa817250cdf22...

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

to

==========
Revert of service_manager: More consistent Service lifecycle API (patchset #3
id:40001 of https://codereview.chromium.org/2700693004/ )

Reason for revert:
CL is the suspected culprit of breaking ContentBrowserTest.BrowserCrashCallStack

Original issue's description:
> service_manager: More consistent Service lifecycle API
> 
> This introduces OnStartFailed() to be invoked in lieu of OnStart()
> if the Service pipe breaks before OnStart() can be invoked.
> 
> This disambiguates the meaning of OnStop(), ensuring that it will
> now only be called in the event that OnStart() happened first.
> 
> BUG=672614
> R=sky@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2700693004
> Cr-Commit-Position: refs/heads/master@{#451252}
> Committed:
https://chromium.googlesource.com/chromium/src/+/6cadd4324682783aa817250cdf22...

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

Review-Url: https://codereview.chromium.org/2700123002
Cr-Commit-Position: refs/heads/master@{#451274}
Committed:
https://chromium.googlesource.com/chromium/src/+/da1a75010d17285002bab1fda9ca...
==========

[commit-bot: I haz the power, 2017-02-17 10:07:40]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/da1a75010d17285002bab1fda9ca...

[Ken Rockot(use gerrit already), 2017-02-17 16:33:38]

On 2017/02/17 at 10:07:40, commit-bot wrote:
> Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/da1a75010d17285002bab1fda9ca...

Please try to include links to example failures when reverting a CL, as
otherwise the author has to spend considerably more time trying to find out
exactly what failed (platform, build config, etc).