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

Unified Diff: webrtc/sdk/android/src/jni/pc/statsobserver_jni.cc

Issue 3009613002: Android: Replace webrtc_jni namespace with nested jni namespace (Closed)
Patch Set: Rebase Created 3 years, 4 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 | « webrtc/sdk/android/src/jni/pc/statsobserver_jni.h ('k') | webrtc/sdk/android/src/jni/pc/video_jni.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/sdk/android/src/jni/pc/statsobserver_jni.cc
diff --git a/webrtc/sdk/android/src/jni/pc/statsobserver_jni.cc b/webrtc/sdk/android/src/jni/pc/statsobserver_jni.cc
index aeb4a44ef2bfa146e5552fae790e132278b15183..f72d1ae901592c6cc7c6228ff1b2f8cd143bec75 100644
--- a/webrtc/sdk/android/src/jni/pc/statsobserver_jni.cc
+++ b/webrtc/sdk/android/src/jni/pc/statsobserver_jni.cc
@@ -12,7 +12,8 @@
#include "webrtc/sdk/android/src/jni/classreferenceholder.h"
-namespace webrtc_jni {
+namespace webrtc {
+namespace jni {
// Convenience, used since callbacks occur on the signaling thread, which may
// be a non-Java thread.
@@ -35,7 +36,7 @@ StatsObserverJni::StatsObserverJni(JNIEnv* jni, jobject j_observer)
"<init>",
"(Ljava/lang/String;Ljava/lang/String;)V")) {}
-void StatsObserverJni::OnComplete(const webrtc::StatsReports& reports) {
+void StatsObserverJni::OnComplete(const StatsReports& reports) {
ScopedLocalRefFrame local_ref_frame(jni());
jobjectArray j_reports = ReportsToJava(jni(), reports);
jmethodID m = GetMethodID(jni(), *j_observer_class_, "onComplete",
@@ -44,9 +45,8 @@ void StatsObserverJni::OnComplete(const webrtc::StatsReports& reports) {
CHECK_EXCEPTION(jni()) << "error during CallVoidMethod";
}
-jobjectArray StatsObserverJni::ReportsToJava(
- JNIEnv* jni,
- const webrtc::StatsReports& reports) {
+jobjectArray StatsObserverJni::ReportsToJava(JNIEnv* jni,
+ const StatsReports& reports) {
jobjectArray reports_array =
jni->NewObjectArray(reports.size(), *j_stats_report_class_, NULL);
int i = 0;
@@ -63,9 +63,8 @@ jobjectArray StatsObserverJni::ReportsToJava(
return reports_array;
}
-jobjectArray StatsObserverJni::ValuesToJava(
- JNIEnv* jni,
- const webrtc::StatsReport::Values& values) {
+jobjectArray StatsObserverJni::ValuesToJava(JNIEnv* jni,
+ const StatsReport::Values& values) {
jobjectArray j_values =
jni->NewObjectArray(values.size(), *j_value_class_, NULL);
int i = 0;
@@ -82,4 +81,5 @@ jobjectArray StatsObserverJni::ValuesToJava(
return j_values;
}
-} // namespace webrtc_jni
+} // namespace jni
+} // namespace webrtc
« no previous file with comments | « webrtc/sdk/android/src/jni/pc/statsobserver_jni.h ('k') | webrtc/sdk/android/src/jni/pc/video_jni.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698