Bug 247814 - Escape non-ASCII characters in console messages
Summary: Escape non-ASCII characters in console messages
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-11-11 09:58 PST by Ahmad Saleem
Modified: 2022-11-18 09:59 PST (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ahmad Saleem 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!
Comment 1 Ahmad Saleem 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
Comment 2 Radar WebKit Bug Importer 2022-11-18 09:59:17 PST 
<rdar://problem/102519095>