OS() and USE() tests are preferred over a series of PLATFORM() tests for "for low-level services" - see also the discussion at bug 107406. JavaScriptCore for OS(DARWIN) is used by PLATFORM(MAC), PLATFORM(QT), PLATFORM(WX) and PLATFORM(GTK), so "PLATFORM(MAC) || PLATFORM(IOS) || (PLATFORM(WX) && OS(DARWIN)) || (PLATFORM(QT) && OS(DARWIN))" can be replaced by "OS(DARWIN) && USE(CF)" which makes the code more readable as well. PLATFORM(MAC), PLATFORM(QT) and PLATFORM(WX) are all enable USE(CF) but not PLATFORM(GTK).
Created attachment 184893 [details] proposed change
Comment on attachment 184893 [details] proposed change LGTM.
Comment on attachment 184893 [details] proposed change Clearing flags on attachment: 184893 Committed r141059: <http://trac.webkit.org/changeset/141059>
All reviewed patches have been landed. Closing bug.