[Webkit-unassigned] [Bug 11844] Code Cleanup for more of the rendering code
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Sat Dec 16 14:07:16 PST 2006
http://bugs.webkit.org/show_bug.cgi?id=11844
mitz at webkit.org changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #11887|review? |review+
Flag| |
------- Comment #4 from mitz at webkit.org 2006-12-16 14:07 PDT -------
(From update of attachment 11887)
r=me, although I'd prefer if you addressed the following comments before
landing:
Use the class name 'RenderText' instead of 'rendertext':
+ // The text runs point to parts of the rendertext's m_str
I'm fine with you just adding these comments for now, but it's wrong to call
these values 'random'. Use 'arbitrary' instead.
+ // FIXME: we should not use a random value like this. Perhaps we should
use INT_MAX.
+ // FIXME: we should not use a random value like this. Perhaps we should
use INT_MAX.
Here I think it's best to leave 'sizes' since otherwise it's not obvious that
the parameter is pointing to an array:
- void setControlSize(NSCell*, const IntSize* sizes, const IntSize&
minSize);
- void setSizeFromFont(RenderStyle*, const IntSize* sizes) const;
- IntSize sizeForFont(RenderStyle*, const IntSize* sizes) const;
- IntSize sizeForSystemFont(RenderStyle*, const IntSize* sizes) const;
+ void setControlSize(NSCell*, const IntSize*, const IntSize& minSize);
+ void setSizeFromFont(RenderStyle*, const IntSize*) const;
+ IntSize sizeForFont(RenderStyle*, const IntSize*) const;
+ IntSize sizeForSystemFont(RenderStyle*, const IntSize*) const;
--
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
More information about the webkit-unassigned
mailing list