Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(656)

Unified Diff: chrome/browser/resources/video_player/js/video_player.js

Issue 209853011: [VideoPlayer] Browser tests for new separated video player app (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Adressed the comment Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/video_player/js/video_player.js
diff --git a/chrome/browser/resources/video_player/js/video_player.js b/chrome/browser/resources/video_player/js/video_player.js
index 3843a8aa03a3bfd08a3d7f068badaad8a51fdaaa..7cffa15cdb745018420a585006601c330a3d8328 100644
--- a/chrome/browser/resources/video_player/js/video_player.js
+++ b/chrome/browser/resources/video_player/js/video_player.js
@@ -50,14 +50,14 @@ function FullWindowVideoControls(
VideoControls.call(this,
controlsContainer,
onPlaybackError,
- loadTimeData.getString.bind(loadTimeData),
- this.toggleFullScreen_.bind(this),
+ loadTimeData.getString.wrap(loadTimeData),
+ this.toggleFullScreen_.wrap(this),
videoContainer);
this.playerContainer_ = playerContainer;
this.updateStyle();
- window.addEventListener('resize', this.updateStyle.bind(this));
+ window.addEventListener('resize', this.updateStyle.wrap(this));
document.addEventListener('keydown', function(e) {
if (e.keyIdentifier == 'U+0020') { // Space
@@ -70,7 +70,7 @@ function FullWindowVideoControls(
false); // Leave the full screen mode.
e.preventDefault();
}
- }.bind(this));
+ }.wrap(this));
// TODO(mtomasz): Simplify. crbug.com/254318.
videoContainer.addEventListener('click', function(e) {
@@ -81,7 +81,7 @@ function FullWindowVideoControls(
} else {
this.togglePlayStateWithFeedback();
}
- }.bind(this));
+ }.wrap(this));
this.inactivityWatcher_ = new MouseInactivityWatcher(playerContainer);
this.__defineGetter__('inactivityWatcher', function() {

Powered by Google App Engine
This is Rietveld 408576698