In https://bugs.webkit.org/show_bug.cgi?id=248745, I tried to fix a bug where tests with multiple periods in their name would resolve the wrong link. I managed to fix this situation in some cases, but didn't cover all of them. This is because I was checking the length of the second "extension" instead of the first. I could fix this bug, but clearly, this code is confusing me and others, so I tracked down when this was added, https://bugs.webkit.org/show_bug.cgi?id=160198. The problem I was trying to fix involved crash logs gathered at the end of a test run not associated to any test, and run-webkit-tests was assigning those crash logs "names" which were the name of the process. If you strip the "extension" from a string like "com.apple.WebKit.WebContent", you basically end up with a bunch of "com.apple.WebKit" processes, which isn't right. Really, We should look for file "names" which are explicitly process names instead of trying to guess based on the number of characters in an extension, which is an opaque way to solve this problem.
<rdar://problem/103326384>
Pull request: https://github.com/WebKit/WebKit/pull/7582
Committed 258104@main (ab68abe37c22): <https://commits.webkit.org/258104@main> Reviewed commits have been landed. Closing PR #7582 and removing active labels.