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.

Sign up for GitHub

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

Jump to bottom

Clarify asynchronicity in duration change algorithm when reducing duration #20

Closed
wolenetz opened this issue Oct 13, 2015 · 3 comments
Closed

Clarify asynchronicity in duration change algorithm when reducing duration #20

wolenetz opened this issue Oct 13, 2015 · 3 comments
Assignees
@jdsmith3000
Labels
needs implementation
Milestone
V1

Comments

@wolenetz
Copy link
Member

Migrated from w3c bugzilla tracker. For history prior to migration, please see:
https://www.w3.org/Bugs/Public/show_bug.cgi?id=27982

It was previously assigned to Adrian Bateman. Editors will sync soon to determine who to take this bug.
@wolenetz wolenetz added this to the V1 milestone Oct 15, 2015
@wolenetz wolenetz assigned wolenetz and jdsmith3000 and unassigned wolenetz Oct 15, 2015
@jdsmith3000 jdsmith3000 mentioned this issue Oct 28, 2015
Closed
@jdsmith3000
Copy link
Contributor

We plan to disallow aborts of a range removal to insure results are deterministic, and separate range removal from duration changes that would currently do an automatic remove. That affects both this issue and Issue 19.

@jdsmith3000
Copy link
Contributor

See latest comment in issue #19. We want to return an error on abort() during range removal.

@jdsmith3000 jdsmith3000 mentioned this issue Apr 18, 2016
Closed
jdsmith3000 added a commit to jdsmith3000/media-source that referenced this issue Apr 19, 2016
@jdsmith3000
Fixed w3c#19, w3c#20, w3c#26: Require remove() for range removal
8adbbf4 
- Require remove() for all Range Removals
- Error on Duration Changes that need remove first
- Error on abort() during Range Removals
@jdsmith3000 jdsmith3000 mentioned this issue Apr 19, 2016
Merged
jdsmith3000 added a commit to jdsmith3000/media-source that referenced this issue May 5, 2016
@jdsmith3000
Fixed w3c#19, w3c#20, w3c#26: Require remove() for range removal
1df21ab 
- Require remove() for all Range Removals
- Error on Duration Changes that need remove first
- Error on abort() during Range Removals

- Updated to address review comments:
-  Change Duration: used starting presentation timestamp in duration
test
-  Expanded note on using remove()
-  Corrected indentation
-  Updated dates
jdsmith3000 added a commit to jdsmith3000/media-source that referenced this issue May 12, 2016
@jdsmith3000
Fixed w3c#19, w3c#20, w3c#26: Require remove() for range removal
1492a31 
- Require remove() for all Range Removals
- Error on Duration Changes that need remove first
- Error on abort() during Range Removals

Updated to address review comments:
- Change Duration: used starting presentation timestamp in duration test
- Expanded note on using remove()
- Corrected indentation
- Updated dates

Updated to address review comments:
-  Added "buffered coded frames" language to Duration Change check for duration reductions.
-  Restored Issue w3c#54 to Revision History and added link
-  Updated publishDate
jdsmith3000 added a commit to jdsmith3000/media-source that referenced this issue May 12, 2016
@jdsmith3000
Fixed w3c#19, w3c#20, w3c#26: Require remove() for range removal
5948949 
- Require remove() for all Range Removals
- Error on Duration Changes that need remove first
- Error on abort() during Range Removals

Updated to address review comments:
- Change Duration: used starting presentation timestamp in duration test
- Expanded note on using remove()
- Corrected indentation
- Updated dates

Updated to address review comments:
- Added "buffered coded frames" language to Duration Change check for duration reductions.
- Restored Issue w3c#54 to Revision History and added link
- Updated publishDate
jdsmith3000 added a commit to jdsmith3000/media-source that referenced this issue May 12, 2016
@jdsmith3000
Fixed w3c#19, w3c#20, w3c#26: Require remove() for range removal
b386c1c 
- Require remove() for all Range Removals
- Error on Duration Changes that need remove first
- Error on abort() during Range Removals

Updated to address review comments:
- Change Duration: used starting presentation timestamp in duration test
- Expanded note on using remove()
- Corrected indentation
- Updated dates

Updated to address review comments:
- Added "buffered coded frames" language to Duration Change check for duration reductions.
- Restored Issue w3c#54 to Revision History and added link
- Updated publishDate
@wolenetz
Copy link
Member Author

@jdsmith3000 fixed this with b386c1c merged in #65

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jdsmith3000 jdsmith3000

Labels
needs implementation
Projects
None yet
Milestone
V1
Development

No branches or pull requests
2 participants
@jdsmith3000 @wolenetz