Bug 249888 - Remove "bgpropertiesAttr"
Summary: Remove "bgpropertiesAttr"
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ahmad Saleem
URL:
Keywords: GoodFirstBug, InRadar
Depends on:
Blocks:
 
Reported: 2022-12-25 14:26 PST by Ahmad Saleem
Modified: 2023-01-09 23:18 PST (History)
7 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-12-25 14:26:30 PST 
Hi Team,

I am raising this bug to remove "bgpropertiesAttr" since I couldn't manage to find any old one. It was never supported by Gecko and was not standardised and removed by Blink as well in 2014.

It is not net positive on code clean-up but WebKit is only one supporting right now and I think it is now time to say Goodbyes:

Blink Commit - https://chromium.googlesource.com/chromium/blink/+/278d9993a71dfaa8f8e324ac021313ea038e5e44

WebKit Source - https://searchfox.org/wubkat/source/Source/WebCore/html/HTMLBodyElement.cpp#70 & https://searchfox.org/wubkat/source/Source/WebCore/html/HTMLBodyElement.cpp#93

Just wanted to raise bug so we get input (if needed) or just get rid of it.

Thanks!
Comment 1 Radar WebKit Bug Importer 2023-01-01 14:27:17 PST 
<rdar://problem/103802245>
Comment 2 Ahmad Saleem 2023-01-09 17:49:23 PST 
PR - https://github.com/WebKit/WebKit/pull/8425
Comment 3 EWS 2023-01-09 23:18:00 PST 
Committed 258704@main (a6647584c855): <https://commits.webkit.org/258704@main>

Reviewed commits have been landed. Closing PR #8425 and removing active labels.