Deprecate PeerConnectionFactory#createVideoSource(VideoCapturer).
This is done in preparation of moving VideoCapturer out of
video_api_java. Clients should update to using
createVideoSource(boolean).
CapturerObserver is moved to a separate file because it needs to stay
in video_api_java to allow VideoSource to depend on it.
Bug: webrtc:9496
Change-Id: I3c93f6bc4df553919dcbe05b00ef4c68f2c9ab60
Reviewed-on: https://webrtc-review.googlesource.com/87305
Reviewed-by: Magnus Jedvert <magjed@webrtc.org>
Commit-Queue: Sami Kalliomäki <sakal@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#23868}diff --git a/sdk/android/api/org/webrtc/CapturerObserver.java b/sdk/android/api/org/webrtc/CapturerObserver.java
new file mode 100644
index 0000000..4edab9a
--- /dev/null
+++ b/sdk/android/api/org/webrtc/CapturerObserver.java
@@ -0,0 +1,29 @@
+/*
+ * Copyright 2018 The WebRTC project authors. All Rights Reserved.
+ *
+ * Use of this source code is governed by a BSD-style license
+ * that can be found in the LICENSE file in the root of the source
+ * tree. An additional intellectual property rights grant can be found
+ * in the file PATENTS. All contributing project authors may
+ * be found in the AUTHORS file in the root of the source tree.
+ */
+
+package org.webrtc;
+
+/**
+ * Interface for observering a capturer. Passed to {@link VideoCapturer#initialize}. Provided by
+ * {@link VideoSource#getCapturerObserver}.
+ *
+ * All callbacks must be executed on a single thread.
+ *
+ * @note This will replace the deprecated VideoCapturer.CapturerObserver interface.
+ */
+public interface CapturerObserver extends VideoCapturer.CapturerObserver {
+ /** Notify if the capturer have been started successfully or not. */
+ @Override void onCapturerStarted(boolean success);
+ /** Notify that the capturer has been stopped. */
+ @Override void onCapturerStopped();
+
+ /** Delivers a captured frame. */
+ @Override void onFrameCaptured(VideoFrame frame);
+}
diff --git a/sdk/android/api/org/webrtc/PeerConnectionFactory.java b/sdk/android/api/org/webrtc/PeerConnectionFactory.java
index 789baf3..dc89aec 100644
--- a/sdk/android/api/org/webrtc/PeerConnectionFactory.java
+++ b/sdk/android/api/org/webrtc/PeerConnectionFactory.java
@@ -357,6 +357,7 @@
return new VideoSource(nativeCreateVideoSource(nativeFactory, isScreencast));
}
+ @Deprecated
public VideoSource createVideoSource(VideoCapturer capturer) {
final EglBase.Context eglContext =
localEglbase == null ? null : localEglbase.getEglBaseContext();
diff --git a/sdk/android/api/org/webrtc/VideoCapturer.java b/sdk/android/api/org/webrtc/VideoCapturer.java
index 99a0729..8292fbd 100644
--- a/sdk/android/api/org/webrtc/VideoCapturer.java
+++ b/sdk/android/api/org/webrtc/VideoCapturer.java
@@ -16,6 +16,7 @@
// Base interface for all VideoCapturers to implement.
public interface VideoCapturer {
// Interface used for providing callbacks to an observer.
+ @Deprecated
public interface CapturerObserver {
// Notify if the camera have been started successfully or not.
// Called on a Java thread owned by VideoCapturer.
@@ -26,6 +27,13 @@
void onFrameCaptured(VideoFrame frame);
}
+ /** Deprecated, implementations should be update to implement the version below. */
+ @Deprecated
+ default void initialize(SurfaceTextureHelper surfaceTextureHelper, Context applicationContext,
+ CapturerObserver capturerObserver) {
+ throw new UnsupportedOperationException("Not implemented.");
+ }
+
/**
* This function is used to initialize the camera thread, the android application context, and the
* capture observer. It will be called only once and before any startCapture() request. The
@@ -37,8 +45,12 @@
* called. This also means that the caller can reuse the SurfaceTextureHelper to initialize a new
* VideoCapturer once the previous VideoCapturer has been disposed.
*/
- void initialize(SurfaceTextureHelper surfaceTextureHelper, Context applicationContext,
- CapturerObserver capturerObserver);
+ // Our version of clang format doesn't understand default and messes up.
+ // clang-format off
+ default void initialize(SurfaceTextureHelper surfaceTextureHelper, Context applicationContext,
+ org.webrtc.CapturerObserver capturerObserver) {
+ initialize(surfaceTextureHelper, applicationContext, (CapturerObserver) capturerObserver);
+ }
/**
* Start capturing frames in a format that is as close as possible to {@code width x height} and
diff --git a/sdk/android/api/org/webrtc/VideoSource.java b/sdk/android/api/org/webrtc/VideoSource.java
index 3a290e3..fb0ad1a 100644
--- a/sdk/android/api/org/webrtc/VideoSource.java
+++ b/sdk/android/api/org/webrtc/VideoSource.java
@@ -40,7 +40,7 @@
nativeAdaptOutputFormat(nativeSource, width, height, fps);
}
- public VideoCapturer.CapturerObserver getCapturerObserver() {
+ public CapturerObserver getCapturerObserver() {
return capturerObserver;
}