Open Bug 1691784 Opened 4 years ago Updated 4 years ago

Handle thread actor reconfigure options using the watcher Configuration actor

Categories

(DevTools :: Framework, task, P3)

task

Tracking

(Not tracked)

People

(Reporter: jdescottes, Unassigned)

References

Details

In Bug 1690698, we introduce a new Configuration actor that will be used to propagate devtools settings from the client to the server.

We will start using it for the browsing-context target options passed via the reconfigure helper.
The thread actor also has a reconfigure method and supports a set of options:

  • ignoreCaughtExceptions
  • logEventBreakpoints
  • observeAsmJS
  • pauseWorkersUntilAttach
  • pauseOnExceptions
  • shouldIncludeAsyncLiveFrames
  • shouldIncludeSavedFrames
  • shouldShowOverlay
  • skipBreakpoints

There is also breakpoints and eventBreakpoints although I am not sure what is there role compare to breakpoints set via the regular setBreakpoint APIs.

I was aiming to tackle most of these flags via bug 1687261.
With explicit methods on explicit actors. Like setBreakpoint on BreakpointListActor.
I planned that based on your original feedback for breakpoints.
Do not expose addWatcherDataEntry straight to the client and instead use explicit method for each data entry.
It looks like your circle back to this original proposal, but with a different naming.

Should we pass all data that is useful across navigation via the configuration actor ?
If not, what is the definition of what is a config versus what shouldn't be a config?
If yes, should we get rid of BreakpointListActor and replace it by a configuration?
Also, if yes:

  • all server side events would be resources,
  • all permanent actions would be configs,
  • only transcient actions and on-demand data fetching would require custom methods
You need to log in before you can comment on or make changes to this bug.