Summary: | Change roundedIntRect to roundedRect | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Levi Weintraub <leviw> | ||||||||
Component: | Layout and Rendering | Assignee: | Levi Weintraub <leviw> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | darin, eae, eric, gustavo, jamesr, simon.fraser, webkit.review.bot, xan.lopez | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 63567 | ||||||||||
Attachments: |
|
Description
Levi Weintraub
2011-07-11 16:46:02 PDT
Created attachment 100390 [details]
Patch
Comment on attachment 100390 [details]
Patch
Seems OK. Could also be RoundedLayoutRect.
Actually I think when used in GraphicsContext methods, it should always be a float-based. Comment on attachment 100390 [details] Patch Attachment 100390 [details] did not pass efl-ews (efl): Output: http://queues.webkit.org/results/9009779 Comment on attachment 100390 [details] Patch Attachment 100390 [details] did not pass gtk-ews (gtk): Output: http://queues.webkit.org/results/9012651 Created attachment 100506 [details]
Patch for landing
(In reply to comment #3) > Actually I think when used in GraphicsContext methods, it should always be a float-based. I think you're right. We plan on auditing all the rounding as part of our process. It'll be one of the most fun parts :) Comment on attachment 100506 [details]
Patch for landing
Just realized I missed one more file. Fixing and re-uploading.
Created attachment 100525 [details]
Patch for landing
Comment on attachment 100525 [details] Patch for landing Clearing flags on attachment: 100525 Committed r90832: <http://trac.webkit.org/changeset/90832> All reviewed patches have been landed. Closing bug. This is a layer violation. Layout units is a higher level concept in the rendering engine, not one that should exist at the platform level. If we can move to floating point for the rounded rectangle arguments for graphics context arguments, then we should do so. It seems like a very bad idea to have the graphics context layer be built on top of the layout units concept. Agree with Darin |