Open Bug 1749725 Opened 3 years ago Updated 3 years ago

Verify if WorkerRunnable::Cancel can just assert against re-entrance

Categories

(Core :: DOM: Workers, task)

task

Tracking

()

People

(Reporter: jstutte, Unassigned)

References

Details

In bug 1741182 we updated all WorkerRunnable::Cancel overrides for correct error handling. During review, :smaug asked if we could instead just assert this.

Here is a try run with this assert.

Even if this is green, it is not immediately clear, if Cancel can be called (or triggered to be called) more than once through JS and having appropriate error handling is actually needed here.

You need to log in before you can comment on or make changes to this bug.