Closed Bug 1224029 Opened 5 years ago Closed 5 years ago
_peer Connection _track Disabling .html | Test timed out .
backlog: --- → webrtc/webaudio+
Priority: -- → P3
This is the #2 top orange overall. Can somebody please take a look?
Short answer: Yes. Longer answer: I plan to do a full scrub of the orange factor list next week and get folks looking at all our (WebRTC/Web Audio) top oranges. (I realize there are more than just this one.) We had been doing pretty well on the orange factor for a while, but we've landed a lot of code recently -- which has changed the timing. Some of my team is on PTO this week, and uplift is Monday. So I can't make this a top priority this week, but after this week, I'll get folks looking at this and all the others so that by the end of March we've driven our orange count way down.
This is permafailing the Linux x64 asan M(2) tests: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-resultStatus=retry&filter-resultStatus=usercancel&filter-resultStatus=runnable&filter-searchStr=5d377d84fbc0b4d3e08a614ad9db7bd2b86ae82f&fromchange=90fae59aad6e&selectedJob=22995359 On aurora, it's the Linux pgo M(2). Please put this on the priority list because of the permafailure. Thank you.
Talked with mreavy about this. I'm working on this one; I have a failure caught in rr, so I have hope to fix this soon.
Assignee: nobody → rjesup
Rank: 35 → 10
Priority: P3 → P1
Cause is that a disabled track *may* render as a 1x1 black image, and drawImage(video, 0, 0) doesn't scale, so we get 1x1 black and sample pixel 50,50 (which is still green) -> timeout
Attachment #8728109 - Flags: review?(matt.woodrow)
Attachment #8728109 - Flags: review?(matt.woodrow) → review+
Comment on attachment 8728109 [details] [diff] [review] ensure video elements that may be disabled (black) are scaled Approval Request Comment [Feature/regressing bug #]: N/A [User impact if declined]: Test failures that need to be starred frequently [Describe test coverage new/current, TreeHerder]: This is a fix for a broken/racy test. No failures reported since this landed (in trees with the patch) from what I can see [Risks and why]: No risk to the product, purely a test script change [String/UUID change made/needed]: none
Comment on attachment 8728109 [details] [diff] [review] ensure video elements that may be disabled (black) are scaled Test-only changes don't need approval :)
You need to log in before you can comment on or make changes to this bug.