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

Let js fire "seeking" #171

Merged
merged 1 commit into from
Jul 21, 2015
Merged

Let js fire "seeking" #171

merged 1 commit into from
Jul 21, 2015

Conversation

dmlap
Copy link
Member

@dmlap dmlap commented Jul 17, 2015

The flash tech has all the information necessary to fire "seeking" and is much easier to validate. Remove this logic from the SWF and let the js wrapper handle it.

The flash tech has all the information necessary to fire "seeking" and is much easier to validate. Remove this logic from the SWF and let the js wrapper handle it.
@gkatsev
Copy link
Member

gkatsev commented Jul 17, 2015

LGTM

@heff
Copy link
Member

heff commented Jul 17, 2015

This is gonna require a major release of the swf, so we'll need to figure out how we're gonna handle that.

@gkatsev
Copy link
Member

gkatsev commented Jul 17, 2015

A major release of the swf doesn't necessarily mean a major release of videojs. Though, since we're updating it as part of 5.0, sounds alright to me.

@heff
Copy link
Member

heff commented Jul 17, 2015

Yeah, I'm not worried about video.js for this. More interested in if we're locking down swf-4.x or if we're going to continue to patch it.

@dmlap
Copy link
Member Author

dmlap commented Jul 17, 2015

I think we might as well do a major release of the SWF for vjs5. I don't think I'll be able to port contrib-hls to a source handler without making this sort of adjustment or adding a whole bunch of hooks into the source handler API to intercept things like seeking. Based on our previous discussions, intercepting all the tech methods doesn't seem like the our preferred route.

@gkatsev
Copy link
Member

gkatsev commented Jul 17, 2015

I'm ok with locking down the swf for vjs-4.x. Though, @bclwhitaker might have a different answer :)

@dmlap
Copy link
Member Author

dmlap commented Jul 21, 2015

Based on the comments in this thread, we're going to merge this into master (starting vjs-swf 5.x) and make stable the 4.x branch until video.js 5.0 is released.

dmlap added a commit that referenced this pull request Jul 21, 2015
@dmlap dmlap merged commit 94bdb08 into videojs:master Jul 21, 2015
@dmlap dmlap deleted the do-not-seeking branch July 21, 2015 18:13
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