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

Unified Diff: content/browser/ppapi_plugin_process_host.cc

Issue 1994013002: Fix "unused variable" warnings. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Sync to latest behavior Created 4 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/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 2bd3d085bbed09d203296165ecbe3ce35be5e81b..0b5e447d09549c35f14ba048b39a96495a18631b 100644
--- a/content/browser/ppapi_plugin_process_host.cc
+++ b/content/browser/ppapi_plugin_process_host.cc
@@ -60,14 +60,16 @@ class PpapiPluginSandboxedProcessLauncherDelegate
PpapiPluginSandboxedProcessLauncherDelegate(bool is_broker,
const PepperPluginInfo& info,
ChildProcessHost* host)
- :
#if defined(OS_WIN)
- info_(info),
-#endif // OS_WIN
-#if defined(OS_POSIX)
- ipc_fd_(host->TakeClientFileDescriptor()),
-#endif // OS_POSIX
- is_broker_(is_broker) {}
+ : info_(info), is_broker_(is_broker) {
+#elif defined(OS_MACOSX) || defined(OS_ANDROID)
+ : ipc_fd_(host->TakeClientFileDescriptor()) {
+#elif defined(OS_POSIX)
+ : ipc_fd_(host->TakeClientFileDescriptor()), is_broker_(is_broker) {
+#else
+ {
+#endif
+ }
~PpapiPluginSandboxedProcessLauncherDelegate() override {}
@@ -140,7 +142,10 @@ class PpapiPluginSandboxedProcessLauncherDelegate
#if defined(OS_POSIX)
base::ScopedFD ipc_fd_;
#endif // OS_POSIX
+#if (defined(OS_POSIX) && !defined(OS_MACOSX) && !defined(OS_ANDROID)) || \
+ defined(OS_WIN)
bool is_broker_;
+#endif
DISALLOW_COPY_AND_ASSIGN(PpapiPluginSandboxedProcessLauncherDelegate);
};

Powered by Google App Engine
This is Rietveld 408576698