RESOLVED FIXED Bug 183450
[Win] Use __debugbreak for WTFBreakpointTrap 
https://bugs.webkit.org/show_bug.cgi?id=183450
Summary [Win] Use __debugbreak for WTFBreakpointTrap
Yusuke Suzuki
Reported 2018-03-08 06:24:46 PST
[Win] Use __debugbreak for WTFBreakpointTrap
Attachments
Patch (1.51 KB, patch)
2018-03-08 06:25 PST, Yusuke Suzuki 		mark.lam: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Yusuke Suzuki
Comment 1 2018-03-08 06:25:51 PST
Created attachment 335296 [details] Patch
Mark Lam
Comment 2 2018-03-08 07:24:54 PST
Comment on attachment 335296 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=335296&action=review r=me > Source/WTF/wtf/Assertions.h:212 > +#else > #if ASAN_ENABLED Change this into an "#elif ASAN_ENABLED" because these cases should all be mutually exclusive. > Source/WTF/wtf/Assertions.h:223 > +#endif // COMPILER(MSVC) Remove this.
Yusuke Suzuki
Comment 3 2018-03-08 07:43:35 PST
Comment on attachment 335296 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=335296&action=review Thank you! >> Source/WTF/wtf/Assertions.h:212 >> #if ASAN_ENABLED > > Change this into an "#elif ASAN_ENABLED" because these cases should all be mutually exclusive. OK, changed. >> Source/WTF/wtf/Assertions.h:223 >> +#endif // COMPILER(MSVC) > > Remove this. Fixed.
Yusuke Suzuki
Comment 4 2018-03-08 07:47:31 PST
Committed r229409: <https://trac.webkit.org/changeset/229409>
Radar WebKit Bug Importer
Comment 5 2018-03-08 07:48:34 PST
<rdar://problem/38262145>
Note You need to log in before you can comment on or make changes to this bug.