Chrome Violation: Non-passive scroll handlers

[tommyh]

Description

Chrome is logging "Violations" for videojs's scroll handlers.

[Violation] Added non-passive event listener to a scroll-blocking 'touchstart' event. Consider marking event handler as 'passive' to make the page more responsive.
external_embed-b5b3e31….js:936
[Violation] Added non-passive event listener to a scroll-blocking 'touchmove' event. Consider marking event handler as 'passive' to make the page more responsive.

Related Links:
https://developers.google.com/web/updates/2016/06/passive-event-listeners
https://developer.mozilla.org/en-US/docs/Web/API/EventTarget/addEventListener#Safely_detecting_option_support

Steps to reproduce

1. Open Chrome and dev tools
2. Set console level to "Verbose"
3. Go to http://videojs.com
4. Look in the console for "Violation" warnings.

Results

Expected

Videojs should add passive: true when adding event listeners, so Chrome (and others) can be more performant. The Violation warnings should go away as well.

Actual

Violation warnings in the dev tools console.

Additional Information

versions

videojs

5.19.2

browsers

Chrome Version 59.0.3071.109 (Official Build) (64-bit)

OSes

I tested on mac

plugins

No