Code cleanup in core/html/

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.
  *
@@ skipping 54 matching lines @@
         else if (equalIgnoringCase(value, "poly"))
             m_shape = Poly;
         else if (equalIgnoringCase(value, "rect"))
             m_shape = Rect;
         invalidateCachedRegion();
     } else if (name == coordsAttr) {
         m_coords = parseHTMLAreaElementCoords(value.string());
         invalidateCachedRegion();
     } else if (name == altAttr || name == accesskeyAttr) {
         // Do nothing.
-    } else
+    } else {
         HTMLAnchorElement::parseAttribute(name, value);
+    }
 }
 
 void HTMLAreaElement::invalidateCachedRegion()
 {
     m_lastSize = LayoutSize(-1, -1);
 }
 
 bool HTMLAreaElement::pointInArea(LayoutPoint location, const LayoutSize& containerSize)
 {
     if (m_lastSize != containerSize) {
@@ skipping 48 matching lines @@
         if (m_coords.size() == 3)
             shape = Circle;
         else if (m_coords.size() == 4)
             shape = Rect;
         else if (m_coords.size() >= 6)
             shape = Poly;
     }
 
     Path path;
     switch (shape) {
-        case Poly:
-            if (m_coords.size() >= 6) {
-                int numPoints = m_coords.size() / 2;
-                path.moveTo(FloatPoint(minimumValueForLength(m_coords[0], width).toFloat(), minimumValueForLength(m_coords[1], height).toFloat()));
-                for (int i = 1; i < numPoints; ++i)
-                    path.addLineTo(FloatPoint(minimumValueForLength(m_coords[i * 2], width).toFloat(), minimumValueForLength(m_coords[i * 2 + 1], height).toFloat()));
-                path.closeSubpath();
-            }
-            break;
-        case Circle:
-            if (m_coords.size() >= 3) {
-                Length radius = m_coords[2];
-                float r = std::min(minimumValueForLength(radius, width).toFloat(), minimumValueForLength(radius, height).toFloat());
-                path.addEllipse(FloatRect(minimumValueForLength(m_coords[0], width).toFloat() - r, minimumValueForLength(m_coords[1], height).toFloat() - r, 2 * r, 2 * r));
-            }
-            break;
-        case Rect:
-            if (m_coords.size() >= 4) {
-                float x0 = minimumValueForLength(m_coords[0], width).toFloat();
-                float y0 = minimumValueForLength(m_coords[1], height).toFloat();
-                float x1 = minimumValueForLength(m_coords[2], width).toFloat();
-                float y1 = minimumValueForLength(m_coords[3], height).toFloat();
-                path.addRect(FloatRect(x0, y0, x1 - x0, y1 - y0));
-            }
-            break;
-        case Default:
-            path.addRect(FloatRect(0, 0, width.toFloat(), height.toFloat()));
-            break;
-        case Unknown:
-            break;
+    case Poly:
+        if (m_coords.size() >= 6) {
+            int numPoints = m_coords.size() / 2;
+            path.moveTo(FloatPoint(minimumValueForLength(m_coords[0], width).toFloat(), minimumValueForLength(m_coords[1], height).toFloat()));
+            for (int i = 1; i < numPoints; ++i)
+                path.addLineTo(FloatPoint(minimumValueForLength(m_coords[i * 2], width).toFloat(), minimumValueForLength(m_coords[i * 2 + 1], height).toFloat()));
+            path.closeSubpath();
+        }
+        break;
+    case Circle:
+        if (m_coords.size() >= 3) {
+            Length radius = m_coords[2];
+            float r = std::min(minimumValueForLength(radius, width).toFloat(), minimumValueForLength(radius, height).toFloat());
+            path.addEllipse(FloatRect(minimumValueForLength(m_coords[0], width).toFloat() - r, minimumValueForLength(m_coords[1], height).toFloat() - r, 2 * r, 2 * r));
+        }
+        break;
+    case Rect:
+        if (m_coords.size() >= 4) {
+            float x0 = minimumValueForLength(m_coords[0], width).toFloat();
+            float y0 = minimumValueForLength(m_coords[1], height).toFloat();
+            float x1 = minimumValueForLength(m_coords[2], width).toFloat();
+            float y1 = minimumValueForLength(m_coords[3], height).toFloat();
+            path.addRect(FloatRect(x0, y0, x1 - x0, y1 - y0));
+        }
+        break;
+    case Default:
+        path.addRect(FloatRect(0, 0, width.toFloat(), height.toFloat()));
+        break;
+    case Unknown:
+        break;
     }
 
     return path;
 }
 
 HTMLImageElement* HTMLAreaElement::imageElement() const
 {
     if (HTMLMapElement* mapElement = Traversal<HTMLMapElement>::firstAncestor(*this))
         return mapElement->imageElement();
     return nullptr;
@@ skipping 42 matching lines @@
         return;
 
     HTMLImageElement* imageElement = this->imageElement();
     if (!imageElement)
         return;
 
     imageElement->updateFocusAppearance(restorePreviousSelection);
 }
 
 }