sel_main_chrome: Remove now-unneeded call to NaClSecureCommandChannel()

sel_main_chrome: Remove now-unneeded call to NaClSecureCommandChannel()

Chromium no longer connects to the trusted SRPC command channel (since
https://codereview.chromium.org/1153293003 was committed), so we can
remove the call that spawns the thread that listens on this channel.

BUG=https://code.google.com/p/chromium/issues/detail?id=496287
TEST=Manually ran some of Chromium's browser_tests

Committed: https://chromium.googlesource.com/native_client/src/native_client/+/495bd47427a144f90c2340890c265bb39eccca12

[Mark Seaborn, 2015-08-27 00:15:28]

mseaborn@chromium.org changed reviewers:
+ dschuff@chromium.org

[Derek Schuff, 2015-08-27 06:26:19]

LGTM, although I don't really have familiarity with this code. IIUC, this
doesn't specifically have to do with the translator's (untrusted) use of SRPC,
other than removing that has now unblocked removing SRPC everywhere?

[Mark Seaborn, 2015-08-27 20:53:26]

On 26 August 2015 at 23:26, <dschuff@chromium.org> wrote:

> LGTM, although I don't really have familiarity with this code. IIUC, this
> doesn't specifically have to do with the translator's (untrusted) use of
> SRPC,
> other than removing that has now unblocked removing SRPC everywhere?


This isn't related to the PNaCl translator.  It was enabled by this change:
https://codereview.chromium.org/1153293003

We haven't removed Chromium's use of SRPC for the PNaCl translator yet.

Cheers,
Mark

-- 
You received this message because you are subscribed to the Google Groups
"Native-Client-Reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to native-client-reviews+unsubscribe@googlegroups.com.
To post to this group, send email to native-client-reviews@googlegroups.com.
Visit this group at http://groups.google.com/group/native-client-reviews.
For more options, visit https://groups.google.com/d/optout.

[Mark Seaborn, 2015-08-27 20:55:02]

The CQ bit was checked by mseaborn@chromium.org

[commit-bot: I haz the power, 2015-08-27 20:55:08]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1245273006/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1245273006/1

[commit-bot: I haz the power, 2015-08-27 22:56:33]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/native_client/src/native_client/+/495bd4742...