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

Unified Diff: content/common/sandbox_mac.mm

Issue 2944623003: Call SetApplicationIsDaemon() in V2 sandbox. (Closed)
Patch Set: Split warmup and initialization into separate phases Created 3 years, 5 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/common/sandbox_mac.mm
diff --git a/content/common/sandbox_mac.mm b/content/common/sandbox_mac.mm
index ec0ea7dcbefaa61ed73038752655dd72e5618121..548e4880d154dafcf35d6eadfb56c6f72f90555e 100644
--- a/content/common/sandbox_mac.mm
+++ b/content/common/sandbox_mac.mm
@@ -41,12 +41,6 @@
#include "ui/base/layout.h"
#include "ui/gl/init/gl_factory.h"
-extern "C" {
-void CGSSetDenyWindowServerConnections(bool);
-void CGSShutdownServerConnections();
-OSStatus SetApplicationIsDaemon(Boolean isDaemon);
-};
-
namespace content {
namespace {
@@ -174,21 +168,6 @@ void Sandbox::SandboxWarmup(int sandbox_type) {
NSColor* color = [NSColor controlTextColor];
[color colorUsingColorSpaceName:NSCalibratedRGBColorSpace];
}
-
- if (sandbox_type == SANDBOX_TYPE_RENDERER) {
- // Now disconnect from WindowServer, after all objects have been warmed up.
- // Shutting down the connection requires connecting to WindowServer,
- // so do this before actually engaging the sandbox. This may cause two log
- // messages to be printed to the system logger on certain OS versions.
- CGSSetDenyWindowServerConnections(true);
- CGSShutdownServerConnections();
-
- // Allow the process to continue without a LaunchServices ASN. The
- // INIT_Process function in HIServices will abort if it cannot connect to
- // launchservicesd to get an ASN. By setting this flag, HIServices skips
- // that.
- SetApplicationIsDaemon(true);
- }
}
// Load the appropriate template for the given sandbox type.

Powered by Google App Engine
This is Rietveld 408576698