Change ServiceManagerConnectionImpl to run service request handlers on the IO thread.

Change ServiceManagerConnectionImpl to run service request handlers on the IO thread.

Currently, ServiceManagerConnectionImpl posts to the main thread to run
service request handlers. This prevents services from being launched
before the main message loop starts. The pref service needs to launch
before the main message loop starts. This CL fixes this by running
service request handlers directly on the IO thread.

BUG=654988
Review-Url: https://codereview.chromium.org/2729733003
Cr-Original-Commit-Position: refs/heads/master@{#455295}
Committed: https://chromium.googlesource.com/chromium/src/+/072bd1f5e01408854503b640c9ad62216d9b6b76
Review-Url: https://codereview.chromium.org/2729733003
Cr-Commit-Position: refs/heads/master@{#456316}
Committed: https://chromium.googlesource.com/chromium/src/+/030f501c3474280e3a0571e8ea71d1c7a1cfed28

[Sam McNally, 2017-03-02 23:29:15]

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

[commit-bot: I haz the power, 2017-03-02 23:29:39]

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

[Sam McNally, 2017-03-02 23:45:26]

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

[commit-bot: I haz the power, 2017-03-02 23:46:09]

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-03 01:15:08]

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

[commit-bot: I haz the power, 2017-03-03 01:15:09]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Sam McNally, 2017-03-03 01:40:33]

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

[commit-bot: I haz the power, 2017-03-03 01:41:08]

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-03 03:15:41]

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

[commit-bot: I haz the power, 2017-03-03 03:15:42]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Sam McNally, 2017-03-03 03:33:08]

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

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

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-03 04:50:20]

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

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

Dry run: This issue passed the CQ dry run.

[Sam McNally, 2017-03-03 05:06:10]

Patchset #3 (id:40001) has been deleted

[Sam McNally, 2017-03-03 05:06:13]

Patchset #2 (id:20001) has been deleted

[Sam McNally, 2017-03-03 05:06:17]

Patchset #1 (id:1) has been deleted

[Sam McNally, 2017-03-03 05:12:35]

Description was changed from

==========
Change ServiceManagerConnectionImpl to run service request handlers on the IO
thread.

Currently, ServiceManagerConnectionImpl posts to the main thread to run
service request handlers. This prevents services from being launched
before the main message loop starts. Prefs needs to launch before the
main message loop starts. This CL fixes this by running service request
handlers directly on the IO thread.

BUG=654988
==========

to

==========
Change ServiceManagerConnectionImpl to run service request handlers on the IO
thread.

Currently, ServiceManagerConnectionImpl posts to the main thread to run
service request handlers. This prevents services from being launched
before the main message loop starts. The pref service needs to launch
before the main message loop starts. This CL fixes this by running
service request handlers directly on the IO thread.

BUG=654988
==========

[Sam McNally, 2017-03-03 05:18:58]

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

[Ken Rockot(use gerrit already), 2017-03-03 17:40:49]

lgtm

[Sam McNally, 2017-03-05 23:22:26]

sammc@chromium.org changed reviewers:
+ jochen@chromium.org

[Sam McNally, 2017-03-05 23:22:27]

+jochen for //content/browser/browser_context.cc

[jochen (gone - plz use gerrit), 2017-03-06 12:57:02]

can you add a test please?

[Sam McNally, 2017-03-06 23:52:41]

Patchset #2 (id:80001) has been deleted

[Sam McNally, 2017-03-06 23:55:21]

Patchset #3 (id:120001) has been deleted

[Sam McNally, 2017-03-06 23:55:28]

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

[commit-bot: I haz the power, 2017-03-06 23:56:01]

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

[Sam McNally, 2017-03-07 01:53:47]

On 2017/03/06 12:57:02, jochen wrote:
> can you add a test please?

Done.

[commit-bot: I haz the power, 2017-03-07 03:03:42]

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

[commit-bot: I haz the power, 2017-03-07 03:03:43]

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_...)

[jochen (gone - plz use gerrit), 2017-03-07 11:50:51]

lgtm

[Sam McNally, 2017-03-07 22:29:22]

The CQ bit was checked by sammc@chromium.org

[Sam McNally, 2017-03-07 22:29:22]

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

[commit-bot: I haz the power, 2017-03-07 22:30:42]

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-07 23:50:25]

CQ is committing da patch.

Bot data: {"patchset_id": 140001, "attempt_start_ts": 1488925762057960,
"parent_rev": "fec2991e99c468610b90b26f1aa017cfe2834af8", "commit_rev":
"072bd1f5e01408854503b640c9ad62216d9b6b76"}

[commit-bot: I haz the power, 2017-03-07 23:51:36]

Description was changed from

==========
Change ServiceManagerConnectionImpl to run service request handlers on the IO
thread.

Currently, ServiceManagerConnectionImpl posts to the main thread to run
service request handlers. This prevents services from being launched
before the main message loop starts. The pref service needs to launch
before the main message loop starts. This CL fixes this by running
service request handlers directly on the IO thread.

BUG=654988
==========

to

