Refactoring: Remove HTMLFormControlElement::autofocus
authortkent@chromium.org <tkent@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 10 Dec 2012 11:34:26 +0000 (11:34 +0000)
committertkent@chromium.org <tkent@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 10 Dec 2012 11:34:26 +0000 (11:34 +0000)
https://bugs.webkit.org/show_bug.cgi?id=104524

Reviewed by Kentaro Hara.

autofocus function is used only by HTMLFormControlElement itself, no
sub-classes override it, and it is not exposed via an IDL attribute,

No new tests. This doesn't make any behavior changes.

* html/HTMLFormControlElement.cpp:
(WebCore::shouldAutofocus): Use fastHasAttribute instead of autofocus().
* html/HTMLFormControlElement.h: Remove autofocus().

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@137131 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/html/HTMLFormControlElement.cpp
Source/WebCore/html/HTMLFormControlElement.h

index fd24908..04ca652 100644 (file)
@@ -1,3 +1,19 @@
+2012-12-10  Kent Tamura  <tkent@chromium.org>
+
+        Refactoring: Remove HTMLFormControlElement::autofocus
+        https://bugs.webkit.org/show_bug.cgi?id=104524
+
+        Reviewed by Kentaro Hara.
+
+        autofocus function is used only by HTMLFormControlElement itself, no
+        sub-classes override it, and it is not exposed via an IDL attribute,
+
+        No new tests. This doesn't make any behavior changes.
+
+        * html/HTMLFormControlElement.cpp:
+        (WebCore::shouldAutofocus): Use fastHasAttribute instead of autofocus().
+        * html/HTMLFormControlElement.h: Remove autofocus().
+
 2012-12-10  Jonathan Liu  <net147@gmail.com>
 
         Fix GC3Dintptr and GC3Dsizeiptr typedefs for Win64
 2012-12-10  Jonathan Liu  <net147@gmail.com>
 
         Fix GC3Dintptr and GC3Dsizeiptr typedefs for Win64
index 6a4eeb4..73383e0 100644 (file)
@@ -166,7 +166,7 @@ void HTMLFormControlElement::requiredAttributeChanged()
 
 static bool shouldAutofocus(HTMLFormControlElement* element)
 {
 
 static bool shouldAutofocus(HTMLFormControlElement* element)
 {
-    if (!element->autofocus())
+    if (!element->fastHasAttribute(autofocusAttr))
         return false;
     if (!element->renderer())
         return false;
         return false;
     if (!element->renderer())
         return false;
@@ -278,11 +278,6 @@ bool HTMLFormControlElement::disabled() const
     return m_ancestorDisabledState == AncestorDisabledStateDisabled;
 }
 
     return m_ancestorDisabledState == AncestorDisabledStateDisabled;
 }
 
-bool HTMLFormControlElement::autofocus() const
-{
-    return hasAttribute(autofocusAttr);
-}
-
 bool HTMLFormControlElement::isRequired() const
 {
     return m_isRequired;
 bool HTMLFormControlElement::isRequired() const
 {
     return m_isRequired;
index f1fccd3..44777c0 100644 (file)
@@ -71,9 +71,6 @@ public:
     virtual bool isFocusable() const;
     virtual bool isEnumeratable() const { return false; }
 
     virtual bool isFocusable() const;
     virtual bool isEnumeratable() const { return false; }
 
-    // Determines whether or not a control will be automatically focused.
-    virtual bool autofocus() const;
-
     bool isRequired() const;
 
     const AtomicString& type() const { return formControlType(); }
     bool isRequired() const;
 
     const AtomicString& type() const { return formControlType(); }