[webkit-reviews] review denied: [Bug 23588] Add inputmode attribute support, as per XHTML Basic 1.1 : [Attachment 58455] Resubmission after addressing reviewer's comments
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Sun Jun 20 23:12:12 PDT 2010
Kent Tamura <tkent at chromium.org> has denied Charles Wei
<charles.wei at torchmobile.com.cn>'s request for review:
Bug 23588: Add inputmode attribute support, as per XHTML Basic 1.1
https://bugs.webkit.org/show_bug.cgi?id=23588
Attachment 58455: Resubmission after addressing reviewer's comments
https://bugs.webkit.org/attachment.cgi?id=58455&action=review
------- Additional Comments from Kent Tamura <tkent at chromium.org>
You need to change more files to introduce a feature flag. e.g.
*/FeatureDefines.xcconfig, features.gypi. Bug#40878 has a good example.
WebCore/html/HTMLInputElement.idl:103
+ attribute [ConvertNullToNullString, Reflect] DOMString
inputmode;
You don't need ConvertNullToNullString anymore. Reflect implies it.
WebCore/html/HTMLTextAreaElement.idl:56
+ attribute [ConvertNullToNullString, Reflect] DOMString
inputmode;
ditto.
WebKitTools/ChangeLog:5
+ Add feature "XHTML Basic 1.1 support with ENABLE_XHTMLBASIC
Unclosed "
More information about the webkit-reviews
mailing list