Convert AudioContext tests to testharness |
||
Issue descriptionConvert AudioContext tests to use testharness
,
Jan 23 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3aee9f268a3a471fa539d704e622aee1ce68f0ce commit 3aee9f268a3a471fa539d704e622aee1ce68f0ce Author: rtoy <rtoy@chromium.org> Date: Mon Jan 23 23:00:50 2017 Convert AudioContext Audit tests to testharness Conversion done using this script: for f in `grep -l Audit *.html` do # Replace js-test with testharness and add testharnessreport.js sed -i 's;\(.*\)\(<script.*\)js-test.js\(.*\);\1\2testharness.js\3\ \1\2testharnessreport.js\3 ;' $f # Remove old js-test methods that no longer exist. sed -i '/^[ ]*\(description(.*)\|finishJSTest()\);[ ]*$/d' $f # Remove jsTestIsAsync stuff sed -i '/^[ ]*window.jsTestIsAsync *= *true;[ ]*$/d' $f # Remove expected results file. base=`basename $f .html` git rm $base-expected.txt done Since the original tests didn't actually use any of the old Audit.Should() functions, we took this opportunity to convert it to the new Audit.should() because we had to convert the old js-test functions anyway. BUG= 678701 TEST=AudioContext/audiocontext-close.html, AudioContext/audiocontext-suspend-resume.html Review-Url: https://codereview.chromium.org/2611133003 Cr-Commit-Position: refs/heads/master@{#445529} [delete] https://crrev.com/5d8596504544ee64049242d88b15fc3a79295519/third_party/WebKit/LayoutTests/webaudio/AudioContext/audiocontext-close-expected.txt [modify] https://crrev.com/3aee9f268a3a471fa539d704e622aee1ce68f0ce/third_party/WebKit/LayoutTests/webaudio/AudioContext/audiocontext-close.html [delete] https://crrev.com/5d8596504544ee64049242d88b15fc3a79295519/third_party/WebKit/LayoutTests/webaudio/AudioContext/audiocontext-suspend-resume-expected.txt [modify] https://crrev.com/3aee9f268a3a471fa539d704e622aee1ce68f0ce/third_party/WebKit/LayoutTests/webaudio/AudioContext/audiocontext-suspend-resume.html
,
Jan 23 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ec77ce738b4415bb5d2bf6aa2779931a06676dfe commit ec77ce738b4415bb5d2bf6aa2779931a06676dfe Author: rtoy <rtoy@chromium.org> Date: Mon Jan 23 23:55:38 2017 Convert audiocontext-close-basic and max-contexts to testharness Manually converted test to use testharness and new Audit. BUG= 678701 TEST=AudioContext/audiocontext-close-basic.html, audiocontext-max-contexts.html Review-Url: https://codereview.chromium.org/2616623006 Cr-Commit-Position: refs/heads/master@{#445549} [delete] https://crrev.com/c9eeee6e21fa8b8ec1829f9eac2d343640d4d01c/third_party/WebKit/LayoutTests/webaudio/AudioContext/audiocontext-close-basic-expected.txt [modify] https://crrev.com/ec77ce738b4415bb5d2bf6aa2779931a06676dfe/third_party/WebKit/LayoutTests/webaudio/AudioContext/audiocontext-close-basic.html [delete] https://crrev.com/c9eeee6e21fa8b8ec1829f9eac2d343640d4d01c/third_party/WebKit/LayoutTests/webaudio/AudioContext/audiocontext-max-contexts-expected.txt [modify] https://crrev.com/ec77ce738b4415bb5d2bf6aa2779931a06676dfe/third_party/WebKit/LayoutTests/webaudio/AudioContext/audiocontext-max-contexts.html
,
Mar 23 2017
AudioContext/audiocontext-lock-threading-race.html contains a call to gc(). We're not going to convert this file since it's not portable anyway. |
||
►
Sign in to add a comment |
||
Comment 1 by bugdroid1@chromium.org
, Jan 23 2017