<fieldset> should be ignored in interactive validation

Source/core/html/HTMLFormControlElement.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 Apple Inc. All rights reserved.
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  *
  * 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
@@ skipping 506 matching lines @@
         return false;
     // An event handler can deref this object.
     RefPtrWillBeRawPtr<HTMLFormControlElement> protector(this);
     RefPtrWillBeRawPtr<Document> originalDocument(document());
     bool needsDefaultAction = dispatchEvent(Event::createCancelable(EventTypeNames::invalid));
     if (needsDefaultAction && unhandledInvalidControls && inDocument() && originalDocument == document())
         unhandledInvalidControls->append(this);
     return false;
 }
 
+bool HTMLFormControlElement::isSubmittableElement()
+{
+    if (this->formControlType() == "fieldset")

[tkent, 2015/06/25 11:30:02]

No, HTMLFormControlElement should not have knowledge of specific elements.
isSubmittableElement() should be a virtual function. 
HTMLFormControlElement::isSubmittableElement() should return true, and
HTMLFieldSetElement::isSubmittableElement() should return false.

[tanay.c, 2015/06/25 12:04:56]

Done.

+        return false;
+
+    return true;
+}
+
 void HTMLFormControlElement::showValidationMessage()
 {
     scrollIntoViewIfNeeded(false);
     RefPtrWillBeRawPtr<HTMLFormControlElement> protector(this);
     focus();
     updateVisibleValidationMessage();
 }
 
 bool HTMLFormControlElement::reportValidity()
 {
@@ skipping 104 matching lines @@
         dispatchFormControlChangeEvent();
 }
 
 void HTMLFormControlElement::copyNonAttributePropertiesFromElement(const Element& source)
 {
     HTMLElement::copyNonAttributePropertiesFromElement(source);
     setNeedsValidityCheck();
 }
 
 } // namespace blink