Move some basic early process init into Service Manager

Move some basic early process init into Service Manager

Introduces the beginnings of an embedder API for the Service Manager,
consisting of a single entry point (service_manager::Main) which in
turn delegates to its embedder for arbitrary initialization and
process execution logic.

This is the first of several incremental steps to remove content from
the generic process startup flow. Future patches will rework various
main entry points to go through service_manager::Main directly, rather
than going ContentMain -> service_manager::Main.

This will also allow us to introduce new process types which run the
Service Manager or arbitrary services directly without touching any
part of content.

BUG=654986
Review-Url: https://codereview.chromium.org/2613653003
Cr-Original-Commit-Position: refs/heads/master@{#458252}
Committed: https://chromium.googlesource.com/chromium/src/+/c6026704ff452d402924ce2d88b0168baf227b1e
Review-Url: https://codereview.chromium.org/2613653003
Cr-Commit-Position: refs/heads/master@{#458331}
Committed: https://chromium.googlesource.com/chromium/src/+/60d92c2f3ff60c2fcc0b3cddd05738403fb1f0c3

[Ken Rockot(use gerrit already), 2017-01-04 00:41:16]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-04 00:41:49]

Dry run: 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-04 01:49:09]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-04 01:49:10]

Dry run: This issue passed the CQ dry run.

[Ken Rockot(use gerrit already), 2017-01-05 17:26:20]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-05 17:26:50]

Dry run: 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-05 18:06:28]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-05 18:06:29]

Dry run: Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[Ken Rockot(use gerrit already), 2017-03-14 17:33:52]

Patchset #1 (id:1) has been deleted

[Ken Rockot(use gerrit already), 2017-03-14 17:33:56]

Patchset #1 (id:20001) has been deleted

[Ken Rockot(use gerrit already), 2017-03-14 17:35:07]

Description was changed from

==========
Introduce Service Manager embedder API

Introduces a basic embedder API for the Service Manager, consisting
of a single entry point (service_manager::Main) which in turn delegates
initialization logic to the embedder.

This is the first of several incremental steps to transplant process
control logic from content to service_manager.

BUG=654986
==========

to

==========
Introduce Service Manager embedder API

Introduces a basic embedder API for the Service Manager, consisting
of a single entry point (service_manager::Main) which in turn delegates
initialization logic to the embedder.

Moves some early process environment setup from content into
service_manager.

This is the first of several incremental steps to transplant process
control from content to service_manager.

BUG=654986
==========

[Ken Rockot(use gerrit already), 2017-03-14 17:35:16]

Description was changed from

==========
Introduce Service Manager embedder API

Introduces a basic embedder API for the Service Manager, consisting
of a single entry point (service_manager::Main) which in turn delegates
initialization logic to the embedder.

Moves some early process environment setup from content into
service_manager.

This is the first of several incremental steps to transplant process
control from content to service_manager.

BUG=654986
==========

to

==========
Introduce Service Manager embedder API

Introduces a basic embedder API for the Service Manager, consisting
of a single entry point (service_manager::Main) which in turn delegates
initialization logic to the embedder.

Moves some early process environment setup from content into
service_manager.

BUG=654986
==========

[Ken Rockot(use gerrit already), 2017-03-14 19:47:10]

Description was changed from

==========
Introduce Service Manager embedder API

Introduces a basic embedder API for the Service Manager, consisting
of a single entry point (service_manager::Main) which in turn delegates
initialization logic to the embedder.

Moves some early process environment setup from content into
service_manager.

BUG=654986
==========

to

==========
Introduce Service Manager embedder API

Introduces a basic embedder API for the Service Manager, consisting
of a single entry point (service_manager::Main) which in turn delegates
initialization logic to the embedder.

Moves some early process environment setup from content into
service_manager.

This is the first of several incremental steps to remove content from
the generic process startup flow.

BUG=654986
==========

[Ken Rockot(use gerrit already), 2017-03-14 19:49:56]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-14 19:50:26]

Dry run: 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-03-14 19:55:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-14 19:55:40]

Dry run: Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Ken Rockot(use gerrit already), 2017-03-14 20:00:16]

Patchset #1 (id:40001) has been deleted

[Ken Rockot(use gerrit already), 2017-03-14 20:00:33]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[Ken Rockot(use gerrit already), 2017-03-14 20:00:42]

Patchset #1 (id:60001) has been deleted

[commit-bot: I haz the power, 2017-03-14 20:01:05]

Dry run: 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-03-14 20:13:18]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-14 20:13:18]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Ken Rockot(use gerrit already), 2017-03-14 20:38:42]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-14 20:39:33]

Dry run: 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-03-14 20:51:53]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-14 20:51:54]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[Ken Rockot(use gerrit already), 2017-03-15 00:02:00]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-15 00:02:51]

