Implement open_resource in non-SFI mode.

ppapi/nacl_irt/manifest_service.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ppapi/nacl_irt/manifest_service.h"
 
 #include "base/message_loop/message_loop_proxy.h"
 #include "ipc/ipc_channel_handle.h"
 #include "ipc/ipc_channel_proxy.h"
 #include "ipc/ipc_sync_message_filter.h"
 #include "ppapi/proxy/ppapi_messages.h"
 
 namespace ppapi {
 
+const char kFilePrefix[] = "files/";
+
 ManifestService::ManifestService(
     const IPC::ChannelHandle& handle,
     scoped_refptr<base::MessageLoopProxy> io_message_loop,
     base::WaitableEvent* shutdown_event) {
   filter_ = new IPC::SyncMessageFilter(shutdown_event);
   channel_.reset(new IPC::ChannelProxy(handle,
                                        IPC::Channel::MODE_SERVER,
-                                       NULL, // Listener
+                                       NULL,  // Listener
                                        io_message_loop));
   channel_->AddFilter(filter_.get());
 }
 
 ManifestService::~ManifestService() {
 }
 
 void ManifestService::StartupInitializationComplete() {
   filter_->Send(new PpapiHostMsg_StartupInitializationComplete);
 }
 
+bool ManifestService::OpenResource(const char* file, int* fd) {
+  ppapi::proxy::SerializedHandle ipc_fd;
+  if (!filter_->Send(new PpapiHostMsg_OpenResource(
+          std::string(kFilePrefix) + file, &ipc_fd))) {

[dmichael (off chromium), 2014/04/25 21:06:58]

Why this magical files/ prefix? Shouldn't "file" just be a relative URL?

[Mark Seaborn, 2014/04/26 00:18:59]

This is matching existing behaviour, I guess (which is not well-documented). 
Note that this is not a URL.

[hidehiko, 2014/04/28 08:44:27]

Exactly as Mark said, this is just to be compatible with the current NaCl (i.e.
NaCl in SFI-mode).
IMHO, eventually we can clean this up, but at least I'd like to split the task
from this CL.

+    LOG(ERROR) << "ManifestService::OpenResource failed:" << file;
+    *fd = -1;
+    return false;
+  }
+
+  *fd = IPC::PlatformFileForTransitToPlatformFile(ipc_fd.descriptor());
+  return true;
+}
+
 }  // namespace ppapi