Open issues

• Sources (file tree)
  • List inline classic scripts Devtools: initial Debugger > Sources panel #36164
  • List inline module scripts Devtools: initial Debugger > Sources panel #36164
  • List external classic scripts Devtools: initial Debugger > Sources panel #36164
  • List external module scripts
  • List external classic worker scripts
    • Devtools: Support worker scripts in Debugger > Source panel #36562
    • DevTools: worker_id should be id & thread for workers should be thread #36631
    • DevTools: Add resource_available as a common shared util  #36632
    • Devtools: No worker files in Debugger> Source panel anymore #37012
    • Devtools: Fix url parsing for worker urls #36727
    • devtools: Fix worker targets in debugger tab #43981
  • List external module worker scripts
  • Support files created by source maps
• Source contents
  • DevTools: Move Source related code to dedicated source.rs file #36667
  • DevTools: Implement support for showing source_content in Debugger > Source panel #36774
  • DevTools: Fix empty debugger > source panel #37197
  • Devtools: refactor source actor state #37528
  • Devtools: sources for HTML files should be the whole HTML file #36874
  • DevTools: sources for HTML files should be the whole HTML file #37456
  • devtools: Make contentType optional in source actor source responses #38330
  • devtools: Fix source contents tests and fix a race #38359
  • Allow embedder to indicate whether scripts are inline mozjs#603
  • script: Tell SpiderMonkey whether scripts are inline #38363
  • devtools: Expose introductionType to devtools clients #38541
  • script: Set correct introductionType values in more places #38550
  • devtools: Create source actors from Debugger API notifications #38334
  • Support source maps
• Set breakpoints
  • breakpoint: remove breakpoint
    • devtools: Handle removeBreakpoint on breakpoint-list actor #38797
  • thread: getAvailableEventBreakpoints request handler
  • source: getBreakableLines and getBreakpointPositionsCompressed request handlers
    • devtools: Show clients where they can set breakpoints #37667
    • devtools: Fix available breakpoint positions with nested scripts #38826
  • Create debugger global and debugger.js
    • script: Add support for creating globals in isolated compartments #38236
    • script: Create a debugger script for the SpiderMonkey Debugger API #38331
    • script: Add new Window globals as debuggees #38333
    • script: Remove logging in debugger script #38475
    • script: Add new worker globals as debuggees #38551
    • script: Rename DebuggerEvent to DebuggerAddDebuggeeEvent #38571
    • Iterating on embedded resources is slow #38382
    • devtools: Get the correct debuggee in debugger.js #42774
  • Debugger API
    • saveJobQueue() Implement saveJobQueue() for the SpiderMonkey Debugger API mozjs#595
      script: Implement jsglue traps for saveJobQueue() #38232
    • runJobs() Implement runJobs() for the SpiderMonkey Debugger API mozjs#597
      script: Implement jsglue trap for runJobs() #38265
    • SetScriptEnvironmentPreparer() Expose ScriptEnvironmentPreparer to Rust mozjs#598
  • Breakpoint List actor
    • setActiveEventBreakpoints
    • setXHRBreakpoint
    • removeXHRBreakpoint
  • devtools: Handle the setBreakpoint message. #41925
  • devtools: implement clearBreakpoint #42154
  • devtools: implement pause and resume in debugger #42580
• Pause, resume, and step threads
  • Block synchronously while pumping a nested event loop. More details on zulip
  • Stop blocking in response to other devtools messages
  • thread: interupt
    • devtools: Handle pause in the debugger #42007
    • devtools: Unify pause handling for breakpoints and interrupt  #42599
  • thread: resume
    • resume
    • step
    • next
    • finish
    • devtools: Implement initial support for Step in, Step out and Step over #42907
    • devtools: Make why attribute use PauseReasons #42878
    • devtools: Pass steppingType to onPop hook #43995
  • thread: sources request handler
  • thread: source request handler
  • thread: release
  • thread: attach / detach
  • thread: frames request handler
    • devtools: implement listing all frames #43015
  • new actor type frame
    • devtools: Register frame actor before pause #42844
    • devtools: Use getOffsetMetadata in Frame #42874
    • devtools: Avoid creating unnecessary new frame actors #42906
    • devtools: Use actual values fromFrameState #44744
• Black-boxing
  • devtools: Implement getBlackboxingActor #44142
  • source: blackbox request handler
  • blackbox requests on blackboxing actors
  • source: unblackbox request handler
  • getBlackboxingActor requests on watcher actors
  • unblackbox requests on blackboxing actors
• Implement scopes panel
  • getEnvironment message
  • devtools: Do not cache environment actor values #44765
• Implement call stack panel
  • Basic call stack support
  • Support async functions
  • unwatchResources requests on watcher actors
  • unwatchTargets requests on watcher actors
  • detach requests on target actors

Other

• devtools: improve ID naming for better readability and context #35942
• devtools: Use webview_id as browser_id #35956
• DevTools: Improve resource_available to handle multiple connections #36933
• Devtools: send error replies instead of ignoring messages #37683
  • Devtools: send error replies instead of ignoring messages #37686
• Gate pull requests on mozjs-sys bump if there are any C++ changes mozjs#601
• devtools: Warn when sending common protocol errors #38548
• devtools: Fix getBreakpointListActor handler in watcher actor #38624
• devtools: Fix recursion in debugger #44024
• devtools: Move debugger_value_to_json to lib.rs #44160

Related

• Automated tests devtools: Improve testing #44256
• Console tab devtools: Console tab #39858
• Style Editor tab devtools: Style Editor tab #44266

• In Firefox, sources are seen in resources-available-array of type network-event, source
  • See also console-message, if there were any past console messages
• We can send existing sources in resources-available-array for source (see watcher.js:531)
• But client never requests them in watchResources for source despite firefox.js:84
  • Tried setting sources to true in RootActor traits
  • Tried setting source to true in SessionContext supported_resources
  • Maybe need to send empty resources-available-array for console-message and error-message? Servo currently just ignores those watchResources requests, the internal BrowsingContextActor::resource_available takes and sends exactly one resource

Other TODO's:

(cc @atbrakhi)