Closed Bug 1454488 Opened 7 years ago Closed 7 years ago

[wpt-sync] Sync PR 10492 - [vertical-scroll]: Do not propagate "scrollIntoView()" from frames with 'vertical-scroll' disabled to frames with the feature enabled

Categories

(Core :: DOM: Core & HTML, enhancement, P4)

Product:
Core
Component:
DOM: Core & HTML
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla61
Tracking Flags:
Tracking Status
firefox61 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 10492 into mozilla-central (this bug is closed when the sync is complete). PR: https://github.com/w3c/web-platform-tests/pull/10492 Details from upstream follow. Ehsan Karamad <ekaramad@chromium.org> wrote: > [vertical-scroll]: Do not propagate "scrollIntoView()" from frames with 'vertical-scroll' disabled to frames with the feature enabled > > If a 'vertical-scroll' is disabled for an <iframe>, then it should not > be able to affect the vertical scroll position. One way to do that is > using scripted scrolling by calling "element.scrollIntoView()". > > To block such frames (whose feature's disabled), programmatic > recusive scroll calls are not forwarded to parent frame (unless the > parent frame has the feature disabled as well). In ther words, if a > given <iframe> is the first frame in a branch which does not have the > feature, then calls to programmatic scrollIntoView() from inside the > <iframe>: > * Will not change the scroll offset in any of the ancestor frames, > * Will work within the scope of <iframe>, i.e., elements become > visible inside <iframe>'s bounds. > > Link to explainer/design document for "vertical-scroll": > https://docs.google.com/document/d/1qiWelnMlsOHuT_CQ0Zm_qEAf54HS5DhoIvEDHBlfqps/edit?usp=sharing > > Bug: 611982 > Change-Id: I0e06b399ad890e263128b997cfbb04eb3bdd1494 > Reviewed-on: https://chromium-review.googlesource.com/1014191 > WPT-Export-Revision: a00fd27160fab6aef8aef60fc3c63f394dc55575
Component: web-platform-tests → DOM
Product: Testing → Core
Try push failed: decision task returned error
Ran 1 tests and 2 subtests OK : 1 PASS : 1 FAIL : 1 New tests that have failures or other problems: /feature-policy/experimental-features/vertical-scroll-scrollintoview.tentative.sub.html Calling 'scrollIntoView()' inside a <iframe> with 'vertical-scroll none;'will not propagate upwards.: FAIL
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/mozilla-inbound/rev/77eaba20ef96 [wpt PR 10492] - 'vertical-scroll' for programmatic scrolling, a=testonly https://hg.mozilla.org/integration/mozilla-inbound/rev/5f9e45532ac0 [wpt PR 10492] - Update wpt metadata, a=testonly
https://hg.mozilla.org/mozilla-central/rev/77eaba20ef96 https://hg.mozilla.org/mozilla-central/rev/5f9e45532ac0
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox61: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.