Chromium Code Reviews| Index: Source/core/html/forms/InputType.cpp |
| diff --git a/Source/core/html/forms/InputType.cpp b/Source/core/html/forms/InputType.cpp |
| index eabbbe9bae142ddff80eb2a1dc308f3b82bef93d..65ba6ea43779d26aa19026c89f694211d05cb3e0 100644 |
| --- a/Source/core/html/forms/InputType.cpp |
| +++ b/Source/core/html/forms/InputType.cpp |
| @@ -985,4 +985,12 @@ void InputType::countUsageIfVisible(UseCounter::Feature feature) const |
| } |
| } |
| +Decimal InputType::findStepBase(const Decimal& defaultValue) const |
|
yosin_UTC9
2013/11/27 01:28:05
The word "find" leads me the function has loop. Ho
|
| +{ |
| + Decimal stepBase = parseToNumber(element().fastGetAttribute(minAttr), Decimal::nan()); |
|
yosin_UTC9
2013/11/27 01:28:05
nit: const Decimal stepBase = ...;
|
| + if (!stepBase.isFinite()) |
| + stepBase = parseToNumber(element().fastGetAttribute(valueAttr), defaultValue); |
|
yosin_UTC9
2013/11/27 01:28:05
nit: rather than copying, just |return parseToNumb
|
| + return stepBase; |
| +} |
| + |
| } // namespace WebCore |