Clarifying the parameters ownership of BrowserChildProcessHost::Launch()

Making the ownership of parameters in BrowserChildProcessHost::Launch and
ChildProcessLauncher::Launch() explicit.

BUG=675294
Committed: https://crrev.com/d20cc0bc639b07b29e1e50ae5f321fceb8a9daa7
Cr-Commit-Position: refs/heads/master@{#440333}

[Jay Civelli, 2016-12-17 01:16:02]

Description was changed from

==========
Clarifying the parameters ownership in ChildProcessLauncher::Launch()

BUG=
==========

to

==========
Making the ownership of parameters in BrowserChildProcessHost::Launch and
ChildProcessLauncher::Launch() explicit.

BUG=
==========

[Jay Civelli, 2016-12-17 01:23:18]

Description was changed from

==========
Making the ownership of parameters in BrowserChildProcessHost::Launch and
ChildProcessLauncher::Launch() explicit.

BUG=
==========

to

==========
Making the ownership of parameters in BrowserChildProcessHost::Launch and
ChildProcessLauncher::Launch() explicit.

BUG=675294
==========

[Jay Civelli, 2016-12-17 01:23:27]

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

[commit-bot: I haz the power, 2016-12-17 01:24:23]

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

[Jay Civelli, 2016-12-17 01:26:52]

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

[commit-bot: I haz the power, 2016-12-17 01:58:47]

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

[commit-bot: I haz the power, 2016-12-17 01:58:48]

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...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Jay Civelli, 2016-12-19 18:10:48]

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

[commit-bot: I haz the power, 2016-12-19 18:11: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, 2016-12-19 18:54:47]

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

[commit-bot: I haz the power, 2016-12-19 18:54:47]

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

[jam, 2016-12-19 19:06:24]

lgtm

[Jay Civelli, 2016-12-19 19:16:27]

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

[commit-bot: I haz the power, 2016-12-19 19:17:03]

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, 2016-12-19 19:57:33]

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

[commit-bot: I haz the power, 2016-12-19 19:57: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_...)

[Jay Civelli, 2016-12-19 20:21:28]

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

[commit-bot: I haz the power, 2016-12-19 20:22:11]

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, 2016-12-19 20:59:55]

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

[commit-bot: I haz the power, 2016-12-19 20:59:55]

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

[Jay Civelli, 2016-12-19 21:55:25]

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

[commit-bot: I haz the power, 2016-12-19 21:56:02]

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, 2016-12-19 22:39:31]

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

[commit-bot: I haz the power, 2016-12-19 22:39:32]

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...)
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Jay Civelli, 2016-12-19 22:50:17]

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

[commit-bot: I haz the power, 2016-12-19 22:50:54]

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, 2016-12-20 00:52:27]

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

[commit-bot: I haz the power, 2016-12-20 00:52:28]

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

[Jay Civelli, 2016-12-22 02:06:59]

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

[commit-bot: I haz the power, 2016-12-22 02:07:11]

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, 2016-12-22 03:40:55]

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

[commit-bot: I haz the power, 2016-12-22 03:40:56]

Dry run: This issue passed the CQ dry run.

[Jay Civelli, 2016-12-22 03:46:17]

The CQ bit was checked by jcivelli@chromium.org

[Jay Civelli, 2016-12-22 03:46:18]

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/2585963002/#ps120001
(title: "Fixed crasher on Windows.")

[commit-bot: I haz the power, 2016-12-22 03:46:30]

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, 2016-12-22 03:51:45]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1482378377949830,
"parent_rev": "a7916885f13b3d9336f3195e970117e5d7622b82", "commit_rev":
"3f7ea6540b3c405702c25b0f8442519c261f84c5"}

[commit-bot: I haz the power, 2016-12-22 03:52:26]

Description was changed from

==========
Making the ownership of parameters in BrowserChildProcessHost::Launch and
ChildProcessLauncher::Launch() explicit.

BUG=675294
==========

to

==========
Making the ownership of parameters in BrowserChildProcessHost::Launch and
ChildProcessLauncher::Launch() explicit.

BUG=675294

Review-Url: https://codereview.chromium.org/2585963002
==========

[commit-bot: I haz the power, 2016-12-22 03:52:27]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2016-12-22 03:53:54]

Description was changed from

==========
Making the ownership of parameters in BrowserChildProcessHost::Launch and
ChildProcessLauncher::Launch() explicit.

BUG=675294

Review-Url: https://codereview.chromium.org/2585963002
==========

to

==========
Making the ownership of parameters in BrowserChildProcessHost::Launch and
ChildProcessLauncher::Launch() explicit.

BUG=675294

Committed: https://crrev.com/d20cc0bc639b07b29e1e50ae5f321fceb8a9daa7
Cr-Commit-Position: refs/heads/master@{#440333}
==========

[commit-bot: I haz the power, 2016-12-22 03:53:55]

Patchset 7 (id:??) landed as
https://crrev.com/d20cc0bc639b07b29e1e50ae5f321fceb8a9daa7
Cr-Commit-Position: refs/heads/master@{#440333}