FIELDSET element should not check inner controls in checkValidity().

third_party/WebKit/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 444 matching lines @@
 }
 
 void HTMLFormControlElement::setNeedsWillValidateCheck()
 {
     // We need to recalculate willValidate immediately because willValidate change can causes style change.
     bool newWillValidate = recalcWillValidate();
     if (m_willValidateInitialized && m_willValidate == newWillValidate)
         return;
     m_willValidateInitialized = true;
     m_willValidate = newWillValidate;
+    // Needs to force setNeedsValidityCheck() to invalidate validity state of
+    // FORM/FIELDSET. If this element updates willValidate twice and
+    // isValidElement() is not called between them, the second call of this
+    // function still has m_validityIsDirty==true, which means
+    // setNeedsValidityCheck() doesn't invalidate validity state of
+    // FORM/FIELDSET.
+    m_validityIsDirty = false;
     setNeedsValidityCheck();
     // No need to trigger style recalculation here because
     // setNeedsValidityCheck() does it in the right away. This relies on
     // the assumption that valid() is always true if willValidate() is false.
 
     if (!m_willValidate)
         hideVisibleValidationMessage();
 }
 
 void HTMLFormControlElement::findCustomValidationMessageTextDirection(const String& message, TextDirection &messageDir, String& subMessage, TextDirection &subMessageDir)
@@ skipping 49 matching lines @@
 {
     Page* page = document().page();
     if (!page)
         return nullptr;
 
     return &page->validationMessageClient();
 }
 
 bool HTMLFormControlElement::checkValidity(HeapVector<Member<HTMLFormControlElement>>* unhandledInvalidControls, CheckValidityEventBehavior eventBehavior)
 {
+    if (!willValidate())
+        return true;
     if (isValidElement())
         return true;
     if (eventBehavior != CheckValidityDispatchInvalidEvent)
         return false;
     Document* originalDocument = &document();
     DispatchEventResult dispatchResult = dispatchEvent(Event::createCancelable(EventTypeNames::invalid));
     if (dispatchResult == DispatchEventResult::NotCanceled && unhandledInvalidControls && isConnected() && originalDocument == document())
         unhandledInvalidControls->append(this);
     return false;
 }
@@ skipping 109 matching lines @@
         dispatchFormControlChangeEvent();
 }
 
 void HTMLFormControlElement::copyNonAttributePropertiesFromElement(const Element& source)
 {
     HTMLElement::copyNonAttributePropertiesFromElement(source);
     setNeedsValidityCheck();
 }
 
 } // namespace blink