Remove unused Document::ignoreAutofocus().
authorakling@apple.com <akling@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 22 Nov 2014 10:11:13 +0000 (10:11 +0000)
committerakling@apple.com <akling@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 22 Nov 2014 10:11:13 +0000 (10:11 +0000)
<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):

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

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

index 874fecc..bab0992 100644 (file)
@@ -1,3 +1,20 @@
+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>
 
         URTBF after r176459 to fix GTK build.
index 31c177c..1c9ff5d 100644 (file)
@@ -421,7 +421,6 @@ Document::Document(Frame* frame, const URL& url, unsigned documentClasses, unsig
     , m_wellFormed(false)
     , m_printing(false)
     , m_paginatedForScreen(false)
-    , m_ignoreAutofocus(false)
     , m_compatibilityMode(DocumentCompatibilityMode::NoQuirksMode)
     , m_compatibilityModeLocked(false)
     , m_textColor(Color::black)
index 92ef746..8f3b22c 100644 (file)
@@ -752,11 +752,6 @@ public:
     UserActionElementSet& userActionElements()  { return m_userActionElements; }
     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*);
     void elementInActiveChainDidDetach(Element*);
@@ -1421,8 +1416,6 @@ private:
     bool m_printing;
     bool m_paginatedForScreen;
 
-    bool m_ignoreAutofocus;
-
     DocumentCompatibilityMode m_compatibilityMode;
     bool m_compatibilityModeLocked; // This is cheaper than making setCompatibilityMode virtual.
 
index 22c533c..22c1636 100644 (file)
@@ -181,8 +181,6 @@ static bool shouldAutofocus(HTMLFormControlElement* element)
         return false;
     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.
         element->document().addConsoleMessage(MessageSource::Security, MessageLevel::Error, ASCIILiteral("Blocked autofocusing on a form control because the form's frame is sandboxed and the 'allow-scripts' permission is not set."));