[webkit-reviews] review denied: [Bug 19911] :read-only and :read-write pseudoclasses support : [Attachment 22107] Support for :read-only and :read-write pseudoclasses
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Jul 7 10:02:35 PDT 2008
Adele Peterson <adele at apple.com> has denied Michelangelo De Simone
<m.des at mac.com>'s request for review:
Bug 19911: :read-only and :read-write pseudoclasses support
https://bugs.webkit.org/show_bug.cgi?id=19911
Attachment 22107: Support for :read-only and :read-write pseudoclasses
https://bugs.webkit.org/attachment.cgi?id=22107&action=edit
------- Additional Comments from Adele Peterson <adele at apple.com>
Looks pretty good- I have just a few comments:
atomic strings in CSSSelector.cpp should be readOnly and readWrite
you shouldn't need the check && !e->isInputTypeHidden()
you've left out checks for search fields and isindex - instead of using the
specific checks for the type, you can use isTextField().
More information about the webkit-reviews
mailing list