Don't always include DataDetectorsUI headers
Created attachment 311339 [details] Patch
Comment on attachment 311339 [details] Patch r=me
Created attachment 311416 [details] Disables data detections entirely
<rdar://problem/22739993>
Comment on attachment 311416 [details] Disables data detections entirely View in context: https://bugs.webkit.org/attachment.cgi?id=311416&action=review r=me > Source/WTF/wtf/Platform.h:552 > #define HAVE_AVKIT 1 > #define HAVE_PARENTAL_CONTROLS 1 > +#define ENABLE_DATA_DETECTION 1 Nit: Please alphabetize by full macro name (so ENABLE_* would come before HAVE_*). > Source/WebKit2/UIProcess/API/Cocoa/WKWebView.mm:56 > +#if ENABLE(DATA_DETECTION) > #import "WKDataDetectorTypesInternal.h" > +#endif Are we still putting these in a separate section after the default list of #import statements? > Source/WebKit2/WebProcess/InjectedBundle/API/Cocoa/WKWebProcessPlugInRangeHandle.mm:34 > +#if ENABLE(DATA_DETECTION) > #import "WKDataDetectorTypesInternal.h" > +#endif Move to after default list of #import statements?
Created attachment 311521 [details] Patch for landing
Comment on attachment 311521 [details] Patch for landing Wait for EWS.
Thanks for the review!
Comment on attachment 311521 [details] Patch for landing Clearing flags on attachment: 311521 Committed r217580: <http://trac.webkit.org/changeset/217580>
All reviewed patches have been landed. Closing bug.