browsersetting: don't send update if old value and new value is same

Signed-off-by: Rashesh <rashesh.padia@collabora.com> Change-Id: I89b64a7c014ec52898906b88829d08edda18894f
CollaboraOnline · Feb 27, 2025 · 4ee56ae · 4ee56ae
1 parent 6a4779d
commit 4ee56ae
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/browser/js/global.js b/browser/js/global.js
@@ -739,8 +739,9 @@ function getInitializerClass() {
 		set: function(key, value) {
 			value = String(value); // NOT "new String(...)". We cannot use .toString here because value could be null/undefined
 			if (global.prefs.useBrowserSetting) {
+				const oldValue = global.prefs._userBrowserSetting[key];
 				global.prefs._userBrowserSetting[key] = value;
-				if (global.socket && (global.socket instanceof WebSocket) && global.socket.readyState === 1)
+				if (global.socket && (global.socket instanceof WebSocket) && global.socket.readyState === 1 && oldValue !== value)
 					global.socket.send('browsersetting action=update key=' + key + ' value=' + value);
 			}
 			if (global.prefs.canPersist) {