Fix leak-checking for iOS Simulators
authorjbedard@apple.com <jbedard@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Aug 2017 15:46:30 +0000 (15:46 +0000)
committerjbedard@apple.com <jbedard@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Aug 2017 15:46:30 +0000 (15:46 +0000)
https://bugs.webkit.org/show_bug.cgi?id=175735

Reviewed by David Kilzer.

Follow-up fix to r220942. _proc may be undefined.

* Scripts/webkitpy/port/server_process.py:
(ServerProcess._wait_for_stop): Handle case where _proc is undefined.

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

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

index 6df37d1..415c94d 100644 (file)
@@ -1,3 +1,15 @@
+2017-08-22  Jonathan Bedard  <jbedard@apple.com>
+
+        Fix leak-checking for iOS Simulators
+        https://bugs.webkit.org/show_bug.cgi?id=175735
+
+        Reviewed by David Kilzer.
+
+        Follow-up fix to r220942. _proc may be undefined.
+
+        * Scripts/webkitpy/port/server_process.py:
+        (ServerProcess._wait_for_stop): Handle case where _proc is undefined.
+
 2017-08-22  Carlos Alberto Lopez Perez  <clopez@igalia.com>
 
         [GTK][WPE] install-dependencies should install the required gstreamer packages needed for production builds.
index 36016ff..a00e541 100644 (file)
@@ -359,18 +359,18 @@ class ServerProcess(object):
         killed = False
         if timeout_secs:
             deadline = now + timeout_secs
-            while self._proc.poll() is None and time.time() < deadline:
+            while self._proc and self._proc.poll() is None and time.time() < deadline:
                 time.sleep(0.01)
-            if self._proc.poll() is None:
+            if self._proc and self._proc.poll() is None:
                 _log.warning('stopping %s(pid %d) timed out, killing it' % (self._name, self._proc.pid))
 
-        if self._proc.poll() is None:
+        if self._proc and self._proc.poll() is None:
             self._kill()
             killed = True
             _log.debug('killed pid %d' % self._proc.pid)
 
         # read any remaining data on the pipes and return it.
-        if not killed:
+        if self._proc and not killed:
             if self._use_win32_apis:
                 self._wait_for_data_and_update_buffers_using_win32_apis(now)
             else: