Remove src/trusted/manifest_name_service_proxy/ (old open_resource() impl)

Remove src/trusted/manifest_name_service_proxy/ (old open_resource() impl)

This was part of the old SRPC-based implementation of open_resource(),
which is no longer used.

 * This removes the hookup in nacl_secure_service.c where
   "ManifestNameService" was registered.

 * This removes srpc_nameservice_test.c, which checked that it was
   possible to get a "ManifestNameService" connection and initialize
   SRPC on it (but tested it no further than that).

   srpc_nameservice_test.c also contained some dead code: There was a
   "rngdump" method for testing the "SecureRandom" service (which was
   removed earlier), but "rngdump" was never called.

BUG=https://code.google.com/p/nativeclient/issues/detail?id=3864
TEST=Chromium trybots + NaCl trybots
R=jvoung@chromium.org, teravest@chromium.org

Committed: https://src.chromium.org/viewvc/native_client?view=rev&revision=da8dfa4e7c67c2a59861b97c94e28e0cf6ee9cdd

[Mark Seaborn, 2014-12-17 23:42:33]

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

[Mark Seaborn, 2014-12-18 00:07:17]

Looks like I sent this out too soon; sorry.  There are some NaCl-side tests that
depend on this.  Don't review this yet.

[Mark Seaborn, 2015-01-08 21:53:05]

This is ready for commit now.  Please take a look.

[teravest, 2015-01-09 17:34:29]

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

[teravest, 2015-01-09 17:34:29]

lgtm

[jvoung (off chromium), 2015-01-09 17:36:40]

not terribly familiar with this code =), but LGTM

https://codereview.chromium.org/797843003/diff/60001/tests/nameservice/srpc_n...
File tests/nameservice/srpc_nameservice_test.c (left):

https://codereview.chromium.org/797843003/diff/60001/tests/nameservice/srpc_n...
tests/nameservice/srpc_nameservice_test.c:178: "ManifestNameService", O_RDWR,
I see a couple of chrome side tests that also use this "ManifestNameService"
rpc. I assume you'll do something with those tests too?

[Mark Seaborn, 2015-01-10 07:00:36]

https://codereview.chromium.org/797843003/diff/60001/tests/nameservice/srpc_n...
File tests/nameservice/srpc_nameservice_test.c (left):

https://codereview.chromium.org/797843003/diff/60001/tests/nameservice/srpc_n...
tests/nameservice/srpc_nameservice_test.c:178: "ManifestNameService", O_RDWR,
On 2015/01/09 17:36:40, jvoung wrote:
> I see a couple of chrome side tests that also use this "ManifestNameService"
> rpc. I assume you'll do something with those tests too?

Thanks for catching that.  Done: https://codereview.chromium.org/842363002/

[Mark Seaborn, 2015-01-10 07:00:43]

The CQ bit was checked by mseaborn@chromium.org

[commit-bot: I haz the power, 2015-01-10 07:01:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/797843003/60001

[commit-bot: I haz the power, 2015-01-10 07:01:49]

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

[commit-bot: I haz the power, 2015-01-10 07:01:50]

CLs for remote refs other than refs/heads/master must contain NOTRY=true in
order for the CQ to process them

[Mark Seaborn, 2015-01-10 07:09:02]

The CQ bit was checked by mseaborn@chromium.org

[commit-bot: I haz the power, 2015-01-10 07:09:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/797843003/60001

[commit-bot: I haz the power, 2015-01-10 07:09:52]

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

[commit-bot: I haz the power, 2015-01-10 07:09:53]

Failed to apply the patch.

[Mark Seaborn, 2015-01-11 17:13:15]

Committed patchset #4 (id:60001) manually as
da8dfa4e7c67c2a59861b97c94e28e0cf6ee9cdd (presubmit successful).