Closed Bug 801682 Opened 9 years ago Closed 9 years ago

Enable media crashtests on the alder branch (and disable leak checks)

Categories

(Core :: WebRTC, defect, P1)

19 Branch
defect

Tracking

()

RESOLVED FIXED

People

(Reporter: whimboo, Assigned: whimboo)

References

Details

(Whiteboard: [WebRTC][do not merge to m-c!!][blocking-webrtc-][qa-])

Attachments

(1 file, 1 obsolete file)

Crashtests for /dom/media/ can't be checked in right now due to leakage. Given that we will have to do the following:

* We need a patch which enables the tests. This has to happen in testing/crashtest/crashtests.list by adding:

"include ../../dom/media/tests/crashtests/crashtests.list"

Keep in mind that my first crashtest on bug 798873 has to land first.

* Disable the processing of the leak log:

http://mxr.mozilla.org/mozilla-central/source/layout/tools/reftest/runreftest.py#136

Eric, can you take that?
Depends on: 798873
I will take care once my patch on bug 798873 is on alder.
Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Severity: normal → major
Most of work is happening on central now, right, given that we did the large import of the code now? Is this really essential now that most development is happening on central?
We can't run crashtests at all on m-c due to the leak situation. So we need a way to get those executed. This was discussed in detail yesterday on IRC and is the only viable option we have. As long as we have those leaks we should continue with merging from m-c to the alder branch. The tests for the branch builds will show us if something regressed.
Depends on: 791270
No longer depends on: 798873
Can anyone please merge m-c into alder so we can get this implemented? I will have a patch ready in a bit but it will not apply until the latest code from m-c is on the alder branch.
Whiteboard: [WebRTC] → [WebRTC][do not merge to m-c until all leaks are fixed]
Due to the change in runreftests.py we actually should never merge this patch into m-c.
Whiteboard: [WebRTC][do not merge to m-c until all leaks are fixed] → [WebRTC][do not merge to m-c!!]
Attached patch Patch v1 (obsolete) — Splinter Review
For testing purposes I have pushed it to try for the m-c branch:
https://tbpl.mozilla.org/?tree=Try&rev=ec267581775c
Attachment #672207 - Flags: review?(rjesup)
Priority: -- → P1
Whiteboard: [WebRTC][do not merge to m-c!!] → [WebRTC][do not merge to m-c!!][blocking-webrtc-]
Comment on attachment 672207 [details] [diff] [review]
Patch v1

Review of attachment 672207 [details] [diff] [review]:
-----------------------------------------------------------------

::: layout/tools/reftest/runreftest.py
@@ +133,5 @@
>                                   # give the JS harness 30 seconds to deal
>                                   # with its own timeouts
>                                   timeout=options.timeout + 30.0)
> +      # Disable leak checks until we have time to fix those leaks
> +      # processLeakLog(self.leakLogFile, options.leakThreshold)

Add a comment that this should not get to m-c
Attachment #672207 - Flags: review?(rjesup) → review+
Attached patch Patch v1.1Splinter Review
Updated patch with the comment added. Taking over r+ from last version.

Randell, do you have any update regarding how you want to handle merges from m-c to alder? When can we land this patch? Would you mind doing so? I do not have a local alder checkout present anymore. Thanks!
Attachment #672207 - Attachment is obsolete: true
Attachment #675107 - Flags: review+
Comment on attachment 675107 [details] [diff] [review]
Patch v1.1

This has been landed today:
https://hg.mozilla.org/projects/alder/rev/67ebbc2cf0ff

Randell, shall we keep the bug open for a likely backout later?
Attachment #675107 - Flags: checkin+
Summary: Enable media crashtests on the alder branch → Enable media crashtests on the alder branch (and disable leak checks)
Depends on: 811873
Leave open for eventual landing on m-c
At some point we'll likely re-image alder rather than back things out.
(In reply to Randell Jesup [:jesup] from comment #10)
> Leave open for eventual landing on m-c
> At some point we'll likely re-image alder rather than back things out.

The patch here should never land on m-c! We would bust the whole reftest run. :) the only option we have here is to backout the patch or re-image as you have proposed.
This bug has been fixed and we don't plan to do anymore work on it. Once we can enable leak checks we will reopen for a partly backout.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Depends on: 812648
Whiteboard: [WebRTC][do not merge to m-c!!][blocking-webrtc-] → [WebRTC][do not merge to m-c!!][blocking-webrtc-][qa-]
You need to log in before you can comment on or make changes to this bug.