Network service: start the process as early as possible.

Network service: start the process as early as possible.

That significantly improves the startup performance: if a URL is loaded using
commandline flag, starting the network service process early could reduce the
timeToOnLoad number (from navigation start to onload event) by 0.5 ~ 1 second.

BUG=598073
Review-Url: https://codereview.chromium.org/2854713002
Cr-Commit-Position: refs/heads/master@{#468496}
Committed: https://chromium.googlesource.com/chromium/src/+/7fef43e36006a1871f571f1404b87b642a88f13d

[yzshen1, 2017-05-01 21:35:18]

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

[yzshen1, 2017-05-01 21:35:48]

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

[yzshen1, 2017-05-01 21:35:49]

Hi, John.

Would you please take a look? Thanks!

[yzshen1, 2017-05-01 21:36:01]

Description was changed from

==========
Network service: start the process as early as possible.

That significantly improvesthe startup performance: if a URL is loaded using
commandline flag, starting the network service process early could reduce the
timeToOnLoad number (from navigation start to onload event) by 0.5 ~ 1 second.

BUG=598073
==========

to

==========
Network service: start the process as early as possible.

That significantly improves the startup performance: if a URL is loaded using
commandline flag, starting the network service process early could reduce the
timeToOnLoad number (from navigation start to onload event) by 0.5 ~ 1 second.

BUG=598073
==========

[commit-bot: I haz the power, 2017-05-01 21:36:35]

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-05-01 22:51:47]

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

[commit-bot: I haz the power, 2017-05-01 22:51:49]

Dry run: This issue passed the CQ dry run.

[jam, 2017-05-01 23:46:49]

lgtm for now to get the perf numbers up

Seems like this will be a common pattern, and perhaps we need to describe which
surfaces are started immediately in chrome/content's manifests?

[yzshen1, 2017-05-01 23:50:50]

On 2017/05/01 23:46:49, jam wrote:
> lgtm for now to get the perf numbers up
> 
> Seems like this will be a common pattern, and perhaps we need to describe
which
> surfaces are started immediately in chrome/content's manifests?

Yeah, agreed that it is nice to support describing this in the manifests.

[yzshen1, 2017-05-01 23:50:54]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2017-05-01 23:51:24]

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-05-01 23:57:40]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493682654654380, "parent_rev":
"9fb1c8b9feec6f43c99eca8cecf09a5582c254f7", "commit_rev":
"7fef43e36006a1871f571f1404b87b642a88f13d"}

[commit-bot: I haz the power, 2017-05-01 23:57:47]

Description was changed from

==========
Network service: start the process as early as possible.

That significantly improves the startup performance: if a URL is loaded using
commandline flag, starting the network service process early could reduce the
timeToOnLoad number (from navigation start to onload event) by 0.5 ~ 1 second.

BUG=598073
==========

to

==========
Network service: start the process as early as possible.

That significantly improves the startup performance: if a URL is loaded using
commandline flag, starting the network service process early could reduce the
timeToOnLoad number (from navigation start to onload event) by 0.5 ~ 1 second.

BUG=598073

Review-Url: https://codereview.chromium.org/2854713002
Cr-Commit-Position: refs/heads/master@{#468496}
Committed:
https://chromium.googlesource.com/chromium/src/+/7fef43e36006a1871f571f1404b8...
==========

[commit-bot: I haz the power, 2017-05-01 23:57:48]

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