1766711 - A shutdown crash on a WPT crashtest still marks the test as pass

Status: NEW

(Testing :: web-platform-tests, defect)

Description

[Kagami Rosylight [:saschanaz] (they/them)]

Can it wait for the process to do a full shutdown before marking it as pass?

Comment 1

[James Graham [:jgraham]]

We don't restart the process after each passing test, so it seems hard to attribute a crash to a specific test. Do Gecko crashtests actually restart once per test, or how does the accounting work?

Comment 2

[Kagami Rosylight [:saschanaz] (they/them)]

I'm not sure, all I know at this point is that Gecko crashtests do attribute it to something (maybe just to the whole command). I'm on PTO now, I'll check next week.

Comment 3

[Kagami Rosylight [:saschanaz] (they/them)]

ᅟSo:

load shutdown-crash.html
load empty.html

Crashtest attributes it to empty.html. 🥲

It would be great if it detect shutdown crash (by checking mozilla::dom::ContentProcess::CleanUp in the stack?) and attribute it to nothing, but still marking it as a test failure. That will help detecting regressions at least.

Comment 4

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:jgraham, could you have a look please?

For more information, please visit auto_nag documentation.