Patch forthcoming.
Created attachment 261973 [details] the patch
Created attachment 261974 [details] the patch
Created attachment 261975 [details] the patch
Comment on attachment 261975 [details] the patch View in context: https://bugs.webkit.org/attachment.cgi?id=261975&action=review > Source/WTF/wtf/MainThread.h:71 > +inline bool isMainThreadOrGCThread() { return true; } Since isMainThreadOrGCThread() is only used in assertions and this version is useless, can you just omit it altogether in the ASSERT_DISABLED case?
(In reply to comment #4) > Comment on attachment 261975 [details] > the patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=261975&action=review > > > Source/WTF/wtf/MainThread.h:71 > > +inline bool isMainThreadOrGCThread() { return true; } > > Since isMainThreadOrGCThread() is only used in assertions and this version > is useless, can you just omit it altogether in the ASSERT_DISABLED case? Yeah, that's better! I'll try that.
Created attachment 261976 [details] patch for landing
Created attachment 261977 [details] patch for landing
Meh, we don't need this.