WebUnitTests.cpp is an intentional layering violation and includes some headers from base/ in the Chromium code. "config.h" is largely unneeded (it was used for ASSERT, and attempting to run a null test suite isn't going to get very far anyway) and conflicted with base/logging.h, so it'd be nice to remove it rather than teach certain files not to #include base/logging.h.
Created attachment 186671 [details] Patch
Comment on attachment 186671 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=186671&action=review > Source/WebKit/chromium/tests/WebUnitTests.cpp:33 > #include <base/test/test_suite.h> Please add a FIXME about this include since it's a layering violation.
Committed r142027: <http://trac.webkit.org/changeset/142027>