Move calls to DisableAllBreakpointSites() and m_thread_list.DiscardThreadPlans() into base Process::Destroy() instead of in subclass DoDestroy() methods.

llvm-svn: 204752

lldb/source/Plugins/Process/FreeBSD/ProcessFreeBSD.cpp

@@ -133,8 +133,6 @@ ProcessFreeBSD::DoDetach(bool keep_stopped)
         return error;
     }
 
-    DisableAllBreakpointSites();
-
     error = m_monitor->Detach(GetID());
 
     if (error.Success())

lldb/source/Plugins/Process/MacOSX-Kernel/ProcessKDP.cpp

@@ -599,10 +599,6 @@ ProcessKDP::DoDetach(bool keep_stopped)
     }
     else
     {
-        DisableAllBreakpointSites ();
-        
-        m_thread_list.DiscardThreadPlans();
-        
         // If we are going to keep the target stopped, then don't send the disconnect message.
         if (!keep_stopped && m_comm.IsConnected())
         {

lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp

@@ -1866,10 +1866,6 @@ ProcessGDBRemote::DoDetach(bool keep_stopped)
     if (log)
         log->Printf ("ProcessGDBRemote::DoDetach(keep_stopped: %i)", keep_stopped);
  
-    DisableAllBreakpointSites ();
-
-    m_thread_list.DiscardThreadPlans();
-
     error = m_gdb_comm.Detach (keep_stopped);
     if (log)
     {

lldb/source/Target/Process.cpp

@@ -3692,6 +3692,9 @@ Process::Detach (bool keep_stopped)
             }
         }
     
+        m_thread_list.DiscardThreadPlans();
+        DisableAllBreakpointSites();
+
         error = DoDetach(keep_stopped);
         if (error.Success())
         {