The debugging prints for DEBUG_AUDIONODE_REFERENCES use printf, but it's nicer to fprintf instead so that the debugging messages aren't all buffered up and printed whenever stdio decides to output the buffer. It makes it easier to follow things as they happen.
Created attachment 148297 [details] Patch
Attachment 148297 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1 Source/WebCore/ChangeLog:10: You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible. [changelog/nonewtests] [5] Total errors found: 1 in 3 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 148310 [details] Patch
Comment on attachment 148310 [details] Patch LGTM, r=me.
Comment on attachment 148310 [details] Patch Clearing flags on attachment: 148310 Committed r120707: <http://trac.webkit.org/changeset/120707>
All reviewed patches have been landed. Closing bug.
(In reply to comment #4) > (From update of attachment 148310 [details]) > LGTM, r=me. A bit too late, but doesn't AudioNode.cpp need to include <stdio.h> when DEBUG_AUDIONODE_REFERENCES is non-zero? I think I needed this on linux.