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 |
Computing file changes ...
