diff options
| author | morrita <morrita@chromium.org> | 2014-10-20 16:53:25 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2014-10-20 23:53:50 +0000 |
| commit | a409ccc3182ae1cb91fe7937a6c60f8d80755436 (patch) | |
| tree | de8d2996aa21b74beefef4d5bc85cfae6ede8bec /components/nacl/browser | |
| parent | 71187e343e2a8005dff92649b3013ba5019f974a (diff) | |
| download | chromium_src-a409ccc3182ae1cb91fe7937a6c60f8d80755436.zip chromium_src-a409ccc3182ae1cb91fe7937a6c60f8d80755436.tar.gz chromium_src-a409ccc3182ae1cb91fe7937a6c60f8d80755436.tar.bz2 | |
Refactoring: Make IPC::Channel::TakeClientFileDescriptor() a ScopedFD.
By using base::ScopedFD, the ownership of the returned FD becomes
clearer.
BUG=415294
R=brettw@chromium.org, jam@chromium.org, yzshen@chromium.org, sehr@chromium.org
TBR=gene@chromium.org
Review URL: https://codereview.chromium.org/621613002
Cr-Commit-Position: refs/heads/master@{#300381}
Diffstat (limited to 'components/nacl/browser')
| -rw-r--r-- | components/nacl/browser/nacl_process_host.cc | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/components/nacl/browser/nacl_process_host.cc b/components/nacl/browser/nacl_process_host.cc index 3b6ddcf..00c1201 100644 --- a/components/nacl/browser/nacl_process_host.cc +++ b/components/nacl/browser/nacl_process_host.cc @@ -179,14 +179,14 @@ class NaClSandboxedProcessLauncherDelegate virtual bool ShouldUseZygote() override { return true; } - virtual int GetIpcFd() override { - return ipc_fd_; + virtual base::ScopedFD TakeIpcFd() override { + return ipc_fd_.Pass(); } #endif // OS_WIN private: #if defined(OS_POSIX) - int ipc_fd_; + base::ScopedFD ipc_fd_; #endif // OS_POSIX }; |