Implement specced parsing algorithm for <area coords>

third_party/WebKit/Source/core/html/HTMLAreaElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2009, 2011 Apple Inc. All rights reserved.
  *
  * 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,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Library General Public License for more details.
  *
  * You should have received a copy of the GNU Library General Public License
  * along with this library; see the file COPYING.LIB.  If not, write to
  * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
  * Boston, MA 02110-1301, USA.
  */
 
 #include "core/html/HTMLAreaElement.h"
 
 #include "core/HTMLNames.h"
 #include "core/dom/ElementTraversal.h"
 #include "core/html/HTMLImageElement.h"
 #include "core/html/HTMLMapElement.h"
+#include "core/html/parser/HTMLParserIdioms.h"
 #include "core/layout/HitTestResult.h"
 #include "core/layout/LayoutImage.h"
 #include "core/layout/LayoutView.h"
 #include "platform/graphics/Path.h"
 #include "platform/transforms/AffineTransform.h"
 
 namespace blink {
 
 namespace {
 
-// Adapt a Length to the allowed range of a LayoutUnit.
-float clampCoordinate(const Length& length)
+// Adapt a double to the allowed range of a LayoutUnit and narrow it to float precision.
+float clampCoordinate(double value)
 {
-    ASSERT(length.isFixed());
-    return LayoutUnit(length.value()).toFloat();
+    return LayoutUnit(value).toFloat();
 }
 
 }
 
 using namespace HTMLNames;
 
 inline HTMLAreaElement::HTMLAreaElement(Document& document)
     : HTMLAnchorElement(areaTag, document)
     , m_lastSize(-1, -1)
     , m_shape(Rect)
@@ skipping 20 matching lines @@
             m_shape = Circle;
         } else if (equalIgnoringASCIICase(value, "polygon") || equalIgnoringASCIICase(value, "poly")) {
             m_shape = Poly;
         } else {
             // The missing (and implicitly invalid) value default for the
             // 'shape' attribute is 'rect'.
             m_shape = Rect;
         }
         invalidateCachedRegion();
     } else if (name == coordsAttr) {
-        m_coords = parseHTMLAreaElementCoords(value.string());
+        m_coords = parseHTMLListOfFloatingPointNumbers(value.string());
         invalidateCachedRegion();
     } else if (name == altAttr || name == accesskeyAttr) {
         // Do nothing.
     } else {
         HTMLAnchorElement::parseAttribute(name, oldValue, value);
     }
 }
 
 void HTMLAreaElement::invalidateCachedRegion()
 {
@@ skipping 51 matching lines @@
         if (m_coords.size() >= 6) {
             int numPoints = m_coords.size() / 2;
             path.moveTo(FloatPoint(clampCoordinate(m_coords[0]), clampCoordinate(m_coords[1])));
             for (int i = 1; i < numPoints; ++i)
                 path.addLineTo(FloatPoint(clampCoordinate(m_coords[i * 2]), clampCoordinate(m_coords[i * 2 + 1])));
             path.closeSubpath();
             path.setWindRule(RULE_EVENODD);
         }
         break;
     case Circle:
-        if (m_coords.size() >= 3 && m_coords[2].value() > 0) {
+        if (m_coords.size() >= 3 && m_coords[2] > 0) {
             float r = clampCoordinate(m_coords[2]);
             path.addEllipse(FloatRect(clampCoordinate(m_coords[0]) - r, clampCoordinate(m_coords[1]) - r, 2 * r, 2 * r));
         }
         break;
     case Rect:
         if (m_coords.size() >= 4) {
             float x0 = clampCoordinate(m_coords[0]);
             float y0 = clampCoordinate(m_coords[1]);
             float x1 = clampCoordinate(m_coords[2]);
             float y1 = clampCoordinate(m_coords[3]);
@@ skipping 55 matching lines @@
 void HTMLAreaElement::updateFocusAppearance(SelectionBehaviorOnFocus selectionBehavior)
 {
     if (!isFocusable())
         return;
 
     if (HTMLImageElement* imageElement = this->imageElement())
         imageElement->updateFocusAppearance(selectionBehavior);
 }
 
 } // namespace blink