From 466d73a693da344b01ce4fff1976b17d1a461c10 Mon Sep 17 00:00:00 2001
From: Henning Koch <henning.koch@makandra.de>
Date: Fri, 21 Jun 2024 14:55:47 +0200
Subject: [PATCH] Don't look at iframeElement.contentDocument.readyState --
 This may be iframe's pending state (its initial children)

---
 CHANGELOG.md                    | 1 -
 lib/capybara-lockstep/helper.js | 7 +------
 2 files changed, 1 insertion(+), 7 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index dc2ffee..48724de 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -7,7 +7,6 @@ This project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html
 
 - We now only wait for `<script>` elements with a JavaScript type
 - We only wait for `<iframe>` elements with a `[src]` attribute
-- We no longer wait for `<iframe>` elements that have already finished loading
 - We no longer wait for `<video>` and `<audio>` elements to load their metadata. This did not work consistently, and would sometimes cause capybara-lockstep to wait indefinitely.
 
 
diff --git a/lib/capybara-lockstep/helper.js b/lib/capybara-lockstep/helper.js
index 5614a76..7cd4179 100644
--- a/lib/capybara-lockstep/helper.js
+++ b/lib/capybara-lockstep/helper.js
@@ -190,12 +190,7 @@ window.CapybaraLockstep = (function() {
   function isTrackableIFrame(element) {
     return element.matches('iframe[src]') &&
       !hasDataSource(element) &&
-      element.getAttribute('loading') !== 'lazy' &&
-      !hasIFrameFinishedLoading(element)
-  }
-
-  function hasIFrameFinishedLoading(element) {
-    return element.contentDocument && element.contentDocument.readyState === 'complete'
+      element.getAttribute('loading') !== 'lazy'
   }
 
   function isTrackableScriptType(type) {