Try runs, with the test version that's in phabricator, rooted on a commit... - before Bug 1961975's fix: https://treeherder.mozilla.org/jobs?repo=try&revision=b16e7709d932716f37d96f0f97802323c30639fe - after Bug 1961975's fix: https://treeherder.mozilla.org/jobs?repo=try&revision=30630f92e6c4fdbe8dffcf33a01358bc73bd5eca Note the crash on linux debug c1 at least. I don't know offhand how we're managing to avoid crashing on windows debug test runs there, but I'm not going to worry about it too much, since this is kind of an afterthought anyway and we've now got *some* test coverage for this.
Bug 1962274 Comment 2 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
Try runs, with the test version that's in phabricator, rooted on a commit... - before Bug 1961975's fix: https://treeherder.mozilla.org/jobs?repo=try&revision=b16e7709d932716f37d96f0f97802323c30639fe - after Bug 1961975's fix: https://treeherder.mozilla.org/jobs?repo=try&revision=30630f92e6c4fdbe8dffcf33a01358bc73bd5eca The test does trigger a crash on linux debug c1 in the "before" try run (which is good and indicates that it's giving us the test coverage that we need). I don't know offhand how we're managing to avoid crashing on windows debug test runs there, but I'm not going to worry about it too much, since this is kind of an afterthought anyway and we've now got *some* test coverage for this.
Try runs, with the test version that's in phabricator, rooted on a commit... - before Bug 1961975's fix: https://treeherder.mozilla.org/jobs?repo=try&revision=b16e7709d932716f37d96f0f97802323c30639fe - after Bug 1961975's fix: https://treeherder.mozilla.org/jobs?repo=try&revision=30630f92e6c4fdbe8dffcf33a01358bc73bd5eca The test does trigger a crash on linux debug c1 in the "before" try run (which is good and indicates that it's giving us the test coverage that we need). I don't know offhand how we're managing to avoid crashing on windows debug test runs in that same "before" Try run, but I'm not going to worry about it too much, since this is kind of an afterthought anyway and we've now got *some* test coverage for this.