Debugger: Move watchHandler()->notifyUpdateStarted() to doUpdateLocals

This was it also gets triggered when expanding tooltips.

Change-Id: I90ad39e9e82abdcd40e663b4ba4998205563009b
Reviewed-by: David Schulz <david.schulz@theqtcompany.com>

src/plugins/debugger/cdb/cdbengine.cpp

@@ -1341,6 +1341,9 @@ void CdbEngine::doUpdateLocals(const UpdateParameters &updateParameters)
         watchHandler()->removeAllData();
         return;
     }
+
+    watchHandler()->notifyUpdateStarted();
+
     /* Watchers: Forcibly discard old symbol group as switching from
      * thread 0/frame 0 -> thread 1/assembly -> thread 0/frame 0 will otherwise re-use it
      * and cause errors as it seems to go 'stale' when switching threads.

src/plugins/debugger/debuggerengine.cpp

@@ -1551,7 +1551,6 @@ void DebuggerEngine::createSnapshot()
 void DebuggerEngine::updateLocals()
 {
     watchHandler()->resetValueCache();
-    watchHandler()->notifyUpdateStarted();
     doUpdateLocals(UpdateParameters());
 }
 

src/plugins/debugger/gdb/gdbengine.cpp

@@ -4660,6 +4660,8 @@ void GdbEngine::doUpdateLocals(const UpdateParameters &params)
 {
     m_pendingBreakpointRequests = 0;
 
+    watchHandler()->notifyUpdateStarted();
+
     DebuggerCommand cmd("showData");
     watchHandler()->appendFormatRequests(&cmd);
 

src/plugins/debugger/lldb/lldbengine.cpp

@@ -864,6 +864,8 @@ void LldbEngine::doUpdateLocals(const UpdateParameters &params)
         return;
     }
 
+    watchHandler()->notifyUpdateStarted();
+
     DebuggerCommand cmd("updateData");
     cmd.arg("nativeMixed", isNativeMixedActive());
     watchHandler()->appendFormatRequests(&cmd);