[webkit-reviews] review granted: [Bug 6812] Missing focus ring on new text fields : [Attachment 6522] patch w/ Changelog

bugzilla-request-daemon at opendarwin.org bugzilla-request-daemon at opendarwin.org
Thu Feb 16 12:59:05 PST 2006


Darin Adler <darin at apple.com> has granted Adele Peterson <adele at apple.com>'s
request for review:
Bug 6812: Missing focus ring on new text fields
http://bugzilla.opendarwin.org/show_bug.cgi?id=6812

Attachment 6522: patch w/ Changelog
http://bugzilla.opendarwin.org/attachment.cgi?id=6522&action=edit

------- Additional Comments from Darin Adler <darin at apple.com>
Looks fine.

RenderTheme::engineShouldDrawFocusRing could be written as just a return
statement, since if (x) return true; return false; is the same as return x;



More information about the webkit-reviews mailing list