Remove two superfluous overrides of supportsFocus
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 19 Jun 2013 03:31:58 +0000 (03:31 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 19 Jun 2013 03:31:58 +0000 (03:31 +0000)
https://bugs.webkit.org/show_bug.cgi?id=117770

Reviewed by Benjamin Poulain.

Merge https://chromium.googlesource.com/chromium/blink/+/4228964a63e3a889a40efce0d01866629fdcd9b9.

* html/HTMLOptGroupElement.cpp:
* html/HTMLOptGroupElement.h:
* html/HTMLOptionElement.cpp:
* html/HTMLOptionElement.h:

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

Source/WebCore/ChangeLog
Source/WebCore/html/HTMLOptGroupElement.cpp
Source/WebCore/html/HTMLOptGroupElement.h
Source/WebCore/html/HTMLOptionElement.cpp
Source/WebCore/html/HTMLOptionElement.h

index b5ff69b077e430a35a2a99b21b97f91c3ff32ae1..88d8fe088ede334b3fb1f4025a8f5a438d3f45f2 100644 (file)
@@ -1,3 +1,17 @@
+2013-06-18  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Remove two superfluous overrides of supportsFocus
+        https://bugs.webkit.org/show_bug.cgi?id=117770
+
+        Reviewed by Benjamin Poulain.
+
+        Merge https://chromium.googlesource.com/chromium/blink/+/4228964a63e3a889a40efce0d01866629fdcd9b9.
+
+        * html/HTMLOptGroupElement.cpp:
+        * html/HTMLOptGroupElement.h:
+        * html/HTMLOptionElement.cpp:
+        * html/HTMLOptionElement.h:
+
 2013-06-18  Darin Adler  <darin@apple.com>
 
         Improve style of null checks in FrameView::setFrameRect
index 0d0d93380350bab721da271205de9230f45ca36e..07989ce517c6eee43e3b24118251c7a0829acd1a 100644 (file)
@@ -55,11 +55,6 @@ bool HTMLOptGroupElement::isDisabledFormControl() const
     return fastHasAttribute(disabledAttr);
 }
 
-bool HTMLOptGroupElement::supportsFocus() const
-{
-    return HTMLElement::supportsFocus();
-}
-
 bool HTMLOptGroupElement::isFocusable() const
 {
     // Optgroup elements do not have a renderer so we check the renderStyle instead.
index 9fcfdcb1c7709aae3c82da74ddea2d1c52789daf..242b8c100b95b35f9739644369806d2e81d9f8b9 100644 (file)
@@ -43,7 +43,6 @@ private:
     HTMLOptGroupElement(const QualifiedName&, Document*);
 
     virtual const AtomicString& formControlType() const;
-    virtual bool supportsFocus() const OVERRIDE;
     virtual bool isFocusable() const OVERRIDE;
     virtual void parseAttribute(const QualifiedName&, const AtomicString&) OVERRIDE;
     virtual bool rendererIsNeeded(const NodeRenderingContext&) { return false; }
index 59a6225df82ba912b843421f92f1e5fbac7ea301..66ff999ca9302753a1fe0861ab43fb09daa62586 100644 (file)
@@ -106,11 +106,6 @@ void HTMLOptionElement::detach(const AttachContext& context)
     HTMLElement::detach(context);
 }
 
-bool HTMLOptionElement::supportsFocus() const
-{
-    return HTMLElement::supportsFocus();
-}
-
 bool HTMLOptionElement::isFocusable() const
 {
     // Option elements do not have a renderer so we check the renderStyle instead.
index 8c6675348b1721e86893c9718cc94dd00f47aab1..5df52f192e3f4c28ba3e632982a51bd7981df04b 100644 (file)
@@ -69,7 +69,6 @@ public:
 private:
     HTMLOptionElement(const QualifiedName&, Document*);
 
-    virtual bool supportsFocus() const OVERRIDE;
     virtual bool isFocusable() const OVERRIDE;
     virtual bool rendererIsNeeded(const NodeRenderingContext&) { return false; }
     virtual void attach(const AttachContext& = AttachContext()) OVERRIDE;