warning_ and error_ functions are now wired to onunmute and onmute events

chrome/test/data/webrtc/manual/peerconnection_manual.js

Index: chrome/test/data/webrtc/manual/peerconnection_manual.js
diff --git a/chrome/test/data/webrtc/manual/peerconnection_manual.js b/chrome/test/data/webrtc/manual/peerconnection_manual.js
index 40ac84a4205235244caef9c7159c1aac27066bd7..a631f05a546aaf625ce8562a8a9552a2b203e156 100644
--- a/chrome/test/data/webrtc/manual/peerconnection_manual.js
+++ b/chrome/test/data/webrtc/manual/peerconnection_manual.js
@@ -916,6 +916,12 @@ function success_(method) {
 }
 
 /** @private */
+function warning_(message) {

[mcasas, 2014/08/21 10:53:17]

It seems that there are:
- function print_(message)
- function debug_(message)
- function error_(message)
  (these three around l.1143
- function warning_(message)
- function failure_(method, error)

and they are all very similar. Could you bring them together and consolidate the
common parts?

+  $('debug').innerHTML += '<span style="color:orange;">' + message +
+                          '</span><br>';
+}
+
+/** @private */
 function failure_(method, error) {
   error_(method + '() failed: ' + JSON.stringify(error));
 }
@@ -1007,13 +1013,13 @@ function getUserMediaOkCallback_(stream) {
     };
 
     // Print information on track going to mute or back from it.
-    // TODO(mcasas): add a warning_() function and move the following print_()
-    // notifications to error_() and warning_(), respectively.
     stream.getVideoTracks()[0].onmute = function() {
-      print_(global.localStream + ' track onmute event has fired');
+      error_(global.localStream + ' MediaStreamTrack.onmute event has fired, ' +
+             'no frames to the track.');
     };
     stream.getVideoTracks()[0].onunmute = function() {
-      print_(global.localStream + ' track onunmute event has fired');
+      warning_(global.localStream + ' MediaStreamTrack.onunmute event has ' +
+               'fired.');
     };
   }
 }