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

Unified Diff: content/app/content_main_runner.cc

Issue 1137453002: content: Pass IOSurface references using Mach IPC. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rsesek's review Created 5 years, 7 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: content/app/content_main_runner.cc
diff --git a/content/app/content_main_runner.cc b/content/app/content_main_runner.cc
index 279536b09b983deec9ae5f22856e5c0bbb72f83a..a7faac2d8560ef810580abca45fe6f2eb2a7866c 100644
--- a/content/app/content_main_runner.cc
+++ b/content/app/content_main_runner.cc
@@ -86,7 +86,9 @@
#if !defined(OS_IOS)
#include "base/power_monitor/power_monitor_device_source.h"
#include "content/app/mac/mac_init.h"
+#include "content/browser/browser_io_surface_manager_mac.h"
#include "content/browser/mach_broker_mac.h"
+#include "content/child/child_io_surface_manager_mac.h"
#include "content/common/sandbox_init_mac.h"
#endif // !OS_IOS
#endif // OS_WIN
@@ -664,6 +666,18 @@ class ContentMainRunnerImpl : public ContentMainRunner {
(!delegate_ || delegate_->ShouldSendMachPort(process_type))) {
MachBroker::ChildSendTaskPortToParent();
}
+
+ if (!command_line.HasSwitch(switches::kSingleProcess) &&
+ !process_type.empty() && (process_type == switches::kRendererProcess ||
+ process_type == switches::kGpuProcess)) {
+ base::mac::ScopedMachSendRight service_port =
+ BrowserIOSurfaceManager::LookupServicePort(getppid());
+ if (service_port.is_valid()) {
+ ChildIOSurfaceManager::GetInstance()->set_service_port(
+ service_port.release());
reveman 2015/05/27 05:03:18 Note: this could be a lot cleaner if move semantic
Robert Sesek 2015/05/28 18:25:11 A lot of places use the implicit |operator mach_po
+ IOSurfaceManager::SetInstance(ChildIOSurfaceManager::GetInstance());
+ }
+ }
#elif defined(OS_WIN)
SetupCRT(command_line);
#endif
« no previous file with comments | « no previous file | content/browser/browser_io_surface_manager_mac.h » ('j') | content/browser/browser_io_surface_manager_mac.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698