- Revision
- 124174
- Author
- [email protected]
- Date
- 2012-07-30 22:46:43 -0700 (Mon, 30 Jul 2012)
Log Message
Unreviewed, rolling out r124171.
http://trac.webkit.org/changeset/124171
https://bugs.webkit.org/show_bug.cgi?id=92712
Start failing fast/css/square-button-appearance.html
(Requested by yosin on #webkit).
Patch by Sheriff Bot <[email protected]> on 2012-07-30
* css/SelectorChecker.cpp:
(WebCore::SelectorChecker::checkOneSelector):
* dom/Element.cpp:
* dom/Element.h:
(Element):
(WebCore::Element::isReadOnlyFormControl):
* html/HTMLFormControlElement.h:
(WebCore::HTMLFormControlElement::isReadOnlyFormControl):
* html/shadow/SliderThumbElement.cpp:
(WebCore::SliderThumbElement::isReadOnlyFormControl):
(WebCore::SliderThumbElement::willRespondToMouseMoveEvents):
(WebCore::SliderThumbElement::willRespondToMouseClickEvents):
* html/shadow/SliderThumbElement.h:
(SliderThumbElement):
* html/shadow/TextControlInnerElements.cpp:
(WebCore::SearchFieldCancelButtonElement::willRespondToMouseClickEvents):
(WebCore::SpinButtonElement::willRespondToMouseMoveEvents):
(WebCore::SpinButtonElement::willRespondToMouseClickEvents):
(WebCore::InputFieldSpeechButtonElement::willRespondToMouseClickEvents):
* html/shadow/TextControlInnerElements.h:
(WebCore::SpinButtonElement::isReadOnlyFormControl):
* rendering/RenderTheme.cpp:
(WebCore::RenderTheme::isReadOnlyControl):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (124173 => 124174)
--- trunk/Source/WebCore/ChangeLog 2012-07-31 05:28:37 UTC (rev 124173)
+++ trunk/Source/WebCore/ChangeLog 2012-07-31 05:46:43 UTC (rev 124174)
@@ -1,3 +1,36 @@
+2012-07-30 Sheriff Bot <[email protected]>
+
+ Unreviewed, rolling out r124171.
+ http://trac.webkit.org/changeset/124171
+ https://bugs.webkit.org/show_bug.cgi?id=92712
+
+ Start failing fast/css/square-button-appearance.html
+ (Requested by yosin on #webkit).
+
+ * css/SelectorChecker.cpp:
+ (WebCore::SelectorChecker::checkOneSelector):
+ * dom/Element.cpp:
+ * dom/Element.h:
+ (Element):
+ (WebCore::Element::isReadOnlyFormControl):
+ * html/HTMLFormControlElement.h:
+ (WebCore::HTMLFormControlElement::isReadOnlyFormControl):
+ * html/shadow/SliderThumbElement.cpp:
+ (WebCore::SliderThumbElement::isReadOnlyFormControl):
+ (WebCore::SliderThumbElement::willRespondToMouseMoveEvents):
+ (WebCore::SliderThumbElement::willRespondToMouseClickEvents):
+ * html/shadow/SliderThumbElement.h:
+ (SliderThumbElement):
+ * html/shadow/TextControlInnerElements.cpp:
+ (WebCore::SearchFieldCancelButtonElement::willRespondToMouseClickEvents):
+ (WebCore::SpinButtonElement::willRespondToMouseMoveEvents):
+ (WebCore::SpinButtonElement::willRespondToMouseClickEvents):
+ (WebCore::InputFieldSpeechButtonElement::willRespondToMouseClickEvents):
+ * html/shadow/TextControlInnerElements.h:
+ (WebCore::SpinButtonElement::isReadOnlyFormControl):
+ * rendering/RenderTheme.cpp:
+ (WebCore::RenderTheme::isReadOnlyControl):
+
2012-07-30 Yoshifumi Inoue <[email protected]>
[CSS] Change Element::isReadOnlyFormControl to shouldMatchReadWriteSelector or HTMLTextFormControlElement::readOnly
Modified: trunk/Source/WebCore/css/SelectorChecker.cpp (124173 => 124174)
--- trunk/Source/WebCore/css/SelectorChecker.cpp 2012-07-31 05:28:37 UTC (rev 124173)
+++ trunk/Source/WebCore/css/SelectorChecker.cpp 2012-07-31 05:46:43 UTC (rev 124174)
@@ -1066,11 +1066,11 @@
case CSSSelector::PseudoReadOnly:
if (!element || !element->isFormControlElement())
return false;
- return element->isTextFormControl() && !element->shouldMatchReadWriteSelector();
+ return element->isTextFormControl() && element->isReadOnlyFormControl();
case CSSSelector::PseudoReadWrite:
if (!element || !element->isFormControlElement())
return false;
- return element->isTextFormControl() && element->shouldMatchReadWriteSelector();
+ return element->isTextFormControl() && !element->isReadOnlyFormControl();
case CSSSelector::PseudoOptional:
return element && element->isOptionalFormControl();
case CSSSelector::PseudoRequired:
Modified: trunk/Source/WebCore/dom/Element.cpp (124173 => 124174)
--- trunk/Source/WebCore/dom/Element.cpp 2012-07-31 05:28:37 UTC (rev 124173)
+++ trunk/Source/WebCore/dom/Element.cpp 2012-07-31 05:46:43 UTC (rev 124174)
@@ -1783,12 +1783,6 @@
return count;
}
-
-bool Element::shouldMatchReadWriteSelector() const
-{
- return false;
-}
-
bool Element::webkitMatchesSelector(const String& selector, ExceptionCode& ec)
{
if (selector.isEmpty()) {
Modified: trunk/Source/WebCore/dom/Element.h (124173 => 124174)
--- trunk/Source/WebCore/dom/Element.h 2012-07-31 05:28:37 UTC (rev 124173)
+++ trunk/Source/WebCore/dom/Element.h 2012-07-31 05:46:43 UTC (rev 124174)
@@ -350,7 +350,6 @@
Element* nextElementSibling() const;
unsigned childElementCount() const;
- virtual bool shouldMatchReadWriteSelector() const;
bool webkitMatchesSelector(const String& selectors, ExceptionCode&);
DOMTokenList* classList();
@@ -374,6 +373,7 @@
virtual bool isFormControlElement() const { return false; }
virtual bool isEnabledFormControl() const { return true; }
+ virtual bool isReadOnlyFormControl() const { return false; }
virtual bool isSpinButtonElement() const { return false; }
virtual bool isTextFormControl() const { return false; }
virtual bool isOptionalFormControl() const { return false; }
Modified: trunk/Source/WebCore/html/HTMLFormControlElement.h (124173 => 124174)
--- trunk/Source/WebCore/html/HTMLFormControlElement.h 2012-07-31 05:28:37 UTC (rev 124173)
+++ trunk/Source/WebCore/html/HTMLFormControlElement.h 2012-07-31 05:46:43 UTC (rev 124174)
@@ -81,7 +81,7 @@
virtual const AtomicString& formControlType() const OVERRIDE = 0;
virtual bool isEnabledFormControl() const { return !disabled(); }
- virtual bool shouldMatchReadWriteSelector() const OVERRIDE { return !readOnly(); }
+ virtual bool isReadOnlyFormControl() const { return readOnly(); }
virtual bool canTriggerImplicitSubmission() const { return false; }
Modified: trunk/Source/WebCore/html/shadow/SliderThumbElement.cpp (124173 => 124174)
--- trunk/Source/WebCore/html/shadow/SliderThumbElement.cpp 2012-07-31 05:28:37 UTC (rev 124173)
+++ trunk/Source/WebCore/html/shadow/SliderThumbElement.cpp 2012-07-31 05:46:43 UTC (rev 124174)
@@ -214,9 +214,9 @@
return hostInput()->isEnabledFormControl();
}
-bool SliderThumbElement::shouldMatchReadWriteSelector() const
+bool SliderThumbElement::isReadOnlyFormControl() const
{
- return hostInput()->shouldMatchReadWriteSelector();
+ return hostInput()->isReadOnlyFormControl();
}
Node* SliderThumbElement::focusDelegate()
@@ -334,7 +334,7 @@
bool SliderThumbElement::willRespondToMouseMoveEvents()
{
const HTMLInputElement* input = hostInput();
- if (input && !input->readOnly() && input->isEnabledFormControl() && m_inDragMode)
+ if (input && !input->isReadOnlyFormControl() && input->isEnabledFormControl() && m_inDragMode)
return true;
return HTMLDivElement::willRespondToMouseMoveEvents();
@@ -343,7 +343,7 @@
bool SliderThumbElement::willRespondToMouseClickEvents()
{
const HTMLInputElement* input = hostInput();
- if (input && !input->readOnly() && input->isEnabledFormControl())
+ if (input && !input->isReadOnlyFormControl() && input->isEnabledFormControl())
return true;
return HTMLDivElement::willRespondToMouseClickEvents();
Modified: trunk/Source/WebCore/html/shadow/SliderThumbElement.h (124173 => 124174)
--- trunk/Source/WebCore/html/shadow/SliderThumbElement.h 2012-07-31 05:28:37 UTC (rev 124173)
+++ trunk/Source/WebCore/html/shadow/SliderThumbElement.h 2012-07-31 05:46:43 UTC (rev 124174)
@@ -66,7 +66,7 @@
virtual RenderObject* createRenderer(RenderArena*, RenderStyle*);
virtual PassRefPtr<Element> cloneElementWithoutAttributesAndChildren();
virtual bool isEnabledFormControl() const;
- virtual bool shouldMatchReadWriteSelector() const OVERRIDE;
+ virtual bool isReadOnlyFormControl() const;
virtual Node* focusDelegate();
void startDragging();
void stopDragging();
Modified: trunk/Source/WebCore/html/shadow/TextControlInnerElements.cpp (124173 => 124174)
--- trunk/Source/WebCore/html/shadow/TextControlInnerElements.cpp 2012-07-31 05:28:37 UTC (rev 124173)
+++ trunk/Source/WebCore/html/shadow/TextControlInnerElements.cpp 2012-07-31 05:46:43 UTC (rev 124174)
@@ -242,7 +242,7 @@
bool SearchFieldCancelButtonElement::willRespondToMouseClickEvents()
{
const HTMLInputElement* input = static_cast<HTMLInputElement*>(shadowHost());
- if (!input->disabled() && !input->readOnly())
+ if (!input->disabled() && !input->isReadOnlyFormControl())
return true;
return HTMLDivElement::willRespondToMouseClickEvents();
@@ -345,7 +345,7 @@
bool SpinButtonElement::willRespondToMouseMoveEvents()
{
const HTMLInputElement* input = static_cast<HTMLInputElement*>(shadowHost());
- if (renderBox() && !input->disabled() && !input->readOnly())
+ if (renderBox() && !input->disabled() && !input->isReadOnlyFormControl())
return true;
return HTMLDivElement::willRespondToMouseMoveEvents();
@@ -354,7 +354,7 @@
bool SpinButtonElement::willRespondToMouseClickEvents()
{
const HTMLInputElement* input = static_cast<HTMLInputElement*>(shadowHost());
- if (renderBox() && !input->disabled() && !input->readOnly())
+ if (renderBox() && !input->disabled() && !input->isReadOnlyFormControl())
return true;
return HTMLDivElement::willRespondToMouseClickEvents();
@@ -514,7 +514,7 @@
bool InputFieldSpeechButtonElement::willRespondToMouseClickEvents()
{
const HTMLInputElement* input = static_cast<HTMLInputElement*>(shadowHost());
- if (!input->disabled() && !input->readOnly())
+ if (!input->disabled() && !input->isReadOnlyFormControl())
return true;
return HTMLDivElement::willRespondToMouseClickEvents();
Modified: trunk/Source/WebCore/html/shadow/TextControlInnerElements.h (124173 => 124174)
--- trunk/Source/WebCore/html/shadow/TextControlInnerElements.h 2012-07-31 05:28:37 UTC (rev 124173)
+++ trunk/Source/WebCore/html/shadow/TextControlInnerElements.h 2012-07-31 05:46:43 UTC (rev 124174)
@@ -125,7 +125,7 @@
virtual void detach();
virtual bool isSpinButtonElement() const { return true; }
virtual bool isEnabledFormControl() const { return shadowHost()->isEnabledFormControl(); }
- virtual bool shouldMatchReadWriteSelector() const OVERRIDE { return shadowHost()->shouldMatchReadWriteSelector(); }
+ virtual bool isReadOnlyFormControl() const { return shadowHost()->isReadOnlyFormControl(); }
virtual void defaultEventHandler(Event*);
void doStepAction(int);
void startRepeatingTimer();
Modified: trunk/Source/WebCore/rendering/RenderTheme.cpp (124173 => 124174)
--- trunk/Source/WebCore/rendering/RenderTheme.cpp 2012-07-31 05:28:37 UTC (rev 124173)
+++ trunk/Source/WebCore/rendering/RenderTheme.cpp 2012-07-31 05:46:43 UTC (rev 124174)
@@ -828,7 +828,7 @@
Node* node = o->node();
if (!node || !node->isElementNode())
return false;
- return !static_cast<Element*>(node)->shouldMatchReadWriteSelector();
+ return static_cast<Element*>(node)->isReadOnlyFormControl();
}
bool RenderTheme::isHovered(const RenderObject* o) const
