loliball/media3
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Split AdTagLoader out of ImaAdsLoader

Browse Source 
In a later change ImaAdsLoader will use multiple AdTagLoaders.

This change shouldn't have any substantial changes in behavior (it's almost
entirely moving code around). An exception is that ImaSdkSettings is configured
when making a request rather than at construction time.

Issue: #3750
PiperOrigin-RevId: 339891712
This commit is contained in:
andrewlewis 2020-10-30 17:06:59 +00:00 committed by Oliver Woodman
parent 27707e9c65
commit 32b710712e
3 changed files with 1424 additions and 1294 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1397
extensions/ima/src/main/java/com/google/android/exoplayer2/ext/ima/AdTagLoader.java Normal file
View File

File diff suppressed because it is too large Load Diff

1320
extensions/ima/src/main/java/com/google/android/exoplayer2/ext/ima/ImaAdsLoader.java
View File

File diff suppressed because it is too large Load Diff

1
extensions/ima/src/test/java/com/google/android/exoplayer2/ext/ima/ImaAdsLoaderTest.java
View File

@ -172,6 +172,7 @@ public final class ImaAdsLoaderTest {
public void builder_overridesPlayerType() {
when(mockImaSdkSettings.getPlayerType()).thenReturn("test player type");
setupPlayback(CONTENT_TIMELINE, PREROLL_CUE_POINTS_SECONDS);
imaAdsLoader.start(adsLoaderListener, adViewProvider);
verify(mockImaSdkSettings).setPlayerType("google/exo.ext.ima");
}