Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(8351)

Unified Diff: chrome/renderer/render_process.cc

Issue 39208: POSIX: Rewrite IPC's interaction with FileDescriptor (Closed)
Patch Set: Created 11 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/renderer/render_process.cc
diff --git a/chrome/renderer/render_process.cc b/chrome/renderer/render_process.cc
index 80ec5fb61639f115182f00c4a96d52b3b84902ee..8e7c31b1eb9c569aad53b3831a2e424ba11c4f7e 100644
--- a/chrome/renderer/render_process.cc
+++ b/chrome/renderer/render_process.cc
@@ -194,13 +194,13 @@ TransportDIB* RenderProcess::CreateTransportDIB(size_t size) {
#elif defined(OS_MACOSX) // defined(OS_WIN) || defined(OS_LINUX)
// Mac creates transport DIBs in the browser, so we need to do a sync IPC to
// get one.
- IPC::Maybe<TransportDIB::Handle> mhandle;
- IPC::Message* msg = new ViewHostMsg_AllocTransportDIB(size, &mhandle);
+ TransportDIB::Handle handle;
+ IPC::Message* msg = new ViewHostMsg_AllocTransportDIB(size, &handle);
if (!child_thread()->Send(msg))
return NULL;
- if (!mhandle.valid)
+ if (handle.fd < 0)
return NULL;
- return TransportDIB::Map(mhandle.value);
+ return TransportDIB::Map(handle);
#endif // defined(OS_MACOSX)
}
« chrome/common/ipc_message_utils.h ('K') | « chrome/common/render_messages_internal.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698