[all-commits] [llvm/llvm-project] a22714: [lldb-dap] Refactor handling progress events. (#22...

[Ebuka Ezike via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a22714d1867d7941681c129ea1a37c85541a1ad4
      https://github.com/llvm/llvm-project/commit/a22714d1867d7941681c129ea1a37c85541a1ad4
  Author: Ebuka Ezike <e_ezike at apple.com>
  Date:   2026-09-22 (Tue, 22 Sep 2026)

  Changed paths:
    M lldb/test/API/tools/lldb-dap/progress/Progress_emitter.py
    M lldb/test/API/tools/lldb-dap/progress/TestDAP_Progress.py
    M lldb/tools/lldb-dap/DAP.cpp
    M lldb/tools/lldb-dap/DAP.h
    M lldb/tools/lldb-dap/ProgressEvent.cpp
    M lldb/tools/lldb-dap/ProgressEvent.h
    M lldb/tools/lldb-dap/Protocol/ProtocolEvents.cpp
    M lldb/tools/lldb-dap/Protocol/ProtocolEvents.h
    M lldb/unittests/DAP/CMakeLists.txt
    A lldb/unittests/DAP/ProgressEventTest.cpp
    M lldb/unittests/DAP/ProtocolEventsTest.cpp

  Log Message:
  -----------
  [lldb-dap] Refactor handling progress events. (#224957)

lldb-dap now only uses one thread to receive and throttle received
progress events.
For every progress with a progressId, we delay reporting the progress
for 1000ms until we have recieved a new progress and only send an update
if the progress has a new version after 250ms.

This means for every progress integration test we do nothing for at
least 1250ms. The previous `TestDAP_progress` runs for at least 8s
regardless of how fast the computer is.

Moved most of the test to unittest where we can simulate progress delays
and time passing. We now have only one `TestDAP_progress` test to verify
the client receives progress Events.

When the ProgressEventThread reports a new progressEvent, the reporter
then.
- Create or update the `PendingProgress` with that progressId.
- Determines the action to take based on the ProgressState,
k_start_delay and k_update_interval.
- Sends the apropriate Event based on the action calculated.

Migrate the API test to the new infrastructure.
Add unit tests for the ProgressEventReporter and protocol progressXXXX
POD.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications