 Chromium Code Reviews
 Chromium Code Reviews Issue 460343004:
  Fix update of validity cache value, so that it reflects the correct state of any FormControlElement.  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/blink.git@master
    
  
    Issue 460343004:
  Fix update of validity cache value, so that it reflects the correct state of any FormControlElement.  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/blink.git@master| OLD | NEW | 
|---|---|
| 1 /* | 1 /* | 
| 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) | 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) | 
| 3 * (C) 1999 Antti Koivisto (koivisto@kde.org) | 3 * (C) 1999 Antti Koivisto (koivisto@kde.org) | 
| 4 * (C) 2000 Dirk Mueller (mueller@kde.org) | 4 * (C) 2000 Dirk Mueller (mueller@kde.org) | 
| 5 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved. | 5 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved. | 
| 6 * | 6 * | 
| 7 * This library is free software; you can redistribute it and/or | 7 * This library is free software; you can redistribute it and/or | 
| 8 * modify it under the terms of the GNU Library General Public | 8 * modify it under the terms of the GNU Library General Public | 
| 9 * License as published by the Free Software Foundation; either | 9 * License as published by the Free Software Foundation; either | 
| 10 * version 2 of the License, or (at your option) any later version. | 10 * version 2 of the License, or (at your option) any later version. | 
| (...skipping 163 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 174 | 174 | 
| 175 // The initial value of m_willValidate depends on the derived class. We can' t | 175 // The initial value of m_willValidate depends on the derived class. We can' t | 
| 176 // initialize it with a virtual function in the constructor. m_willValidate | 176 // initialize it with a virtual function in the constructor. m_willValidate | 
| 177 // is not deterministic as long as m_willValidateInitialized is false. | 177 // is not deterministic as long as m_willValidateInitialized is false. | 
| 178 mutable bool m_willValidateInitialized: 1; | 178 mutable bool m_willValidateInitialized: 1; | 
| 179 mutable bool m_willValidate : 1; | 179 mutable bool m_willValidate : 1; | 
| 180 | 180 | 
| 181 // Cache of valid(). | 181 // Cache of valid(). | 
| 182 // But "candidate for constraint validation" doesn't affect m_isValid. | 182 // But "candidate for constraint validation" doesn't affect m_isValid. | 
| 183 bool m_isValid : 1; | 183 bool m_isValid : 1; | 
| 184 bool m_validIsDirty : 1; | |
| 
tkent
2014/09/03 02:16:35
The name is weird.
'valid' is an adjective.  It sh
 
spartha
2014/09/03 07:58:40
Done.
 | |
| 184 | 185 | 
| 185 bool m_wasChangedSinceLastFormControlChangeEvent : 1; | 186 bool m_wasChangedSinceLastFormControlChangeEvent : 1; | 
| 186 bool m_wasFocusedByMouse : 1; | 187 bool m_wasFocusedByMouse : 1; | 
| 187 }; | 188 }; | 
| 188 | 189 | 
| 189 inline bool isHTMLFormControlElement(const Element& element) | 190 inline bool isHTMLFormControlElement(const Element& element) | 
| 190 { | 191 { | 
| 191 return element.isFormControlElement(); | 192 return element.isFormControlElement(); | 
| 192 } | 193 } | 
| 193 | 194 | 
| 194 DEFINE_HTMLELEMENT_TYPE_CASTS_WITH_FUNCTION(HTMLFormControlElement); | 195 DEFINE_HTMLELEMENT_TYPE_CASTS_WITH_FUNCTION(HTMLFormControlElement); | 
| 195 DEFINE_TYPE_CASTS(HTMLFormControlElement, FormAssociatedElement, control, contro l->isFormControlElement(), control.isFormControlElement()); | 196 DEFINE_TYPE_CASTS(HTMLFormControlElement, FormAssociatedElement, control, contro l->isFormControlElement(), control.isFormControlElement()); | 
| 196 | 197 | 
| 197 } // namespace | 198 } // namespace | 
| 198 | 199 | 
| 199 #endif | 200 #endif | 
| OLD | NEW |