IPC: Get rid of FileDescriptor usage from FileDescriptorSet and Message

ipc/ipc_message_utils.cc

Index: ipc/ipc_message_utils.cc
diff --git a/ipc/ipc_message_utils.cc b/ipc/ipc_message_utils.cc
index f5543ff71496b26fa6af8a116f278fbeab6d9c20..7870342e994514fd142460af255761b716868db8 100644
--- a/ipc/ipc_message_utils.cc
+++ b/ipc/ipc_message_utils.cc
@@ -462,8 +462,14 @@ void ParamTraits<base::FileDescriptor>::Write(Message* m, const param_type& p) {
   const bool valid = p.fd >= 0;
   WriteParam(m, valid);
 
-  if (valid) {
-    if (!m->WriteFileDescriptor(p))
+  if (!valid)
+    return;
+
+  if (p.auto_close) {
+    if (!m->WriteFile(base::File(p.fd)))
+      NOTREACHED();
+  } else {
+    if (!m->WriteBorrowingFile(p.fd))
       NOTREACHED();
   }
 }
@@ -471,17 +477,21 @@ void ParamTraits<base::FileDescriptor>::Write(Message* m, const param_type& p) {
 bool ParamTraits<base::FileDescriptor>::Read(const Message* m,
                                              PickleIterator* iter,
                                              param_type* r) {
+  *r = base::FileDescriptor();
+
   bool valid;
   if (!ReadParam(m, iter, &valid))
     return false;
 
-  if (!valid) {

[agl, 2014/09/18 01:17:26]

One hopes this change is correct, but it looks reasonable.

[Hajime Morrita, 2014/09/18 01:29:08]

Well, I'll preserve existing code for now and change it in another CL.

-    r->fd = -1;
-    r->auto_close = false;
-    return true;
-  }
+  if (!valid)
+    return false;
 
-  return m->ReadFileDescriptor(iter, r);
+  base::File fd;
+  if (!m->ReadFile(iter, &fd))
+    return false;
+
+  *r = base::FileDescriptor(fd.Pass());
+  return true;
 }
 
 void ParamTraits<base::FileDescriptor>::Log(const param_type& p,