content/public/browser/content_browser_client.h - Issue 1185333003: Implement GetSandboxType() on all platforms and implement for all process types.

Unified Diff: content/public/browser/content_browser_client.h

Issue 1185333003: Implement GetSandboxType() on all platforms and implement for all process types. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Created 5 years, 6 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

« content/common/sandbox_win.cc ('K') | « content/content_common.gypi ('k') | content/public/browser/content_browser_client.cc » ('j') | content/public/common/sandboxed_process_launcher_delegate.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: content/public/browser/content_browser_client.h

diff --git a/content/public/browser/content_browser_client.h b/content/public/browser/content_browser_client.h

index 0a97ef27fbaac3e36a95c58c36a55be0f3f7733b..1c750b63187c80e572af866c66b0204025c04e9e 100644

--- a/content/public/browser/content_browser_client.h

+++ b/content/public/browser/content_browser_client.h

@@ -640,6 +640,12 @@ class CONTENT_EXPORT ContentBrowserClient {

// policy.

virtual void PreSpawnRenderer(sandbox::TargetPolicy* policy,

bool* success) {}

+ // Returns the App Container SID for the specified sandboxed process type, or

+ // empty string if this sandboxed process type does not support living inside

+ // an App Container.

+ virtual base::string16 GetAppContainerSidForSandboxType(

nasko 2015/06/19 12:18:19 I don't see an implementation of this interface fo

Will Harris 2015/06/23 16:21:11 There's quite a few subclasses of ContentBrowserCl

nasko 2015/06/24 08:04:08 Indeed, that sounds like the right approach to me.

Will Harris 2015/06/24 11:37:09 Done.

+ int sandbox_type) const;

#endif

#if defined(VIDEO_HOLE)