Commit cb0394b9 by WeiChungChang

'improve_far_seek_of_chunkSampleStream'

parent deefe50a
...@@ -229,10 +229,26 @@ public final class DefaultTrackOutput implements TrackOutput { ...@@ -229,10 +229,26 @@ public final class DefaultTrackOutput implements TrackOutput {
* Attempts to skip to the keyframe before the specified time, if it's present in the buffer. * Attempts to skip to the keyframe before the specified time, if it's present in the buffer.
* *
* @param timeUs The seek time. * @param timeUs The seek time.
* @param skipToLastKey Skip to last key regardless the seek time is out of range .
* @return Whether the skip was successful.
*/
public boolean skipToKeyframeBefore(long timeUs, boolean skipToLastKey) {
long nextOffset = infoQueue.skipToKeyframeBefore(timeUs, skipToLastKey);
if (nextOffset == C.POSITION_UNSET) {
return false;
}
dropDownstreamTo(nextOffset);
return true;
}
/**
* Attempts to skip to the keyframe before the specified time, if it's present in the buffer.
*
* @param timeUs The seek time.
* @return Whether the skip was successful. * @return Whether the skip was successful.
*/ */
public boolean skipToKeyframeBefore(long timeUs) { public boolean skipToKeyframeBefore(long timeUs) {
long nextOffset = infoQueue.skipToKeyframeBefore(timeUs); long nextOffset = infoQueue.skipToKeyframeBefore(timeUs, false);
if (nextOffset == C.POSITION_UNSET) { if (nextOffset == C.POSITION_UNSET) {
return false; return false;
} }
...@@ -781,12 +797,12 @@ public final class DefaultTrackOutput implements TrackOutput { ...@@ -781,12 +797,12 @@ public final class DefaultTrackOutput implements TrackOutput {
* @return The offset of the keyframe's data if the keyframe was present. * @return The offset of the keyframe's data if the keyframe was present.
* {@link C#POSITION_UNSET} otherwise. * {@link C#POSITION_UNSET} otherwise.
*/ */
public synchronized long skipToKeyframeBefore(long timeUs) { public synchronized long skipToKeyframeBefore(long timeUs, boolean skipToLastKey) {
if (queueSize == 0 || timeUs < timesUs[relativeReadIndex]) { if (queueSize == 0 || timeUs < timesUs[relativeReadIndex]) {
return C.POSITION_UNSET; return C.POSITION_UNSET;
} }
if (timeUs > largestQueuedTimestampUs) { if (timeUs > largestQueuedTimestampUs && !skipToLastKey) {
return C.POSITION_UNSET; return C.POSITION_UNSET;
} }
......
...@@ -115,6 +115,20 @@ public class ChunkSampleStream<T extends ChunkSource> implements SampleStream, S ...@@ -115,6 +115,20 @@ public class ChunkSampleStream<T extends ChunkSource> implements SampleStream, S
} }
/** /**
* Check if the input position locates within the chunk we are downloading now.
*
* @param positionUs The target position in microseconds.
* @return Whether the input position locates within the chunk we are downloading now.
*/
public boolean isWithinLastChunk(long positionUs) {
if (isPendingReset() || loadingFinished) {
return false;
} else {
return ((positionUs >= getCurrentLoadPositionUs()) && (positionUs < getNextLoadPositionUs()));
}
}
/**
* Seeks to the specified position in microseconds. * Seeks to the specified position in microseconds.
* *
* @param positionUs The seek position in microseconds. * @param positionUs The seek position in microseconds.
...@@ -122,7 +136,8 @@ public class ChunkSampleStream<T extends ChunkSource> implements SampleStream, S ...@@ -122,7 +136,8 @@ public class ChunkSampleStream<T extends ChunkSource> implements SampleStream, S
public void seekToUs(long positionUs) { public void seekToUs(long positionUs) {
lastSeekPositionUs = positionUs; lastSeekPositionUs = positionUs;
// If we're not pending a reset, see if we can seek within the sample queue. // If we're not pending a reset, see if we can seek within the sample queue.
boolean seekInsideBuffer = !isPendingReset() && sampleQueue.skipToKeyframeBefore(positionUs); boolean seekInsideBuffer = !isPendingReset() &&
sampleQueue.skipToKeyframeBefore(positionUs, isWithinLastChunk(positionUs));
if (seekInsideBuffer) { if (seekInsideBuffer) {
// We succeeded. All we need to do is discard any chunks that we've moved past. // We succeeded. All we need to do is discard any chunks that we've moved past.
while (mediaChunks.size() > 1 while (mediaChunks.size() > 1
...@@ -284,6 +299,14 @@ public class ChunkSampleStream<T extends ChunkSource> implements SampleStream, S ...@@ -284,6 +299,14 @@ public class ChunkSampleStream<T extends ChunkSource> implements SampleStream, S
return true; return true;
} }
public long getCurrentLoadPositionUs() {
if (isPendingReset()) {
return pendingResetPositionUs;
} else {
return loadingFinished ? C.TIME_END_OF_SOURCE : mediaChunks.getLast().startTimeUs;
}
}
@Override @Override
public long getNextLoadPositionUs() { public long getNextLoadPositionUs() {
if (isPendingReset()) { if (isPendingReset()) {
......
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