==========
Change ServiceManagerConnectionImpl to run service request handlers on the IO
thread.

Currently, ServiceManagerConnectionImpl posts to the main thread to run
service request handlers. This prevents services from being launched
before the main message loop starts. The pref service needs to launch
before the main message loop starts. This CL fixes this by running
service request handlers directly on the IO thread.

BUG=654988

Review-Url: https://codereview.chromium.org/2729733003
Cr-Commit-Position: refs/heads/master@{#455295}
Committed:
https://chromium.googlesource.com/chromium/src/+/072bd1f5e01408854503b640c9ad...
==========

[commit-bot: I haz the power, 2017-03-07 23:51:38]

Committed patchset #3 (id:140001) as
https://chromium.googlesource.com/chromium/src/+/072bd1f5e01408854503b640c9ad...

[Sam McNally, 2017-03-13 00:12:42]

Description was changed from

==========
Change ServiceManagerConnectionImpl to run service request handlers on the IO
thread.

Currently, ServiceManagerConnectionImpl posts to the main thread to run
service request handlers. This prevents services from being launched
before the main message loop starts. The pref service needs to launch
before the main message loop starts. This CL fixes this by running
service request handlers directly on the IO thread.

BUG=654988

Review-Url: https://codereview.chromium.org/2729733003
Cr-Commit-Position: refs/heads/master@{#455295}
Committed:
https://chromium.googlesource.com/chromium/src/+/072bd1f5e01408854503b640c9ad...
==========

to

==========
Change ServiceManagerConnectionImpl to run service request handlers on the IO
thread.

Currently, ServiceManagerConnectionImpl posts to the main thread to run
service request handlers. This prevents services from being launched
before the main message loop starts. The pref service needs to launch
before the main message loop starts. This CL fixes this by running
service request handlers directly on the IO thread.

BUG=654988

Review-Url: https://codereview.chromium.org/2729733003
Cr-Commit-Position: refs/heads/master@{#455295}
Committed:
https://chromium.googlesource.com/chromium/src/+/072bd1f5e01408854503b640c9ad...
==========

[Sam McNally, 2017-03-13 00:15:04]

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

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

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

[Sam McNally, 2017-03-13 00:17:52]

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

[commit-bot: I haz the power, 2017-03-13 00:18:06]

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

[Sam McNally, 2017-03-13 00:45:15]

Patchset #4 (id:160001) has been deleted

[commit-bot: I haz the power, 2017-03-13 01:07:11]

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

[commit-bot: I haz the power, 2017-03-13 01:07:12]

Dry run: This issue passed the CQ dry run.

[Sam McNally, 2017-03-13 02:24:45]

The CQ bit was checked by sammc@chromium.org

[Sam McNally, 2017-03-13 02:24:45]

The patchset sent to the CQ was uploaded after l-g-t-m from rockot@chromium.org,
jochen@chromium.org
Link to the patchset: https://codereview.chromium.org/2729733003/#ps200001
(title: "Fix startup race.")

[commit-bot: I haz the power, 2017-03-13 02:24:53]

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-13 02:28:49]

CQ is committing da patch.

Bot data: {"patchset_id": 200001, "attempt_start_ts": 1489371885059290,
"parent_rev": "fd92641a50fb746ad1cee17fc3bcbcdb849fd9ee", "commit_rev":
"030f501c3474280e3a0571e8ea71d1c7a1cfed28"}

[commit-bot: I haz the power, 2017-03-13 02:29:35]

Description was changed from

==========
Change ServiceManagerConnectionImpl to run service request handlers on the IO
thread.

Currently, ServiceManagerConnectionImpl posts to the main thread to run
service request handlers. This prevents services from being launched
before the main message loop starts. The pref service needs to launch
before the main message loop starts. This CL fixes this by running
service request handlers directly on the IO thread.

BUG=654988

Review-Url: https://codereview.chromium.org/2729733003
Cr-Commit-Position: refs/heads/master@{#455295}
Committed:
https://chromium.googlesource.com/chromium/src/+/072bd1f5e01408854503b640c9ad...
==========

to

==========
Change ServiceManagerConnectionImpl to run service request handlers on the IO
thread.

Currently, ServiceManagerConnectionImpl posts to the main thread to run
service request handlers. This prevents services from being launched
before the main message loop starts. The pref service needs to launch
before the main message loop starts. This CL fixes this by running
service request handlers directly on the IO thread.

BUG=654988

Review-Url: https://codereview.chromium.org/2729733003
Cr-Original-Commit-Position: refs/heads/master@{#455295}
Committed:
https://chromium.googlesource.com/chromium/src/+/072bd1f5e01408854503b640c9ad...
Review-Url: https://codereview.chromium.org/2729733003
Cr-Commit-Position: refs/heads/master@{#456316}
Committed:
https://chromium.googlesource.com/chromium/src/+/030f501c3474280e3a0571e8ea71...
==========

[commit-bot: I haz the power, 2017-03-13 02:29:36]

Committed patchset #5 (id:200001) as
https://chromium.googlesource.com/chromium/src/+/030f501c3474280e3a0571e8ea71...