Debounce config saves

osu.Framework/Configuration/ConfigManager.cs

@@ -3,6 +3,8 @@
 
 using System;
 using System.Collections.Generic;
+using System.Threading;
+using System.Threading.Tasks;
 using osu.Framework.Configuration.Tracking;
 
 namespace osu.Framework.Configuration
@@ -119,7 +121,7 @@ public Bindable<U> Set<U>(T lookup, U value)
         protected virtual void AddBindable<TBindable>(T lookup, Bindable<TBindable> bindable)
         {
             ConfigStore[lookup] = bindable;
-            bindable.ValueChanged += _ => Save();
+            bindable.ValueChanged += _ => backgroundSave();
         }
 
         private Bindable<U> set<U>(T lookup, U value)
@@ -154,15 +156,14 @@ protected Bindable<U> GetOriginalBindable<U>(T lookup)
 
         #region IDisposable Support
 
-        private bool disposedValue; // To detect redundant calls
+        private bool isDisposed;
 
         protected virtual void Dispose(bool disposing)
         {
-            if (!disposedValue)
+            if (!isDisposed)
             {
                 Save();
-
-                disposedValue = true;
+                isDisposed = true;
             }
         }
 
@@ -204,10 +205,27 @@ public void Load()
             hasLoaded = true;
         }
 
+        private int lastBackgroundSave;
+
+        /// <summary>
+        /// Perform a save with debounce.
+        /// </summary>
+        private void backgroundSave()
+        {
+            var current = Interlocked.Increment(ref lastBackgroundSave);
+            Task.Delay(100).ContinueWith(task =>
+            {
+                if (current == lastBackgroundSave) Save();
+            });
+        }
+
+        private readonly object saveLock = new object();
+
         public bool Save()
         {
             if (!hasLoaded) return false;
-            return PerformSave();
+            lock (saveLock)
+                return PerformSave();
         }
 
         protected abstract void PerformLoad();