RESOLVED FIXED 58195
new-run-webkit-tests: implement support for audio, take two 
https://bugs.webkit.org/show_bug.cgi?id=58195
Summary new-run-webkit-tests: implement support for audio, take two
Dirk Pranke
Reported 2011-04-09 19:11:19 PDT
new-run-webkit-tests: implement support for audio, take two
Attachments
Patch (36.87 KB, patch)
2011-04-09 19:28 PDT, Dirk Pranke 		tony: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Dirk Pranke
Comment 1 2011-04-09 19:28:01 PDT
Created attachment 88940 [details] Patch
Dirk Pranke
Comment 2 2011-04-09 19:32:08 PDT
Comment on attachment 88940 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=88940&action=review This is a revised version of the patch(es) landed in bug 58101. Those patches were rolled out because they were crashing new-run-webkit-tests. The regression turned out to be a result of passing None instead of '' to Port.diff_image(). I have added comments to show the diffs from the previous patch. > Tools/Scripts/webkitpy/layout_tests/port/chromium.py:143 > + return True This chunk is the fix for the crash on diff_image() in chromium ports. This really needs unit tests, but to test it properly requires changing too many other things, so I'll post the unit tests in a separate patch for clarity. > Tools/Scripts/webkitpy/layout_tests/port/webkit.py:115 > + if not actual_contents or not expected_contents: And this is the fix for the webkit-base ports. Again, I will post unit tests in a separate patch for clarity.
Dirk Pranke
Comment 3 2011-04-09 19:52:10 PDT
unit tests posted in bug 58196.
Dirk Pranke
Comment 4 2011-04-11 12:18:13 PDT
Committed r83475: <http://trac.webkit.org/changeset/83475>
Note You need to log in before you can comment on or make changes to this bug.