[webkit-reviews] review requested: [Bug 42815] getBoundingClientRect Broken for SVG Elements : [Attachment 65517] Fix and layout test

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Aug 25 19:35:30 PDT 2010


Cosmin Truta <ctruta at chromium.org> has asked  for review:
Bug 42815: getBoundingClientRect Broken for SVG Elements
https://bugs.webkit.org/show_bug.cgi?id=42815

Attachment 65517: Fix and layout test
https://bugs.webkit.org/attachment.cgi?id=65517&action=review

------- Additional Comments from Cosmin Truta <ctruta at chromium.org>
(In reply to comment #1)
I wonder if it's necessary to create this separate code path for SVG inside
Element::getBoundingClientRect, when it's simpler to just add box model object
capabilities to the SVG render. I replaced the base class of
RenderSVGModelObject with RenderPath (which is a subclass of
RenderBoxModelObject), and everything just starts working.


More information about the webkit-reviews mailing list