Dry run: 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-03-15 00:13:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-15 00:13:34]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Ken Rockot(use gerrit already), 2017-03-15 20:25:47]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-15 20:26:16]

Dry run: 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-03-15 21:55:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-15 21:55:04]

Dry run: This issue passed the CQ dry run.

[Ken Rockot(use gerrit already), 2017-03-15 22:10:33]

Description was changed from

==========
Introduce Service Manager embedder API

Introduces a basic embedder API for the Service Manager, consisting
of a single entry point (service_manager::Main) which in turn delegates
initialization logic to the embedder.

Moves some early process environment setup from content into
service_manager.

This is the first of several incremental steps to remove content from
the generic process startup flow.

BUG=654986
==========

to

==========
Move some basic early process init into Service Manager

Introduces the beginnings of an embedder API for the Service Manager,
consisting of a single entry point (service_manager::Main) which in
turn delegates to its embedder for arbitrary initialization and
process execution logic.

This is the first of several incremental steps to remove content from
the generic process startup flow. Future patches will rework various
main entry points to go through service_manager::Main directly, rather
than going ContentMain -> service_manager::Main.

This will also allow us to introduce new process types which run the
Service Manager or arbitrary services directly without touching any
part of content.

BUG=654986
==========

[Ken Rockot(use gerrit already), 2017-03-15 22:11:51]

rockot@chromium.org changed reviewers:
+ jam@chromium.org

[Ken Rockot(use gerrit already), 2017-03-15 22:11:52]

jam@ could you do an overall review pass and specifically stamp for content
ownership?

https://codereview.chromium.org/2613653003/diff/80001/build/sanitizers/lsan_s...
File build/sanitizers/lsan_suppressions.cc (right):

https://codereview.chromium.org/2613653003/diff/80001/build/sanitizers/lsan_s...
build/sanitizers/lsan_suppressions.cc:88:
"leak:service_manager::SetProcessTitleFromCommandLine\n"
this is the only meaningful change in this file. clang-format really wants to
format the file, so I let it.

[Ken Rockot(use gerrit already), 2017-03-15 22:29:29]

Patchset #2 (id:100001) has been deleted

[Ken Rockot(use gerrit already), 2017-03-15 22:29:34]

Patchset #2 (id:120001) has been deleted

[jam, 2017-03-20 15:06:16]

https://codereview.chromium.org/2613653003/diff/140001/content/app/android/co...
File content/app/android/content_main.cc (right):

https://codereview.chromium.org/2613653003/diff/140001/content/app/android/co...
content/app/android/content_main.cc:36: class ContentServiceManagerMainDelegate
: public service_manager::MainDelegate {
why isn't this shared with the version in content/app/content_main.cc?

https://codereview.chromium.org/2613653003/diff/140001/services/service_manag...
File services/service_manager/embedder/main_delegate.h (right):

https://codereview.chromium.org/2613653003/diff/140001/services/service_manag...
services/service_manager/embedder/main_delegate.h:21: class
SERVICE_MANAGER_EMBEDDER_EXPORT MainDelegate {
nit: the export thing isn't needed for non-copyable interfaces afaik

[Ken Rockot(use gerrit already), 2017-03-20 17:18:58]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-20 17:19:26]

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

[Ken Rockot(use gerrit already), 2017-03-20 17:23:38]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[Ken Rockot(use gerrit already), 2017-03-20 17:23:47]

Patchset #3 (id:160001) has been deleted

[commit-bot: I haz the power, 2017-03-20 17:24:21]

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

[Ken Rockot(use gerrit already), 2017-03-20 17:27:06]

https://codereview.chromium.org/2613653003/diff/140001/content/app/android/co...
File content/app/android/content_main.cc (right):

https://codereview.chromium.org/2613653003/diff/140001/content/app/android/co...
content/app/android/content_main.cc:36: class ContentServiceManagerMainDelegate
: public service_manager::MainDelegate {
On 2017/03/20 at 15:06:16, jam wrote:
> why isn't this shared with the version in content/app/content_main.cc?

There were reasons, but in retrospect they weren't great reasons. Done

https://codereview.chromium.org/2613653003/diff/140001/services/service_manag...
File services/service_manager/embedder/main_delegate.h (right):

https://codereview.chromium.org/2613653003/diff/140001/services/service_manag...
services/service_manager/embedder/main_delegate.h:21: class
SERVICE_MANAGER_EMBEDDER_EXPORT MainDelegate {
On 2017/03/20 at 15:06:16, jam wrote:
> nit: the export thing isn't needed for non-copyable interfaces afaik

Done

https://codereview.chromium.org/2613653003/diff/180001/content/app/content_ma...
File content/app/content_main.cc (left):

https://codereview.chromium.org/2613653003/diff/180001/content/app/content_ma...
content/app/content_main.cc:19: base::debug::GlobalActivityTracker* tracker =
nit: This GlobalActivityTracker stuff is new since rebasing. I've moved it into
service_manager::Main as well.

[commit-bot: I haz the power, 2017-03-20 17:44:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-20 17:44:49]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[Ken Rockot(use gerrit already), 2017-03-20 18:33:08]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-20 18:34:07]

Dry run: 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-03-20 19:21:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-20 19:21:13]

Dry run: Try jobs failed on following builders:
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[jam, 2017-03-20 22:23:45]

lgtm

https://codereview.chromium.org/2613653003/diff/200001/content/app/content_se...
File content/app/content_service_manager_main_delegate.h (right):

https://codereview.chromium.org/2613653003/diff/200001/content/app/content_se...
content/app/content_service_manager_main_delegate.h:20: class CONTENT_EXPORT
ContentServiceManagerMainDelegate
are you sure the export is needed? I don't think it is in this case.

[Ken Rockot(use gerrit already), 2017-03-20 22:56:56]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[Ken Rockot(use gerrit already), 2017-03-20 22:57:31]

https://codereview.chromium.org/2613653003/diff/200001/content/app/content_se...
File content/app/content_service_manager_main_delegate.h (right):

https://codereview.chromium.org/2613653003/diff/200001/content/app/content_se...
content/app/content_service_manager_main_delegate.h:20: class CONTENT_EXPORT
ContentServiceManagerMainDelegate
On 2017/03/20 at 22:23:45, jam wrote:
> are you sure the export is needed? I don't think it is in this case.

Shouldn't be, removed

[commit-bot: I haz the power, 2017-03-20 22:58:08]

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

[Ken Rockot(use gerrit already), 2017-03-21 00:17:15]

The CQ bit was unchecked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2017-03-21 00:17:17]

