New issue
Advanced search Search tips
Starred by 5 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 16
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Feature

Blocking:
issue 709494
issue 869492



Sign in to add a comment
link

Issue 873358: Implement "lazyload" attribute support for frames.

Reported by sclit...@chromium.org, Aug 10 Project Member

Issue description

Implement support for the "lazyload" attribute on iframe elements.

https://github.com/whatwg/html/pull/3752
 

Comment 1 by sclit...@chromium.org, Aug 10

Blocking: 709494

Comment 2 by ekaramad@chromium.org, Aug 13

Blocking: 869492

Comment 3 by bugdroid1@chromium.org, Aug 16

Project Member
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b5278c3633150f6030882dc717426f690ffadeca

commit b5278c3633150f6030882dc717426f690ffadeca
Author: Scott Little <sclittle@chromium.org>
Date: Thu Aug 16 22:01:48 2018

LazyLoad: Implement support for "lazyload" attribute on frames.

This CL implements support for the "lazyload" attribute on frames,
according to https://github.com/whatwg/html/pull/3752, and as part of
the LazyLoad feature. The accepted values are:

"on", which causes the browser to lazily load a frame even if it's
same-origin or nested inside another lazyloaded frame,

"off", which causes the browser to avoid lazily loading this frame or
any of it's children (unless those children are marked with
lazyload="on"),

"auto", which activates the default behavior, and is therefore not
explicitly handled in code.

Bug:  873358 
Change-Id: I2fde65adb15216260291b08e39888a2363f44d4a
Reviewed-on: https://chromium-review.googlesource.com/1176293
Commit-Queue: Scott Little <sclittle@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583841}
[modify] https://crrev.com/b5278c3633150f6030882dc717426f690ffadeca/third_party/blink/renderer/core/html/html_attribute_names.json5
[modify] https://crrev.com/b5278c3633150f6030882dc717426f690ffadeca/third_party/blink/renderer/core/html/html_frame_owner_element.cc
[modify] https://crrev.com/b5278c3633150f6030882dc717426f690ffadeca/third_party/blink/renderer/core/html/html_frame_owner_element.h
[modify] https://crrev.com/b5278c3633150f6030882dc717426f690ffadeca/third_party/blink/renderer/core/html/lazy_load_frame_observer.cc
[modify] https://crrev.com/b5278c3633150f6030882dc717426f690ffadeca/third_party/blink/renderer/core/html/lazy_load_frame_observer.h
[modify] https://crrev.com/b5278c3633150f6030882dc717426f690ffadeca/third_party/blink/renderer/core/html/lazy_load_frame_observer_test.cc

Comment 4 by sclit...@chromium.org, Aug 16

Status: Fixed (was: Started)

Sign in to add a comment