Summary: | Need WTFCrash workaround for shipping SafariForWebKitDevelopment binaries. | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Mark Lam <mark.lam> | ||||||||
Component: | Web Template Framework | Assignee: | Mark Lam <mark.lam> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | benjamin, cdumez, cmarcelo, commit-queue, ggaren, hi, joepeck | ||||||||
Priority: | P2 | ||||||||||
Version: | WebKit Local Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Mark Lam
2016-02-11 13:19:18 PST
Created attachment 271079 [details]
proposed patch.
Comment on attachment 271079 [details] proposed patch. View in context: https://bugs.webkit.org/attachment.cgi?id=271079&action=review > Source/WTF/wtf/Assertions.cpp:507 > +// This is a workaround for presently shipping (crica early 2016) SafariForWebKitDevelopment Nit: 'circa' Comment on attachment 271079 [details]
proposed patch.
I've talked with Geoff offline, and am going to re-implement this in a hopefully cleaner way. Stay tuned.
Created attachment 271103 [details]
proposed patch 2 based on Geoff's suggestion.
Comment on attachment 271103 [details] proposed patch 2 based on Geoff's suggestion. View in context: https://bugs.webkit.org/attachment.cgi?id=271103&action=review > Source/WTF/wtf/Assertions.h:157 > +ALWAYS_INLINE NO_RETURN_DUE_TO_CRASH void CRASH() It doesn't seem proper to introduce a new symbol into the global namespace that is not WTF prefixed. All other symbols in WTF introduces in Assertions.h/cpp are prefixed by "WTF". Can CRASH stay a macro, but just call to WTFCrashImpl()? > Source/WTF/wtf/Assertions.h:177 > +#else /* not defined(NDEBUG) && OS(DARWIN) */ > + > +WTF_EXPORT_PRIVATE NO_RETURN_DUE_TO_CRASH void WTFCrashImpl(); > +#define CRASH() WTFCrashImpl() And it would be pretty confusing for CRASH to be a macro on some ports, but a symbol on others. It would be nice to be consistent. Created attachment 271107 [details]
proposed patch 3 with Joe's feedback.
Comment on attachment 271107 [details]
proposed patch 3 with Joe's feedback.
Will commit manually.
Thanks for the review. Landed in r196458: <http://trac.webkit.org/r196458>. |