Convert Analyser tests to use testharness

Convert Analyser tests to use testharness

The follow bash script converted the tests using Audit to use
testharness instead of js-test:

for f in `grep -l Audit *.html`
do
  # Replace js-test.js with testharness.js and add testharnessreport.js
  sed -i 's;\(.*\)\(<script.*\)js-test.js\(.*\);\1\2testharness.js\3\
\1\2testharnessreport.js\3 ;' $f
  # Remove functions that don't exist anymore.
  sed -i '/^[ 	]*\(description(".*")\|finishJSTest()\);[ 	]*/d' $f
done

However, a few files needed manual adjustment because they used
testPassed()/testFailed(), which no longer exists.  These were either
removed or replaced by Should().summarize().

The expected results files were manually removed.

BUG=675987
TEST=Analyser/* passes

Committed: https://crrev.com/bdbfd895a9da23367f3d34465b78047cdc1de4e3
Cr-Commit-Position: refs/heads/master@{#440166}

[Raymond Toy, 2016-12-20 17:26:38]

rtoy@chromium.org changed reviewers:
+ hongchan@chromium.org

[Raymond Toy, 2016-12-20 17:26:38]

PTAL.  This is the first conversion to testharness. I plan on following a
similar approach for the other directories, only converting Audit tests for now.

[hongchan, 2016-12-20 18:45:42]

On 2016/12/20 17:26:38, Raymond Toy wrote:
> PTAL.  This is the first conversion to testharness. I plan on following a
> similar approach for the other directories, only converting Audit tests for
now.

Can we use the new audit library? I would like to deprecate the old audit
library (i.e. audio-testing.js) - at least we should not use it in the future
CLs.

[Raymond Toy, 2016-12-20 18:47:41]

On 2016/12/20 18:45:42, hongchan wrote:
> On 2016/12/20 17:26:38, Raymond Toy wrote:
> > PTAL.  This is the first conversion to testharness. I plan on following a
> > similar approach for the other directories, only converting Audit tests for
> now.
> 
> Can we use the new audit library? I would like to deprecate the old audit
> library (i.e. audio-testing.js) - at least we should not use it in the future
> CLs.

Too many changes for my liking, at least for the current conversion to
testharness. I'd rather do that in a separate CL.

[Raymond Toy, 2016-12-20 22:01:30]

Description was changed from

==========
Convert Analyser tests to use testharness

The follow bash script converted the tests using Audit to use
testharness instead of js-test:

for f in `grep -l Audit *.html`
do
  sed -i 's;\(.*\)\(<script.*\)js-test.js\(.*\);\1\2testharness.js\3\
\1\2testharnessreport.js\3 ;' $f
  sed -i '/^[ 	]*\(description(".*")\|finishJSTest()\);[ 	]*/d' $f
done

However, a few files needed manual adjustment because they used
testPassed()/testFailed(), which no longer exists.  These were either
removed or replaced by Should().summarize().

The expected results files were manually removed.

BUG=675987
TEST=Analyser/* passes
==========

to

==========
Convert Analyser tests to use testharness

The follow bash script converted the tests using Audit to use
testharness instead of js-test:

for f in `grep -l Audit *.html`
do
  # Replace js-test.js with testharness.js and add testharnessreport.js
  sed -i 's;\(.*\)\(<script.*\)js-test.js\(.*\);\1\2testharness.js\3\
\1\2testharnessreport.js\3 ;' $f
  # Remove functions that don't exist anymore.
  sed -i '/^[ 	]*\(description(".*")\|finishJSTest()\);[ 	]*/d' $f
done

However, a few files needed manual adjustment because they used
testPassed()/testFailed(), which no longer exists.  These were either
removed or replaced by Should().summarize().

The expected results files were manually removed.

