Bug 264291 - Make execCommand("insertImage", false, "") not add a src attribute
Summary: Make execCommand("insertImage", false, "") not add a src attribute
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-11-06 14:29 PST by Ahmad Saleem
Modified: 2023-11-13 14:30 PST (History)
2 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 2023-11-06 14:29:14 PST 
Hi Team,

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

Test Case: https://jsfiddle.net/qbsuLan9/show

Safari Technology Preview 182 fails all, while Chrome Canary 121 pass both and Firefox Nightly 121 fail only two.

WebKit Source: https://searchfox.org/wubkat/source/Source/WebCore/editing/EditorCommand.cpp#482

Just wanted to raise, so we can fix. Although from quick looks that we already have this change applied but I think it would be still ideal to fix the test case.

Thanks!
Comment 1 Ahmad Saleem 2023-11-06 18:08:56 PST 
NOTE: We might have it on WPT as well - https://wpt.fyi/results/editing/run/insertimage.html?label=master&label=experimental&aligned&q=safari%3Afail

Although, Chrome and Safari are failing similar tests.
Comment 2 Radar WebKit Bug Importer 2023-11-13 14:30:17 PST 
<rdar://problem/118356377>