Revision 343fd25cd72c64ed30da54b650220cca7338cd57 authored by Sandra Sun on 14 March 2018, 17:24:07 UTC, committed by Blink WPT Bot on 14 March 2018, 18:11:13 UTC
According to spec, https://www.w3.org/TR/css-scroll-snap-1/#snap-scope we should only consider the snap points of visible snap areas. This patch implements the visibility requirement by adding a field of visible region for each snap point. It also handles the case of visibility conflict when snapping to two axes individually. Bug: 778257 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Change-Id: I7a3d2aa6b9c1293df3f05118152f105fa3265d29 Reviewed-on: https://chromium-review.googlesource.com/814874 Commit-Queue: Sandra Sun <sunyunjia@chromium.org> Reviewed-by: Jeremy Roman <jbroman@chromium.org> Reviewed-by: Ali Juma <ajuma@chromium.org> Reviewed-by: Majid Valipour <majidvp@chromium.org> Cr-Commit-Position: refs/heads/master@{#543111}
1 parent ec54ac4
File | Mode | Size |
---|---|---|
Gyroscope-disabled-by-feature-policy.https.html | -rw-r--r-- | 402 bytes |
Gyroscope-disabled-by-feature-policy.https.html.headers | -rw-r--r-- | 33 bytes |
Gyroscope-enabled-by-feature-policy-attribute-redirect-on-load.https.html | -rw-r--r-- | 460 bytes |
Gyroscope-enabled-by-feature-policy-attribute.https.html | -rw-r--r-- | 426 bytes |
Gyroscope-enabled-by-feature-policy.https.html | -rw-r--r-- | 400 bytes |
Gyroscope-enabled-by-feature-policy.https.html.headers | -rw-r--r-- | 28 bytes |
Gyroscope-enabled-on-self-origin-by-feature-policy.https.html | -rw-r--r-- | 430 bytes |
Gyroscope-enabled-on-self-origin-by-feature-policy.https.html.headers | -rw-r--r-- | 33 bytes |
Gyroscope.https.html | -rw-r--r-- | 428 bytes |
Gyroscope_insecure_context.html | -rw-r--r-- | 569 bytes |
Gyroscope_onerror-manual.https.html | -rw-r--r-- | 570 bytes |
OWNERS | -rw-r--r-- | 63 bytes |
idlharness.https.html | -rw-r--r-- | 1.1 KB |
![swh spinner](/static/img/swh-spinner.gif)
Computing file changes ...