BUG=675987
TEST=Analyser/* passes
==========

[hongchan, 2016-12-20 23:04:31]

How about replace 'var ' with 'let ' as a proof of a test file using
'testharness' officially? Just a thought.

https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-byte-data.html
(right):

https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-byte-data.html:14:
window.jsTestIsAsync = true;
Remove this since it's from js-test.

https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-downmix.html
(right):

https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-downmix.html:16:
window.jsTestIsAsync = true;
ditto.

https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-float-data.html
(right):

https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-float-data.html:14:
window.jsTestIsAsync = true;
ditto.

https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-freq-data-smoothing.html
(right):

https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-freq-data-smoothing.html:17:
window.jsTestIsAsync = true;
ditto.

https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-freq-data.html
(right):

https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-freq-data.html:16:
window.jsTestIsAsync = true;
ditto.

https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-multiple-calls.html
(right):

https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-multiple-calls.html:14:
window.jsTestIsAsync = true;
ditto.

https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-zero.html
(right):

https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-zero.html:14:
window.jsTestIsAsync = true;
ditto.

[Raymond Toy, 2016-12-20 23:16:57]

On 2016/12/20 23:04:31, hongchan wrote:
> How about replace 'var ' with 'let ' as a proof of a test file using
> 'testharness' officially? Just a thought.

I'd prefer not to do that. I'd like the conversion to testharness to be as small
as possible.
> 
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> File
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-byte-data.html
> (right):
> 
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-byte-data.html:14:
> window.jsTestIsAsync = true;
> Remove this since it's from js-test.
> 
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> File
> third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-downmix.html
> (right):
> 
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-downmix.html:16:
> window.jsTestIsAsync = true;
> ditto.
> 
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> File
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-float-data.html
> (right):
> 
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-float-data.html:14:
> window.jsTestIsAsync = true;
> ditto.
> 
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> File
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-freq-data-smoothing.html
> (right):
> 
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-freq-data-smoothing.html:17:
> window.jsTestIsAsync = true;
> ditto.
> 
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> File
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-freq-data.html
> (right):
> 
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-freq-data.html:16:
> window.jsTestIsAsync = true;
> ditto.
> 
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> File
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-multiple-calls.html
> (right):
> 
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-multiple-calls.html:14:
> window.jsTestIsAsync = true;
> ditto.
> 
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> File
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-zero.html
> (right):
> 
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-zero.html:14:
> window.jsTestIsAsync = true;
> ditto.

[blink-reviews, 2016-12-20 23:20:54]

lgtm

On Tue, Dec 20, 2016 at 3:16 PM rtoy@chromium.org via
codereview.chromium.org <reply@chromiumcodereview-hr.appspotmail.com> wrote:

> On 2016/12/20 23:04:31, hongchan wrote:
> > How about replace 'var ' with 'let ' as a proof of a test file using
> > 'testharness' officially? Just a thought.
>
> I'd prefer not to do that. I'd like the conversion to testharness to be as
> small
> as possible.
>
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> > File
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-byte-data.html
> > (right):
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-byte-data.html:14:
> > window.jsTestIsAsync = true;
> > Remove this since it's from js-test.
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> > File
> >
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-downmix.html
> > (right):
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-downmix.html:16:
> > window.jsTestIsAsync = true;
> > ditto.
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> > File
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-float-data.html
> > (right):
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-float-data.html:14:
> > window.jsTestIsAsync = true;
> > ditto.
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> > File
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-freq-data-smoothing.html
> > (right):
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-freq-data-smoothing.html:17:
> > window.jsTestIsAsync = true;
> > ditto.
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> > File
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-freq-data.html
> > (right):
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-freq-data.html:16:
> > window.jsTestIsAsync = true;
> > ditto.
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> > File
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-multiple-calls.html
> > (right):
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-multiple-calls.html:14:
> > window.jsTestIsAsync = true;
> > ditto.
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> > File
> third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-zero.html
> > (right):
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-zero.html:14:
> > window.jsTestIsAsync = true;
> > ditto.
>
>
>
> https://codereview.chromium.org/2595633002/
>

-- 
You received this message because you are subscribed to the Google Groups "Blink
Reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

[chromium-reviews, 2016-12-20 23:21:08]

lgtm

On Tue, Dec 20, 2016 at 3:16 PM rtoy@chromium.org via
codereview.chromium.org <reply@chromiumcodereview-hr.appspotmail.com> wrote:

> On 2016/12/20 23:04:31, hongchan wrote:
> > How about replace 'var ' with 'let ' as a proof of a test file using
> > 'testharness' officially? Just a thought.
>
> I'd prefer not to do that. I'd like the conversion to testharness to be as
> small
> as possible.
>
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> > File
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-byte-data.html
> > (right):
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-byte-data.html:14:
> > window.jsTestIsAsync = true;
> > Remove this since it's from js-test.
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> > File
> >
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-downmix.html
> > (right):
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-downmix.html:16:
> > window.jsTestIsAsync = true;
> > ditto.
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> > File
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-float-data.html
> > (right):
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-float-data.html:14:
> > window.jsTestIsAsync = true;
> > ditto.
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> > File
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-freq-data-smoothing.html
> > (right):
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-freq-data-smoothing.html:17:
> > window.jsTestIsAsync = true;
> > ditto.
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> > File
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-freq-data.html
> > (right):
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-freq-data.html:16:
> > window.jsTestIsAsync = true;
> > ditto.
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> > File
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-multiple-calls.html
> > (right):
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-multiple-calls.html:14:
> > window.jsTestIsAsync = true;
> > ditto.
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> > File
> third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-zero.html
> > (right):
> >
> >
>
>
https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
> >
>
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-zero.html:14:
> > window.jsTestIsAsync = true;
> > ditto.
>
>
>
> https://codereview.chromium.org/2595633002/
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Raymond Toy, 2016-12-21 16:16:33]

https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-byte-data.html
(right):

https://codereview.chromium.org/2595633002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-byte-data.html:14:
window.jsTestIsAsync = true;
On 2016/12/20 23:04:30, hongchan wrote:
> Remove this since it's from js-test.

Done.

[hongchan, 2016-12-21 16:22:10]

lgtm with nits

https://codereview.chromium.org/2595633002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-downmix.html
(right):

https://codereview.chromium.org/2595633002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-downmix.html:16:

Let's remove this empty line. I guess all the instances of jsTestIsAsync is
followed by an empty line.

[Raymond Toy, 2016-12-21 16:23:34]

On 2016/12/21 16:22:10, hongchan wrote:
> lgtm with nits
> 
>
https://codereview.chromium.org/2595633002/diff/20001/third_party/WebKit/Layo...
> File
> third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-downmix.html
> (right):
> 
>
https://codereview.chromium.org/2595633002/diff/20001/third_party/WebKit/Layo...
>
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-downmix.html:16:
> 
> Let's remove this empty line. I guess all the instances of jsTestIsAsync is
> followed by an empty line.

I knew you were going to say this, because I used a sed script to remove that
line.

[Raymond Toy, 2016-12-21 16:42:09]

The CQ bit was checked by rtoy@chromium.org

[Raymond Toy, 2016-12-21 16:42:09]

The patchset sent to the CQ was uploaded after l-g-t-m from
hongchan@chromium.org
Link to the patchset: https://codereview.chromium.org/2595633002/#ps40001
(title: "Remove blank line")

[commit-bot: I haz the power, 2016-12-21 16:42:36]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-21 18:45:50]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1482338529205500,
"parent_rev": "b3b85c113b2d4b19d228b6a89da5daf69568882f", "commit_rev":
"dda90ed802c98104a48d56155fa6c36fcc40326f"}

[commit-bot: I haz the power, 2016-12-21 18:46:17]

Description was changed from

==========
Convert Analyser tests to use testharness

The follow bash script converted the tests using Audit to use
testharness instead of js-test:

for f in `grep -l Audit *.html`
do
  # Replace js-test.js with testharness.js and add testharnessreport.js
  sed -i 's;\(.*\)\(<script.*\)js-test.js\(.*\);\1\2testharness.js\3\
\1\2testharnessreport.js\3 ;' $f
  # Remove functions that don't exist anymore.
  sed -i '/^[ 	]*\(description(".*")\|finishJSTest()\);[ 	]*/d' $f
