Commit a8d4ad94 by aquilescanta Committed by Oliver Woodman

Fix DefaultTrackSelector#Parameter withSelectUndeterminedTextLanguage

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=177170994
parent 0e0300b8
...@@ -243,8 +243,7 @@ public class DefaultTrackSelector extends MappingTrackSelector { ...@@ -243,8 +243,7 @@ public class DefaultTrackSelector extends MappingTrackSelector {
/** /**
* Returns an instance with the provided {@link #selectUndeterminedTextLanguage}. * Returns an instance with the provided {@link #selectUndeterminedTextLanguage}.
*/ */
public Parameters withSelectUndeterminedTextLanguageAsFallback( public Parameters withSelectUndeterminedTextLanguage(boolean selectUndeterminedTextLanguage) {
boolean selectUndeterminedTextLanguage) {
if (selectUndeterminedTextLanguage == this.selectUndeterminedTextLanguage) { if (selectUndeterminedTextLanguage == this.selectUndeterminedTextLanguage) {
return this; return this;
} }
......
...@@ -561,8 +561,7 @@ public final class DefaultTrackSelectorTest { ...@@ -561,8 +561,7 @@ public final class DefaultTrackSelectorTest {
wrapFormats(spanish, german, undeterminedUnd, undeterminedNull)); wrapFormats(spanish, german, undeterminedUnd, undeterminedNull));
assertThat(result.selections.get(0)).isNull(); assertThat(result.selections.get(0)).isNull();
trackSelector.setParameters( trackSelector.setParameters(DEFAULT_PARAMETERS.withSelectUndeterminedTextLanguage(true));
DEFAULT_PARAMETERS.withSelectUndeterminedTextLanguageAsFallback(true));
result = trackSelector.selectTracks(textRendererCapabilites, result = trackSelector.selectTracks(textRendererCapabilites,
wrapFormats(spanish, german, undeterminedUnd, undeterminedNull)); wrapFormats(spanish, german, undeterminedUnd, undeterminedNull));
assertThat(result.selections.get(0).getFormat(0)).isSameAs(undeterminedUnd); assertThat(result.selections.get(0).getFormat(0)).isSameAs(undeterminedUnd);
...@@ -577,7 +576,7 @@ public final class DefaultTrackSelectorTest { ...@@ -577,7 +576,7 @@ public final class DefaultTrackSelectorTest {
assertThat(result.selections.get(0)).isNull(); assertThat(result.selections.get(0)).isNull();
trackSelector.setParameters( trackSelector.setParameters(
trackSelector.getParameters().withSelectUndeterminedTextLanguageAsFallback(true)); trackSelector.getParameters().withSelectUndeterminedTextLanguage(true));
result = trackSelector.selectTracks(textRendererCapabilites, result = trackSelector.selectTracks(textRendererCapabilites,
wrapFormats(german, undeterminedUnd, undeterminedNull)); wrapFormats(german, undeterminedUnd, undeterminedNull));
assertThat(result.selections.get(0).getFormat(0)).isSameAs(undeterminedUnd); assertThat(result.selections.get(0).getFormat(0)).isSameAs(undeterminedUnd);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment