[Webkit-unassigned] [Bug 12228] Cleanup assorted CSS classes

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=12228


darin at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #12395|review?                     |review+
               Flag|                            |




------- Comment #2 from darin at apple.com  2007-01-13 07:25 PDT -------
(From update of attachment 12395)
The word "Hand" really shouldn't be capitalized in "shorthand".

Do we really want braces for the for statements that don't need them? I wish
our style guidelines were clear on that.

If anything I'd like to see you applying even more of the m_ prefix. Hyatt was
just talking about how it's hard to work on certain classes because it's so
hard to tell the members apart from local variables.

Fine to land this.

r=me


-- 
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.