done

However, a few files needed manual adjustment because they used
testPassed()/testFailed(), which no longer exists.  These were either
removed or replaced by Should().summarize().

The expected results files were manually removed.

BUG=675987
TEST=Analyser/* passes
==========

to

==========
Convert Analyser tests to use testharness

The follow bash script converted the tests using Audit to use
testharness instead of js-test:

for f in `grep -l Audit *.html`
do
  # Replace js-test.js with testharness.js and add testharnessreport.js
  sed -i 's;\(.*\)\(<script.*\)js-test.js\(.*\);\1\2testharness.js\3\
\1\2testharnessreport.js\3 ;' $f
  # Remove functions that don't exist anymore.
  sed -i '/^[ 	]*\(description(".*")\|finishJSTest()\);[ 	]*/d' $f
done

However, a few files needed manual adjustment because they used
testPassed()/testFailed(), which no longer exists.  These were either
removed or replaced by Should().summarize().

The expected results files were manually removed.

BUG=675987
TEST=Analyser/* passes

Review-Url: https://codereview.chromium.org/2595633002
==========

[commit-bot: I haz the power, 2016-12-21 18:46:18]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-12-21 18:48:32]

Description was changed from

==========
Convert Analyser tests to use testharness

The follow bash script converted the tests using Audit to use
testharness instead of js-test:

for f in `grep -l Audit *.html`
do
  # Replace js-test.js with testharness.js and add testharnessreport.js
  sed -i 's;\(.*\)\(<script.*\)js-test.js\(.*\);\1\2testharness.js\3\
\1\2testharnessreport.js\3 ;' $f
  # Remove functions that don't exist anymore.
  sed -i '/^[ 	]*\(description(".*")\|finishJSTest()\);[ 	]*/d' $f
done

However, a few files needed manual adjustment because they used
testPassed()/testFailed(), which no longer exists.  These were either
removed or replaced by Should().summarize().

The expected results files were manually removed.

BUG=675987
TEST=Analyser/* passes

Review-Url: https://codereview.chromium.org/2595633002
==========

to

==========
Convert Analyser tests to use testharness

The follow bash script converted the tests using Audit to use
testharness instead of js-test:

for f in `grep -l Audit *.html`
do
  # Replace js-test.js with testharness.js and add testharnessreport.js
  sed -i 's;\(.*\)\(<script.*\)js-test.js\(.*\);\1\2testharness.js\3\
\1\2testharnessreport.js\3 ;' $f
  # Remove functions that don't exist anymore.
  sed -i '/^[ 	]*\(description(".*")\|finishJSTest()\);[ 	]*/d' $f
done

However, a few files needed manual adjustment because they used
testPassed()/testFailed(), which no longer exists.  These were either
removed or replaced by Should().summarize().

The expected results files were manually removed.

BUG=675987
TEST=Analyser/* passes

Committed: https://crrev.com/bdbfd895a9da23367f3d34465b78047cdc1de4e3
Cr-Commit-Position: refs/heads/master@{#440166}
==========

[commit-bot: I haz the power, 2016-12-21 18:48:33]

Patchset 3 (id:??) landed as
https://crrev.com/bdbfd895a9da23367f3d34465b78047cdc1de4e3
Cr-Commit-Position: refs/heads/master@{#440166}