New issue
Advanced search Search tips

Issue 826445 link

Starred by 3 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Iframes that LazyLoad decides not to defer should block the document's load event.

Project Member Reported by sclit...@chromium.org, Mar 27 2018

Issue description

LazyLoad uses heuristics to identify frames on the page that are likely analytics or communication iframes. Since, on a normal page load, the document's load event would wait for these iframes to finish loading, some pages may rely on this ordering.

LazyLoad should make the document's load event deterministically wait for these iframes to load, while still not blocking the document's load event on iframes that did end up getting deferred.
 
Components: -Blink>Previews Blink>Loader>LazyLoad
Blocking: 709494
Blocking: -709494
Labels: -Pri-2 Pri-3
Status: Assigned (was: Started)
I'm lowering the priority of this since in practice, the intersection observer will almost always fire before the document's load event, causing the frame to start loading which in turn blocks the document's load event. There are still some edge cases where we should ensure that there's a strict ordering, like if the document load event would fire before the intersection observer would fire to start loading a hidden frame, but these cases are rarer and lower priority.

Sign in to add a comment