1364001 - Don't throttle readahead until download is fast enough

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, enhancement, P1)

Description

[JW Wang [:jwwang] [:jw_wang]]

Per bug 1347174 comment 37, we want to download as much as possible for slow connections to get a decent playback quality.

Therefore, we should only turn on throttling when download is fast enough to continue playback without stopping for buffering.

Comment 1

[JW Wang [:jwwang] [:jw_wang]]

Attachment: Bug 1364001. P1 - add an API to throttle download.

Review commit: https://reviewboard.mozilla.org/r/139148/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/139148/

Comment 2

[JW Wang [:jwwang] [:jw_wang]]

Attachment: Bug 1364001. P2 - throttle download when we can play through.

Review commit: https://reviewboard.mozilla.org/r/139150/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/139150/

Comment 3

[Chris Pearce [:cpearce (Not reading bugmail)]]

Comment on attachment 8867595 [details]
Bug 1364001. P1 - add an API to throttle download.

https://reviewboard.mozilla.org/r/139148/#review142390

Good, thanks!

Comment 4

[Chris Pearce [:cpearce (Not reading bugmail)]]

Comment on attachment 8867596 [details]
Bug 1364001. P2 - throttle download when we can play through.

https://reviewboard.mozilla.org/r/139150/#review142392

Comment 5

[JW Wang [:jwwang] [:jw_wang]]

Thanks!

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/baa6989bbb1c
P1 - add an API to throttle download. r=cpearce
https://hg.mozilla.org/integration/autoland/rev/8518f6af32c2
P2 - throttle download when we can play through. r=cpearce

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/baa6989bbb1c
https://hg.mozilla.org/mozilla-central/rev/8518f6af32c2

Comment 8

[u589863]

(In reply to JW Wang [:jwwang] [:jw_wang] from comment #0)
> Per bug 1347174 comment 37, we want to download as much as possible for slow
> connections to get a decent playback quality.
> 
> Therefore, we should only turn on throttling when download is fast enough to
> continue playback without stopping for buffering.

(In reply to Chris Pearce (:cpearce) from comment #3)
> Comment on attachment 8867595 [details]
> Bug 1364001. P1 - add an API to throttle download.
> 
> https://reviewboard.mozilla.org/r/139148/#review142390
> 
> Good, thanks!

is there any preference to load entire media w/o checking/bypassing and throttling?

"media.cache_resume_threshold"
and "media.cache_readahead_limit" to 999999.


can we set it to -1 = unlimited? so all the media is loaded always

I ask because in our country speeds fluctuates a lot,
so we basically just open tabs in background and after some time switch to it and watch it so it's not buffering or anything and smooth as our connection is very unreliable, sometimes fast sometimes very slow

Comment 9

[Chris Pearce [:cpearce (Not reading bugmail)]]

jigsawmode: with the change in this bug, we should only throttle the download if we estimate that we can download it at a rate faster than it would be played. This change will be in tomorrow's Nightly build.

It would be great if you could download a Nightly build tomorrow from https://www.mozilla.org/en-US/firefox/channel/desktop/ and test that and confirm whether you see this behaviour on your network?

Comment 10

[u589863]

(In reply to Chris Pearce (:cpearce) from comment #9)
> jigsawmode: with the change in this bug, we should only throttle the
> download if we estimate that we can download it at a rate faster than it
> would be played. This change will be in tomorrow's Nightly build.
> 
> It would be great if you could download a Nightly build tomorrow from
> https://www.mozilla.org/en-US/firefox/channel/desktop/ and test that and
> confirm whether you see this behaviour on your network?

Thank you Sure will do that but 
Irrelevant of the connection? is there any preference to load entire media w/o checking/bypassing and throttling?
Just for eg say want to play at 3x normal speed but connection starts with good speeds but **** while watching(connection is flaky like i said that's why want full download/buffered in background)

"media.cache_resume_threshold"
and "media.cache_readahead_limit" to 999999.


can we set it to -1 = unlimited? so all the media is loaded always irrespective of connection speeds?

Comment 11

[Chris Pearce [:cpearce (Not reading bugmail)]]

(In reply to jigsawmode from comment #10)
> (In reply to Chris Pearce (:cpearce) from comment #9)
> > jigsawmode: with the change in this bug, we should only throttle the
> > download if we estimate that we can download it at a rate faster than it
> > would be played. This change will be in tomorrow's Nightly build.
> > 
> > It would be great if you could download a Nightly build tomorrow from
> > https://www.mozilla.org/en-US/firefox/channel/desktop/ and test that and
> > confirm whether you see this behaviour on your network?
> 
> Thank you Sure will do that but 
> Irrelevant of the connection? is there any preference to load entire media
> w/o checking/bypassing and throttling?
> Just for eg say want to play at 3x normal speed but connection starts with
> good speeds but **** while watching(connection is flaky like i said that's
> why want full download/buffered in background)
> 
> "media.cache_resume_threshold"
> and "media.cache_readahead_limit" to 999999.
> 
> 
> can we set it to -1 = unlimited? so all the media is loaded always
> irrespective of connection speeds?

Setting those prefs to a large number should give you the behaviour you want.

Comment 12

[u589863]

(In reply to Chris Pearce (:cpearce) from comment #11)
> (In reply to jigsawmode from comment #10)
> > (In reply to Chris Pearce (:cpearce) from comment #9)
> > > jigsawmode: with the change in this bug, we should only throttle the
> > > download if we estimate that we can download it at a rate faster than it
> > > would be played. This change will be in tomorrow's Nightly build.
> > > 
> > > It would be great if you could download a Nightly build tomorrow from
> > > https://www.mozilla.org/en-US/firefox/channel/desktop/ and test that and
> > > confirm whether you see this behaviour on your network?
> > 
> > Thank you Sure will do that but 
> > Irrelevant of the connection? is there any preference to load entire media
> > w/o checking/bypassing and throttling?
> > Just for eg say want to play at 3x normal speed but connection starts with
> > good speeds but **** while watching(connection is flaky like i said that's
> > why want full download/buffered in background)
> > 
> > "media.cache_resume_threshold"
> > and "media.cache_readahead_limit" to 999999.
> > 
> > 
> > can we set it to -1 = unlimited? so all the media is loaded always
> > irrespective of connection speeds?
> 
> Setting those prefs to a large number should give you the behaviour you want.

what will be default value in tomorrow's build?

you forgot to answer two important things,
Can i set it to -1 as unlimited or what should it be the maximum to trigger that behavior if -1 is not an option?
Sorry for asking stupid questions.

Comment 13

[JW Wang [:jwwang] [:jw_wang]]

(In reply to jigsawmode from comment #12)
> what will be default value in tomorrow's build?
"media.cache_readahead_limit" = 60, "media.cache_resume_threshold"=30.

 
> you forgot to answer two important things,
> Can i set it to -1 as unlimited
No.

> or what should it be the maximum to trigger
> that behavior if -1 is not an option?
> Sorry for asking stupid questions.
Set "media.cache_readahead_limit" to 999999.

Comment 14

[Danial Horton]

It seems on some html5 players, these settings might result in high cpu util in a universal crt thread responsible for interacting with the windows decoders.