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

Unified Diff: webrtc/api/android/java/src/org/webrtc/VideoRenderer.java

Issue 2377003002: Format all Java in WebRTC. (Closed)
Patch Set: Rebase. Created 4 years, 3 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/api/android/java/src/org/webrtc/VideoRenderer.java
diff --git a/webrtc/api/android/java/src/org/webrtc/VideoRenderer.java b/webrtc/api/android/java/src/org/webrtc/VideoRenderer.java
index 819b77dfb715ad81228c979abcd2156974e5e081..8ee4d27e7a3fee831b343929d75e6ff8a0b3add6 100644
--- a/webrtc/api/android/java/src/org/webrtc/VideoRenderer.java
+++ b/webrtc/api/android/java/src/org/webrtc/VideoRenderer.java
@@ -61,11 +61,13 @@ public class VideoRenderer {
// top-left corner of the image, but in glTexImage2D() the first element corresponds to the
// bottom-left corner. This discrepancy is corrected by setting a vertical flip as sampling
// matrix.
+ // clang-format off
samplingMatrix = new float[] {
1, 0, 0, 0,
0, -1, 0, 0,
0, 0, 1, 0,
0, 1, 0, 1};
+ // clang-format on
}
/**
@@ -97,14 +99,13 @@ public class VideoRenderer {
@Override
public String toString() {
- return width + "x" + height + ":" + yuvStrides[0] + ":" + yuvStrides[1] +
- ":" + yuvStrides[2];
+ return width + "x" + height + ":" + yuvStrides[0] + ":" + yuvStrides[1] + ":" + yuvStrides[2];
}
}
// Helper native function to do a video frame plane copying.
- public static native void nativeCopyPlane(ByteBuffer src, int width,
- int height, int srcStride, ByteBuffer dst, int dstStride);
+ public static native void nativeCopyPlane(
+ ByteBuffer src, int width, int height, int srcStride, ByteBuffer dst, int dstStride);
/** The real meat of VideoSinkInterface. */
public static interface Callbacks {
@@ -115,17 +116,17 @@ public class VideoRenderer {
public void renderFrame(I420Frame frame);
}
- /**
- * This must be called after every renderFrame() to release the frame.
- */
- public static void renderFrameDone(I420Frame frame) {
- frame.yuvPlanes = null;
- frame.textureId = 0;
- if (frame.nativeFramePointer != 0) {
- releaseNativeFrame(frame.nativeFramePointer);
- frame.nativeFramePointer = 0;
- }
- }
+ /**
+ * This must be called after every renderFrame() to release the frame.
+ */
+ public static void renderFrameDone(I420Frame frame) {
+ frame.yuvPlanes = null;
+ frame.textureId = 0;
+ if (frame.nativeFramePointer != 0) {
+ releaseNativeFrame(frame.nativeFramePointer);
+ frame.nativeFramePointer = 0;
+ }
+ }
long nativeVideoRenderer;

Powered by Google App Engine
This is Rietveld 408576698