Changeset 194136 in webkit

Timestamp:

Dec 16, 2015 1:16:29 AM (8 years ago)

Author:

youenn.fablet@crf.canon.fr

Message:

Align WebKit testharness.js with WPT testharness.js
​https://bugs.webkit.org/show_bug.cgi?id=152297

Reviewed by Darin Adler.

Cosmetic changes (output:true is overriden by WK testharnessreport.js)
To remove the warning "WPT tests are not using the same testharness.js file as other WebKit Layout tests."

• media/track/getCueAsHTMLCrash.html: Adding testharnessreport.js and removing call to dumpAsText.
• media/track/getCueAsHTMLCrash-expected.txt: Rebasing as tests are now outputting results.
• resources/testharness.js:

Location:

trunk/LayoutTests

Files:

• ChangeLog (modified) (1 diff)
• media/track/getCueAsHTMLCrash-expected.txt (modified) (1 diff)
• media/track/getCueAsHTMLCrash.html (modified) (1 diff)
• resources/testharness.js (modified) (1 diff)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2015-12-16  Youenn Fablet  <youenn.fablet@crf.canon.fr>
+
+        Align WebKit testharness.js with WPT testharness.js
+        https://bugs.webkit.org/show_bug.cgi?id=152297
+
+        Reviewed by Darin Adler.
+
+        Cosmetic changes (output:true is overriden by WK testharnessreport.js)
+        To remove the warning "WPT tests are not using the same testharness.js file as other WebKit Layout tests."
+
+        * media/track/getCueAsHTMLCrash.html: Adding testharnessreport.js and removing call to dumpAsText.
+        * media/track/getCueAsHTMLCrash-expected.txt: Rebasing as tests are now outputting results.
+        * resources/testharness.js:
+
 2015-12-16  Yusuke Suzuki  <utatane.tea@gmail.com>
 

trunk/LayoutTests/media/track/getCueAsHTMLCrash-expected.txt

@@ -1 +1 @@
 Test passes if it does not induce a crash.
+
+PASS , creating the cue 
+FAIL , > assert_false: hasChildNodes() expected false got true
+PASS ,  
+PASS , x\0 
+

trunk/LayoutTests/media/track/getCueAsHTMLCrash.html

@@ -3 +3 @@
 <div>Test passes if it does not induce a crash.</div>
 <script src="../../resources/testharness.js"></script>
+<script src="../../resources/testharnessreport.js"></script>
 <script>
-if (window.testRunner)
-    testRunner.dumpAsText();
-
 test(function() {
     var cue = new TextTrackCue(0, 1, '<c>x\0');

trunk/LayoutTests/resources/testharness.js

@@ -16 +16 @@
 {
     var debug = false;
-
     // default timeout is 10 seconds, test can override if needed
     var settings = {
-        output:false,
+        output:true,
         harness_timeout:{
             "normal":10000,