Bug 247814

Summary: Escape non-ASCII characters in console messages
Product: WebKit Reporter: Ahmad Saleem <ahmad.saleem792>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: NEW    
Severity: Normal CC: sabouhallawa, webkit-bug-importer, zimmermann
Priority: P2 Keywords: InRadar
Version: Safari Technology Preview   
Hardware: Unspecified   
OS: Unspecified   

Ahmad Saleem
Reported 2022-11-11 09:58:29 PST
Hi Team, Just going through Blink commits, I come across another potential merge candidate: Blink Commit - https://chromium.googlesource.com/chromium/blink/+/a2db82039c609e1aac58a17e660aa10235331d23 Webkit GitHub - https://github.com/WebKit/WebKit/blob/4808f5e2de9f553275955f6b2cf652f11f355d1b/Source/WebCore/svg/SVGElement.cpp#L246 ___ Just wanted to raise the bug as future candidate for merging to align with blink. Thanks!
Attachments
Add attachment proposed patch, testcase, etc.
Ahmad Saleem
Comment 1 2022-11-11 10:58:06 PST
I tried but Webkit JSONValues is not smarter and don't have "quoteString" and we have to either use "escapeString" (which leads to argument issue) or create our own function for double quotes similar to Blink. PR - https://github.com/WebKit/WebKit/pull/6402
Radar WebKit Bug Importer
Comment 2 2022-11-18 09:59:17 PST
<rdar://problem/102519095>
Note You need to log in before you can comment on or make changes to this bug.