[SVG] SVGAnimatedPointList migration to new SVG property impl.

Source/core/svg/SVGTextContentElement.cpp

 /*
  * Copyright (C) 2004, 2005, 2007, 2008 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Rob Buis <buis@kde.org>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 92 matching lines @@
         exceptionState.throwUninformativeAndGenericDOMException(IndexSizeError);
         return 0.0f;
     }
 
     if (nchars > numberOfChars - charnum)
         nchars = numberOfChars - charnum;
 
     return SVGTextQuery(renderer()).subStringLength(charnum, nchars);
 }
 
-SVGPoint SVGTextContentElement::getStartPositionOfChar(unsigned charnum, ExceptionState& exceptionState)
+PassRefPtr<SVGPointTearOff> SVGTextContentElement::getStartPositionOfChar(unsigned charnum, ExceptionState& exceptionState)
 {
     document().updateLayoutIgnorePendingStylesheets();
 
     if (charnum > getNumberOfChars()) {
         exceptionState.throwUninformativeAndGenericDOMException(IndexSizeError);
-        return FloatPoint();
+        return 0;
     }
 
-    return SVGTextQuery(renderer()).startPositionOfCharacter(charnum);
+    FloatPoint point = SVGTextQuery(renderer()).startPositionOfCharacter(charnum);
+    return SVGPointTearOff::create(SVGPoint::create(point), 0, PropertyIsNotAnimVal);
 }
 
-SVGPoint SVGTextContentElement::getEndPositionOfChar(unsigned charnum, ExceptionState& exceptionState)
+PassRefPtr<SVGPointTearOff> SVGTextContentElement::getEndPositionOfChar(unsigned charnum, ExceptionState& exceptionState)
 {
     document().updateLayoutIgnorePendingStylesheets();
 
     if (charnum > getNumberOfChars()) {
         exceptionState.throwUninformativeAndGenericDOMException(IndexSizeError);
-        return FloatPoint();
+        return 0;
     }
 
-    return SVGTextQuery(renderer()).endPositionOfCharacter(charnum);
+    FloatPoint point = SVGTextQuery(renderer()).endPositionOfCharacter(charnum);
+    return SVGPointTearOff::create(SVGPoint::create(point), 0, PropertyIsNotAnimVal);
 }
 
 PassRefPtr<SVGRectTearOff> SVGTextContentElement::getExtentOfChar(unsigned charnum, ExceptionState& exceptionState)
 {
     document().updateLayoutIgnorePendingStylesheets();
 
     if (charnum > getNumberOfChars()) {
         exceptionState.throwUninformativeAndGenericDOMException(IndexSizeError);
         return 0;
     }
 
     FloatRect rect = SVGTextQuery(renderer()).extentOfCharacter(charnum);
     return SVGRectTearOff::create(SVGRect::create(rect), 0, PropertyIsNotAnimVal);
 }
 
 float SVGTextContentElement::getRotationOfChar(unsigned charnum, ExceptionState& exceptionState)
 {
     document().updateLayoutIgnorePendingStylesheets();
 
     if (charnum > getNumberOfChars()) {
         exceptionState.throwUninformativeAndGenericDOMException(IndexSizeError);
         return 0.0f;
     }
 
     return SVGTextQuery(renderer()).rotationOfCharacter(charnum);
 }
 
-int SVGTextContentElement::getCharNumAtPosition(const SVGPoint& point)
+int SVGTextContentElement::getCharNumAtPosition(PassRefPtr<SVGPointTearOff> point, ExceptionState& exceptionState)
 {
+    // FIXME

[haraken, 2014/01/17 11:50:18]

Add some description.

[kouhei (in TOK), 2014/01/20 01:10:26]

Removed. This was a temporary fix needed before nbarth's strict null checking
CL.

+    if (!point) {
+        exceptionState.throwTypeError("The argument can't be null.");
+        return 0;
+    }
+
     document().updateLayoutIgnorePendingStylesheets();
-    return SVGTextQuery(renderer()).characterNumberAtPosition(point);
+    return SVGTextQuery(renderer()).characterNumberAtPosition(point->target()->value());
 }
 
 void SVGTextContentElement::selectSubString(unsigned charnum, unsigned nchars, ExceptionState& exceptionState)
 {
     unsigned numberOfChars = getNumberOfChars();
     if (charnum >= numberOfChars) {
         exceptionState.throwUninformativeAndGenericDOMException(IndexSizeError);
         return;
     }
 
@@ skipping 101 matching lines @@
     SVGElement* element = toSVGElement(renderer->node());
     ASSERT(element);
 
     if (!element->isTextContent())
         return 0;
 
     return toSVGTextContentElement(element);
 }
 
 }