RESOLVED FIXED 171829
Move offsetParent / offsetLeft / offsetTop / offsetWidth / offsetHeight from Element to HTMLElement 
https://bugs.webkit.org/show_bug.cgi?id=171829
Summary Move offsetParent / offsetLeft / offsetTop / offsetWidth / offsetHeight from ...
Chris Dumez
Reported 2017-05-08 14:38:11 PDT
Move offsetParent / offsetLeft / offsetTop / offsetWidth / offsetHeight from Element to HTMLElement, as per the specification: - https://drafts.csswg.org/cssom-view/#extensions-to-the-htmlelement-interface Both Firefox and Chrome match the specification and also do not have the corresponding API on SVGElement. Bug tracking the similar move in Blink (completed over 1 year ago): - https://bugs.chromium.org/p/chromium/issues/detail?id=463116 Note that for the inner SVG elements like <rect> and <circle> these attributes were returning 0, because those aren't part of the CSS box model. They did return correct values for the <svg> element itself, however.
Attachments
WIP patch (13.27 KB, patch)
2017-05-08 15:28 PDT, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Patch (21.49 KB, patch)
2017-05-08 15:59 PDT, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Patch (23.08 KB, patch)
2017-05-08 16:03 PDT, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2017-05-08 15:28:35 PDT
Created attachment 309421 [details] WIP patch
Chris Dumez
Comment 2 2017-05-08 15:29:14 PDT
Found bug in web-platform-test: https://github.com/w3c/web-platform-tests/pull/5855
Chris Dumez
Comment 3 2017-05-08 15:59:46 PDT
Created attachment 309427 [details] Patch
Chris Dumez
Comment 4 2017-05-08 16:03:33 PDT
Created attachment 309429 [details] Patch
WebKit Commit Bot
Comment 5 2017-05-08 17:16:21 PDT
Comment on attachment 309429 [details] Patch Clearing flags on attachment: 309429 Committed r216466: <http://trac.webkit.org/changeset/216466>
WebKit Commit Bot
Comment 6 2017-05-08 17:16:23 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.