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

Unified Diff: content/browser/ppapi_plugin_process_host.cc

Issue 1609133002: Change Win32k PPAPI lockdown to use finch params for mime type. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: code review changes Created 4 years, 11 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 | « chrome/browser/chrome_content_browser_client.cc ('k') | content/common/content_switches_internal.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/ppapi_plugin_process_host.cc
diff --git a/content/browser/ppapi_plugin_process_host.cc b/content/browser/ppapi_plugin_process_host.cc
index 766a71390626c82162264baea8000721580708f6..4ff33a6bd681f82e7116df658eaebc14194a8c3c 100644
--- a/content/browser/ppapi_plugin_process_host.cc
+++ b/content/browser/ppapi_plugin_process_host.cc
@@ -89,18 +89,23 @@ class PpapiPluginSandboxedProcessLauncherDelegate
if (result != sandbox::SBOX_ALL_OK)
return false;
+ content::ContentBrowserClient* browser_client =
+ GetContentClient()->browser();
+
#if !defined(NACL_WIN64)
- for (const auto& mime_type : info_.mime_types) {
- if (IsWin32kLockdownEnabledForMimeType(mime_type.mime_type)) {
- if (!AddWin32kLockdownPolicy(policy))
- return false;
- break;
+ if (IsWin32kRendererLockdownEnabled()) {
+ for (const auto& mime_type : info_.mime_types) {
+ if (browser_client->IsWin32kLockdownEnabledForMimeType(
+ mime_type.mime_type)) {
+ if (!AddWin32kLockdownPolicy(policy))
+ return false;
+ break;
+ }
}
}
#endif
const base::string16& sid =
- GetContentClient()->browser()->GetAppContainerSidForSandboxType(
- GetSandboxType());
+ browser_client->GetAppContainerSidForSandboxType(GetSandboxType());
if (!sid.empty())
AddAppContainerPolicy(policy, sid.c_str());
« no previous file with comments | « chrome/browser/chrome_content_browser_client.cc ('k') | content/common/content_switches_internal.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698