Skip to content

Commit

Permalink
@forbesjo fixed webkit deprecation warnings. closes videojs#2559
Browse files Browse the repository at this point in the history
  • Loading branch information
jforbes authored and heff committed Sep 10, 2015
1 parent a584104 commit 849b85b
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ CHANGELOG
=========

## HEAD (Unreleased)
_(none)_
* @forbesjo fixed webkit deprecation warnings ([view](https://github.com/videojs/video.js/pull/2559))

--------------------

Expand Down
4 changes: 3 additions & 1 deletion src/js/events.js
Original file line number Diff line number Diff line change
Expand Up @@ -188,7 +188,9 @@ vjs.fixEvent = function(event) {
for (var key in old) {
// Safari 6.0.3 warns you if you try to copy deprecated layerX/Y
// Chrome warns you if you try to copy deprecated keyboardEvent.keyLocation
if (key !== 'layerX' && key !== 'layerY' && key !== 'keyLocation') {
// and webkitMovementX/Y
if (key !== 'layerX' && key !== 'layerY' && key !== 'keyLocation' &&
key !== 'webkitMovementX' && key !== 'webkitMovementY') {
// Chrome 32+ warns if you try to copy deprecated returnValue, but
// we still want to if preventDefault isn't supported (IE8).
if (!(key == 'returnValue' && old.preventDefault)) {
Expand Down

0 comments on commit 849b85b

Please sign in to comment.