<html><head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; color: rgb(0, 0, 0); font-size: 14px; font-family: Calibri, sans-serif; "><div>LayoutTests/resources is fine with me – that was the location I considered using originally and only moved them to LayoutTests/fast/js/resources because that is where js-test-pre and –post are.  </div><div><br></div><div>I'll upload a new patch with the files in LayoutTests/resources.</div><div><br></div><div><br></div><div><br></div><span id="OLK_SRC_BODY_SECTION"><div style="font-family:Calibri; font-size:11pt; text-align:left; color:black; BORDER-BOTTOM: medium none; BORDER-LEFT: medium none; PADDING-BOTTOM: 0in; PADDING-LEFT: 0in; PADDING-RIGHT: 0in; BORDER-TOP: #b5c4df 1pt solid; BORDER-RIGHT: medium none; PADDING-TOP: 3pt"><span style="font-weight:bold">From: </span> Ryosuke Niwa <<a href="mailto:rniwa@webkit.org">rniwa@webkit.org</a>><br><span style="font-weight:bold">Date: </span> Fri, 9 Mar 2012 14:37:18 -0800<br><span style="font-weight:bold">To: </span> Jacob Goldstein <<a href="mailto:jacobg@adobe.com">jacobg@adobe.com</a>><br><span style="font-weight:bold">Cc: </span> "<a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a>" <<a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a>><br><span style="font-weight:bold">Subject: </span> Re: [webkit-dev] Test conversion to use W3C testharness.js<br></div><div><br></div><div class="gmail_quote">On Fri, Mar 9, 2012 at 2:28 PM, Jacob Goldstein <span dir="ltr"><<a href="mailto:jacobg@adobe.com">jacobg@adobe.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="font-size:14px;font-family:Calibri,sans-serif;word-wrap:break-word"><div><div style="text-align:left"><span style="font-size: 15px; font-family: Calibri, Verdana, Helvetica, Arial; ">I recently uploaded a patch to </span><a href="https://bugs.webkit.org/show_bug.cgi?id=80709" target="_blank" style="font-family:Calibri,Verdana,Helvetica,Arial;font-size:15px">https://bugs.webkit.org/show_bug.cgi?id=80709</a><span style="font-size: 15px; font-family: Calibri, Verdana, Helvetica, Arial; "> which converted an existing JavaScript regions parsing test to use the W3C testharness.js in place of js-test-pre.js/js-test-post.</span><span style="font-size: 15px; font-family: Calibri, Verdana, Helvetica, Arial; ">js.  This patch also places testharness.js and a WebKit-specific testharnessreport.js file in LayoutTests/fast/js/resources.</span></div></div></div></blockquote><div><br></div><div>Can we put them in LayoutTests/resources instead? I always find it hard to remember the path fast/js/resources.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="font-size:14px;font-family:Calibri,sans-serif;word-wrap:break-word"><div><span style="font-size: 15px; font-family: Calibri, Verdana, Helvetica, Arial; ">In cases where tests need to be written for both the WebKit and W3C testing suites, having the ability to use testharness.js with WebKit tests would mean that the test file only needs to be written once, and yet can still rely on the functionality from both test harnesses.   As it stands now, if someone needs to write a test for both suites, they either have to implement all functionality from scratch, or write one version of the test to use js-test-pre.js and another to use testharness.js.  The inclusion of testharness.js in the WebKit repository alleviates the need for this duplication of effort.  The testharnessreport.js file was intended for customization of the capabilities provided by testharness.js, I've added a call to layoutTestController.dumpAsText() to this file to allow it to function as a WebKit JavaScript test.<br></span></div></div></blockquote><div><br></div><div>I support the effort to make layout tests more compatible with W3C tests.</div><div><br></div><div>Is the plan to use testharness.js for all new tests? Or only tests that we intend to contribute back to W3C?</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div style="font-family:Calibri,sans-serif;font-size:14px"><span style="font-size: 15px; font-family: Calibri, Verdana, Helvetica, Arial; ">Another concern is that changes to testharness.js in the future that break backward compatibility could then break WebKit tests.  This is an issue I plan to discuss with W3C members to determine if backward compatibility can be ensured.<br></span></div></div></blockquote><div><br></div><div>There is no such a guarantee at the moment? That concerns me. On other hand, we wouldn't be importing ToT version of testharness.js so if such an incompatibility is introduced, we can migrate our tests on time as well.</div><div><br></div><div>- Ryosuke</div><div><br></div></div></span></body></html>