RESOLVED FIXED 203646
[SVG2] Add the 'orient' property of the interface SVGMarkerElement 
https://bugs.webkit.org/show_bug.cgi?id=203646
Summary [SVG2] Add the 'orient' property of the interface SVGMarkerElement
Said Abou-Hallawa
Reported 2019-10-30 16:43:25 PDT
The spec link is https://www.w3.org/TR/SVG2/painting.html#InterfaceSVGMarkerElement
Attachments
Patch (20.75 KB, patch)
2019-10-30 16:49 PDT, Said Abou-Hallawa 		no flags
DetailsFormatted DiffDiff
Patch (33.69 KB, patch)
2019-11-13 13:26 PST, Said Abou-Hallawa 		no flags
DetailsFormatted DiffDiff
Patch (35.11 KB, patch)
2019-11-13 16:03 PST, Said Abou-Hallawa 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Said Abou-Hallawa
Comment 1 2019-10-30 16:49:31 PDT
Created attachment 382391 [details] Patch
Said Abou-Hallawa
Comment 2 2019-11-13 13:26:40 PST
Created attachment 383488 [details] Patch
Simon Fraser (smfr)
Comment 3 2019-11-13 14:54:26 PST
Comment on attachment 383488 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=383488&action=review > Source/WebCore/svg/SVGMarkerElement.h:70 > + void setOrientToAngle(SVGAngle&); const SVGAngle& or Ref<SVGAngle>&& ? Could a null angle indicate "auto"? > Source/WebCore/svg/SVGMarkerTypes.h:85 > + static unsigned highestEnumValue() { return SVGMarkerOrientAngle; } Deserves a comment to explain why it's not SVGMarkerOrientAutoStartReverse
Said Abou-Hallawa
Comment 4 2019-11-13 16:03:20 PST
Created attachment 383508 [details] Patch
Said Abou-Hallawa
Comment 5 2019-11-13 16:06:48 PST
Comment on attachment 383488 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=383488&action=review >> Source/WebCore/svg/SVGMarkerElement.h:70 >> + void setOrientToAngle(SVGAngle&); > > const SVGAngle& or Ref<SVGAngle>&& ? Could a null angle indicate "auto"? Argument type was changed to const SVGAngle&. Some methods of SVGAngle had to be made const. >> Source/WebCore/svg/SVGMarkerTypes.h:85 >> + static unsigned highestEnumValue() { return SVGMarkerOrientAngle; } > > Deserves a comment to explain why it's not SVGMarkerOrientAutoStartReverse A comment was added in SVGMarkerTypes.h where the enum value SVGMarkerOrientAutoStartReverse is defined.
WebKit Commit Bot
Comment 6 2019-11-13 18:21:55 PST
Comment on attachment 383508 [details] Patch Clearing flags on attachment: 383508 Committed r252444: <https://trac.webkit.org/changeset/252444>
WebKit Commit Bot
Comment 7 2019-11-13 18:21:56 PST
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 8 2019-11-13 18:22:23 PST
<rdar://problem/57176727>
Darin Adler
Comment 9 2019-12-04 10:48:15 PST
Comment on attachment 383508 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=383508&action=review > Source/WebCore/svg/SVGMarkerElement.h:67 > + String orient() const; > + void setOrient(const String&); Can we use [Reflect] and omit these? > Source/WebCore/svg/SVGMarkerElement.idl:45 > + attribute DOMString orient; Can we use [Reflect] here?
Note You need to log in before you can comment on or make changes to this bug.