Skip to content
This repository has been archived by the owner on Dec 10, 2020. It is now read-only.

Conditionally trigger loadstart and loadedmetadata for 5.0 #178

Merged
merged 1 commit into from
Aug 26, 2015

Conversation

imbcmdth
Copy link
Member

Only trigger loadstart and loadedmetadata if we are in data generation mode and let the tech emit those events since it has a better knowledge about when to do so.

Only trigger loadstart and loadedmetadata if we are in data generation mode and let the tech emit those events since it has a better knowledge about when to do so.
@imbcmdth
Copy link
Member Author

This was included into stable already (but then reverted) and now being forward-ported to master.

@imbcmdth imbcmdth changed the title Conditionally trigger loadstart and loadedmetadata Conditionally trigger loadstart and loadedmetadata for 5.0 Aug 11, 2015
@dmlap
Copy link
Member

dmlap commented Aug 26, 2015

lgtm

@heff
Copy link
Member

heff commented Aug 26, 2015

lgtm

Only if we're not in data generation mode, yeah?

@dmlap
Copy link
Member

dmlap commented Aug 26, 2015

Yes, I believe that's just a typo.

dmlap added a commit that referenced this pull request Aug 26, 2015
Conditionally trigger loadstart and loadedmetadata for 5.0
@dmlap dmlap merged commit 8fa9d66 into videojs:master Aug 26, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants