Remove HTMLFormControlElement::m_disabled, m_isReadOnly, and m_isRequired.

third_party/WebKit/Source/core/html/HTMLFormControlElement.h

Index: third_party/WebKit/Source/core/html/HTMLFormControlElement.h
diff --git a/third_party/WebKit/Source/core/html/HTMLFormControlElement.h b/third_party/WebKit/Source/core/html/HTMLFormControlElement.h
index 1809c8af089f55ae50906f6cfec6ce4af79e790a..c9934aec2db8ea6517801210d566b1ddfd46e6b2 100644
--- a/third_party/WebKit/Source/core/html/HTMLFormControlElement.h
+++ b/third_party/WebKit/Source/core/html/HTMLFormControlElement.h
@@ -100,8 +100,8 @@ public:
     void setCustomValidity(const String&) final;
     void findCustomValidationMessageTextDirection(const String& message, TextDirection &messageDir, String& subMessage, TextDirection& subMessageDir);
 
-    bool isReadOnly() const { return m_isReadOnly; }
-    bool isDisabledOrReadOnly() const { return isDisabledFormControl() || m_isReadOnly; }
+    bool isReadOnly() const { return fastHasAttribute(HTMLNames::readonlyAttr); }

[kochi, 2015/12/03 03:38:59]

isRequired() is just { return fastHasAttribute(...); } but defined out-of-line.
If not performance critical, how about making them consistent (both inline, or
both out-of-line)?

[tkent, 2015/12/03 04:05:29]

Ok, I took out-of-line.

+    bool isDisabledOrReadOnly() const { return isDisabledFormControl() || isReadOnly(); }
 
     bool isAutofocusable() const;
 
@@ -174,10 +174,7 @@ private:
     // Requests validity recalc for all ancestor fieldsets, if exist.
     void fieldSetAncestorsSetNeedsValidityCheck(Node*);
 
-    bool m_disabled : 1;
     bool m_isAutofilled : 1;

[kochi, 2015/12/03 03:38:59]

Can these 2 bitfields move to below so that it could be grouped as
one integer (skipping 2 enum members below) to get the class smaller?

[tkent, 2015/12/03 04:05:29]

Good suggestion.  Done.

-    bool m_isReadOnly : 1;
-    bool m_isRequired : 1;
     bool m_hasValidationMessage : 1;
 
     enum AncestorDisabledState { AncestorDisabledStateUnknown, AncestorDisabledStateEnabled, AncestorDisabledStateDisabled };