Bug 260677
| Summary: | Remove 'SVGViewSpec' interface | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | SVG | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | karlcow, sabouhallawa, webkit-bug-importer, zimmermann |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=270089 | ||
Ahmad Saleem
Hi Team,
This was removed from Blink and also not in SVG2 spec.
Blink Commit: https://chromium.googlesource.com/chromium/src.git/+/b5891de613454cbb79dbcce852396649bf8c5342
GitHub Spec Commit: https://github.com/w3c/svgwg/commit/4c26fd36937a65192024208d85c144a21071b057
More Spec Context: https://www.w3.org/2015/06/11-svg-minutes.html#item02
"RESOLUTION: remove currentView and useCurrentView properties on SVGSVGElement and remove SVGViewSpec interface"
For first part, we already have bug 226721.
Just raising so we can track this as well to better align with SVG2 spec.
Thanks!
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/114791247>
Karl Dubost
Rob in Bug 270089 mentions about supporting it.
But SVGViewSpec is also not defined in Firefox
https://searchfox.org/firefox-main/search?q=svgviewspec&path=&case=false®exp=false
It's not in Chrome as @Ahmad is mentioning above.
It should be removed from WebKit too.
Referenced in
https://searchfox.org/wubkat/source/Source/WebCore/svg/SVGViewSpec.idl
https://searchfox.org/wubkat/source/Source/WebCore/svg/SVGViewSpec.cpp
https://searchfox.org/wubkat/source/Source/WebCore/svg/SVGViewSpec.h
https://searchfox.org/wubkat/source/Source/WebCore/bindings/js/JSSVGViewSpecCustom.cpp
There is a stalled issue on
https://github.com/w3c/svgwg/issues/747
Gecko has a piece of code
https://searchfox.org/firefox-main/rev/3c52c9c886f7b0423001cef86abfafcf97f2b90a/dom/svg/SVGFragmentIdentifier.cpp#219-220
```cpp
if (ProcessSVGViewSpec(aAnchorName, rootElement)) {
return true;
```
which is referencing
https://searchfox.org/firefox-main/rev/3c52c9c886f7b0423001cef86abfafcf97f2b90a/dom/svg/SVGFragmentIdentifier.cpp#116-155
Also
https://searchfox.org/wubkat/search?q=symbol:T_WebCore%3A%3ASVGViewSpec&redirect=false