After https://trac.webkit.org/changeset/205275, all LOG_WITH_STREAM macros are now enabled in release builds, because LogMacros.h has: #if LOG_DISABLED && RELEASE_LOG_DISABLED ... #else #define LOG_WITH_STREAM(.....) #endif LOG() is OK, so we should make them match. <rdar://problem/28347599>
Disable those macros in RELEASE_LOG_DISABLED mode. With that disabled, there were a couple of downstream effects. First, GraphicsContextCG.cpp would no longer access a local variable in release mode, so that variable needed to be conditionally defined. Second, the inclusion of logFunctionResult also needed to be fiddled with.
Created attachment 289253 [details] Patch
Comment on attachment 289253 [details] Patch Clearing flags on attachment: 289253 Committed r206120: <http://trac.webkit.org/changeset/206120>
All reviewed patches have been landed. Closing bug.