Why does AUDIO_TRACK_LOAD_TIMEOUT follow errorRetry in playlistLoadPolicy?

[jiho-bae]

What do you want to do with Hls.js?

I am using hls.js version v1.5.18.

LEVEL_LOAD_TIMEOUT follows playlistLoadPolicy.timeoutRetry.
But LEVEL_LOAD_ERROR, AUDIO_TRACK_LOAD_ERROR, and AUDIO_TRACK_LOAD_TIMEOUT follow playlistLoadPolicy.errorRetry.

Why does AUDIO_TRACK_LOAD_TIMEOUT also follow errorRetry?

What have you tried so far?

I added the shouldRetry to the playlistLoadPolicy, and I confirmed that shouldRetry of timeoutRetry is executed in LEVEL_LOAD_TIMEOUT, and errorRetry is executed in AUDIO_TRACK_LOAD_TIMEOUT.

[robwalch]

The following cases are missing from isTimeoutError:

    case ErrorDetails.AUDIO_TRACK_LOAD_TIMEOUT:
    case ErrorDetails.SUBTITLE_TRACK_LOAD_TIMEOUT:
    case ErrorDetails.ASSET_LIST_LOAD_TIMEOUT:

hls.js/src/utils/error-helper.ts

Lines 6 to 15 in 99358d8

	export function isTimeoutError(error: ErrorData): boolean {
	switch (error.details) {
	case ErrorDetails.FRAG_LOAD_TIMEOUT:
	case ErrorDetails.KEY_LOAD_TIMEOUT:
	case ErrorDetails.LEVEL_LOAD_TIMEOUT:
	case ErrorDetails.MANIFEST_LOAD_TIMEOUT:
	return true;
	}
	return false;
	}