Closed Bug 1318982 Opened 3 years ago Closed 3 years ago

Enable dormant for most media mochitests

Categories

(Core :: Audio/Video: Playback, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox53 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

There are some test failures with dormant enabled:

test_Eviction_mp4.html
test_PlayEvents.html
test_ResumeAfterClearing_mp4.html

We will fix them in follow-up bugs.
Assignee: nobody → jwwang
Blocks: 1286129
Priority: -- → P3
Comment on attachment 8812641 [details]
Bug 1318982 - Enable dormant for most media mochitests.

https://reviewboard.mozilla.org/r/94288/#review94778

::: dom/media/mediasource/test/test_Eviction_mp4.html:36
(Diff revision 1)
>    });
>  }
>  
> +addMSEPrefs(
> +  ["media.mediasource.eviction_threshold.audio", 524288],
> +  ["media.dormant-on-pause-timeout-ms", -1] // Disable dormant.

not aligned... extra space line 35

::: dom/media/mediasource/test/test_Eviction_mp4.html:36
(Diff revision 1)
>    });
>  }
>  
> +addMSEPrefs(
> +  ["media.mediasource.eviction_threshold.audio", 524288],
> +  ["media.dormant-on-pause-timeout-ms", -1] // Disable dormant.

Could you please add a comment describing for each test where dormant is disabled, on why it's disabled with a bug created for each of those?

I don't see why dormant should be disabled here, seeing that dormant should be invisible to JS anyway
Attachment #8812641 - Flags: review?(jyavenard) → review+
Comment on attachment 8812641 [details]
Bug 1318982 - Enable dormant for most media mochitests.

https://reviewboard.mozilla.org/r/94288/#review94778

> not aligned... extra space line 35

Can you point me at the space? I can't find it.

> Could you please add a comment describing for each test where dormant is disabled, on why it's disabled with a bug created for each of those?
> 
> I don't see why dormant should be disabled here, seeing that dormant should be invisible to JS anyway

Still figuring out why dormant causes test failures. I will file follow-up bugs.
Comment on attachment 8812641 [details]
Bug 1318982 - Enable dormant for most media mochitests.

https://reviewboard.mozilla.org/r/94288/#review94778

> Can you point me at the space? I can't find it.

doh, I don't see it anymore..
Blocks: 1319292
Blocks: 1319293
Blocks: 1319295
Pushed by jwwang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/af9ac4bc7efd
Enable dormant for most media mochitests. r=jya
https://hg.mozilla.org/mozilla-central/rev/af9ac4bc7efd
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.