Changeset 176504 in webkit

Timestamp:

Nov 22, 2014, 2:11:13 AM (12 years ago)

Author:

akling@apple.com

Message:

Remove unused Document::ignoreAutofocus().
<​https://webkit.org/b/139002>

Reviewed by Antti Koivisto.

No one ever actually changes this flag, so let's remove it.

• dom/Document.cpp:

(WebCore::Document::Document):

• dom/Document.h:

(WebCore::Document::ignoreAutofocus): Deleted.
(WebCore::Document::setIgnoreAutofocus): Deleted.

• html/HTMLFormControlElement.cpp:

(WebCore::shouldAutofocus):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• dom/Document.cpp (modified) (1 diff)
• dom/Document.h (modified) (2 diffs)
• html/HTMLFormControlElement.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2014-11-22  Andreas Kling  <akling@apple.com>
+
+        Remove unused Document::ignoreAutofocus().
+        <https://webkit.org/b/139002>
+
+        Reviewed by Antti Koivisto.
+
+        No one ever actually changes this flag, so let's remove it.
+
+        * dom/Document.cpp:
+        (WebCore::Document::Document):
+        * dom/Document.h:
+        (WebCore::Document::ignoreAutofocus): Deleted.
+        (WebCore::Document::setIgnoreAutofocus): Deleted.
+        * html/HTMLFormControlElement.cpp:
+        (WebCore::shouldAutofocus):
+
 2014-11-22  Csaba Osztrogonác  <ossy@webkit.org>
 

trunk/Source/WebCore/dom/Document.cpp

@@ -422 +422 @@
     , m_printing(false)
     , m_paginatedForScreen(false)
-    , m_ignoreAutofocus(false)
     , m_compatibilityMode(DocumentCompatibilityMode::NoQuirksMode)
     , m_compatibilityModeLocked(false)

trunk/Source/WebCore/dom/Document.h

@@ -753 +753 @@
     const UserActionElementSet& userActionElements() const { return m_userActionElements; }
 
-    // The m_ignoreAutofocus flag specifies whether or not the document has been changed by the user enough 
-    // for WebCore to ignore the autofocus attribute on any form controls
-    bool ignoreAutofocus() const { return m_ignoreAutofocus; };
-    void setIgnoreAutofocus(bool shouldIgnore = true) { m_ignoreAutofocus = shouldIgnore; };
-
     void removeFocusedNodeOfSubtree(Node*, bool amongChildrenOnly = false);
     void hoveredElementDidDetach(Element*);
@@ -1422 +1417 @@
     bool m_paginatedForScreen;
 
-    bool m_ignoreAutofocus;
-
     DocumentCompatibilityMode m_compatibilityMode;
     bool m_compatibilityModeLocked; // This is cheaper than making setCompatibilityMode virtual.

trunk/Source/WebCore/html/HTMLFormControlElement.cpp

@@ -182 +182 @@
     if (!element->inDocument() || !element->document().renderView())
         return false;
-    if (element->document().ignoreAutofocus())
-        return false;
     if (element->document().isSandboxed(SandboxAutomaticFeatures)) {
         // FIXME: This message should be moved off the console once a solution to https://bugs.webkit.org/show_bug.cgi?id=103274 exists.