From 58e38905bb99814bc14e936531959f64201c144e Mon Sep 17 00:00:00 2001 From: Jos Demmers Date: Tue, 18 Jun 2024 23:49:51 +0200 Subject: [PATCH] Fixed download issue for D4Builds and Mobalytics imports. --- D4Companion.Services/BuildsManagerD4Builds.cs | 3 +++ D4Companion.Services/BuildsManagerMobalytics.cs | 3 +++ 2 files changed, 6 insertions(+) diff --git a/D4Companion.Services/BuildsManagerD4Builds.cs b/D4Companion.Services/BuildsManagerD4Builds.cs index 0772d2a5..f16ed678 100644 --- a/D4Companion.Services/BuildsManagerD4Builds.cs +++ b/D4Companion.Services/BuildsManagerD4Builds.cs @@ -261,6 +261,9 @@ public void DownloadD4BuildsBuild(string buildIdD4Builds) } finally { + // Note:You need to call driver.close() before driver.quit() otherwise you get lingering chrome processes with high resource usage. + // This is an issue with recent chrome versions (124+). + _webDriver?.Close(); _webDriver?.Quit(); _webDriver = null; _webDriverWait = null; diff --git a/D4Companion.Services/BuildsManagerMobalytics.cs b/D4Companion.Services/BuildsManagerMobalytics.cs index 826cbb76..3df0974f 100644 --- a/D4Companion.Services/BuildsManagerMobalytics.cs +++ b/D4Companion.Services/BuildsManagerMobalytics.cs @@ -287,6 +287,9 @@ public void DownloadMobalyticsBuild(string buildUrl) } finally { + // Note:You need to call driver.close() before driver.quit() otherwise you get lingering chrome processes with high resource usage. + // This is an issue with recent chrome versions (124+). + _webDriver?.Close(); _webDriver?.Quit(); _webDriver = null; _webDriverWait = null;