Bug 264291
| Summary: | Make execCommand("insertImage", false, "") not add a src attribute | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | HTML Editing | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | webkit-bug-importer, wenson_hsieh |
| Priority: | P2 | Keywords: | InRadar |
| Version: | Safari Technology Preview | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ahmad Saleem
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!
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Ahmad Saleem
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.
Radar WebKit Bug Importer
<rdar://problem/118356377>