Hooking MHTML generation to the browser.

base/platform_file_posix.cc

Index: base/platform_file_posix.cc
diff --git a/base/platform_file_posix.cc b/base/platform_file_posix.cc
index a1388d6c3a426f462b46b57c39e521a6019272e4..3a515e496e95eb09f3b5f0dd610e882e95e052c9 100644
--- a/base/platform_file_posix.cc
+++ b/base/platform_file_posix.cc
@@ -191,4 +191,10 @@ bool GetPlatformFileInfo(PlatformFile file, PlatformFileInfo* info) {
   return true;
 }
 
+PlatformFile GetFileHandleForProcess(PlatformFile file,
+                                     ProcessHandle process,
+                                     bool close_source_handle) {
+  return file;

[piman, 2011/06/13 23:19:42]

from the peanut-gallery: this has a race condition.
Imagine something like:
int fd=open("myfile", O_RDONLY);
int remote_fd = GetFileHandleForProcess(fd, remote_process, false);
channel->Send(new MyMessage(FileDescriptor(remote_fd, false)));
close(fd);

Then, since remote_fd==fd, the close(fd) may close remote_fd before the IPC
message gets sent, in which case the receiving process won't receive a valid fd
(or at least not the one it should)

A good way to ensure no race is to always returns a fd that the caller owns:
close_source_handle ? file : dup(file)

And then the caller is responsible for sending a FileDescriptor(remote_fd,
true), avoiding the race in all cases.

[Jay Civelli, 2011/06/14 00:19:27]

Good point!
I moved that code to ipc_platform_file.cc, and this is what it now does.

[piman, 2011/06/14 00:31:23]

Great, LGTM on that.

+}
+
 }  // namespace base