Qt5 currently prints a lot of debug messages similar to: No platform plugin argument was specified, defaulting to "xcb". No platform plugin argument was specified, defaulting to "xcb". Successfully connected to display :7.0 Successfully connected to display :7.0 Information of screen 253: width.........: 1024 height........: 768 depth.........: 24 white pixel...: ffffff black pixel...: 0 Information of screen 253: width.........: 1024 height........: 768 depth.........: 24 white pixel...: ffffff black pixel...: 0 QIBusPlatformInputContext: ibus config file '/home/webkitbuildbot/.config/ibus/bus/124cb6f3e543db1073af49560001668c-unix-7' does not exist. QIBusPlatformInputContext: not connected. Suppress these in the API tests to make the output less messy.
Created attachment 113181 [details] proposed patch
Comment on attachment 113181 [details] proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=113181&action=review r- due to style :). Looks good otherwise. > Source/WebKit2/UIProcess/API/qt/tests/util.h:33 > +#define SUPPRESS_DEBUG_OUTPUT \ Style guide says that this macro should be named like a function and have parentheses at the end. But ... why not simply make it an actual function that's implemented in util.cpp? (instead of "exporting" messageHandler)
Created attachment 113194 [details] proposed patch
Comment on attachment 113194 [details] proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=113194&action=review r=me but add the missing "static" keyword when landing I'd say :) > Source/WebKit2/UIProcess/API/qt/tests/util.cpp:53 > +void messageHandler(QtMsgType type, const char* message) This method should be static.
Committed r98983: <http://trac.webkit.org/changeset/98983>
Comment on attachment 113194 [details] proposed patch Clearing flags.