From 0dcd283e6e872c46560fb7dbf37c366d0a83d8b2 Mon Sep 17 00:00:00 2001 From: Valera Rozuvan Date: Thu, 23 Jan 2014 11:56:19 +0200 Subject: [PATCH] Make video debug messages consistent. In the JavaScript console all messages from the Video player should begin with "[Video info]: ". This way it is easier to see which console.log() messages are from the Video player, and which console.log() messages are temporary, and should be removed before a merge/commit. --- common/lib/xmodule/xmodule/js/spec/video/resizer_spec.js | 2 +- common/lib/xmodule/xmodule/js/src/video/00_resizer.js | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/common/lib/xmodule/xmodule/js/spec/video/resizer_spec.js b/common/lib/xmodule/xmodule/js/spec/video/resizer_spec.js index 0b312db0c9..ea296bfa81 100644 --- a/common/lib/xmodule/xmodule/js/spec/video/resizer_spec.js +++ b/common/lib/xmodule/xmodule/js/spec/video/resizer_spec.js @@ -163,7 +163,7 @@ function (Resizer) { function () { var methods = ['add', 'once'], - errorMessage = 'TypeError: Argument is not a function.', + errorMessage = '[Video info]: TypeError: Argument is not a function.', arg = {}; spyOn(console, 'error'); diff --git a/common/lib/xmodule/xmodule/js/src/video/00_resizer.js b/common/lib/xmodule/xmodule/js/src/video/00_resizer.js index c9fde305ac..0fd34968a6 100644 --- a/common/lib/xmodule/xmodule/js/src/video/00_resizer.js +++ b/common/lib/xmodule/xmodule/js/src/video/00_resizer.js @@ -127,7 +127,7 @@ function () { if ($.isFunction(func)) { callbacksList.push(func); } else { - console.error('TypeError: Argument is not a function.'); + console.error('[Video info]: TypeError: Argument is not a function.'); } return module; @@ -142,7 +142,7 @@ function () { addCallback(decorator); } else { - console.error('TypeError: Argument is not a function.'); + console.error('[Video info]: TypeError: Argument is not a function.'); } return module;