diff options
author | Johannes Doerfert <doerfert@cs.uni-saarland.de> | 2014-08-18 04:40:13 +0400 |
---|---|---|
committer | Johannes Doerfert <doerfert@cs.uni-saarland.de> | 2014-08-18 04:40:13 +0400 |
commit | 9e7b17b0d484861350c4b818697b81e1bc8ac96a (patch) | |
tree | fda45bea5b780281569a3c26e7f77b03e418dcc9 /polly/www | |
parent | 6230691c9186e95cd19dff07074a54fb70719957 (diff) |
Added arcanist linters and cleaned errors and warnings
Arcanist (arc) will now always run linters before uploading any new
commit to Phabricator. All errors/warnings (or their absence) will be
shown in the web interface together with a explanation by the commiter
(arcanist will ask the commiter if the build was not clean).
The linters include:
- clang-format
- spelling check
- permissions check (aka. chmod)
- filename check
- merge conflict marker check
Note, that their scope is sometimes limited (see .arclint for
details).
This commit also fixes all errors and warnings these linters reported,
namely:
- spelling mistakes and typos
- executable permissions for various text files
Differential Revision: http://reviews.llvm.org/D4916
llvm-svn: 215871
Diffstat (limited to 'polly/www')
-rw-r--r-- | polly/www/video-js/video.js | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/polly/www/video-js/video.js b/polly/www/video-js/video.js index c0efccd1a887..9571eeb7e977 100644 --- a/polly/www/video-js/video.js +++ b/polly/www/video-js/video.js @@ -39,7 +39,7 @@ var VideoJS = JRClass.extend({ this.video = element; } // Store reference to player on the video element. - // So you can acess the player later: document.getElementById("video_id").player.play(); + // So you can access the player later: document.getElementById("video_id").player.play(); this.video.player = this; this.values = {}; // Cache video values. this.elements = {}; // Store refs to controls elements. @@ -456,7 +456,7 @@ VideoJS.player.extend({ this.canPlaySourceResult = false; return false; }, - // Check if the extention is compatible, for when type won't work + // Check if the extension is compatible, for when type won't work canPlayExt: function(src){ if (!src) { return ""; } var match = src.match(/\.([^\.]+)$/); @@ -1290,10 +1290,10 @@ VideoJS.player.newBehavior("currentTimeScrubber", function(element){ _V_.addListener(document, "mousemove", this.onCurrentTimeScrubberMouseMove.rEvtContext(this)); _V_.addListener(document, "mouseup", this.onCurrentTimeScrubberMouseUp.rEvtContext(this)); }, - onCurrentTimeScrubberMouseMove: function(event){ // Removeable + onCurrentTimeScrubberMouseMove: function(event){ // Removable this.setCurrentTimeWithScrubber(event); }, - onCurrentTimeScrubberMouseUp: function(event){ // Removeable + onCurrentTimeScrubberMouseUp: function(event){ // Removable _V_.unblockTextSelection(); document.removeEventListener("mousemove", this.onCurrentTimeScrubberMouseMove, false); document.removeEventListener("mouseup", this.onCurrentTimeScrubberMouseUp, false); @@ -1385,11 +1385,11 @@ VideoJS.player.newBehavior("fullscreenToggle", function(element){ } }, - fullscreenOnWindowResize: function(event){ // Removeable + fullscreenOnWindowResize: function(event){ // Removable this.positionControlBars(); }, // Create listener for esc key while in full screen mode - fullscreenOnEscKey: function(event){ // Removeable + fullscreenOnEscKey: function(event){ // Removable if (event.keyCode == 27) { this.exitFullScreen(); } @@ -1718,7 +1718,7 @@ Function.prototype.evtContext = function(obj){ return temp; }; -// Removeable Event listener with Context +// Removable Event listener with Context // Replaces the original function with a version that has context // So it can be removed using the original function name. // In order to work, a version of the function must already exist in the player/prototype |