Bug 249576 - Escape '&' in javascript URLs for innerHTML/outerHTML
Summary: Escape '&' in javascript URLs for innerHTML/outerHTML
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-12-19 05:51 PST by Ahmad Saleem
Modified: 2022-12-19 17:31 PST (History)
1 user (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-12-19 05:51:17 PST 
Hi Team,

While going through Blink's commit, I came across another failing test case:

Test Case - https://jsfiddle.net/c6x9mjt4/show

^ Safari 16.2 & Safari Technology Preview 160 fails more tests than Chrome Canary 111 and Firefox Nightly 110 (which only fail one test and match each other).

Blink Commit - https://chromium.googlesource.com/chromium/blink/+/6ff047e6fd07d74a38cdccca2221b0ba9e72acea

WebKit Source - https://searchfox.org/wubkat/source/Source/WebCore/editing/MarkupAccumulator.cpp#314

I think it is easy to fix and I can give it a try. Just wanted to raise this so we can track it.

Thanks!
Comment 1 EWS 2022-12-19 17:30:46 PST 
Committed 258112@main (aff702358ff4): <https://commits.webkit.org/258112@main>

Reviewed commits have been landed. Closing PR #7859 and removing active labels.
Comment 2 Radar WebKit Bug Importer 2022-12-19 17:31:18 PST 
<rdar://problem/103537944>