Code cleanup in core/html/

Source/core/html/HTMLButtonElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2010 Apple Inc. All rights reserved.
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2007 Samuel Weinig (sam@webkit.org)
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 46 matching lines @@
 }
 
 LayoutObject* HTMLButtonElement::createLayoutObject(const ComputedStyle&)
 {
     return new LayoutButton(this);
 }
 
 const AtomicString& HTMLButtonElement::formControlType() const
 {
     switch (m_type) {
-        case SUBMIT: {
-            DEFINE_STATIC_LOCAL(const AtomicString, submit, ("submit", AtomicString::ConstructFromLiteral));
-            return submit;
-        }
-        case BUTTON: {
-            DEFINE_STATIC_LOCAL(const AtomicString, button, ("button", AtomicString::ConstructFromLiteral));
-            return button;
-        }
-        case RESET: {
-            DEFINE_STATIC_LOCAL(const AtomicString, reset, ("reset", AtomicString::ConstructFromLiteral));
-            return reset;
-        }
+    case SUBMIT: {
+        DEFINE_STATIC_LOCAL(const AtomicString, submit, ("submit", AtomicString::ConstructFromLiteral));
+        return submit;
+    }
+    case BUTTON: {
+        DEFINE_STATIC_LOCAL(const AtomicString, button, ("button", AtomicString::ConstructFromLiteral));
+        return button;
+    }
+    case RESET: {
+        DEFINE_STATIC_LOCAL(const AtomicString, reset, ("reset", AtomicString::ConstructFromLiteral));
+        return reset;
+    }
     }
 
     ASSERT_NOT_REACHED();
     return emptyAtom;
 }
 
 bool HTMLButtonElement::isPresentationAttribute(const QualifiedName& name) const
 {
     if (name == alignAttr) {
         // Don't map 'align' attribute.  This matches what Firefox and IE do, but not Opera.
         // See http://bugs.webkit.org/show_bug.cgi?id=12071
         return false;
     }
 
     return HTMLFormControlElement::isPresentationAttribute(name);
 }
 
 void HTMLButtonElement::parseAttribute(const QualifiedName& name, const AtomicString& value)
 {
     if (name == typeAttr) {
         if (equalIgnoringCase(value, "reset"))
             m_type = RESET;
         else if (equalIgnoringCase(value, "button"))
             m_type = BUTTON;
         else
             m_type = SUBMIT;
         setNeedsWillValidateCheck();
-    } else
+    } else {
         HTMLFormControlElement::parseAttribute(name, value);
+    }
 }
 
 void HTMLButtonElement::defaultEventHandler(Event* event)
 {
     if (event->type() == EventTypeNames::DOMActivate && !isDisabledFormControl()) {
         if (form() && m_type == SUBMIT) {
             m_isActivatedSubmit = true;
             form()->prepareForSubmission(event);
             event->setDefaultHandled();
             m_isActivatedSubmit = false; // Do this in case submission was canceled.
         }
         if (form() && m_type == RESET) {
             form()->reset();
             event->setDefaultHandled();
         }
     }
 
     if (event->isKeyboardEvent()) {
         if (event->type() == EventTypeNames::keydown && toKeyboardEvent(event)->keyIdentifier() == "U+0020") {
             setActive(true);
             // No setDefaultHandled() - IE dispatches a keypress in this case.
             return;
         }
         if (event->type() == EventTypeNames::keypress) {
             switch (toKeyboardEvent(event)->charCode()) {
-                case '\r':
-                    dispatchSimulatedClick(event);
-                    event->setDefaultHandled();
-                    return;
-                case ' ':
-                    // Prevent scrolling down the page.
-                    event->setDefaultHandled();
-                    return;
+            case '\r':
+                dispatchSimulatedClick(event);
+                event->setDefaultHandled();
+                return;
+            case ' ':
+                // Prevent scrolling down the page.
+                event->setDefaultHandled();
+                return;
             }
         }
         if (event->type() == EventTypeNames::keyup && toKeyboardEvent(event)->keyIdentifier() == "U+0020") {
             if (active())
                 dispatchSimulatedClick(event);
             event->setDefaultHandled();
             return;
         }
     }
 
@@ skipping 88 matching lines @@
             argv.append(formactionAttr.toString());
             argv.append(oldValue);
             argv.append(newValue);
             activityLogger->logEvent("blinkSetAttribute", argv.size(), argv.data());
         }
     }
     HTMLFormControlElement::attributeWillChange(name, oldValue, newValue);
 }
 
 } // namespace