Closed Bug 1454488 Opened Last year Closed Last year

[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)

enhancement

Tracking

()

RESOLVED FIXED
mozilla61
Tracking Status
firefox61 --- fixed

People

(Reporter: wptsync, Unassigned)

References

()

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: Last year
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.