SVG doesn't recognize rem units

Source/core/svg/SVGLengthTearOff.cpp

Index: Source/core/svg/SVGLengthTearOff.cpp
diff --git a/Source/core/svg/SVGLengthTearOff.cpp b/Source/core/svg/SVGLengthTearOff.cpp
index e43a606269c486143342e911471eff1a46cbf7b6..baa44f1f7f079b862be56b53477b59147f8f12b6 100644
--- a/Source/core/svg/SVGLengthTearOff.cpp
+++ b/Source/core/svg/SVGLengthTearOff.cpp
@@ -55,7 +55,7 @@ inline bool canResolveRelativeUnits(const SVGElement* contextElement)
 
 SVGLengthType SVGLengthTearOff::unitType()
 {
-    return target()->unitType();
+    return hasExposedLengthUnit() ? target()->unitType() : LengthTypeUnknown;
 }
 
 SVGLengthMode SVGLengthTearOff::unitMode()
@@ -108,7 +108,8 @@ void SVGLengthTearOff::setValueInSpecifiedUnits(float value, ExceptionState& es)
 
 String SVGLengthTearOff::valueAsString()
 {
-    return target()->valueAsString();
+    // TODO: Not all <length> properties have 0 (with no unit) as the default (lacuna) value, Need to return default value instead of 0

[Erik Dahlström (inactive), 2015/04/02 07:58:41]

Nit: should be TODO(name), see
http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#TODO_Comments.

[Shanmuga Pandi, 2015/04/02 13:06:51]

Done.

+    return hasExposedLengthUnit() ? target()->valueAsString() : String::number(0);
 }
 
 void SVGLengthTearOff::setValueAsString(const String& str, ExceptionState& es)
@@ -118,7 +119,16 @@ void SVGLengthTearOff::setValueAsString(const String& str, ExceptionState& es)
         return;
     }
 
+    String oldValue = target()->valueAsString();
+
     target()->setValueAsString(str, es);
+
+    if (!es.hadException() && !hasExposedLengthUnit()) {
+        target()->setValueAsString(oldValue, ASSERT_NO_EXCEPTION); // rollback t o old value

[Erik Dahlström (inactive), 2015/04/02 07:58:41]

Nit: s/t o/to/

[Shanmuga Pandi, 2015/04/02 13:06:51]

Done.

+        es.throwDOMException(SyntaxError, "The value provided ('" + str + "') is invalid.");
+        return;
+    }
+
     commitChange();
 }