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

Unified Diff: webrtc/sdk/android/src/jni/androidhistogram_jni.cc

Issue 2854123003: Build WebRTC with data channel only. (Closed)
Patch Set: CR comments. Created 3 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: webrtc/sdk/android/src/jni/androidhistogram_jni.cc
diff --git a/webrtc/sdk/android/src/jni/androidhistogram_jni.cc b/webrtc/sdk/android/src/jni/androidhistogram_jni.cc
index 74ddcfbb58c596ceff97dfa9167c13845f64e70a..b3c05960857e64797f654fcb06b937b70c59d08b 100644
--- a/webrtc/sdk/android/src/jni/androidhistogram_jni.cc
+++ b/webrtc/sdk/android/src/jni/androidhistogram_jni.cc
@@ -13,7 +13,11 @@
#include "webrtc/sdk/android/src/jni/classreferenceholder.h"
#include "webrtc/sdk/android/src/jni/jni_helpers.h"
-#include "webrtc/sdk/android/src/jni/native_handle_impl.h"
+
+#if defined(HAVE_WEBRTC_VOICE) && defined(HAVE_WEBRTC_VIDEO)
+#include "webrtc/sdk/android/src/jni/native_handle_impl.h" // nogncheck
sakal 2017/05/11 11:06:12 I don't see why this should be here at all. Maybe
Zhi Huang 2017/05/12 20:05:33 Because we don't want to depend on "native_handle_
+#endif
+
#include "webrtc/system_wrappers/include/metrics.h"
// Enables collection of native histograms and creating them.

Powered by Google App Engine
This is Rietveld 408576698