Follow-up fix for r218996
authorjbedard@apple.com <jbedard@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 30 Jun 2017 18:10:37 +0000 (18:10 +0000)
committerjbedard@apple.com <jbedard@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 30 Jun 2017 18:10:37 +0000 (18:10 +0000)
Unreviewed infrastructure fix.

Repeated calls to 'ios_version' make the log noisy, memoize the function.

* Scripts/webkitpy/port/ios_simulator.py:
(IOSSimulatorPort): Memoize ios_version().

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

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

index bbf38b9..f166f84 100644 (file)
@@ -1,3 +1,14 @@
+2017-06-30  Jonathan Bedard  <jbedard@apple.com>
+
+        Follow-up fix for r218996
+
+        Unreviewed infrastructure fix.
+
+        Repeated calls to 'ios_version' make the log noisy, memoize the function.
+
+        * Scripts/webkitpy/port/ios_simulator.py:
+        (IOSSimulatorPort): Memoize ios_version().
+
 2017-06-30  Wenson Hsieh  <wenson_hsieh@apple.com>
 
         [iOS DnD] Drag caret rect is incorrectly computed when dropping in editable content in iframes
index 69b7776..42f99f9 100644 (file)
@@ -100,6 +100,7 @@ class IOSSimulatorPort(IOSPort):
             runtime = Runtime.from_version_string(self.host.platform.xcode_sdk_version('iphonesimulator'))
         return runtime
 
+    @memoized
     def ios_version(self):
         # FIXME: We should replace --runtime with something which makes sense for both Simulator and Device
         # https://bugs.webkit.org/show_bug.cgi?id=173775