Check if documentHeight has changed after all elements initially built (

ampproject#34434)
rochapablo · Aug 30, 2021 · 871e4c6 · 871e4c6
1 parent e5eb534
commit 871e4c6
Showing 1 changed file with 12 additions and 0 deletions.
diff --git a/src/service/resources-impl.js b/src/service/resources-impl.js
@@ -634,6 +634,18 @@ export class ResourcesImpl {
  dev().fine(TAG_, 'document height on load: %s', this.contentHeight_);
  }
 
+ // Once we know the document is fully parsed, we check to see if every AMP Element has been built
+ const firstPassAfterAllBuilt =
+ !this.firstPassAfterDocumentReady_ &&
+ this.firstPassAfterAllBuilt_ &&
+ this.resources_.every(
+ (r) => r.getState() != Resource.NOT_BUILT || r.element.R1()
+ );
+ if (firstPassAfterAllBuilt) {
+ this.firstPassAfterAllBuilt_ = false;
+ this.maybeChangeHeight_ = true;
+ }
+
  const viewportSize = this.viewport_.getSize();
  dev().fine(
  TAG_,