The CQ bit was checked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2017-03-21 00:17:17]

The patchset sent to the CQ was uploaded after l-g-t-m from jam@chromium.org
Link to the patchset: https://codereview.chromium.org/2613653003/#ps220001
(title: ".")

[commit-bot: I haz the power, 2017-03-21 00:17:50]

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-03-21 00:29:39]

CQ is committing da patch.

Bot data: {"patchset_id": 220001, "attempt_start_ts": 1490055437472900,
"parent_rev": "28865543d686d93a737842b56b623f019af8ddac", "commit_rev":
"c6026704ff452d402924ce2d88b0168baf227b1e"}

[commit-bot: I haz the power, 2017-03-21 00:30:28]

Description was changed from

==========
Move some basic early process init into Service Manager

Introduces the beginnings of an embedder API for the Service Manager,
consisting of a single entry point (service_manager::Main) which in
turn delegates to its embedder for arbitrary initialization and
process execution logic.

This is the first of several incremental steps to remove content from
the generic process startup flow. Future patches will rework various
main entry points to go through service_manager::Main directly, rather
than going ContentMain -> service_manager::Main.

This will also allow us to introduce new process types which run the
Service Manager or arbitrary services directly without touching any
part of content.

BUG=654986
==========

to

==========
Move some basic early process init into Service Manager

Introduces the beginnings of an embedder API for the Service Manager,
consisting of a single entry point (service_manager::Main) which in
turn delegates to its embedder for arbitrary initialization and
process execution logic.

This is the first of several incremental steps to remove content from
the generic process startup flow. Future patches will rework various
main entry points to go through service_manager::Main directly, rather
than going ContentMain -> service_manager::Main.

This will also allow us to introduce new process types which run the
Service Manager or arbitrary services directly without touching any
part of content.

BUG=654986

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

[commit-bot: I haz the power, 2017-03-21 00:30:30]

Committed patchset #5 (id:220001) as
https://chromium.googlesource.com/chromium/src/+/c6026704ff452d402924ce2d88b0...

[horo, 2017-03-21 02:04:34]

