[Webkit-unassigned] [Bug 110431] INPUT_MULTIPLE_FIELDS_UI: Unable to enter "24" to hour field
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Feb 21 23:22:41 PST 2013
https://bugs.webkit.org/show_bug.cgi?id=110431
--- Comment #9 from Kunihiko Sakamoto <ksakamoto at chromium.org> 2013-02-21 23:25:04 PST ---
(From update of attachment 189686)
Thank you for reviewing during your absence on leave.
View in context: https://bugs.webkit.org/attachment.cgi?id=189686&action=review
>> Source/WebCore/html/BaseMultipleFieldsDateAndTimeInputType.cpp:376
>> + const AtomicString pattern = m_dateTimeEditElement->getAttribute(HTMLNames::patternAttr);
>
> getAttribute should be fastGetAttribute because it is not a "style" attribute.
> You can remove HTMLNames::.
Changed to fastGetAttribute.
HTMLNames:: is necessary here - or should I add "using namespace HTMLNames" at the top?
>> Source/WebCore/html/BaseMultipleFieldsDateAndTimeInputType.cpp:377
>> + if (!pattern.isNull() && !pattern.isEmpty())
>
> isNull() check is unnecessary. isEmpty means "null or 0-length"
Done.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list