Remove webpagereplay_go_server.SetGoBinaryPath() The last use was removed in crrev.com/c/7817662. Bug: 495366518 Change-Id: I75272dd45f5c13a5d7f5b4d3fa9bdbeb4242d1d3 Reviewed-on: https://chromium-review.googlesource.com/c/catapult/+/7816978 Reviewed-by: John Chen <johnchen@chromium.org>
diff --git a/common/py_utils/py_utils/webpagereplay_go_server.py b/common/py_utils/py_utils/webpagereplay_go_server.py index 1ef346e..18a48f6 100644 --- a/common/py_utils/py_utils/webpagereplay_go_server.py +++ b/common/py_utils/py_utils/webpagereplay_go_server.py
@@ -149,24 +149,6 @@ 'wpr_go', py_utils.GetHostOsName(), py_utils.GetHostArchName()) return ReplayServer._go_binary_path - @classmethod - def SetGoBinaryPath(cls, go_binary_path): - """Overrides the _go_binary_path. - - This allows the server to use WPRGO files retrieved from somewhere - other than GCS via binary_manager, such as test isolation. - - For chromium project to use WPR, it is encourage to use test isolation, - and therefore should call SetGoBinaryPath to set _go_binary_path. - - For Catapult/Telemetry project, the tradition is to download wpr_go - binary via binary_manager. So do not call SetGoBinaryPath. - """ - if not os.path.exists(go_binary_path): - raise ValueError('SetGoBinaryPath could not set {} as it does not exist' - .format(go_binary_path)) - cls._go_binary_path = go_binary_path - @property def http_port(self): return self._started_ports['http']