From 57a202c50ccbe2018838d041cdf9e97090d08aef Mon Sep 17 00:00:00 2001
From: Harold Thetiot <hthetiot@gmail.com>
Date: Tue, 17 Sep 2019 20:20:59 +0200
Subject: [PATCH] Deprecate usage of 'video.src = URL.createObjectURL(stream)'
 in favor of 'video.srcObject = stream' only MediaStream are not Blob anymore.

---
 CHANGELOG.md                          |  6 +++
 FAQ.md                                |  2 +-
 dist/cordova-plugin-iosrtc.js         | 74 +++------------------------
 docs/iosrtc.md                        |  2 +-
 extra/renderer-and-libwebrtc-tests.js |  1 -
 js/MediaStream.js                     |  1 -
 js/videoElementsHandler.js            | 71 +++----------------------
 7 files changed, 22 insertions(+), 135 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 145d3c55..26e8b2a8 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,4 +1,9 @@
+#### Version 5.0.3
+
+* Deprecate usage of `video.src = URL.createObjectURL(stream)` in favor of `video.srcObject = stream` only MediaStream are not Blob anymore.
+
 #### Version 5.0.2
+
 * Set default deployment target to 10.2
 * Remove callback based API
 * Implement RTCPeerConnection.getSenders|getReceivers|addTrack|removeTrack
