Chromium Code Reviews| Index: third_party/WebKit/Source/core/svg/SVGLengthTearOff.cpp |
| diff --git a/third_party/WebKit/Source/core/svg/SVGLengthTearOff.cpp b/third_party/WebKit/Source/core/svg/SVGLengthTearOff.cpp |
| index b27da5d242e7e99f911423d8a761740450f71dfa..3aa80812fe39ccec626dc5542c2206f4dd11a447 100644 |
| --- a/third_party/WebKit/Source/core/svg/SVGLengthTearOff.cpp |
| +++ b/third_party/WebKit/Source/core/svg/SVGLengthTearOff.cpp |
| @@ -166,7 +166,9 @@ void SVGLengthTearOff::setValueInSpecifiedUnits( |
| } |
| String SVGLengthTearOff::valueAsString() { |
| - // TODO(shanmuga.m@samsung.com): Not all <length> properties have 0 (with no unit) as the default (lacuna) value, Need to return default value instead of 0 |
| + // TODO(shanmuga.m@samsung.com): Not all <length> properties have 0 (with no |
| + // unit) as the default (lacuna) value, Need to return default value instead |
| + // of 0 |
|
pdr.
2016/10/04 21:05:30
Nit: lets make this a complete sentence: Add a per
Nico
2016/10/04 21:10:23
Done.
|
| return hasExposedLengthUnit() ? target()->valueAsString() : String::number(0); |
| } |