ModuleInspector: Don't restart inspection if the queue is empty.

StartInspectingModule() was being called regardless of the state of the
queue in the event of a connection error with the UtilWin service.

This was wrong because the connection error can happen even if the
ModuleInspector is idle.

Bug: 1100982
Change-Id: I715a9014e54e143d37ed94a07925fd70d3caf456
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2422259Reviewed-by: Chris Hamilton <chrisha@chromium.org>
Commit-Queue: Chris Hamilton <chrisha@chromium.org>
Commit-Queue: Patrick Monette <pmonette@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809384}

chrome/browser/win/conflicts/module_inspector.cc

@@ -217,9 +217,9 @@ void ModuleInspector::OnUtilWinServiceConnectionError() {
   // Disconnect from the service.
   remote_util_win_.reset();
 
-  // Restart inspection for the current module, only if the retry limit wasn't
+  // Restart the inspection if there is work to do and the retry limit wasn't
   // reached.
-  if (connection_error_retry_count_--)
+  if (!queue_.empty() && connection_error_retry_count_--)
     StartInspectingModule();
 }