New issue
Advanced search Search tips

Issue 650598 link

Starred by 3 users
Status: Fixed
Owner:
f...@opera.com
Closed: Dec 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Launch-OWP
Launch-Accessibility: ----
Launch-Exp-Leadership: ----
Launch-Leadership: ----
Launch-Legal: ----
Launch-M-Approved: ----
Launch-M-Target: ----
Launch-Privacy: ----
Launch-Security: ----
Launch-Test: ----
Launch-UI: ----
Rollout-Type: ----



Sign in to add a comment

Remove SVGCursorElement

Project Member Reported by f...@opera.com, Sep 27 2016 
The <cursor> element is not widely supported (only in "WebKit-derived" browsers), and has been deprecated in SVG2 [1]. Because the "full" semantics have not been implemented (which would allow the <cursor> element to reference another element which would be used as the cursor), the current implementation does not really offer any additional functionality to that of a non-<cursor> reference (i.e a regular image URL.) Regardless, referencing an element could be done directly from the 'cursor' property, so there should be no need for a <cursor> element.

The <cursor> element also happen to be mentioned as "" in the HTML spec's "Dependencies" section [2].

[1] https://svgwg.org/svg2-draft/interact.html#CursorElement
[2] https://html.spec.whatwg.org/multipage/infrastructure.html#dependencies (search for "SVG")

Comment 1 by f...@opera.com, Sep 27 2016 

Currently we only have https://www.chromestatus.com/metrics/feature/timeline/popularity/1112 related to this, and it only measures access to SVGCursorElement.x/.y.

Comment 2 by dtapu...@chromium.org, Sep 27 2016

Components: Blink>SVG

Comment 3 by schenney@chromium.org, Sep 28 2016

Owner: f...@opera.com
Status: Assigned (was: Available)
Assigning while we get usage metrics sorted out. Feel free to un-assign after that.

Comment 5 by f...@opera.com, Nov 23 2016

Labels: -Type-Bug Type-Launch-OWP
Project Member

Comment 6 by bugdroid1@chromium.org, Dec 13 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f63d67df608751f83eaa15a67df069a38714f171

commit f63d67df608751f83eaa15a67df069a38714f171
Author: fs <fs@opera.com>
Date: Tue Dec 13 10:19:54 2016

Remove SVGCursorElement

This allows significant cleanup of CSSCursorImageValue, so do that too.

Intent-thread:

 https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/r0lnh5Rw23M

TBR=timvolodine@chromium.org
BUG= 650598 

Review-Url: https://codereview.chromium.org/2522443002
Cr-Commit-Position: refs/heads/master@{#438116}

[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/android_webview/tools/system_webview_shell/test/data/webexposed/global-interface-listing-expected.txt
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/LayoutTests/fast/events/mouse-cursor-expected.txt
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/LayoutTests/fast/events/mouse-cursor.html
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/LayoutTests/imported/wpt/html/browsers/origin/cross-origin-objects/cross-origin-objects-exceptions-expected.txt
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/LayoutTests/imported/wpt/svg/interfaces-expected.txt
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/LayoutTests/platform/win/virtual/stable/webexposed/global-interface-listing-expected.txt
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/css/cursor-change-href-expected.txt
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/css/cursor-change-href.svg
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/css/cursor-hotspot-override-from-css.html
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/css/svgcursorelement-use-counter.html
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/LayoutTests/svg/custom/global-constructors-expected.txt
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/LayoutTests/svg/dom/SVGStaticListTearOff-contextElement-crash-expected.txt
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/LayoutTests/svg/dom/SVGStaticListTearOff-contextElement-crash.html
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/LayoutTests/svg/dom/svg2-inheritance-expected.txt
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/dynamic-updates/SVGCursorElement-dom-x-attr-expected.png
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/dynamic-updates/SVGCursorElement-dom-x-attr-expected.txt
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/dynamic-updates/SVGCursorElement-dom-x-attr.html
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/dynamic-updates/SVGCursorElement-dom-y-attr-expected.png
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/dynamic-updates/SVGCursorElement-dom-y-attr-expected.txt
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/dynamic-updates/SVGCursorElement-dom-y-attr.html
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/dynamic-updates/SVGCursorElement-svgdom-x-prop-expected.png
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/dynamic-updates/SVGCursorElement-svgdom-x-prop-expected.txt
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/dynamic-updates/SVGCursorElement-svgdom-x-prop.html
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/dynamic-updates/SVGCursorElement-svgdom-y-prop-expected.png
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/dynamic-updates/SVGCursorElement-svgdom-y-prop-expected.txt
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/dynamic-updates/SVGCursorElement-svgdom-y-prop.html
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/dynamic-updates/script-tests/SVGCursorElement-dom-x-attr.js
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/dynamic-updates/script-tests/SVGCursorElement-dom-y-attr.js
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/dynamic-updates/script-tests/SVGCursorElement-svgdom-x-prop.js
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/LayoutTests/svg/dynamic-updates/script-tests/SVGCursorElement-svgdom-y-prop.js
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/LayoutTests/virtual/stable/webexposed/element-instance-property-listing-expected.txt
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/LayoutTests/virtual/stable/webexposed/global-interface-listing-expected.txt
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/LayoutTests/webexposed/element-instance-property-listing-expected.txt
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/LayoutTests/webexposed/global-interface-listing-expected.txt
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/Source/core/core_idl_files.gni
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/Source/core/css/CSSCursorImageValue.cpp
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/Source/core/css/CSSCursorImageValue.h
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/Source/core/css/ComputedStyleCSSValueMapping.cpp
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/Source/core/css/parser/CSSPropertyParser.cpp
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/Source/core/css/resolver/ElementStyleResources.cpp
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/Source/core/css/resolver/ElementStyleResources.h
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/Source/core/css/resolver/StyleBuilderCustom.cpp
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/Source/core/frame/UseCounter.h
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/Source/core/style/StylePendingImage.h
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/Source/core/svg/BUILD.gn
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/Source/core/svg/SVGCursorElement.cpp
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/Source/core/svg/SVGCursorElement.h
[delete] https://crrev.com/c98b5438fcbe802408e6cbec64d3003645e06106/third_party/WebKit/Source/core/svg/SVGCursorElement.idl
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/Source/core/svg/SVGElement.cpp
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/Source/core/svg/SVGElement.h
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/Source/core/svg/SVGElementRareData.cpp
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/Source/core/svg/SVGElementRareData.h
[modify] https://crrev.com/f63d67df608751f83eaa15a67df069a38714f171/third_party/WebKit/Source/core/svg/SVGTagNames.in

Comment 7 by f...@opera.com, Dec 13 2016

Status: Fixed (was: Assigned)

Comment 8 by rbyers@chromium.org, Feb 8 2017

Labels: M-57
►

Sign in to add a comment