Summary: | Move logical dimension getters/setters to FloatingObject from RenderBlock | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Bem Jones-Bey <bjonesbe> | ||||
Component: | New Bugs | Assignee: | Bem Jones-Bey <bjonesbe> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, esprehn+autocc, glenn, hyatt, kondapallykalyan | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Bem Jones-Bey
2013-09-04 09:42:36 PDT
Created attachment 210472 [details]
Patch
Comment on attachment 210472 [details]
Patch
I would like to see you make the horizontal writing mode of the block that owns the floating object be a bit on the FloatingObject. Then you have that information available, and you don't need the boolean arguments to everything.
This doesn't have to hold up the patch though, but I think it would be a good follow-up. Comment on attachment 210472 [details]
Patch
r=me
Comment on attachment 210472 [details] Patch Clearing flags on attachment: 210472 Committed r155050: <http://trac.webkit.org/changeset/155050> All reviewed patches have been landed. Closing bug. |