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

Unified Diff: content/common/sandbox_linux/bpf_gpu_policy_linux.cc

Issue 2146733003: Remove USE_OZONE in video files (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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_linux/bpf_gpu_policy_linux.cc
diff --git a/content/common/sandbox_linux/bpf_gpu_policy_linux.cc b/content/common/sandbox_linux/bpf_gpu_policy_linux.cc
index d356897d9e0740668884c496307960ca5c91dddb..e1879dac0f2d32c3a446851a357b78ded2d63281 100644
--- a/content/common/sandbox_linux/bpf_gpu_policy_linux.cc
+++ b/content/common/sandbox_linux/bpf_gpu_policy_linux.cc
@@ -78,8 +78,8 @@ inline bool IsArchitectureArm() {
#endif
}
-inline bool IsOzone() {
-#if defined(USE_OZONE)
+inline bool UseV4L2Codec() {
+#if defined(USE_V4L2_CODEC)
return true;
#else
return false;
@@ -343,7 +343,7 @@ void GpuProcessPolicy::InitGpuBrokerProcess(
if (!IsChromeOS()) {
permissions.push_back(
BrokerFilePermission::ReadWriteCreateUnlinkRecursive(kDevShm));
- } else if (IsArchitectureArm() || IsOzone()){
+ } else if (IsArchitectureArm() || UseV4L2Codec()){
wuchengli 2016/07/13 08:23:59 We don't need IsArchitectureArm() here. Right?
henryhsu 2016/07/13 08:43:28 Yes
AddV4L2GpuWhitelist(&permissions);
if (UseLibV4L2()) {
dlopen("/usr/lib/libv4l2.so", RTLD_NOW|RTLD_GLOBAL|RTLD_NODELETE);

Powered by Google App Engine
This is Rietveld 408576698