@@ -12,6 +17,7 @@
 * fix TypeError: undefined is not an object (evaluating 'stream.id') when removing stream [PR #383](https://github.com/cordova-rtc/cordova-plugin-iosrtc/pull/383) by @hthetiot via @l7s).
 
 #### Version 5.0.1
+
 * fix typo on iosrtcPlugin.swift
 
 #### Version 5.0.0
diff --git a/FAQ.md b/FAQ.md
index 40087815..d01bde1f 100644
--- a/FAQ.md
+++ b/FAQ.md
@@ -1,6 +1,6 @@
 # FAQ
 
-**Q:** What about `<video>` elements and `video.src = URL.createObjectURL(stream)`? do I need custom HTML tags or functions to display WebRTC videos?
+**Q:** What about `<video>` elements and `video.srcObject = stream`? do I need custom HTML tags or functions to display WebRTC videos?
 
 **A:** No. Just use an HTML video element as usual, really. The plugin will properly place a native *UIView* layer on top of it by respecting (most of) its [CSS properties](docs/videoCSS.md).
 
diff --git a/dist/cordova-plugin-iosrtc.js b/dist/cordova-plugin-iosrtc.js
index 61beb4b3..e6fb2093 100644
--- a/dist/cordova-plugin-iosrtc.js
+++ b/dist/cordova-plugin-iosrtc.js
@@ -143,7 +143,6 @@ var originalMediaStreamTrack = MediaStreamTrack.originalMediaStreamTrack;
 
 /**
  * Expose the MediaStream class.
- * Make MediaStream be a Blob so it can be consumed by URL.createObjectURL().
  */
 function MediaStream(arg, id) {
 	debug('new MediaStream(arg) | [arg:%o]', arg);
@@ -2470,7 +2469,6 @@ function dump() {
 
 }).call(this,typeof global !== "undefined" ? global : typeof self !== "undefined" ? self : typeof window !== "undefined" ? window : {})
 },{"./MediaStream":4,"./MediaStreamTrack":6,"./RTCIceCandidate":9,"./RTCPeerConnection":10,"./RTCSessionDescription":11,"./enumerateDevices":14,"./getUserMedia":15,"./videoElementsHandler":17,"cordova/exec":undefined,"debug":18,"domready":20}],17:[function(_dereq_,module,exports){
-(function (global){
 /**
  * Expose a function that must be called when the library is loaded.
  * And also a helper function.
@@ -2491,12 +2489,6 @@ var
  * Local variables.
  */
 
-	// RegExp for MediaStream blobId.
-	MEDIASTREAM_ID_REGEXP = new RegExp(/^MediaStream_/),
-
-	// RegExp for Blob URI.
-	BLOB_URI_REGEX = new RegExp(/^blob:/),
-
 	// Dictionary of MediaStreamRenderers (provided via module argument).
 	// - key: MediaStreamRenderer id.
 	// - value: MediaStreamRenderer.
@@ -2517,8 +2509,8 @@ var
 			// HTML video element.
 			video = mutation.target;
 
-			// .src or .srcObject removed.
-			if (!video.src && !video.srcObject) {
+			// .srcObject removed.
+			if (!video.srcObject) {
 				// If this video element was previously handling a MediaStreamRenderer, release it.
 				releaseMediaStreamRenderer(video);
 				continue;
@@ -2643,13 +2635,13 @@ function videoElementsHandler(_mediaStreams, _mediaStreamRenderers) {
 function observeVideo(video) {
 	debug('observeVideo()');
 
-	// If the video already has a src/srcObject property but is not yet handled by the plugin
+	// If the video already has a srcObject property but is not yet handled by the plugin
 	// then handle it now.
-	if ((video.src || video.srcObject) && !video._iosrtcMediaStreamRendererId) {
+	if ((video.srcObject) && !video._iosrtcMediaStreamRendererId) {
 		handleVideo(video);
 	}
 
-	// Add .src observer to the video element.
+	// Add .srcObject observer to the video element.
 	videoObserver.observe(video, {
 		// Set to true if additions and removals of the target node's child elements (including text
 		// nodes) are to be observed.
@@ -2668,16 +2660,7 @@ function observeVideo(video) {
 		characterDataOldValue: false,
 		// Set to an array of attribute local names (without namespace) if not all attribute mutations
 		// need to be observed.
-		attributeFilter: ['src', 'srcObject']
-	});
-
-	// Intercept video 'error' events if it's due to the attached MediaStream.
-	video.addEventListener('error', function (event) {
-		if (video.error.code === global.MediaError.MEDIA_ERR_SRC_NOT_SUPPORTED && BLOB_URI_REGEX.test(video.src)) {
-			debug('stopping "error" event propagation for video element');
-
-			event.stopImmediatePropagation();
-		}
+		attributeFilter: ['srcObject']
 	});
 }
 
@@ -2688,51 +2671,10 @@ function observeVideo(video) {
 
 function handleVideo(video) {
 	var
-		xhr = new XMLHttpRequest(),
 		stream;
 
-	// The app has set video.src.
-	if (video.src) {
-		xhr.open('GET', video.src, true);
-		xhr.responseType = 'blob';
-		xhr.onload = function () {
-			if (xhr.status !== 200) {
-				// If this video element was previously handling a MediaStreamRenderer, release it.
-				releaseMediaStreamRenderer(video);
-
-				return;
-			}
-
-			var reader = new FileReader();
-
-			// Some versions of Safari fail to set onloadend property, some others do not react
-			// on 'loadend' event. Try everything here.
-			try {
-				reader.onloadend = onloadend;
-			} catch (error) {
-				reader.addEventListener('loadend', onloadend);
-			}
-			reader.readAsText(xhr.response);
-
-			function onloadend() {
-				var mediaStreamBlobId = reader.result;
-
-				// The retrieved URL does not point to a MediaStream.
-				if (!mediaStreamBlobId || typeof mediaStreamBlobId !== 'string' || !MEDIASTREAM_ID_REGEXP.test(mediaStreamBlobId)) {
-					// If this video element was previously handling a MediaStreamRenderer, release it.
-					releaseMediaStreamRenderer(video);
-
-					return;
-				}
-
-				provideMediaStreamRenderer(video, mediaStreamBlobId);
-			}
-		};
-		xhr.send();
-	}
-
 	// The app has set video.srcObject.
-	else if (video.srcObject) {
+	if (video.srcObject) {
 		stream = video.srcObject;
 
 		if (!stream.getBlobId()) {
@@ -2805,7 +2747,6 @@ function provideMediaStreamRenderer(video, mediaStreamBlobId) {
 	});
 }
 
-
 function releaseMediaStreamRenderer(video) {
 	if (!video._iosrtcMediaStreamRendererId) {
 		return;
@@ -2826,7 +2767,6 @@ function releaseMediaStreamRenderer(video) {
 	delete video.readyState;
 }
 
-}).call(this,typeof global !== "undefined" ? global : typeof self !== "undefined" ? self : typeof window !== "undefined" ? window : {})
 },{"./MediaStreamRenderer":5,"debug":18}],18:[function(_dereq_,module,exports){
 (function (process){
 /* eslint-env browser */
diff --git a/docs/iosrtc.md b/docs/iosrtc.md
index 1beb2ee6..60690a96 100644
--- a/docs/iosrtc.md
+++ b/docs/iosrtc.md
@@ -163,7 +163,7 @@ peerconnection.addEventListener('addstream', function (event) {
   cordova.plugins.iosrtc.observeVideo(video);
 
   // Attach the MediaStream to it.
-  video.src = URL.createObjectURL(event.stream);
+  video.srcObject = event.stream;
 
   // When the stream is ready to be rendered then append the video
   // element to the DOM.
diff --git a/extra/renderer-and-libwebrtc-tests.js b/extra/renderer-and-libwebrtc-tests.js
index ee6bc8ed..88f202e8 100644
--- a/extra/renderer-and-libwebrtc-tests.js
+++ b/extra/renderer-and-libwebrtc-tests.js
@@ -96,7 +96,6 @@ function TestGetUserMedia() {
 
       localStream = stream;
       srcObjectStream = localVideoEl.srcObject = localStream;
-      //localVideoEl.src = window.URL.createObjectURL(stream);
     
     } catch (err) {
       console.log('srcObject.err', err);
diff --git a/js/MediaStream.js b/js/MediaStream.js
index cb0e78f4..c38bceb1 100644
--- a/js/MediaStream.js
+++ b/js/MediaStream.js
@@ -37,7 +37,6 @@ var originalMediaStreamTrack = MediaStreamTrack.originalMediaStreamTrack;
 
 /**
  * Expose the MediaStream class.
- * Make MediaStream be a Blob so it can be consumed by URL.createObjectURL().
  */
 function MediaStream(arg, id) {
 	debug('new MediaStream(arg) | [arg:%o]', arg);
diff --git a/js/videoElementsHandler.js b/js/videoElementsHandler.js
index 036374f8..fce9e42f 100644
--- a/js/videoElementsHandler.js
+++ b/js/videoElementsHandler.js
@@ -18,12 +18,6 @@ var
  * Local variables.
  */
 
-	// RegExp for MediaStream blobId.
-	MEDIASTREAM_ID_REGEXP = new RegExp(/^MediaStream_/),
-
-	// RegExp for Blob URI.
-	BLOB_URI_REGEX = new RegExp(/^blob:/),
-
 	// Dictionary of MediaStreamRenderers (provided via module argument).
 	// - key: MediaStreamRenderer id.
 	// - value: MediaStreamRenderer.
@@ -44,8 +38,8 @@ var
 			// HTML video element.
 			video = mutation.target;
 
-			// .src or .srcObject removed.
-			if (!video.src && !video.srcObject) {
+			// .srcObject removed.
+			if (!video.srcObject) {
 				// If this video element was previously handling a MediaStreamRenderer, release it.
 				releaseMediaStreamRenderer(video);
 				continue;
@@ -170,13 +164,13 @@ function videoElementsHandler(_mediaStreams, _mediaStreamRenderers) {
 function observeVideo(video) {
 	debug('observeVideo()');
 
-	// If the video already has a src/srcObject property but is not yet handled by the plugin
+	// If the video already has a srcObject property but is not yet handled by the plugin
 	// then handle it now.
-	if ((video.src || video.srcObject) && !video._iosrtcMediaStreamRendererId) {
+	if ((video.srcObject) && !video._iosrtcMediaStreamRendererId) {
 		handleVideo(video);
 	}
 
-	// Add .src observer to the video element.
+	// Add .srcObject observer to the video element.
 	videoObserver.observe(video, {
 		// Set to true if additions and removals of the target node's child elements (including text
 		// nodes) are to be observed.
@@ -195,16 +189,7 @@ function observeVideo(video) {
 		characterDataOldValue: false,
 		// Set to an array of attribute local names (without namespace) if not all attribute mutations
 		// need to be observed.
-		attributeFilter: ['src', 'srcObject']
-	});
-
-	// Intercept video 'error' events if it's due to the attached MediaStream.
-	video.addEventListener('error', function (event) {
-		if (video.error.code === global.MediaError.MEDIA_ERR_SRC_NOT_SUPPORTED && BLOB_URI_REGEX.test(video.src)) {
-			debug('stopping "error" event propagation for video element');
-
-			event.stopImmediatePropagation();
-		}
+		attributeFilter: ['srcObject']
 	});
 }
 
@@ -215,51 +200,10 @@ function observeVideo(video) {
 
 function handleVideo(video) {
 	var
-		xhr = new XMLHttpRequest(),
 		stream;
 
-	// The app has set video.src.
-	if (video.src) {
-		xhr.open('GET', video.src, true);
-		xhr.responseType = 'blob';
-		xhr.onload = function () {
-			if (xhr.status !== 200) {
-				// If this video element was previously handling a MediaStreamRenderer, release it.
-				releaseMediaStreamRenderer(video);
-
-				return;
-			}
-
-			var reader = new FileReader();
-
-			// Some versions of Safari fail to set onloadend property, some others do not react
-			// on 'loadend' event. Try everything here.
-			try {
-				reader.onloadend = onloadend;
-			} catch (error) {
-				reader.addEventListener('loadend', onloadend);
-			}
-			reader.readAsText(xhr.response);
-
-			function onloadend() {
-				var mediaStreamBlobId = reader.result;
-
-				// The retrieved URL does not point to a MediaStream.
-				if (!mediaStreamBlobId || typeof mediaStreamBlobId !== 'string' || !MEDIASTREAM_ID_REGEXP.test(mediaStreamBlobId)) {
-					// If this video element was previously handling a MediaStreamRenderer, release it.
-					releaseMediaStreamRenderer(video);
-
-					return;
-				}
-
-				provideMediaStreamRenderer(video, mediaStreamBlobId);
-			}
-		};
-		xhr.send();
-	}
-
 	// The app has set video.srcObject.
-	else if (video.srcObject) {
+	if (video.srcObject) {
 		stream = video.srcObject;
 
 		if (!stream.getBlobId()) {
@@ -332,7 +276,6 @@ function provideMediaStreamRenderer(video, mediaStreamBlobId) {
 	});
 }
 
-
 function releaseMediaStreamRenderer(video) {
 	if (!video._iosrtcMediaStreamRendererId) {
 		return;