From ba9bcbe5498c4468a607a87022e56d856bceeab0 Mon Sep 17 00:00:00 2001
From: Julian Descottes <jdescottes@mozilla.com>
Date: Thu, 2 May 2024 13:00:34 +0000
Subject: [PATCH] Bug 1894305 - [bidi] Add new trace log for
 Navigate:waitForInitialNavigationCompleted r=webdriver-reviewers,whimboo

Depends on D209047

This additional log could help debug what happens on the Github macos-latest workers

Differential Revision: https://phabricator.services.mozilla.com/D209048
---
 remote/shared/Navigate.sys.mjs | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/remote/shared/Navigate.sys.mjs b/remote/shared/Navigate.sys.mjs
index 9b72c0dfbf32..cdb23b54c7dd 100644
--- a/remote/shared/Navigate.sys.mjs
+++ b/remote/shared/Navigate.sys.mjs
@@ -91,9 +91,14 @@ export async function waitForInitialNavigationCompleted(
     isInitial = browsingContext.currentWindowGlobal.isInitialDocument;
   }
 
+  const isLoadingDocument = listener.isLoadingDocument;
+  lazy.logger.trace(
+    lazy.truncate`[${browsingContext.id}] Wait for initial navigation: isInitial=${isInitial}, isLoadingDocument=${isLoadingDocument}`
+  );
+
   // If the current document is not the initial "about:blank" and is also
   // no longer loading, assume the navigation is done and return.
-  if (!isInitial && !listener.isLoadingDocument) {
+  if (!isInitial && !isLoadingDocument) {
     lazy.logger.trace(
       lazy.truncate`[${browsingContext.id}] Document already finished loading: ${browsingContext.currentURI?.spec}`
     );