[Webkit-unassigned] [Bug 57969] Add web audio support to DumpRenderTree (mac port)

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Apr 6 14:03:56 PDT 2011


https://bugs.webkit.org/show_bug.cgi?id=57969





--- Comment #5 from Tony Chang <tony at chromium.org>  2011-04-06 14:03:56 PST ---
(From update of attachment 88485)
View in context: https://bugs.webkit.org/attachment.cgi?id=88485&action=review

I think it's OK to put the layout test and the changes to DRT in the same patch.

> Tools/DumpRenderTree/mac/DumpRenderTree.mm:948
> +        if (gLayoutTestController->dumpAsAudio()) {

How big is the -expected.txt?  How horrible would it be to just console.log (or append to a div) the base64 data in the test?  The main benefit would be not having to introduce dumpAsAudio() or remember to call it.  You could also have setEncodedAudioData() write the base64 data via printf (we do this for other delegates like the editing delegate).

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list