Convert NaCl renderer-loader messages to mojo.

Convert NaCl renderer-loader messages to mojo.

Author: Sam McNally <sammc@chromium.org>;

Reland of https://codereview.chromium.org/2525483002/, with trivial fixes
applied to fix bustage that occurred due to that CL racing with
https://codereview.chromium.org/2589663003/.

BUG=577685
TBR=sammc, dcheng

Committed: https://crrev.com/30c19915c5cc51ebcbb05700cedb0d84e7f60c2a
Cr-Commit-Position: refs/heads/master@{#439783}

[blundell, 2016-12-20 12:18:55]

Description was changed from

==========
Convert NaCl renderer-loader messages to mojo.

Author: Sam McNally <sammc@chromium.org>

Reland of https://codereview.chromium.org/2525483002/, with trivial fixes
applied to fix bustage that occurred due to that CL racing with
https://codereview.chromium.org/2589663003/.

BUG=577685
TBR=sammc
==========

to

==========
Convert NaCl renderer-loader messages to mojo.

Author: Sam McNally <sammc@chromium.org>

Reland of https://codereview.chromium.org/2525483002/, with trivial fixes
applied to fix bustage that occurred due to that CL racing with
https://codereview.chromium.org/2589663003/.

BUG=577685
TBR=sammc, dcheng
==========

[blundell, 2016-12-20 12:18:55]

blundell@chromium.org changed reviewers:
+ dcheng@chromium.org

[blundell, 2016-12-20 12:21:29]

This is just https://codereview.chromium.org/2525483002/ with
s/GetProxy()/MakeRequest() to adjust to my CL, which landed in the interim (the
original CL is PS1 and that trivial fix is PS2).

[blundell, 2016-12-20 12:21:37]

The CQ bit was checked by blundell@chromium.org

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

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

[Sam McNally, 2016-12-20 12:25:34]

sammc@chromium.org changed reviewers:
+ sammc@chromium.org
- sammc@google.com

[commit-bot: I haz the power, 2016-12-20 13:21:09]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1482236497124390,
"parent_rev": "f46a8400e51528bbeaf74d7c6b67850bbdf0488f", "commit_rev":
"ee3b2e1e1de2c60fe230292373c27a85ac3330dc"}

[commit-bot: I haz the power, 2016-12-20 13:21:36]

Description was changed from

==========
Convert NaCl renderer-loader messages to mojo.

Author: Sam McNally <sammc@chromium.org>

Reland of https://codereview.chromium.org/2525483002/, with trivial fixes
applied to fix bustage that occurred due to that CL racing with
https://codereview.chromium.org/2589663003/.

BUG=577685
TBR=sammc, dcheng
==========

to

==========
Convert NaCl renderer-loader messages to mojo.

Author: Sam McNally <sammc@chromium.org>

Reland of https://codereview.chromium.org/2525483002/, with trivial fixes
applied to fix bustage that occurred due to that CL racing with
https://codereview.chromium.org/2589663003/.

BUG=577685
TBR=sammc, dcheng

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

[commit-bot: I haz the power, 2016-12-20 13:21:37]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-20 13:24:22]

Description was changed from

==========
Convert NaCl renderer-loader messages to mojo.

Author: Sam McNally <sammc@chromium.org>

Reland of https://codereview.chromium.org/2525483002/, with trivial fixes
applied to fix bustage that occurred due to that CL racing with
https://codereview.chromium.org/2589663003/.

BUG=577685
TBR=sammc, dcheng

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

to

==========
Convert NaCl renderer-loader messages to mojo.

Author: Sam McNally <sammc@chromium.org>

Reland of https://codereview.chromium.org/2525483002/, with trivial fixes
applied to fix bustage that occurred due to that CL racing with
https://codereview.chromium.org/2589663003/.

BUG=577685
TBR=sammc, dcheng

Committed: https://crrev.com/30c19915c5cc51ebcbb05700cedb0d84e7f60c2a
Cr-Commit-Position: refs/heads/master@{#439783}
==========

[commit-bot: I haz the power, 2016-12-20 13:24:24]

Patchset 2 (id:??) landed as
https://crrev.com/30c19915c5cc51ebcbb05700cedb0d84e7f60c2a
Cr-Commit-Position: refs/heads/master@{#439783}

[dcheng, 2016-12-20 17:33:35]

still lgtm

[Sam McNally, 2016-12-20 23:15:52]

LGTM. Thanks for relanding.

[brucedawson, 2016-12-21 01:12:28]

brucedawson@chromium.org changed reviewers:
+ brucedawson@chromium.org

[brucedawson, 2016-12-21 01:12:29]

https://codereview.chromium.org/2590173002/diff/20001/components/nacl/loader/...
File components/nacl/loader/nacl_trusted_listener.cc (right):

https://codereview.chromium.org/2590173002/diff/20001/components/nacl/loader/...
components/nacl/loader/nacl_trusted_listener.cc:25: NaClExit(0);
VC++ hates having a call to exit() in a destructor, and the PGO builders are
hitting this. I think it's a silly warning so I'm suppressing it.

FYI.