Change WebKit and WebKit2 platform code to use rounded locations and pixel snapped rects and sizes. This avoids having to expose the fractional layout types to the platform code.
Created attachment 134444 [details] Patch
Comment on attachment 134444 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=134444&action=review > Source/WebCore/dom/Node.h:380 > LayoutRect renderRect(bool* isReplaced); > + IntRect pixelSnappedRenderRect(bool* isReplaced) { return pixelSnappedIntRect(renderRect(isReplaced)); } Wouldn't this be easier to call renderRect().pixelSnapped() in the caller instead?
(In reply to comment #2) > (From update of attachment 134444 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=134444&action=review > > > Source/WebCore/dom/Node.h:380 > > LayoutRect renderRect(bool* isReplaced); > > + IntRect pixelSnappedRenderRect(bool* isReplaced) { return pixelSnappedIntRect(renderRect(isReplaced)); } > > Wouldn't this be easier to call renderRect().pixelSnapped() in the caller instead? It would but that would break the encapsulation and expose LayoutRects to the WebKit platform code, something we have been trying to avoid.
Ping?
Comment on attachment 134444 [details] Patch Thank you for the ping.
Comment on attachment 134444 [details] Patch Thanks for the review!
Comment on attachment 134444 [details] Patch Clearing flags on attachment: 134444 Committed r112758: <http://trac.webkit.org/changeset/112758>
All reviewed patches have been landed. Closing bug.