[Webkit-unassigned] [Bug 48110] getBoundingClientRect: Do not truncate the coordinates to integers

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=48110


Cosmin Truta <ctruta at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #74442|0                           |1
        is obsolete|                            |
  Attachment #74599|                            |review?, commit-queue?
               Flag|                            |




--- Comment #25 from Cosmin Truta <ctruta at chromium.org>  2010-11-22 14:37:43 PST ---
Created an attachment (id=74599)
 --> (https://bugs.webkit.org/attachment.cgi?id=74599&action=review)
Fix and layout tests

(In reply to comment #24)
Ok, Niko,
I did everything that you asked, except for the following:

> > LayoutTests/svg/zoom/page/zoom-zoom-coords.xhtml:22
> > +</svg><svg id="svg2" xmlns="http://www.w3.org/2000/svg"
> 
> Please add a newline between </svg> and <svg>.
> 
> > LayoutTests/svg/zoom/page/zoom-zoom-coords.xhtml:28
> > +</svg><svg id="svg3" xmlns="http://www.w3.org/2000/svg"
> 
> Ditto.

If I insert whitespace between </svg><svg>, the renderer will insert a small space between the two SVG images, wreacking havoc in all those well-behaved, easy-to-follow (multiples of 50, etc.) coordinates that I carefully crafted. I'd rather not have that little space, and the small cost of having two contiguous </svg><svg> tags inside the test case is IMO significantly outweighed by its benefit.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.