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

Unified Diff: content/browser/renderer_host/render_message_filter.cc

Issue 2847033003: Get rid of SharedMemory::GiveToProcess. (Closed)
Patch Set: Comments from avi. Created 3 years, 8 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
« no previous file with comments | « base/memory/shared_memory_win.cc ('k') | content/renderer/pepper/ppb_image_data_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/renderer_host/render_message_filter.cc
diff --git a/content/browser/renderer_host/render_message_filter.cc b/content/browser/renderer_host/render_message_filter.cc
index 216d089790e5075ba8da1a59b095a9ca392e5ca4..5f9b4b9fc79825020fbd680c7ee934731fa4bd5e 100644
--- a/content/browser/renderer_host/render_message_filter.cc
+++ b/content/browser/renderer_host/render_message_filter.cc
@@ -240,7 +240,9 @@ void RenderMessageFilter::SendLoadFontReply(IPC::Message* reply,
result->font_data_size = 0;
result->font_id = 0;
} else {
- result->font_data.GiveToProcess(base::GetCurrentProcessHandle(), &handle);
+ handle = result->font_data.handle().Duplicate();
+ result->font_data.Unmap();
+ result->font_data.Close();
}
RenderProcessHostMsg_LoadFont::WriteReplyParams(
reply, result->font_data_size, handle, result->font_id);
« no previous file with comments | « base/memory/shared_memory_win.cc ('k') | content/renderer/pepper/ppb_image_data_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698