Validate finiteness of HTMLMediaElement properties.

LayoutTests/media/video-test.js

 
 var video = null;
 var mediaElement = document; // If not set, an event from any element will trigger a waitForEvent() callback.
 var console = null;
 var printFullTestDetails = true; // This is optionaly switched of by test whose tested values can differ. (see disableFullTestDetailsPrinting())
 var Failed = false;
 
 var track = null; // Current TextTrack being tested.
 var cues = null; // Current TextTrackCueList being tested.
 var numberOfTrackTests = 0;
@@ skipping 207 matching lines @@
     logResult(exception instanceof DOMException && exception.code === eval(exceptionString),
         "TEST(" + testString + ") THROWS(" + exceptionString + ": " + exception.message + ")");
 }
 
 function testException(testString, exceptionString) {
     try {
         eval(testString);
     } catch (ex) {
         var exception = ex;
     }
-    logResult(exception !== undefined && exception == eval(exceptionString),
+    logResult(exception !== undefined && exception + '' === exceptionString,

[acolwell GONE FROM CHROMIUM, 2014/03/29 01:40:08]

Why did you need to make this change? This could potentially effect a bunch of
tests if it changes the output.

[sandersd (OOO until July 31), 2014/03/31 20:41:08]

Looks like I didn't quite understand how the other tests were using this
function (it's not pretty, but cleaning it up is a different CL). I've reverted
this change.

         "TEST(" + testString + ") THROWS(" + exceptionString + ")");
 }
 
 var testEnded = false;
 
 function endTest()
 {
     consoleWrite("END OF TEST");
     testEnded = true;
     if (window.testRunner)
@@ skipping 118 matching lines @@
 
         func();
     }
 
     requiredEvents = [];
     for (var i = 0; i < eventList.length; i++) {
         requiredEvents[i] = eventList[i][1];
         eventList[i][0].addEventListener(requiredEvents[i], _eventCallback, true);
     }
 }