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

Side by Side Diff: talk/app/webrtc/java/src/org/webrtc/VideoCapturer.java

Issue 1652123002: Remove Java PC support. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 10 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * libjingle 2 * libjingle
3 * Copyright 2013 Google Inc. 3 * Copyright 2013 Google Inc.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions are met: 6 * modification, are permitted provided that the following conditions are met:
7 * 7 *
8 * 1. Redistributions of source code must retain the above copyright notice, 8 * 1. Redistributions of source code must retain the above copyright notice,
9 * this list of conditions and the following disclaimer. 9 * this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright notice, 10 * 2. Redistributions in binary form must reproduce the above copyright notice,
(...skipping 10 matching lines...) Expand all
21 * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; 21 * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS;
22 * OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, 22 * OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY,
23 * WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR 23 * WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR
24 * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF 24 * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
25 * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 25 * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
26 */ 26 */
27 27
28 package org.webrtc; 28 package org.webrtc;
29 29
30 /** Java version of cricket::VideoCapturer. */ 30 /** Java version of cricket::VideoCapturer. */
31 public class VideoCapturer { 31 public class VideoCapturer {
AlexG 2016/02/01 21:05:33 Do we need this class now? If there is no create f
perkj_webrtc 2016/02/03 17:00:03 VideoCapturerAndroid inherits from this class. So
AlexG 2016/02/04 00:01:22 Add TODO item?
perkj_webrtc 2016/02/05 13:43:44 Done.
32 private long nativeVideoCapturer; 32 private long nativeVideoCapturer;
33 33
34 protected VideoCapturer() { 34 protected VideoCapturer() {
35 } 35 }
36 36
37 public static VideoCapturer create(String deviceName) {
38 Object capturer = nativeCreateVideoCapturer(deviceName);
39 if (capturer != null)
40 return (VideoCapturer) (capturer);
41 return null;
42 }
43
44 // Sets |nativeCapturer| to be owned by VideoCapturer. 37 // Sets |nativeCapturer| to be owned by VideoCapturer.
45 protected void setNativeCapturer(long nativeCapturer) { 38 protected void setNativeCapturer(long nativeCapturer) {
46 this.nativeVideoCapturer = nativeCapturer; 39 this.nativeVideoCapturer = nativeCapturer;
47 } 40 }
48 41
49 // Package-visible for PeerConnectionFactory. 42 // Package-visible for PeerConnectionFactory.
50 long takeNativeVideoCapturer() { 43 long takeNativeVideoCapturer() {
51 if (nativeVideoCapturer == 0) { 44 if (nativeVideoCapturer == 0) {
52 throw new RuntimeException("Capturer can only be taken once!"); 45 throw new RuntimeException("Capturer can only be taken once!");
53 } 46 }
54 long ret = nativeVideoCapturer; 47 long ret = nativeVideoCapturer;
55 nativeVideoCapturer = 0; 48 nativeVideoCapturer = 0;
56 return ret; 49 return ret;
57 } 50 }
58 51
59 public void dispose() { 52 public void dispose() {
60 // No-op iff this capturer is owned by a source (see comment on 53 // No-op iff this capturer is owned by a source (see comment on
61 // PeerConnectionFactoryInterface::CreateVideoSource()). 54 // PeerConnectionFactoryInterface::CreateVideoSource()).
62 if (nativeVideoCapturer != 0) { 55 if (nativeVideoCapturer != 0) {
63 free(nativeVideoCapturer); 56 free(nativeVideoCapturer);
64 } 57 }
65 } 58 }
66 59
67 private static native Object nativeCreateVideoCapturer(String deviceName);
68
69 private static native void free(long nativeVideoCapturer); 60 private static native void free(long nativeVideoCapturer);
70 } 61 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698