Bugfix: remove missed reference to command_succeeded

openwpm · Aug 23, 2019 · 24c1925 · 24c1925
1 parent d1eb54c
commit 24c1925
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/automation/TaskManager.py b/automation/TaskManager.py
@@ -457,7 +457,6 @@ def _issue_command(self, browser, command_sequence, condition=None):
             browser.current_timeout = timeout
             # passes off command and waits for a success (or failure signal)
             browser.command_queue.put(command)
-            command_succeeded = 0  # 1 success, 0 error, -1 timeout
             command_arguments = command[1] if len(command) > 1 else None
 
             # received reply from BrowserManager, either success or failure
@@ -520,7 +519,7 @@ def _issue_command(self, browser, command_sequence, condition=None):
             if critical_failure:
                 return
 
-            if command_succeeded != 1:
+            if command_status != 'ok':
                 with self.threadlock:
                     self.failurecount += 1
                 if self.failurecount > self.failure_limit: