From 0e0ff18b279f7f0646129adab182435794d8cd4b Mon Sep 17 00:00:00 2001 From: olivier Date: Sat, 28 Jan 2023 14:54:01 +0100 Subject: [PATCH 1/3] fix(ios): ensure audio and subtitle tracks are well reported --- ios/Video/Features/RCTVideoUtils.swift | 11 +++++++++-- ios/Video/RCTVideo.swift | 6 ++++-- 2 files changed, 13 insertions(+), 4 deletions(-) diff --git a/ios/Video/Features/RCTVideoUtils.swift b/ios/Video/Features/RCTVideoUtils.swift index 50df8e3f..3953c9d0 100644 --- a/ios/Video/Features/RCTVideoUtils.swift +++ b/ios/Video/Features/RCTVideoUtils.swift @@ -103,10 +103,14 @@ enum RCTVideoUtils { title = value as! String } let language:String! = currentOption?.extendedLanguageTag ?? "" + + let selectedOption: AVMediaSelectionOption? = player.currentItem?.currentMediaSelection.selectedMediaOption(in: group!) + let audioTrack = [ "index": NSNumber(value: i), "title": title, - "language": language + "language": language ?? "", + "selected": currentOption?.displayName == selectedOption?.displayName ] as [String : Any] audioTracks.add(audioTrack) } @@ -129,10 +133,13 @@ enum RCTVideoUtils { title = value as! String } let language:String! = currentOption?.extendedLanguageTag ?? "" + let selectedOpt = player.currentItem?.currentMediaSelection + let selectedOption: AVMediaSelectionOption? = player.currentItem?.currentMediaSelection.selectedMediaOption(in: group!) let textTrack = TextTrack([ "index": NSNumber(value: i), "title": title, - "language": language + "language": language, + "selected": currentOption?.displayName == selectedOption?.displayName ]) textTracks.append(textTrack) } diff --git a/ios/Video/RCTVideo.swift b/ios/Video/RCTVideo.swift index baca4d1a..ce9c99ac 100644 --- a/ios/Video/RCTVideo.swift +++ b/ios/Video/RCTVideo.swift @@ -1007,6 +1007,8 @@ class RCTVideo: UIView, RCTVideoPlayerViewControllerDelegate, RCTPlayerObserverH } if _videoLoadStarted { + let audioTracks = RCTVideoUtils.getAudioTrackInfo(_player) + let textTracks = RCTVideoUtils.getTextTrackInfo(_player).map(\.json) onVideoLoad?(["duration": NSNumber(value: duration), "currentTime": NSNumber(value: Float(CMTimeGetSeconds(_playerItem.currentTime()))), "canPlayReverse": NSNumber(value: _playerItem.canPlayReverse), @@ -1020,8 +1022,8 @@ class RCTVideo: UIView, RCTVideoPlayerViewControllerDelegate, RCTPlayerObserverH "height": width != nil ? NSNumber(value: height!) : "undefinded", "orientation": orientation ], - "audioTracks": RCTVideoUtils.getAudioTrackInfo(_player), - "textTracks": _textTracks ?? RCTVideoUtils.getTextTrackInfo(_player), + "audioTracks": audioTracks, + "textTracks": textTracks, "target": reactTag as Any]) } _videoLoadStarted = false From 57b4a767759e665ed2e24c3fbbdf073da9c83b65 Mon Sep 17 00:00:00 2001 From: olivier Date: Tue, 7 Feb 2023 23:14:50 +0100 Subject: [PATCH 2/3] fix: add stub for IMA and option to enable it on demand --- android/build.gradle | 29 ++++++++- .../ads/interactivemedia/v3/api/AdEvent.java | 11 ++++ .../exoplayer2/ext/ima/ImaAdsLoader.java | 65 +++++++++++++++++++ examples/basic/android/build.gradle | 2 + 4 files changed, 106 insertions(+), 1 deletion(-) create mode 100644 android/src/main/java/com/google/ads/interactivemedia/v3/api/AdEvent.java create mode 100644 android/src/main/java/com/google/android/exoplayer2/ext/ima/ImaAdsLoader.java diff --git a/android/build.gradle b/android/build.gradle index fdd257bf..a2ade2e6 100644 --- a/android/build.gradle +++ b/android/build.gradle @@ -4,6 +4,18 @@ def safeExtGet(prop, fallback) { rootProject.ext.has(prop) ? rootProject.ext.get(prop) : fallback } +def useExoplayerIMA = safeExtGet("RNVUseExoplayerIMA", false) + +println "useExoplayerIMA:" + useExoplayerIMA + +// This string is used to define build path. +// As react native build output directory is react-native path of the module. +// We need to force a new path on each configuration change. +// If you add a new build parameter, please add the new value in this string +def configStringPath = ( + 'useExoplayerIMA' + useExoplayerIMA \ +).md5() + android { compileSdkVersion safeExtGet('compileSdkVersion', 31) buildToolsVersion safeExtGet('buildToolsVersion', '30.0.2') @@ -24,6 +36,19 @@ android { sourceCompatibility JavaVersion.VERSION_1_8 targetCompatibility JavaVersion.VERSION_1_8 } + + buildDir 'buildOutput_' + configStringPath + + sourceSets { + main { + java { + if (useExoplayerIMA) { + exclude 'com/google/ads/interactivemedia/v3/api' + exclude 'com/android/exoplayer2/ext/ima' + } + } + } + } } repositories { @@ -45,7 +70,9 @@ dependencies { implementation('com.google.android.exoplayer:extension-okhttp:2.18.1') { exclude group: 'com.squareup.okhttp3', module: 'okhttp' } - implementation 'com.google.android.exoplayer:extension-ima:2.18.1' + if (useExoplayerIMA) { + implementation 'com.google.android.exoplayer:extension-ima:2.18.1' + } implementation "com.squareup.okhttp3:okhttp:" + '$OKHTTP_VERSION' } diff --git a/android/src/main/java/com/google/ads/interactivemedia/v3/api/AdEvent.java b/android/src/main/java/com/google/ads/interactivemedia/v3/api/AdEvent.java new file mode 100644 index 00000000..1913df29 --- /dev/null +++ b/android/src/main/java/com/google/ads/interactivemedia/v3/api/AdEvent.java @@ -0,0 +1,11 @@ +package com.google.ads.interactivemedia.v3.api; + +import androidx.annotation.InspectableProperty; + +public abstract class AdEvent { + public abstract InspectableProperty getType(); + + public interface AdEventListener { + public void onAdEvent(AdEvent adEvent); + } +} diff --git a/android/src/main/java/com/google/android/exoplayer2/ext/ima/ImaAdsLoader.java b/android/src/main/java/com/google/android/exoplayer2/ext/ima/ImaAdsLoader.java new file mode 100644 index 00000000..d23e7dd6 --- /dev/null +++ b/android/src/main/java/com/google/android/exoplayer2/ext/ima/ImaAdsLoader.java @@ -0,0 +1,65 @@ +package com.google.android.exoplayer2.ext.ima; + +import androidx.annotation.Nullable; + +import com.facebook.react.uimanager.ThemedReactContext; +import com.google.android.exoplayer2.ExoPlayer; +import com.google.android.exoplayer2.Player; +import com.google.android.exoplayer2.source.ads.AdsLoader; +import com.google.android.exoplayer2.source.ads.AdsMediaSource; +import com.google.android.exoplayer2.ui.AdViewProvider; +import com.google.android.exoplayer2.upstream.DataSpec; + +import java.io.IOException; + +public class ImaAdsLoader implements AdsLoader { + public void setPlayer(ExoPlayer player) { + } + + @Override + public void setPlayer(@Nullable Player player) { + + } + + public void release() { + } + + @Override + public void setSupportedContentTypes(int... ints) { + + } + + @Override + public void start(AdsMediaSource adsMediaSource, DataSpec dataSpec, Object o, AdViewProvider adViewProvider, EventListener eventListener) { + + } + + @Override + public void stop(AdsMediaSource adsMediaSource, EventListener eventListener) { + + } + + @Override + public void handlePrepareComplete(AdsMediaSource adsMediaSource, int i, int i1) { + + } + + @Override + public void handlePrepareError(AdsMediaSource adsMediaSource, int i, int i1, IOException e) { + + } + + public static class Builder { + public Builder(ThemedReactContext themedReactContext) { + + } + + public Builder setAdEventListener(Object reactExoplayerView) { + return this; + } + + public ImaAdsLoader build() { + return null; + } + } +} diff --git a/examples/basic/android/build.gradle b/examples/basic/android/build.gradle index 8569fee3..533ca601 100644 --- a/examples/basic/android/build.gradle +++ b/examples/basic/android/build.gradle @@ -14,6 +14,8 @@ buildscript { // Otherwise we default to the side-by-side NDK version from AGP. ndkVersion = "21.4.7075529" } + + RNVUseExoplayerIMA = true } repositories { google() From a3b4b94f71f7bb6c2a4512688473a644b9abd7fc Mon Sep 17 00:00:00 2001 From: olivier Date: Tue, 7 Feb 2023 23:15:05 +0100 Subject: [PATCH 3/3] chore: fix build of andrpoid sample --- examples/basic/metro.config.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/examples/basic/metro.config.js b/examples/basic/metro.config.js index bede29ff..fa377047 100644 --- a/examples/basic/metro.config.js +++ b/examples/basic/metro.config.js @@ -5,7 +5,7 @@ * @format */ const path = require('path'); -const blacklist = require('metro-config/src/defaults/blacklist'); +const blacklist = require('metro-config/src/defaults/exclusionList'); module.exports = { resolver: {