[REGRESSION] run-webkit-tests: No PID defined when searching for simulator crashlogs
authorjbedard@apple.com <jbedard@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 12 Aug 2019 23:43:55 +0000 (23:43 +0000)
committerjbedard@apple.com <jbedard@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 12 Aug 2019 23:43:55 +0000 (23:43 +0000)
https://bugs.webkit.org/show_bug.cgi?id=200644

Reviewed by Aakash Jain.

* Scripts/webkitpy/port/simulator_process.py:
(SimulatorProcess._start): Define system PID after launching app.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@248583 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/webkitpy/port/simulator_process.py

index 4fa7a94..9b16bda 100644 (file)
@@ -1,3 +1,13 @@
+2019-08-12  Jonathan Bedard  <jbedard@apple.com>
+
+        [REGRESSION] run-webkit-tests: No PID defined when searching for simulator crashlogs
+        https://bugs.webkit.org/show_bug.cgi?id=200644
+
+        Reviewed by Aakash Jain.
+
+        * Scripts/webkitpy/port/simulator_process.py:
+        (SimulatorProcess._start): Define system PID after launching app.
+
 2019-08-12  Sam Weinig  <weinig@apple.com>
 
         Replace multiparameter overloads of append() in StringBuilder as a first step toward standardizinging on the flexibleAppend() implementation
index f280d69..4762578 100644 (file)
@@ -94,6 +94,7 @@ class SimulatorProcess(ServerProcess):
         # 3 client connections will be accepted for stdin, stdout and stderr in that order.
         self._target_host.listening_socket.listen(3)
         self._pid = self._target_host.launch_app(self._bundle_id, self._cmd[1:], env=self._env)
+        self._system_pid = self._pid
 
         with Timeout(15, RuntimeError('Timed out waiting for pid {} to connect at port {}'.format(self._pid, self._target_host.listening_port()))):
             stdin = None