The arguments RefPtr is being released and later used. I have some other tweaks in this function too.
Created attachment 183017 [details] Proposed Change
Comment on attachment 183017 [details] Proposed Change r=me Although it might be nice to make a test for this. I'm surprised that it wasn't covered by existing tests.
Comment on attachment 183017 [details] Proposed Change Clearing flags on attachment: 183017 Committed r139927: <http://trac.webkit.org/changeset/139927>
All reviewed patches have been landed. Closing bug.
Comment on attachment 183017 [details] Proposed Change View in context: https://bugs.webkit.org/attachment.cgi?id=183017&action=review > Source/WebCore/page/Console.cpp:218 > + String argAsString = arguments->argumentAt(i).toString(arguments->globalState()); > + printf(" %s", argAsString.utf8().data()); Could all be one line now without a local variable.