From 0dd043c64e5e315663fb167dd8298ddaf20655cd Mon Sep 17 00:00:00 2001
From: wheaney <42350981+wheaney@users.noreply.github.com>
Date: Sun, 23 Jun 2024 13:01:18 -0700
Subject: [PATCH] Add more debug logging

---
 gnome/src/extension.js      | 6 ++++++
 gnome/src/monitormanager.js | 8 ++++++--
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/gnome/src/extension.js b/gnome/src/extension.js
index 54ea1b3..51b33da 100644
--- a/gnome/src/extension.js
+++ b/gnome/src/extension.js
@@ -115,6 +115,7 @@ export default class BreezyDesktopExtension extends Extension {
             const target_monitor = this._monitor_manager.getMonitorPropertiesList()?.find(
                 monitor => SUPPORTED_MONITOR_PRODUCTS.includes(monitor.product));
             if (target_monitor !== undefined) {
+                Globals.logger.log_debug(`BreezyDesktopExtension _find_supported_monitor - Identified supported monitor: ${target_monitor.connector}`);
                 return {
                     monitor: this._monitor_manager.getMonitors()[target_monitor.index],
                     connector: target_monitor.connector,
@@ -123,6 +124,7 @@ export default class BreezyDesktopExtension extends Extension {
             }
 
             if (this.settings.get_boolean('developer-mode')) {
+                Globals.logger.log_debug('BreezyDesktopExtension _find_supported_monitor - Using dummy monitor');
                 // allow testing XR devices with just USB, no video needed
                 return {
                     monitor: this._monitor_manager.getMonitors()[0],
@@ -132,6 +134,7 @@ export default class BreezyDesktopExtension extends Extension {
                 };
             }
 
+            Globals.logger.log_debug('BreezyDesktopExtension _find_supported_monitor - No supported monitor found');
             return null;
         } catch (e) {
             Globals.logger.log(`ERROR: BreezyDesktopExtension _find_supported_monitor ${e.message}\n${e.stack}`);
@@ -157,8 +160,11 @@ export default class BreezyDesktopExtension extends Extension {
                 if (target_monitor.is_dummy || !this._monitor_manager.checkOptimalMode(target_monitor.connector)) {
                     Globals.logger.log('Ready, enabling XR effect');
                     this._effect_enable();
+                } else {
+                    Globals.logger.log_debug('BreezyDesktopExtension _setup - driver running but optimal mode check needed');
                 }
             } else {
+                Globals.logger.log_debug('BreezyDesktopExtension _setup - driver no running, starting poller');
                 this._poll_for_ready();
             }
         }
diff --git a/gnome/src/monitormanager.js b/gnome/src/monitormanager.js
index 3d52d8e..ae289c8 100644
--- a/gnome/src/monitormanager.js
+++ b/gnome/src/monitormanager.js
@@ -279,11 +279,13 @@ export const MonitorManager = GObject.registerClass({
                 } else {
                     if (configChanged) {
                         Globals.logger.log(`Switched to optimal mode for monitor ${monitorConnector}`);
-                    } else if (this._changeHookFn !== null) {
+                    } else if (!!this._changeHookFn) {
                         Globals.logger.log_debug('MonitorManager checkOptimalMode: no config change');
                         
                         // no config change means this won't be triggered automatically, so trigger it manually
                         this._changeHookFn();
+                    } else {
+                        Globals.logger.log('MonitorManager checkOptimalMode: can\'t trigger change hook, no hook set!');
                     }
                 }
             }).bind(this));
@@ -321,8 +323,10 @@ export const MonitorManager = GObject.registerClass({
                 }
             }
             this._monitorProperties = monitorProperties;
-            if (this._changeHookFn !== null) {
+            if (!!this._changeHookFn) {
                 this._changeHookFn();
+            } else {
+                Globals.logger.log('MonitorManager _on_monitors_change: can\'t trigger change hook, no hook set!');
             }
         }).bind(this));
     }