Bug 227897

Summary: consider removing support for removed SVG properties
Product: WebKit Reporter: Cameron McCormack (:heycam) <heycam>
Component: SVGAssignee: Rob Buis <rbuis>
Status: RESOLVED FIXED    
Severity: Normal CC: changseok, dino, esprehn+autocc, ews-watchlist, fmalita, glenn, gyuyoung.kim, joepeck, kondapallykalyan, macpherson, menard, pdr, rbuis, sabouhallawa, schenney, sergio, simon.fraser, webkit-bug-importer, zimmermann
Priority: P2 Keywords: InRadar
Version: WebKit Local Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 191292    
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch none

Description Cameron McCormack (:heycam) 2021-07-12 23:31:33 PDT 
There are four properties that were removed from the SVG 2 spec, and which are not supported in other browsers:

* color-profile -- we recognize the property but don't do anything with it
* color-rendering -- we parse and expose this but don't do anything else with it
* glyph-orientation-horizontal -- we do use this in our SVG text layout
* kerning -- we do use this too in our SVG text layout

Don't see why we shouldn't remove the first two, and we should probably try to remove the second two as well.  (The text-orientation and letter-spacing properties are the official replacements for glyph-orientation-horizontal and kerning.)
Comment 1 Cameron McCormack (:heycam) 2021-07-12 23:42:07 PDT 
Also, glyph-orientation-vertical is deprecated, but still is in the spec.  But WebKit is the only engine to implement it too.
Comment 2 Cameron McCormack (:heycam) 2021-07-12 23:46:41 PDT 
Relatedly, enable-background is another one that was removed (though from the Filters spec), and which we recognize but don't do anything with.
Comment 3 Radar WebKit Bug Importer 2021-07-19 23:32:16 PDT 
<rdar://problem/80821560>
Comment 4 Rob Buis 2021-08-08 11:27:44 PDT 
Created attachment 435149 [details]
Patch
Comment 5 Rob Buis 2021-08-08 13:24:35 PDT 
Created attachment 435152 [details]
Patch
Comment 6 Rob Buis 2021-08-08 14:16:17 PDT 
Created attachment 435157 [details]
Patch
Comment 7 Rob Buis 2021-08-08 22:43:31 PDT 
Created attachment 435166 [details]
Patch
Comment 8 Rob Buis 2021-08-09 00:35:55 PDT 
Created attachment 435173 [details]
Patch
Comment 9 Rob Buis 2021-08-09 04:57:47 PDT 
Created attachment 435183 [details]
Patch
Comment 10 Rob Buis 2021-08-09 09:10:26 PDT 
This seems reasonable to me for color-profile, color-rendering and enable-background.
Comment 11 Rob Buis 2021-10-18 06:54:08 PDT 
@heycam WDYT, looks reasonable?
Comment 12 Cameron McCormack (:heycam) 2021-10-18 23:10:46 PDT 
Comment on attachment 435183 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=435183&action=review

Non-reviewer r=me.

> Source/WebCore/css/SVGCSSComputedStyleDeclaration.cpp:183
>          // the above properties are not yet implemented in the engine

"this property is not yet implemented in the engine"
Comment 13 Cameron McCormack (:heycam) 2021-10-18 23:12:01 PDT 
Once this patch lands, please file a separate bug for the other properties I mentioned that we should try to remove.
Comment 14 Rob Buis 2021-10-18 23:46:38 PDT 
Created attachment 441692 [details]
Patch
Comment 15 Rob Buis 2021-10-19 06:03:00 PDT 
(In reply to Cameron McCormack (:heycam) from comment #13)
> Once this patch lands, please file a separate bug for the other properties I
> mentioned that we should try to remove.

Sure, I filed https://bugs.webkit.org/show_bug.cgi?id=231948 for this.
Comment 16 EWS 2021-10-19 06:40:50 PDT 
Committed r284447 (243207@main): <https://commits.webkit.org/243207@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 441692 [details].