Open Bug 1773683 Opened 2 years ago Updated 9 months ago

DelazifyTask are interruptible, we should plug it into mozilla::Task::RequestInterrupt.

Categories

(Core :: JavaScript Engine, enhancement, P3)

Product:
Core
Component:
JavaScript Engine
Type:
enhancement

Tracking

()

People

(Reporter: nbp, Unassigned)

References

(Blocks 1 open bug)

Details

Bug 1773682 changes the DelazifyTask to parse all selected function in a single schedule.
While this works fine, we should play nicely with the rest of Task Controller tasks, as the main thread can take over any forgotten functions.

The simplest solution, is to make the DelazifyTask interruptible, and honoring the mozilla::Task::RequestInterrupt function in the case of DelazifyTask.

Possible options are:

  • (a) override in RequestInterrupt in HelperThreadTaskHandler, and let it handle DelazifyTask's case
  • (b) Use different Task subclass for DelazifyTask and let it override RequestInterrupt

(a) is simpler if there can be any other interruptible tasks.

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