LGTM2

On Thu, Jun 9, 2022 at 9:18 AM Rick Byers <[email protected]> wrote:

> Oh yikes, these got exposed to the web and we never noticed? Scary. I may
> have had a hand in that a long time ago, sorry! I forget the details, but
> remember something about NaCL and maybe Flash support for touch scrolling.
> Presumably none of that matters anymore now that Flash and NaCL are gone.
>
> I can imagine scenarios where they might be useful to have, but I guess
> being main thread only makes them pretty useless (almost all gesture
> scrolls are threaded these days). Anyway, given the near zero usage, lack
> of a spec, main-thread only behavior and lack of evidence
> <https://www.google.com/search?q=gesturescrollstart+event&oq=gesturescrollstart+event&aqs=chrome..69i57l2j0i546l2j69i65j69i60l3.4102j1j7&gs_lcrp=EgZjaHJvbWUyBggAEEUYOTIGCAEQRRg5MgcIAhAAGKIEMgcIAxAAGKIEMgYIBBBFGEEyBggFEEUYPDIGCAYQRRg8MgYIBxBFGDzSAQg0MTAyajFqN6gCALACAA&sourceid=chrome&ie=UTF-8>
> of anyone talking about these events I'd consider this extremely safe to
> remove (more of a bug fix than a deprecation).  LGTM1
>
> Thanks for discovering this and cleaning it up!
>
> On Thu, Jun 9, 2022 at 9:11 AM Mehdi Kazemi <[email protected]> wrote:
>
>> Contact emails
>>
>>
>> *[email protected] <[email protected]>,[email protected]
>> <[email protected]>,[email protected]
>> <[email protected]>*Explainer
>>
>>
>> *None*Specification
>>
>> None. Not a standard feature.
>>
>> Summary
>>
>> Gesture Scroll DOM events, namely “gesturescrollstart”,
>> “gesturescrollupdate” and “gesturescrollend” are non-standard APIs, which
>> were added to Blink for use in plugins, but it appears they were also
>> exposed to the web unintentionally. Plugins are no longer web-exposed since
>> the deprecation of Google Native Client (NaCl).
>>
>> WebKit Bug: https://bugs.webkit.org/show_bug.cgi?id=92281
>>
>> Changelog:
>> https://bugs.webkit.org/attachment.cgi?id=155046&action=prettypatch
>>
>> Motivation
>>
>> Currently, this API doesn’t work in all situations. It only works when
>> there is a *non-composited scroller*. These events are related to
>> compositing state, and compositing state is not meant to have observable
>> behavior impact and in Blink will vary depending on display type and other
>> factors.
>>
>>
>> Blink component
>>
>> Blink>Scroll
>> <https://bugs.chromium.org/p/chromium/issues/list?q=component%3ABlink%3EScroll&can=2>
>>
>>
>> TAG review
>>
>>
>> TAG review status
>>
>> Not applicable
>>
>>
>> Risks
>>
>> No other engine supports these events, so we do not expect
>> interoperability issues.
>>
>> As for compatibility, usage data from Canary, Dev and Beta channels show
>> that usage is very low, around 0.000015% (gesturescrollstart
>> <https://chromestatus.com/metrics/feature/timeline/popularity/4220>
>> gesturescrollupdate
>> <https://chromestatus.com/metrics/feature/timeline/popularity/4221>
>> gesturescrollend
>> <https://chromestatus.com/metrics/feature/timeline/popularity/4222>).
>> For this reason we would like to just remove it, without any deprecation
>> period.
>>
>>
>> Is this feature fully tested by web-platform-tests
>> <https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md>
>> ?
>>
>> No
>>
>>
>> Requires code in //chrome?
>>
>> False
>>
>>
>> Tracking bug
>>
>> https://crbug.com/1293994
>> <https://bugs.chromium.org/p/chromium/issues/detail?id=1293994>
>>
>>
>> Estimated milestones
>>
>> No milestones specified
>>
>>
>>
>>
>>
>>
>> --
>> You received this message because you are subscribed to the Google Groups
>> "blink-dev" group.
>> To unsubscribe from this group and stop receiving emails from it, send an
>> email to [email protected].
>> To view this discussion on the web visit
>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAASN_%3DmnFnKhp7WQNYiVztF%3Dhr1jpzSTo1x02nXZtysnO2GUMQ%40mail.gmail.com
>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAASN_%3DmnFnKhp7WQNYiVztF%3Dhr1jpzSTo1x02nXZtysnO2GUMQ%40mail.gmail.com?utm_medium=email&utm_source=footer>
>> .
>>
> --
> You received this message because you are subscribed to the Google Groups
> "blink-dev" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to [email protected].
> To view this discussion on the web visit
> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAFUtAY_BmVOXxKibPwSF%2BNn9u76AUXdE09fjE7ohWVF7iqUvMg%40mail.gmail.com
> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAFUtAY_BmVOXxKibPwSF%2BNn9u76AUXdE09fjE7ohWVF7iqUvMg%40mail.gmail.com?utm_medium=email&utm_source=footer>
> .
>

-- 
You received this message because you are subscribed to the Google Groups 
"blink-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAOMQ%2Bw8qg6SbsZNX0SU6JK%3D_7WhsBkuGLpEHpKGpEAXNXnN%3DLA%40mail.gmail.com.

Reply via email to