A revert of this CL (patchset #5 id:220001) has been created in
https://codereview.chromium.org/2763883002/ by horo@chromium.org.

The reason for reverting is: Introduced crashes of webkit_tests on Mac.

BUG=703465

.

[Ken Rockot(use gerrit already), 2017-03-21 02:07:35]

On 2017/03/21 at 02:04:34, horo wrote:
> A revert of this CL (patchset #5 id:220001) has been created in
https://codereview.chromium.org/2763883002/ by horo@chromium.org.
> 
> The reason for reverting is: Introduced crashes of webkit_tests on Mac.
> 
> BUG=703465
> 
> .

Do you have any links to these crash logs?

[horo, 2017-03-21 02:09:00]

On 2017/03/21 02:07:35, Ken Rockot wrote:
> On 2017/03/21 at 02:04:34, horo wrote:
> > A revert of this CL (patchset #5 id:220001) has been created in
> https://codereview.chromium.org/2763883002/ by mailto:horo@chromium.org.
> > 
> > The reason for reverting is: Introduced crashes of webkit_tests on Mac.
> > 
> > BUG=703465
> > 
> > .
> 
> Do you have any links to these crash logs?

See https://bugs.chromium.org/p/chromium/issues/detail?id=703465

I verified the crash on my local Mac.

[Ken Rockot(use gerrit already), 2017-03-21 05:10:47]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-21 05:11:01]

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

[Ken Rockot(use gerrit already), 2017-03-21 05:11:07]

On 2017/03/21 at 02:09:00, horo wrote:
> On 2017/03/21 02:07:35, Ken Rockot wrote:
> > On 2017/03/21 at 02:04:34, horo wrote:
> > > A revert of this CL (patchset #5 id:220001) has been created in
> > https://codereview.chromium.org/2763883002/ by mailto:horo@chromium.org.
> > > 
> > > The reason for reverting is: Introduced crashes of webkit_tests on Mac.
> > > 
> > > BUG=703465
> > > 
> > > .
> > 
> > Do you have any links to these crash logs?
> 
> See https://bugs.chromium.org/p/chromium/issues/detail?id=703465
> 
> I verified the crash on my local Mac.

Thanks, I see the problem now.

[Ken Rockot(use gerrit already), 2017-03-21 06:21:59]

The CQ bit was unchecked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2017-03-21 06:22:01]

The CQ bit was checked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2017-03-21 06:22:02]

The patchset sent to the CQ was uploaded after l-g-t-m from jam@chromium.org
Link to the patchset: https://codereview.chromium.org/2613653003/#ps240001
(title: "fix mac shutdown order")

[commit-bot: I haz the power, 2017-03-21 06:22:05]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-21 06:22:06]

Dry run: This issue passed the CQ dry run.

[Ken Rockot(use gerrit already), 2017-03-21 06:24:17]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2017-03-21 06:24:37]

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-03-21 06:35:28]

CQ is committing da patch.

Bot data: {"patchset_id": 240001, "attempt_start_ts": 1490077457809110,
"parent_rev": "1d2d74238c94507cfd774ccb3b50626c9f89c9cf", "commit_rev":
"029f9feabaf10170db19efc4dc2966a466d38739"}

[commit-bot: I haz the power, 2017-03-21 06:40:03]

CQ is committing da patch.

Bot data: {"patchset_id": 240001, "attempt_start_ts": 1490077457809110,
"parent_rev": "fa791ce4bcf3885b3325406ccbbdc6e974d2357b", "commit_rev":
"01e7ae7b68e252a6d2f1a1fbb2cea8b487f7241d"}

[commit-bot: I haz the power, 2017-03-21 06:44:47]

CQ is committing da patch.

Bot data: {"patchset_id": 240001, "attempt_start_ts": 1490077457809110,
"parent_rev": "afa8fb380609d492b233088453c6a601f4c14762", "commit_rev":
"60d92c2f3ff60c2fcc0b3cddd05738403fb1f0c3"}

[commit-bot: I haz the power, 2017-03-21 06:45:43]

Description was changed from

==========
Move some basic early process init into Service Manager

Introduces the beginnings of an embedder API for the Service Manager,
consisting of a single entry point (service_manager::Main) which in
turn delegates to its embedder for arbitrary initialization and
process execution logic.

This is the first of several incremental steps to remove content from
the generic process startup flow. Future patches will rework various
main entry points to go through service_manager::Main directly, rather
than going ContentMain -> service_manager::Main.

This will also allow us to introduce new process types which run the
Service Manager or arbitrary services directly without touching any
part of content.

BUG=654986

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

to

==========
Move some basic early process init into Service Manager

Introduces the beginnings of an embedder API for the Service Manager,
consisting of a single entry point (service_manager::Main) which in
turn delegates to its embedder for arbitrary initialization and
process execution logic.

This is the first of several incremental steps to remove content from
the generic process startup flow. Future patches will rework various
main entry points to go through service_manager::Main directly, rather
than going ContentMain -> service_manager::Main.

This will also allow us to introduce new process types which run the
Service Manager or arbitrary services directly without touching any
part of content.

BUG=654986

Review-Url: https://codereview.chromium.org/2613653003
Cr-Original-Commit-Position: refs/heads/master@{#458252}
Committed:
https://chromium.googlesource.com/chromium/src/+/c6026704ff452d402924ce2d88b0...
Review-Url: https://codereview.chromium.org/2613653003
Cr-Commit-Position: refs/heads/master@{#458331}
Committed:
https://chromium.googlesource.com/chromium/src/+/60d92c2f3ff60c2fcc0b3cddd057...
==========

[commit-bot: I haz the power, 2017-03-21 06:45:45]

Committed patchset #6 (id:240001) as
https://chromium.googlesource.com/chromium/src/+/60d92c2f3ff60c2fcc0b3cddd057...