Automerge: [lldb][NFC] Address follow up comments in ExecutionContext (#153110)

The PR llvm/llvm-project#152020 got merged by
accident. This commit addresses some follow up review comments.

Author: felipepiovezan

lldb/include/lldb/Target/ExecutionContext.h

@@ -592,9 +592,9 @@ struct StoppedExecutionContext : ExecutionContext {
   }
 
   /// Clears this context, unlocking the ProcessRunLock and returning the
-  /// locked API lock. Like after a move operation, this object is no longer
-  /// usable.
-  [[nodiscard]] std::unique_lock<std::recursive_mutex> Destroy();
+  /// locked API lock, allowing callers to resume the process. Similar to
+  /// a move operation, this object is no longer usable.
+  [[nodiscard]] std::unique_lock<std::recursive_mutex> AllowResume();
 
 private:
   std::unique_lock<std::recursive_mutex> m_api_lock;

lldb/source/API/SBThread.cpp

@@ -525,7 +525,7 @@ static Status ResumeNewPlan(StoppedExecutionContext exe_ctx,
   process->GetThreadList().SetSelectedThreadByID(thread->GetID());
 
   // Release the run lock but keep the API lock.
-  std::unique_lock<std::recursive_mutex> api_lock = exe_ctx.Destroy();
+  std::unique_lock<std::recursive_mutex> api_lock = exe_ctx.AllowResume();
   if (process->GetTarget().GetDebugger().GetAsyncExecution())
     return process->Resume();
   return process->ResumeSynchronous(nullptr);

lldb/source/Target/ExecutionContext.cpp

@@ -138,33 +138,33 @@ lldb_private::GetStoppedExecutionContext(
     const ExecutionContextRef *exe_ctx_ref_ptr) {
   if (!exe_ctx_ref_ptr)
     return llvm::createStringError(
-        "ExecutionContext created with an empty ExecutionContextRef");
+        "StoppedExecutionContext created with an empty ExecutionContextRef");
 
   lldb::TargetSP target_sp = exe_ctx_ref_ptr->GetTargetSP();
   if (!target_sp)
     return llvm::createStringError(
-        "ExecutionContext created with a null target");
+        "StoppedExecutionContext created with a null target");
 
   auto api_lock =
       std::unique_lock<std::recursive_mutex>(target_sp->GetAPIMutex());
 
   auto process_sp = exe_ctx_ref_ptr->GetProcessSP();
   if (!process_sp)
     return llvm::createStringError(
-        "ExecutionContext created with a null process");
+        "StoppedExecutionContext created with a null process");
 
   ProcessRunLock::ProcessRunLocker stop_locker;
   if (!stop_locker.TryLock(&process_sp->GetRunLock()))
     return llvm::createStringError(
-        "attempted to create an ExecutionContext with a running process");
+        "attempted to create a StoppedExecutionContext with a running process");
 
   auto thread_sp = exe_ctx_ref_ptr->GetThreadSP();
   auto frame_sp = exe_ctx_ref_ptr->GetFrameSP();
   return StoppedExecutionContext(target_sp, process_sp, thread_sp, frame_sp,
                                  std::move(api_lock), std::move(stop_locker));
 }
 
-std::unique_lock<std::recursive_mutex> StoppedExecutionContext::Destroy() {
+std::unique_lock<std::recursive_mutex> StoppedExecutionContext::AllowResume() {
   Clear();
   m_stop_locker = ProcessRunLock::ProcessRunLocker();
   return std::move(m_api_lock);