- 23 Dec, 2020 38 commits
-
-
PiperOrigin-RevId: 348789555
olly committed -
PiperOrigin-RevId: 348784475
olly committed -
https://github.com/google/ExoPlayer/commit/dd7b379dc0344316488eb816e06638182a9eed90
*** Original commit *** DataSource.open() throws if already opened. Update DataSource implementations to throw an error if open() is called when the DataSource is already open. *** PiperOrigin-RevId: 348783425
olly committed -
PiperOrigin-RevId: 348760170
ibaker committed -
Also add a MockWebServer Dispatcher that can be customised with different resources and behaviours for different paths. PiperOrigin-RevId: 348759662
ibaker committed -
The length needs to change depending on the positition/length parameters passed to DataSpec, so it doesn't really make sense to keep this private method - it's clearer to in-line the calls in each test. PiperOrigin-RevId: 348749254
ibaker committed -
PiperOrigin-RevId: 348688419
bachinger committed -
Issue: #4904 Issue: #5011 PiperOrigin-RevId: 348655288
tonihei committed -
These two things were reported by the nullness checker as warnings. PiperOrigin-RevId: 348650535
tonihei committed -
Update DataSource implementations to throw an error if open() is called when the DataSource is already open. PiperOrigin-RevId: 348609860
christosts committed -
PiperOrigin-RevId: 348462189
tonihei committed -
Estimating the playback frame-rate, querying the display refresh rate, and setting the surface frame-rate, are all closely related to one another. In particular because setting the surface frame-rate can directly cause the display refresh rate to change. It therefore makes sense to move surface frame-rate adjustment into the helper. This also makes it easier to re-use the logic in other video renderers. PiperOrigin-RevId: 348455864
olly committed -
We currently use INDEX_UNBOUNDED for the calculation and don't offset the index by the firstSegmentNum if not unbounded. PiperOrigin-RevId: 348445797
tonihei committed -
The media specifies a URL that uses http rather than https, which the demo app doesn't permit by default. The URL here is the same except it uses https instead. #minor-release PiperOrigin-RevId: 348445571
olly committed -
#exofixit PiperOrigin-RevId: 348444280
bachinger committed -
PiperOrigin-RevId: 348443305
ibaker committed -
#exofixit PiperOrigin-RevId: 348441436
bachinger committed -
PiperOrigin-RevId: 348440799
Oliver Woodman committed -
This replaces all the duplicated logic previously implemented in FakeSampleStream and more closely follows the pattern of how SampleStreams are used from real MediaPeriods. Some tests needed adjustments because using real the SampleQueue improved behaviour: - Waiting for isLoading is only needed once even across period boundaries because the real SampleQueue doesn't have the on/off pattern. - AnalyticsCollectorTest.playlistOperations() was wrongly asserting that some pre-buffering events. The new version is more intuitively correct we pre-buffer the second item during the initial loading phase (thus period1seq1) and keep the buffer in the queue after the removal operation. PiperOrigin-RevId: 348440255
tonihei committed -
The delta updates loose information about previous init segments. Until this is properly fixed, we can avoid the problem by not using delta updates. Issue: #5011 PiperOrigin-RevId: 348023895
tonihei committed -
We're now using CapturingRenderersFactory everywhere PiperOrigin-RevId: 348018988
ibaker committed -
PiperOrigin-RevId: 348018409
ibaker committed -
PiperOrigin-RevId: 348017841
ibaker committed -
PiperOrigin-RevId: 348017268
ibaker committed -
This avoids a NullPointerException later when the format is used. Fixes https://github.com/google/ExoPlayer/issues/8283. PiperOrigin-RevId: 348017149
krocard committed -
PiperOrigin-RevId: 348016819
ibaker committed -
PiperOrigin-RevId: 348016339
ibaker committed -
PiperOrigin-RevId: 348015859
ibaker committed -
PiperOrigin-RevId: 348015304
ibaker committed -
PiperOrigin-RevId: 348014814
ibaker committed -
PiperOrigin-RevId: 348014322
ibaker committed -
PiperOrigin-RevId: 348011243
christosts committed -
With @Ignore but not @test the method is still executed by JUnit: https://github.com/junit-team/junit4/issues/695 PiperOrigin-RevId: 348009981
ibaker committed -
PiperOrigin-RevId: 348008973
kimvde committed -
If we have trailing parts the available window should reach to the end of all trailing parts and not only to the last finished segment. Issue: #5011 PiperOrigin-RevId: 347996626
tonihei committed -
PiperOrigin-RevId: 347994829
ibaker committed -
I decided not to migrate all the tests in one CL to keep the diff manageable. I'll make follow-up CLs to migrate the tests, and eventually delete TeeCodec and all associated logic. I couldn't completely remove the dump diff because ShadowMediaCodec.getCodecInfo() (which would give me access to the MIME type) doesn't seem to work properly - it returned video/avc when name=exotest.audio.aac, and looking into the code it looks like there's some native methods that are missing shadow implementations. PiperOrigin-RevId: 347991956
ibaker committed -
The adaptive period currently extends the base (non-adaptive) period to share common MediaPeriod boilerplate code. However, once we start using the real SampleQueue in FakeMediaPeriod the common code becomes even less and the overhead to support multiple stream implementation from the base class is no longer worth it. Thus, this change removes the class hierarchy and copies the common parts to FakeAdaptiveMediaPeriod. PiperOrigin-RevId: 347990468
tonihei committed
-
- 17 Dec, 2020 2 commits