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

Unified Diff: webrtc/sdk/android/src/jni/native_handle_impl.h

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/jni_onload.cc ('k') | webrtc/sdk/android/src/jni/native_handle_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/sdk/android/src/jni/native_handle_impl.h
diff --git a/webrtc/sdk/android/src/jni/native_handle_impl.h b/webrtc/sdk/android/src/jni/native_handle_impl.h
index fc674db9d4996691837cb11714f2e5cea80d3a47..3ad1b6a0fa606a4b4e0d57cc90267bece34575ae 100644
--- a/webrtc/sdk/android/src/jni/native_handle_impl.h
+++ b/webrtc/sdk/android/src/jni/native_handle_impl.h
@@ -19,7 +19,8 @@
#include "webrtc/rtc_base/callback.h"
#include "webrtc/sdk/android/src/jni/jni_helpers.h"
-namespace webrtc_jni {
+namespace webrtc {
+namespace jni {
// Open gl texture matrix, in column-major order. Operations are
// in-place.
@@ -37,7 +38,7 @@ class Matrix {
float crop_x,
float crop_y);
- void Rotate(webrtc::VideoRotation rotation);
+ void Rotate(VideoRotation rotation);
private:
Matrix() {}
@@ -62,7 +63,7 @@ struct NativeHandleImpl {
// Java-based frames.
// TODO(sakal): Remove this and AndroidTextureBuffer once they are no longer
// needed.
-class AndroidVideoFrameBuffer : public webrtc::VideoFrameBuffer {
+class AndroidVideoFrameBuffer : public VideoFrameBuffer {
public:
enum class AndroidType { kTextureBuffer, kJavaBuffer };
@@ -85,7 +86,7 @@ class AndroidTextureBuffer : public AndroidVideoFrameBuffer {
int width() const override;
int height() const override;
- rtc::scoped_refptr<webrtc::I420BufferInterface> ToI420() override;
+ rtc::scoped_refptr<I420BufferInterface> ToI420() override;
AndroidType android_type() override { return AndroidType::kTextureBuffer; }
@@ -137,7 +138,7 @@ class AndroidVideoBuffer : public AndroidVideoFrameBuffer {
int width() const override;
int height() const override;
- rtc::scoped_refptr<webrtc::I420BufferInterface> ToI420() override;
+ rtc::scoped_refptr<I420BufferInterface> ToI420() override;
AndroidType android_type() override { return AndroidType::kJavaBuffer; }
@@ -152,9 +153,9 @@ class AndroidVideoBufferFactory {
public:
explicit AndroidVideoBufferFactory(JNIEnv* jni);
- webrtc::VideoFrame CreateFrame(JNIEnv* jni,
- jobject j_video_frame,
- uint32_t timestamp_rtp) const;
+ VideoFrame CreateFrame(JNIEnv* jni,
+ jobject j_video_frame,
+ uint32_t timestamp_rtp) const;
// Wraps a buffer to AndroidVideoBuffer without incrementing the reference
// count.
@@ -183,13 +184,14 @@ class JavaVideoFrameFactory {
public:
JavaVideoFrameFactory(JNIEnv* jni);
- jobject ToJavaFrame(JNIEnv* jni, const webrtc::VideoFrame& frame) const;
+ jobject ToJavaFrame(JNIEnv* jni, const VideoFrame& frame) const;
private:
ScopedGlobalRef<jclass> j_video_frame_class_;
jmethodID j_video_frame_constructor_id_;
};
-} // namespace webrtc_jni
+} // namespace jni
+} // namespace webrtc
#endif // WEBRTC_SDK_ANDROID_SRC_JNI_NATIVE_HANDLE_IMPL_H_
« no previous file with comments | « webrtc/sdk/android/src/jni/jni_onload.cc ('k') | webrtc/sdk/android/src/jni/native_handle_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698