Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changes to enable progress event when mediadiskcache is set to FALSE. #58

Closed
wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost commented May 24, 2016

No description provided.

if (m_cachedPosition)
loaded = m_cachedPosition;
else
loaded = m_mediaDuration;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't the value of m_mediaDuration be checked before using it?

@philn
Copy link

philn commented May 24, 2016

I'm not sure I like the logic of this patch... Also in theory it's possible that loaded is set again in the code following down the patch and there's no log statement notifying about it.

…– Added check & Log notification suggested by Philippe.
@philn philn closed this May 25, 2016
@philn philn deleted the onprogress branch May 25, 2016 10:57
dwrobel pushed a commit to dwrobel/WPEWebKit that referenced this pull request Jan 26, 2022
WebPlatformForEmbedded#58)

It is matter of race result if seek completion is already detected.
Some applications do not recover from waiting event without additional
seeked or palying, but these do not follow on playback rate change.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant