Bug 261760

Summary: Websites Crashing/Reloading in STP 178
Product: WebKit Reporter: Jack Wellborn <w0nka>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Major CC: ahmad.saleem792, ap, sgill26, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Safari Technology Preview   
Hardware: Mac (Apple Silicon)   
OS: Other   
Attachments:
Description Flags
Demonstration of Ars Technica Crashing/Reloading
none
Webkit WebContent Crash Report none

Jack Wellborn
Reported 2023-09-19 11:15:51 PDT
Created attachment 467761 [details] Demonstration of Ars Technica Crashing/Reloading Hello, Some of the sites I commonly frequent seem to crash in Safari Technology Preview 178, most notably https://arstechnica.com/. This occurs even in a Private Browsing window. Environment: MacBook Pro with M1 Pro macOS Sonoma 14.0 (23A339) Safari Technology Preview 178
Attachments
Demonstration of Ars Technica Crashing/Reloading (119.69 MB, video/quicktime)
2023-09-19 11:15 PDT, Jack Wellborn
no flags
Webkit WebContent Crash Report (56.55 KB, text/plain)
2023-09-20 04:48 PDT, Jack Wellborn
no flags
Ahmad Saleem
Comment 1 2023-09-19 12:18:22 PDT
Can you attach 'Crash Logs' from Console.app? I am unable to reproduce the bug on macOS Ventura with STP178 and it does not crash. I do have ad blocker and other extensions though.
Jack Wellborn
Comment 2 2023-09-19 12:58:09 PDT
Weird. I also experienced the issue on https://www.washingtonpost.com earlier today, but I am not reproducing on either site now. My guess is some common bit of ad tech was causing the issue. I will close and reopen if it rears its ugly head again.
Jack Wellborn
Comment 3 2023-09-20 04:48:35 PDT
Created attachment 467785 [details] Webkit WebContent Crash Report This is the request Crash Report.
Jack Wellborn
Comment 4 2023-09-20 04:49:16 PDT
Issue started happening again on washingtonpost.com and I have attached the crash report.
Sam Sneddon [:gsnedders]
Comment 5 2023-09-20 07:46:03 PDT
Crashing in WebCore::ShorthandSerializer::serializeGridTemplate, which Sammy recently touched in Bug 260494. rdar://115783828
Sammy Gill
Comment 6 2023-09-20 13:28:37 PDT
Thanks for the report! Given the crash signature, I think there is a decent chance that this is a dupe of https://bugs.webkit.org/show_bug.cgi?id=261421 and that patch may fix the issue. Is there a way to reproduce the issue reliably? That would help us determine if this truly is a dupe because I have not been able to reproduce it on either of the sites yet
Jack Wellborn
Comment 7 2023-09-21 06:05:40 PDT
Unfortunately, I can't reproduce easily as I suspect the offending code is coming from some ad tech on sites. That said, I also strongly suspect it might be the same `element.TextContent = ""` behavior from the linked issue as that is exactly the kind of thing an ad related script might do. I am going to see if I can reproduce before updating to 179, then see if I can reproduce after.
Sammy Gill
Comment 8 2023-09-21 09:36:19 PDT
If I'm reading the release notes correctly it seems like that the change might not have gotten picked up by STP yet :(
Sammy Gill
Comment 9 2023-09-25 11:45:40 PDT
I am going to mark this as a duplicate since I think that the underlying cause of the bug is the same. Once the fix gets into STP please try it again and reopen the bugzilla if the issue still reproduces! *** This bug has been marked as a duplicate of bug 261421 ***
Note You need to log in before you can comment on or make changes to this bug.