[webkit-reviews] review requested: [Bug 13478] RenderObject::absoluteBoundingBoxRect() should ignore rects outside the bounds of the document : [Attachment 14310] Patch v4

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed May 2 10:06:37 PDT 2007


David Kilzer (ddkilzer) <ddkilzer at webkit.org> has asked  for review:
Bug 13478: RenderObject::absoluteBoundingBoxRect() should ignore rects outside
the bounds of the document
http://bugs.webkit.org/show_bug.cgi?id=13478

Attachment 14310: Patch v4
http://bugs.webkit.org/attachment.cgi?id=14310&action=edit

------- Additional Comments from David Kilzer (ddkilzer) <ddkilzer at webkit.org>
Addressing issues in Comment #10:
- Updated all absoluteRects() methods to clip rects added to document bounds.
- Added Document::bounds() method to return a rect with coordinates (0,0) and
the height/width set to the contents.
- Added RenderObject::addRectClippedToBounds() method to avoid code
duplication.

Questions:
- Is Document the correct place to put the bounds() method?  I thought about
putting it on the FrameView instead, but wanted to avoid calling
document()->view()->bounds() instead of just document()->bounds().
- Is bounds() a good name for Document::bounds()?  Others:  boundsRect(),
contentsRect(), contents()?
- Is addRectClippedToBounds() a good name?  Should it be more generic?	Is
there other code that could be using this method?
- Does the way WebCoreAXObject.mm uses absoluteRects() cause this patch to
prevent users with special needs from accessing elements on the page, or does
it only deal with the visible (rendered) part of the page and we're only
clipping non-visible content anyway?



More information about the webkit-reviews mailing list