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