Title: [151717] trunk/Source/WebCore
Revision
151717
Author
[email protected]
Date
2013-06-18 20:31:58 -0700 (Tue, 18 Jun 2013)

Log Message

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:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (151716 => 151717)


--- trunk/Source/WebCore/ChangeLog	2013-06-19 02:02:15 UTC (rev 151716)
+++ trunk/Source/WebCore/ChangeLog	2013-06-19 03:31:58 UTC (rev 151717)
@@ -1,3 +1,17 @@
+2013-06-18  Ryosuke Niwa  <[email protected]>
+
+        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  <[email protected]>
 
         Improve style of null checks in FrameView::setFrameRect

Modified: trunk/Source/WebCore/html/HTMLOptGroupElement.cpp (151716 => 151717)


--- trunk/Source/WebCore/html/HTMLOptGroupElement.cpp	2013-06-19 02:02:15 UTC (rev 151716)
+++ trunk/Source/WebCore/html/HTMLOptGroupElement.cpp	2013-06-19 03:31:58 UTC (rev 151717)
@@ -55,11 +55,6 @@
     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.

Modified: trunk/Source/WebCore/html/HTMLOptGroupElement.h (151716 => 151717)


--- trunk/Source/WebCore/html/HTMLOptGroupElement.h	2013-06-19 02:02:15 UTC (rev 151716)
+++ trunk/Source/WebCore/html/HTMLOptGroupElement.h	2013-06-19 03:31:58 UTC (rev 151717)
@@ -43,7 +43,6 @@
     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; }

Modified: trunk/Source/WebCore/html/HTMLOptionElement.cpp (151716 => 151717)


--- trunk/Source/WebCore/html/HTMLOptionElement.cpp	2013-06-19 02:02:15 UTC (rev 151716)
+++ trunk/Source/WebCore/html/HTMLOptionElement.cpp	2013-06-19 03:31:58 UTC (rev 151717)
@@ -106,11 +106,6 @@
     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.

Modified: trunk/Source/WebCore/html/HTMLOptionElement.h (151716 => 151717)


--- trunk/Source/WebCore/html/HTMLOptionElement.h	2013-06-19 02:02:15 UTC (rev 151716)
+++ trunk/Source/WebCore/html/HTMLOptionElement.h	2013-06-19 03:31:58 UTC (rev 151717)
@@ -69,7 +69,6 @@
 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;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to