Revert of [Service Manager] Get rid of dynamic service discovery

Revert of [Service Manager] Get rid of dynamic service discovery (patchset #10 id:180001 of https://codereview.chromium.org/2645973006/ )

Reason for revert:
Seems to have broken content_unittests on both
https://build.chromium.org/p/chromium.linux/builders/Android%20Tests%20%28dbg%29/builds/39329
and
https://build.chromium.org/p/chromium.android/builders/Lollipop%20Phone%20Tester/builds/9385

with the following error:

CommandFailedError: Failed to list tests on any device

caused by a crash when listing tests:

[FATAL:catalog.cc(167)] Check failed: result.

Original issue's description:
> [Service Manager] Get rid of dynamic service discovery
> 
> Dynamic service discovery introduces complexity and nondeterminism in
> the Service Manager, and its implementation was never quite production-
> ready.
> 
> This CL removes all of that support, instead requiring a static
> service catalog configuration to be provided. Support remains for
> dynamic manifest discovery via ManifestProvider in order to satisfy
> content layering requirements.
> 
> The interesting and interdependent high-level changes covered here:
> 
>   - No more Packages directory; service binaries live in root_out_dir
>   - Catalog no longer touches disk for any metadata
>   - service_manifest and catalog target settings have less redundancy
>   - service_manifest and catalog outputs go in the target_gen_dir now
>   - service_manifest_overlay behavior has been merged into service_manifest
>   - Service Manager may be constructed with a static catalog manifest
>   - BackgroundServiceManager no longer owns EDK init/shutdown
>   - SM::StartEmbedderService -> SM::RegisterService, more generic API
>   - The "service overrides" concept is no longer useful and thus removed
>   - All SM embedders, including test suites, are now configured with
>     their own static catalog manifest at runtime.
> 
> BUG=677194, 672614
> TEST=standalone mash_session works, chrome --mash works, mash_browser_tests, various mus and SM-related unit tests all operating normally
> TBR=sky@chromium.org
> TBR=xhwang@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2645973006
> Cr-Commit-Position: refs/heads/master@{#445678}
> Committed: https://chromium.googlesource.com/chromium/src/+/f5a497cd5ce83519a9c71bd1ef877f85ec022d38

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

[johnme, 2017-01-24 12:05:51]

The CQ bit was checked by johnme@chromium.org

[johnme, 2017-01-24 12:05:52]

Created Revert of [Service Manager] Get rid of dynamic service discovery

[commit-bot: I haz the power, 2017-01-24 12:06:09]

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-01-24 12:08:00]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485259551182730, "parent_rev":
"c7ee9d5e15a8d98d5f5fe2afcff74bec2f3ae7d6", "commit_rev":
"415bd5606cf989923a835e5977c645313d65cc45"}

[commit-bot: I haz the power, 2017-01-24 12:08:29]

Description was changed from

==========
Revert of [Service Manager] Get rid of dynamic service discovery (patchset #10
id:180001 of https://codereview.chromium.org/2645973006/ )

Reason for revert:
Seems to have broken content_unittests on both
https://build.chromium.org/p/chromium.linux/builders/Android%20Tests%20%28dbg...
and
https://build.chromium.org/p/chromium.android/builders/Lollipop%20Phone%20Tes...

with the following error:

CommandFailedError: Failed to list tests on any device

caused by a crash when listing tests:

[FATAL:catalog.cc(167)] Check failed: result.

Original issue's description:
> [Service Manager] Get rid of dynamic service discovery
> 
> Dynamic service discovery introduces complexity and nondeterminism in
> the Service Manager, and its implementation was never quite production-
> ready.
> 
> This CL removes all of that support, instead requiring a static
> service catalog configuration to be provided. Support remains for
> dynamic manifest discovery via ManifestProvider in order to satisfy
> content layering requirements.
> 
> The interesting and interdependent high-level changes covered here:
> 
>   - No more Packages directory; service binaries live in root_out_dir
>   - Catalog no longer touches disk for any metadata
>   - service_manifest and catalog target settings have less redundancy
>   - service_manifest and catalog outputs go in the target_gen_dir now
>   - service_manifest_overlay behavior has been merged into service_manifest
>   - Service Manager may be constructed with a static catalog manifest
>   - BackgroundServiceManager no longer owns EDK init/shutdown
>   - SM::StartEmbedderService -> SM::RegisterService, more generic API
>   - The "service overrides" concept is no longer useful and thus removed
>   - All SM embedders, including test suites, are now configured with
>     their own static catalog manifest at runtime.
> 
> BUG=677194,672614
> TEST=standalone mash_session works, chrome --mash works, mash_browser_tests,
various mus and SM-related unit tests all operating normally
> TBR=sky@chromium.org
> TBR=xhwang@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2645973006
> Cr-Commit-Position: refs/heads/master@{#445678}
> Committed:
https://chromium.googlesource.com/chromium/src/+/f5a497cd5ce83519a9c71bd1ef87...

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

to

==========
Revert of [Service Manager] Get rid of dynamic service discovery (patchset #10
id:180001 of https://codereview.chromium.org/2645973006/ )

Reason for revert:
Seems to have broken content_unittests on both
https://build.chromium.org/p/chromium.linux/builders/Android%20Tests%20%28dbg...
and
https://build.chromium.org/p/chromium.android/builders/Lollipop%20Phone%20Tes...

with the following error:

CommandFailedError: Failed to list tests on any device

caused by a crash when listing tests:

[FATAL:catalog.cc(167)] Check failed: result.

Original issue's description:
> [Service Manager] Get rid of dynamic service discovery
> 
> Dynamic service discovery introduces complexity and nondeterminism in
> the Service Manager, and its implementation was never quite production-
> ready.
> 
> This CL removes all of that support, instead requiring a static
> service catalog configuration to be provided. Support remains for
> dynamic manifest discovery via ManifestProvider in order to satisfy
> content layering requirements.
> 
> The interesting and interdependent high-level changes covered here:
> 
>   - No more Packages directory; service binaries live in root_out_dir
>   - Catalog no longer touches disk for any metadata
>   - service_manifest and catalog target settings have less redundancy
>   - service_manifest and catalog outputs go in the target_gen_dir now
>   - service_manifest_overlay behavior has been merged into service_manifest
>   - Service Manager may be constructed with a static catalog manifest
>   - BackgroundServiceManager no longer owns EDK init/shutdown
>   - SM::StartEmbedderService -> SM::RegisterService, more generic API
>   - The "service overrides" concept is no longer useful and thus removed
>   - All SM embedders, including test suites, are now configured with
>     their own static catalog manifest at runtime.
> 
> BUG=677194,672614
> TEST=standalone mash_session works, chrome --mash works, mash_browser_tests,
various mus and SM-related unit tests all operating normally
> TBR=sky@chromium.org
> TBR=xhwang@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2645973006
> Cr-Commit-Position: refs/heads/master@{#445678}
> Committed:
https://chromium.googlesource.com/chromium/src/+/f5a497cd5ce83519a9c71bd1ef87...

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

Review-Url: https://codereview.chromium.org/2651953002
Cr-Commit-Position: refs/heads/master@{#445707}
Committed:
https://chromium.googlesource.com/chromium/src/+/415bd5606cf989923a835e5977c6...
==========

[commit-bot: I haz the power, 2017-01-24 12:08:30]

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