This rolls outs https://trac.webkit.org/r233773 which was landed for https://bugs.webkit.org/show_bug.cgi?id=187160. <rdar://problem/54622500>
Created attachment 377086 [details] proposed patch.
Comment on attachment 377086 [details] proposed patch. View in context: https://bugs.webkit.org/attachment.cgi?id=377086&action=review > Source/bmalloc/bmalloc/PerThread.h:2 > + * Copyright (C) 2014-2017 Apple Inc. All rights reserved. Is this intentional? Did you mean 2019? > Source/bmalloc/bmalloc/ProcessCheck.mm:-47 > - || [processName isEqualToString:@"DumpRenderTree"] Why?
Comment on attachment 377086 [details] proposed patch. View in context: https://bugs.webkit.org/attachment.cgi?id=377086&action=review >> Source/bmalloc/bmalloc/PerThread.h:2 >> + * Copyright (C) 2014-2017 Apple Inc. All rights reserved. > > Is this intentional? Did you mean 2019? This came from the rollout. I can change this to 2019. >> Source/bmalloc/bmalloc/ProcessCheck.mm:-47 >> - || [processName isEqualToString:@"DumpRenderTree"] > > Why? This came straight from rolling out Michael's patch. My understanding is that this was added to workaround a DRT issue that came with this patch. Would like Michael to comment on whether this should be left in.
Comment on attachment 377086 [details] proposed patch. View in context: https://bugs.webkit.org/attachment.cgi?id=377086&action=review >>> Source/bmalloc/bmalloc/ProcessCheck.mm:-47 >>> - || [processName isEqualToString:@"DumpRenderTree"] >> >> Why? > > This came straight from rolling out Michael's patch. My understanding is that this was added to workaround a DRT issue that came with this patch. Would like Michael to comment on whether this should be left in. I think we should keep this in.
Thanks for the reviews. Landed in r249065: <http://trac.webkit.org/r249065>.