Remove "const" from obviously-non-const accessors
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 20 Jul 2011 21:12:30 +0000 (21:12 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 20 Jul 2011 21:12:30 +0000 (21:12 +0000)
https://bugs.webkit.org/show_bug.cgi?id=62302

Patch by Peter Kasting <pkasting@google.com> on 2011-07-20
Reviewed by Darin Adler.

Source/WebCore:

No functional change, so no new tests.

* WebCore.exp.in: Adjust function names to reflect non-constness.
* dom/Element.cpp:
(WebCore::Element::cloneElementWithoutAttributesAndChildren): Not const, calls non-const function Document::createElement().
(WebCore::Element::scrollLeft): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::Element::scrollTop): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::Element::scrollWidth): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::Element::scrollHeight): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::Element::boundsInWindowSpace): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::Element::getClientRects): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::Element::getBoundingClientRect): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::Element::innerText): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::Element::outerText): Not const, calls newly non-const function Element::innerText().
* dom/Element.h:
* dom/Node.cpp:
(WebCore::Node::isContentEditable): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::Node::shouldUseInputMethod): Not const, calls newly non-const function Node::isContentEditable().
* dom/Node.h:
* dom/OptionElement.h: Virtual function OptionElement::selected() not const because HTMLOptionElement implementation is non-const.
* html/HTMLAnchorElement.cpp:
(WebCore::HTMLAnchorElement::text): Not const, calls newly non-const function Element::innerText().
* html/HTMLAnchorElement.h:
* html/HTMLAppletElement.cpp:
(WebCore::HTMLAppletElement::renderWidgetForJSBindings): Not const, calls non-const function RenderApplet::createWidgetIfNecessary().
* html/HTMLAppletElement.h:
* html/HTMLAreaElement.cpp:
(WebCore::HTMLAreaElement::imageElement): Not const, calls newly non-const function HTMLMapElement::imageElement().
* html/HTMLAreaElement.h:
* html/HTMLBodyElement.cpp:
(WebCore::HTMLBodyElement::scrollLeft): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::HTMLBodyElement::scrollTop): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::HTMLBodyElement::scrollHeight): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::HTMLBodyElement::scrollWidth): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
* html/HTMLBodyElement.h:
* html/HTMLEmbedElement.cpp:
(WebCore::HTMLEmbedElement::renderWidgetForJSBindings): Not const, calls non-const function Document::createElement().
* html/HTMLEmbedElement.h:
* html/HTMLFrameElementBase.cpp:
(WebCore::HTMLFrameElementBase::width): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::HTMLFrameElementBase::height): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
* html/HTMLFrameElementBase.h:
* html/HTMLImageElement.cpp:
(WebCore::HTMLImageElement::width): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::HTMLImageElement::height): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
* html/HTMLImageElement.h:
* html/HTMLInputElement.cpp:
(WebCore::HTMLInputElement::shouldUseInputMethod): Virtual function not const because Node implementation is non-const.
* html/HTMLInputElement.h:
* html/HTMLKeygenElement.cpp:
(WebCore::KeygenSelectElement::cloneElementWithoutAttributesAndChildren): Virtual function not const because Element implementation is non-const.
* html/HTMLMapElement.cpp:
(WebCore::HTMLMapElement::imageElement): Not const, calls non-const function Document::images().
* html/HTMLMapElement.h:
* html/HTMLObjectElement.cpp:
(WebCore::HTMLObjectElement::renderWidgetForJSBindings): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
* html/HTMLObjectElement.h:
* html/HTMLOptionElement.cpp:
(WebCore::HTMLOptionElement::selected): Not const, calls non-const function HTMLSelectElement::recalcListItemsIfNeeded().
* html/HTMLOptionElement.h:
* html/HTMLPlugInElement.cpp: Virtual function HTMLPlugInElement::renderWidgetForJSBindings() not const because various subclass implementations are non-const.
(WebCore::HTMLPlugInElement::getInstance): Not const, calls newly non-const function HTMLPlugInElement::pluginWidget().
(WebCore::HTMLPlugInElement::pluginWidget): Not const, calls newly non-const function HTMLPlugInElement::renderWidgetForJSBindings().
* html/HTMLPlugInElement.h:
* html/HTMLScriptElement.cpp:
(WebCore::HTMLScriptElement::cloneElementWithoutAttributesAndChildren): Virtual function not const because Element implementation is non-const.
* html/HTMLScriptElement.h:
* html/HTMLTextAreaElement.cpp:
(WebCore::HTMLTextAreaElement::shouldUseInputMethod): Virtual function not const because Node implementation is non-const.
* html/HTMLTextAreaElement.h:
* html/shadow/SliderThumbElement.h:
(WebCore::SliderThumbElement::cloneElementWithoutAttributesAndChildren): Virtual function not const because Element implementation is non-const.
* rendering/svg/SVGShadowTreeElements.cpp:
(WebCore::SVGShadowTreeContainerElement::cloneElementWithoutAttributesAndChildren): Virtual function not const because Element implementation is non-const.
* rendering/svg/SVGShadowTreeElements.h:
* svg/SVGAnimateElement.cpp:
(WebCore::SVGAnimateElement::hasValidAttributeType): Not const, calls newly non-const function SVGSMILElement::targetElement().
* svg/SVGAnimateElement.h:
* svg/SVGAnimateMotionElement.cpp:
(WebCore::SVGAnimateMotionElement::hasValidAttributeType): Not const, calls newly non-const function SVGSMILElement::targetElement().
* svg/SVGAnimateMotionElement.h:
* svg/SVGAnimateTransformElement.cpp:
(WebCore::SVGAnimateTransformElement::hasValidAttributeType): Not const, calls newly non-const function SVGSMILElement::targetElement().
* svg/SVGAnimateTransformElement.h:
* svg/SVGAnimationElement.cpp:
(WebCore::SVGAnimationElement::currentValuesForValuesAnimation): Not const, calls newly non-const function SVGSMILElement::targetElement().
* svg/SVGAnimationElement.h:
* svg/SVGElement.cpp:
(WebCore::SVGElement::accessDocumentSVGExtensions): Not const, calls non-const function Document::accessSVGExtensions().
(WebCore::SVGElement::boundingBox): Not const, calls newly non-const function SVGStyledLocatableElement::getBBox().
* svg/SVGElement.h:
* svg/SVGLocatable.cpp:
(WebCore::SVGLocatable::getBBox): Argument not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets() on argument.
(WebCore::SVGLocatable::computeCTM): Argument not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets() on argument.
(WebCore::SVGLocatable::getTransformToElement): Not const, calls newly non-const function SVGLocatable::getCTM().
* svg/SVGLocatable.h: Virtual functions SVGLocatable::getBBox(), SVGLocatable::getCTM(), and SVGLocatable::getScreenCTM() not const because various subclass implementations are non-const.
* svg/SVGScriptElement.cpp:
(WebCore::SVGScriptElement::cloneElementWithoutAttributesAndChildren): Virtual function not const because Element implementation is non-const.
* svg/SVGScriptElement.h:
* svg/SVGStyledElement.cpp:
(WebCore::SVGStyledElement::buildPendingResourcesIfNeeded): Not const, calls non-const function Document::accessSVGExtensions().
* svg/SVGStyledElement.h:
* svg/SVGStyledLocatableElement.cpp:
(WebCore::SVGStyledLocatableElement::getBBox): Not const, calls newly non-const static function SVGLocatable::getBBox().
(WebCore::SVGStyledLocatableElement::getCTM): Not const, calls newly non-const static function SVGLocatable::computeCTM().
(WebCore::SVGStyledLocatableElement::getScreenCTM): Not const, calls newly non-const static function SVGLocatable::computeCTM().
* svg/SVGStyledLocatableElement.h:
* svg/SVGStyledTransformableElement.cpp:
(WebCore::SVGStyledTransformableElement::getCTM): Not const, calls newly non-const static function SVGLocatable::computeCTM().
(WebCore::SVGStyledTransformableElement::getScreenCTM): Not const, calls newly non-const static function SVGLocatable::computeCTM().
(WebCore::SVGStyledTransformableElement::getBBox): Not const, calls newly non-const static function SVGLocatable::getBBox().
(WebCore::SVGStyledTransformableElement::toClipPath): Virtual function not const because SVGUseElement implementation is non-const.
* svg/SVGStyledTransformableElement.h:
* svg/SVGTextContentElement.cpp:
(WebCore::SVGTextContentElement::getNumberOfChars): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::SVGTextContentElement::getComputedTextLength): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::SVGTextContentElement::getSubStringLength): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::SVGTextContentElement::getStartPositionOfChar): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::SVGTextContentElement::getEndPositionOfChar): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::SVGTextContentElement::getExtentOfChar): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::SVGTextContentElement::getRotationOfChar): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::SVGTextContentElement::getCharNumAtPosition): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::SVGTextContentElement::selectSubString): Not const, calls newly non-const static function SVGTextContentElement::getNumberOfChars().
* svg/SVGTextContentElement.h:
* svg/SVGTextElement.cpp:
(WebCore::SVGTextElement::getBBox): Not const, calls newly non-const static function SVGLocatable::getBBox().
(WebCore::SVGTextElement::getCTM): Not const, calls newly non-const static function SVGLocatable::computeCTM().
(WebCore::SVGTextElement::getScreenCTM): Not const, calls newly non-const static function SVGLocatable::computeCTM().
* svg/SVGTextElement.h:
* svg/SVGUseElement.cpp:
(WebCore::SVGUseElement::instanceRoot): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
(WebCore::isDirectReference):
(WebCore::SVGUseElement::toClipPath): Not const, calls non-const function Document::accessSVGExtensions().
* svg/SVGUseElement.h:
* svg/animation/SVGSMILElement.cpp:
(WebCore::SVGSMILElement::eventBaseFor): Not const, calls newly non-const function SVGSMILElement::targetElement().
(WebCore::SVGSMILElement::targetElement): Not const, calls non-const function Document::accessSVGExtensions().
* svg/animation/SVGSMILElement.h: Virtual function SVGSMILElement::hasValidAttributeType() not const because various subclass implementations are non-const.

Source/WebKit/chromium:

* public/WebElement.h:
* src/WebElement.cpp:
(WebKit::WebElement::innerText): Not const due to call to newly-non-const Element::innerText().
* src/WebSearchableFormData.cpp: Use non-const Node*s for code that calls newly non-const accessors.
(HTMLNames::IsSelectInDefaultState):
(HTMLNames::IsInDefaultState):
* src/WebViewImpl.cpp: Use non-const Node*s for code that calls newly non-const accessors.
(WebKit::WebViewImpl::setComposition):
(WebKit::WebViewImpl::confirmComposition):
(WebKit::WebViewImpl::textInputType):

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

72 files changed:
Source/WebCore/ChangeLog
Source/WebCore/WebCore.exp.in
Source/WebCore/dom/Element.cpp
Source/WebCore/dom/Element.h
Source/WebCore/dom/Node.cpp
Source/WebCore/dom/Node.h
Source/WebCore/dom/OptionElement.h
Source/WebCore/html/HTMLAnchorElement.cpp
Source/WebCore/html/HTMLAnchorElement.h
Source/WebCore/html/HTMLAppletElement.cpp
Source/WebCore/html/HTMLAppletElement.h
Source/WebCore/html/HTMLAreaElement.cpp
Source/WebCore/html/HTMLAreaElement.h
Source/WebCore/html/HTMLBodyElement.cpp
Source/WebCore/html/HTMLBodyElement.h
Source/WebCore/html/HTMLEmbedElement.cpp
Source/WebCore/html/HTMLEmbedElement.h
Source/WebCore/html/HTMLFrameElementBase.cpp
Source/WebCore/html/HTMLFrameElementBase.h
Source/WebCore/html/HTMLImageElement.cpp
Source/WebCore/html/HTMLImageElement.h
Source/WebCore/html/HTMLInputElement.cpp
Source/WebCore/html/HTMLInputElement.h
Source/WebCore/html/HTMLKeygenElement.cpp
Source/WebCore/html/HTMLMapElement.cpp
Source/WebCore/html/HTMLMapElement.h
Source/WebCore/html/HTMLObjectElement.cpp
Source/WebCore/html/HTMLObjectElement.h
Source/WebCore/html/HTMLOptionElement.cpp
Source/WebCore/html/HTMLOptionElement.h
Source/WebCore/html/HTMLPlugInElement.cpp
Source/WebCore/html/HTMLPlugInElement.h
Source/WebCore/html/HTMLScriptElement.cpp
Source/WebCore/html/HTMLScriptElement.h
Source/WebCore/html/HTMLTextAreaElement.cpp
Source/WebCore/html/HTMLTextAreaElement.h
Source/WebCore/html/shadow/SliderThumbElement.h
Source/WebCore/rendering/svg/SVGShadowTreeElements.cpp
Source/WebCore/rendering/svg/SVGShadowTreeElements.h
Source/WebCore/svg/SVGAnimateElement.cpp
Source/WebCore/svg/SVGAnimateElement.h
Source/WebCore/svg/SVGAnimateMotionElement.cpp
Source/WebCore/svg/SVGAnimateMotionElement.h
Source/WebCore/svg/SVGAnimateTransformElement.cpp
Source/WebCore/svg/SVGAnimateTransformElement.h
Source/WebCore/svg/SVGAnimationElement.cpp
Source/WebCore/svg/SVGAnimationElement.h
Source/WebCore/svg/SVGElement.cpp
Source/WebCore/svg/SVGElement.h
Source/WebCore/svg/SVGLocatable.cpp
Source/WebCore/svg/SVGLocatable.h
Source/WebCore/svg/SVGScriptElement.cpp
Source/WebCore/svg/SVGScriptElement.h
Source/WebCore/svg/SVGStyledElement.cpp
Source/WebCore/svg/SVGStyledElement.h
Source/WebCore/svg/SVGStyledLocatableElement.cpp
Source/WebCore/svg/SVGStyledLocatableElement.h
Source/WebCore/svg/SVGStyledTransformableElement.cpp
Source/WebCore/svg/SVGStyledTransformableElement.h
Source/WebCore/svg/SVGTextContentElement.cpp
Source/WebCore/svg/SVGTextContentElement.h
Source/WebCore/svg/SVGTextElement.cpp
Source/WebCore/svg/SVGTextElement.h
Source/WebCore/svg/SVGUseElement.cpp
Source/WebCore/svg/SVGUseElement.h
Source/WebCore/svg/animation/SVGSMILElement.cpp
Source/WebCore/svg/animation/SVGSMILElement.h
Source/WebKit/chromium/ChangeLog
Source/WebKit/chromium/public/WebElement.h
Source/WebKit/chromium/src/WebElement.cpp
Source/WebKit/chromium/src/WebSearchableFormData.cpp
Source/WebKit/chromium/src/WebViewImpl.cpp

index 5296074..3b380fc 100644 (file)
@@ -1,3 +1,149 @@
+2011-07-20  Peter Kasting  <pkasting@google.com>
+
+        Remove "const" from obviously-non-const accessors
+        https://bugs.webkit.org/show_bug.cgi?id=62302
+
+        Reviewed by Darin Adler.
+
+        No functional change, so no new tests.
+
+        * WebCore.exp.in: Adjust function names to reflect non-constness.
+        * dom/Element.cpp:
+        (WebCore::Element::cloneElementWithoutAttributesAndChildren): Not const, calls non-const function Document::createElement().
+        (WebCore::Element::scrollLeft): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::Element::scrollTop): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::Element::scrollWidth): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::Element::scrollHeight): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::Element::boundsInWindowSpace): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::Element::getClientRects): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::Element::getBoundingClientRect): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::Element::innerText): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::Element::outerText): Not const, calls newly non-const function Element::innerText().
+        * dom/Element.h:
+        * dom/Node.cpp:
+        (WebCore::Node::isContentEditable): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::Node::shouldUseInputMethod): Not const, calls newly non-const function Node::isContentEditable().
+        * dom/Node.h:
+        * dom/OptionElement.h: Virtual function OptionElement::selected() not const because HTMLOptionElement implementation is non-const.
+        * html/HTMLAnchorElement.cpp:
+        (WebCore::HTMLAnchorElement::text): Not const, calls newly non-const function Element::innerText().
+        * html/HTMLAnchorElement.h:
+        * html/HTMLAppletElement.cpp:
+        (WebCore::HTMLAppletElement::renderWidgetForJSBindings): Not const, calls non-const function RenderApplet::createWidgetIfNecessary().
+        * html/HTMLAppletElement.h:
+        * html/HTMLAreaElement.cpp:
+        (WebCore::HTMLAreaElement::imageElement): Not const, calls newly non-const function HTMLMapElement::imageElement().
+        * html/HTMLAreaElement.h:
+        * html/HTMLBodyElement.cpp:
+        (WebCore::HTMLBodyElement::scrollLeft): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::HTMLBodyElement::scrollTop): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::HTMLBodyElement::scrollHeight): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::HTMLBodyElement::scrollWidth): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        * html/HTMLBodyElement.h:
+        * html/HTMLEmbedElement.cpp:
+        (WebCore::HTMLEmbedElement::renderWidgetForJSBindings): Not const, calls non-const function Document::createElement().
+        * html/HTMLEmbedElement.h:
+        * html/HTMLFrameElementBase.cpp:
+        (WebCore::HTMLFrameElementBase::width): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::HTMLFrameElementBase::height): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        * html/HTMLFrameElementBase.h:
+        * html/HTMLImageElement.cpp:
+        (WebCore::HTMLImageElement::width): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::HTMLImageElement::height): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        * html/HTMLImageElement.h:
+        * html/HTMLInputElement.cpp:
+        (WebCore::HTMLInputElement::shouldUseInputMethod): Virtual function not const because Node implementation is non-const.
+        * html/HTMLInputElement.h:
+        * html/HTMLKeygenElement.cpp:
+        (WebCore::KeygenSelectElement::cloneElementWithoutAttributesAndChildren): Virtual function not const because Element implementation is non-const.
+        * html/HTMLMapElement.cpp:
+        (WebCore::HTMLMapElement::imageElement): Not const, calls non-const function Document::images().
+        * html/HTMLMapElement.h:
+        * html/HTMLObjectElement.cpp:
+        (WebCore::HTMLObjectElement::renderWidgetForJSBindings): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        * html/HTMLObjectElement.h:
+        * html/HTMLOptionElement.cpp:
+        (WebCore::HTMLOptionElement::selected): Not const, calls non-const function HTMLSelectElement::recalcListItemsIfNeeded().
+        * html/HTMLOptionElement.h:
+        * html/HTMLPlugInElement.cpp: Virtual function HTMLPlugInElement::renderWidgetForJSBindings() not const because various subclass implementations are non-const.
+        (WebCore::HTMLPlugInElement::getInstance): Not const, calls newly non-const function HTMLPlugInElement::pluginWidget().
+        (WebCore::HTMLPlugInElement::pluginWidget): Not const, calls newly non-const function HTMLPlugInElement::renderWidgetForJSBindings().
+        * html/HTMLPlugInElement.h:
+        * html/HTMLScriptElement.cpp:
+        (WebCore::HTMLScriptElement::cloneElementWithoutAttributesAndChildren): Virtual function not const because Element implementation is non-const.
+        * html/HTMLScriptElement.h:
+        * html/HTMLTextAreaElement.cpp:
+        (WebCore::HTMLTextAreaElement::shouldUseInputMethod): Virtual function not const because Node implementation is non-const.
+        * html/HTMLTextAreaElement.h:
+        * html/shadow/SliderThumbElement.h:
+        (WebCore::SliderThumbElement::cloneElementWithoutAttributesAndChildren): Virtual function not const because Element implementation is non-const.
+        * rendering/svg/SVGShadowTreeElements.cpp:
+        (WebCore::SVGShadowTreeContainerElement::cloneElementWithoutAttributesAndChildren): Virtual function not const because Element implementation is non-const.
+        * rendering/svg/SVGShadowTreeElements.h:
+        * svg/SVGAnimateElement.cpp:
+        (WebCore::SVGAnimateElement::hasValidAttributeType): Not const, calls newly non-const function SVGSMILElement::targetElement().
+        * svg/SVGAnimateElement.h:
+        * svg/SVGAnimateMotionElement.cpp:
+        (WebCore::SVGAnimateMotionElement::hasValidAttributeType): Not const, calls newly non-const function SVGSMILElement::targetElement().
+        * svg/SVGAnimateMotionElement.h:
+        * svg/SVGAnimateTransformElement.cpp:
+        (WebCore::SVGAnimateTransformElement::hasValidAttributeType): Not const, calls newly non-const function SVGSMILElement::targetElement().
+        * svg/SVGAnimateTransformElement.h:
+        * svg/SVGAnimationElement.cpp:
+        (WebCore::SVGAnimationElement::currentValuesForValuesAnimation): Not const, calls newly non-const function SVGSMILElement::targetElement().
+        * svg/SVGAnimationElement.h:
+        * svg/SVGElement.cpp:
+        (WebCore::SVGElement::accessDocumentSVGExtensions): Not const, calls non-const function Document::accessSVGExtensions().
+        (WebCore::SVGElement::boundingBox): Not const, calls newly non-const function SVGStyledLocatableElement::getBBox().
+        * svg/SVGElement.h:
+        * svg/SVGLocatable.cpp:
+        (WebCore::SVGLocatable::getBBox): Argument not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets() on argument.
+        (WebCore::SVGLocatable::computeCTM): Argument not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets() on argument.
+        (WebCore::SVGLocatable::getTransformToElement): Not const, calls newly non-const function SVGLocatable::getCTM().
+        * svg/SVGLocatable.h: Virtual functions SVGLocatable::getBBox(), SVGLocatable::getCTM(), and SVGLocatable::getScreenCTM() not const because various subclass implementations are non-const.
+        * svg/SVGScriptElement.cpp:
+        (WebCore::SVGScriptElement::cloneElementWithoutAttributesAndChildren): Virtual function not const because Element implementation is non-const.
+        * svg/SVGScriptElement.h:
+        * svg/SVGStyledElement.cpp:
+        (WebCore::SVGStyledElement::buildPendingResourcesIfNeeded): Not const, calls non-const function Document::accessSVGExtensions().
+        * svg/SVGStyledElement.h:
+        * svg/SVGStyledLocatableElement.cpp:
+        (WebCore::SVGStyledLocatableElement::getBBox): Not const, calls newly non-const static function SVGLocatable::getBBox().
+        (WebCore::SVGStyledLocatableElement::getCTM): Not const, calls newly non-const static function SVGLocatable::computeCTM().
+        (WebCore::SVGStyledLocatableElement::getScreenCTM): Not const, calls newly non-const static function SVGLocatable::computeCTM().
+        * svg/SVGStyledLocatableElement.h:
+        * svg/SVGStyledTransformableElement.cpp:
+        (WebCore::SVGStyledTransformableElement::getCTM): Not const, calls newly non-const static function SVGLocatable::computeCTM().
+        (WebCore::SVGStyledTransformableElement::getScreenCTM): Not const, calls newly non-const static function SVGLocatable::computeCTM().
+        (WebCore::SVGStyledTransformableElement::getBBox): Not const, calls newly non-const static function SVGLocatable::getBBox().
+        (WebCore::SVGStyledTransformableElement::toClipPath): Virtual function not const because SVGUseElement implementation is non-const.
+        * svg/SVGStyledTransformableElement.h:
+        * svg/SVGTextContentElement.cpp:
+        (WebCore::SVGTextContentElement::getNumberOfChars): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::SVGTextContentElement::getComputedTextLength): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::SVGTextContentElement::getSubStringLength): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::SVGTextContentElement::getStartPositionOfChar): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::SVGTextContentElement::getEndPositionOfChar): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::SVGTextContentElement::getExtentOfChar): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::SVGTextContentElement::getRotationOfChar): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::SVGTextContentElement::getCharNumAtPosition): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::SVGTextContentElement::selectSubString): Not const, calls newly non-const static function SVGTextContentElement::getNumberOfChars().
+        * svg/SVGTextContentElement.h:
+        * svg/SVGTextElement.cpp:
+        (WebCore::SVGTextElement::getBBox): Not const, calls newly non-const static function SVGLocatable::getBBox().
+        (WebCore::SVGTextElement::getCTM): Not const, calls newly non-const static function SVGLocatable::computeCTM().
+        (WebCore::SVGTextElement::getScreenCTM): Not const, calls newly non-const static function SVGLocatable::computeCTM().
+        * svg/SVGTextElement.h:
+        * svg/SVGUseElement.cpp:
+        (WebCore::SVGUseElement::instanceRoot): Not const, calls non-const function Document::updateLayoutIgnorePendingStylesheets().
+        (WebCore::isDirectReference):
+        (WebCore::SVGUseElement::toClipPath): Not const, calls non-const function Document::accessSVGExtensions().
+        * svg/SVGUseElement.h:
+        * svg/animation/SVGSMILElement.cpp:
+        (WebCore::SVGSMILElement::eventBaseFor): Not const, calls newly non-const function SVGSMILElement::targetElement().
+        (WebCore::SVGSMILElement::targetElement): Not const, calls non-const function Document::accessSVGExtensions().
+        * svg/animation/SVGSMILElement.h: Virtual function SVGSMILElement::hasValidAttributeType() not const because various subclass implementations are non-const.
+
 2011-07-20  Andy Estes  <aestes@apple.com>
 
         Fix the Mac build after r91392.
index 8faa97f..74c834e 100644 (file)
@@ -820,8 +820,10 @@ __ZN7WebCore6WidgetD2Ev
 __ZN7WebCore6toNodeEN3JSC7JSValueE
 __ZN7WebCore7Console21shouldPrintExceptionsEv
 __ZN7WebCore7Console24setShouldPrintExceptionsEb
+__ZN7WebCore7Element19boundsInWindowSpaceEv
 __ZN7WebCore7Element16ensureShadowRootEv
 __ZN7WebCore7Element16removeShadowRootEv
+__ZN7WebCore7Element9innerTextEv
 __ZN7WebCore7IntRect5scaleEf
 __ZN7WebCore7IntRect5uniteERKS0_
 __ZN7WebCore7IntRect9intersectERKS0_
@@ -1284,8 +1286,6 @@ __ZNK7WebCore6Widget9frameRectEv
 __ZNK7WebCore7Element10screenRectEv
 __ZNK7WebCore7Element10shadowRootEv
 __ZNK7WebCore7Element12getAttributeERKNS_13QualifiedNameE
-__ZNK7WebCore7Element19boundsInWindowSpaceEv
-__ZNK7WebCore7Element9innerTextEv
 __ZNK7WebCore7IntRect10intersectsERKS0_
 __ZNK7WebCore7IntRect8containsERKS0_
 __ZNK7WebCore7IntRectcv6CGRectEv
index 79eea9e..2ee7a84 100644 (file)
@@ -210,7 +210,7 @@ PassRefPtr<Element> Element::cloneElementWithoutChildren()
     return clone.release();
 }
 
-PassRefPtr<Element> Element::cloneElementWithoutAttributesAndChildren() const
+PassRefPtr<Element> Element::cloneElementWithoutAttributesAndChildren()
 {
     return document()->createElement(tagQName(), false);
 }
@@ -465,7 +465,7 @@ int Element::clientHeight()
     return 0;
 }
 
-int Element::scrollLeft() const
+int Element::scrollLeft()
 {
     document()->updateLayoutIgnorePendingStylesheets();
     if (RenderBox* rend = renderBox())
@@ -473,7 +473,7 @@ int Element::scrollLeft() const
     return 0;
 }
 
-int Element::scrollTop() const
+int Element::scrollTop()
 {
     document()->updateLayoutIgnorePendingStylesheets();
     if (RenderBox* rend = renderBox())
@@ -495,7 +495,7 @@ void Element::setScrollTop(int newTop)
         rend->setScrollTop(static_cast<int>(newTop * rend->style()->effectiveZoom()));
 }
 
-int Element::scrollWidth() const
+int Element::scrollWidth()
 {
     document()->updateLayoutIgnorePendingStylesheets();
     if (RenderBox* rend = renderBox())
@@ -503,7 +503,7 @@ int Element::scrollWidth() const
     return 0;
 }
 
-int Element::scrollHeight() const
+int Element::scrollHeight()
 {
     document()->updateLayoutIgnorePendingStylesheets();
     if (RenderBox* rend = renderBox())
@@ -511,7 +511,7 @@ int Element::scrollHeight() const
     return 0;
 }
 
-IntRect Element::boundsInWindowSpace() const
+IntRect Element::boundsInWindowSpace()
 {
     document()->updateLayoutIgnorePendingStylesheets();
 
@@ -523,7 +523,7 @@ IntRect Element::boundsInWindowSpace() const
 #if ENABLE(SVG)
     if (isSVGElement() && renderer()) {
         // Get the bounding rectangle from the SVG model.
-        const SVGElement* svgElement = static_cast<const SVGElement*>(this);
+        SVGElement* svgElement = static_cast<SVGElement*>(this);
         FloatRect localRect;
         if (svgElement->boundingBox(localRect))
             quads.append(renderer()->localToAbsoluteQuad(localRect));
@@ -546,7 +546,7 @@ IntRect Element::boundsInWindowSpace() const
     return result;
 }
 
-PassRefPtr<ClientRectList> Element::getClientRects() const
+PassRefPtr<ClientRectList> Element::getClientRects()
 {
     document()->updateLayoutIgnorePendingStylesheets();
 
@@ -579,7 +579,7 @@ PassRefPtr<ClientRectList> Element::getClientRects() const
     return ClientRectList::create(quads);
 }
 
-PassRefPtr<ClientRect> Element::getBoundingClientRect() const
+PassRefPtr<ClientRect> Element::getBoundingClientRect()
 {
     document()->updateLayoutIgnorePendingStylesheets();
 
@@ -587,7 +587,7 @@ PassRefPtr<ClientRect> Element::getBoundingClientRect() const
 #if ENABLE(SVG)
     if (isSVGElement() && renderer()) {
         // Get the bounding rectangle from the SVG model.
-        const SVGElement* svgElement = static_cast<const SVGElement*>(this);
+        SVGElement* svgElement = static_cast<SVGElement*>(this);
         FloatRect localRect;
         if (svgElement->boundingBox(localRect))
             quads.append(renderer()->localToAbsoluteQuad(localRect));
@@ -1696,7 +1696,7 @@ void Element::blur()
     }
 }
 
-String Element::innerText() const
+String Element::innerText()
 {
     // We need to update layout, since plainText uses line boxes in the render tree.
     document()->updateLayoutIgnorePendingStylesheets();
@@ -1707,7 +1707,7 @@ String Element::innerText() const
     return plainText(rangeOfContents(const_cast<Element*>(this)).get());
 }
 
-String Element::outerText() const
+String Element::outerText()
 {
     // Getting outerText is the same as getting innerText, only
     // setting is different. You would think this should get the plain
index d389bfe..89dcc60 100644 (file)
@@ -160,17 +160,17 @@ public:
     int clientTop();
     int clientWidth();
     int clientHeight();
-    virtual int scrollLeft() const;
-    virtual int scrollTop() const;
+    virtual int scrollLeft();
+    virtual int scrollTop();
     virtual void setScrollLeft(int);
     virtual void setScrollTop(int);
-    virtual int scrollWidth() const;
-    virtual int scrollHeight() const;
+    virtual int scrollWidth();
+    virtual int scrollHeight();
 
-    IntRect boundsInWindowSpace() const;
+    IntRect boundsInWindowSpace();
 
-    PassRefPtr<ClientRectList> getClientRects() const;
-    PassRefPtr<ClientRect> getBoundingClientRect() const;
+    PassRefPtr<ClientRectList> getClientRects();
+    PassRefPtr<ClientRect> getBoundingClientRect();
     
     // Returns the absolute bounding box translated into screen coordinates:
     IntRect screenRect() const;
@@ -262,8 +262,8 @@ public:
     virtual void updateFocusAppearance(bool restorePreviousSelection);
     void blur();
 
-    String innerText() const;
-    String outerText() const;
+    String innerText();
+    String outerText();
  
     virtual String title() const;
 
@@ -423,7 +423,7 @@ private:
     // cloneNode is private so that non-virtual cloneElementWithChildren and cloneElementWithoutChildren
     // are used instead.
     virtual PassRefPtr<Node> cloneNode(bool deep);
-    virtual PassRefPtr<Element> cloneElementWithoutAttributesAndChildren() const;
+    virtual PassRefPtr<Element> cloneElementWithoutAttributesAndChildren();
 
     QualifiedName m_tagName;
     virtual NodeRareData* createRareData();
index 2f30bb7..6285921 100644 (file)
@@ -762,7 +762,7 @@ const AtomicString& Node::virtualNamespaceURI() const
     return nullAtom;
 }
 
-bool Node::isContentEditable() const
+bool Node::isContentEditable()
 {
     document()->updateLayoutIgnorePendingStylesheets();
     return rendererIsEditable(Editable);
@@ -795,7 +795,7 @@ bool Node::rendererIsEditable(EditableLevel editableLevel) const
     return false;
 }
 
-bool Node::shouldUseInputMethod() const
+bool Node::shouldUseInputMethod()
 {
     return isContentEditable();
 }
index f1d3795..e4b0525 100644 (file)
@@ -337,11 +337,11 @@ public:
     virtual bool isMouseFocusable() const;
     virtual Node* focusDelegate();
 
-    bool isContentEditable() const;
+    bool isContentEditable();
 
     bool rendererIsEditable() const { return rendererIsEditable(Editable); }
     bool rendererIsRichlyEditable() const { return rendererIsEditable(RichlyEditable); }
-    virtual bool shouldUseInputMethod() const;
+    virtual bool shouldUseInputMethod();
     virtual IntRect getRect() const;
     IntRect renderRect(bool* isReplaced);
 
index dea885f..c31cf80 100644 (file)
@@ -35,7 +35,7 @@ public:
     virtual ~OptionElement() { }
 
     virtual bool disabled() const = 0;
-    virtual bool selected() const = 0;
+    virtual bool selected() = 0;
     virtual void setSelectedState(bool) = 0;
 
     virtual String text() const = 0;
index d6515e4..aa72ad7 100644 (file)
@@ -467,7 +467,7 @@ void HTMLAnchorElement::setSearch(const String& value)
     setHref(url.string());
 }
 
-String HTMLAnchorElement::text() const
+String HTMLAnchorElement::text()
 {
     return innerText();
 }
index 607d1f5..860ad35 100644 (file)
@@ -86,7 +86,7 @@ public:
 
     String getParameter(const String&) const;
 
-    String text() const;
+    String text();
 
     String toString() const;
 
index 8520c2c..1ba3448 100644 (file)
@@ -150,7 +150,7 @@ void HTMLAppletElement::defaultEventHandler(Event* event)
     widget->handleEvent(event);
 }
 
-RenderWidget* HTMLAppletElement::renderWidgetForJSBindings() const
+RenderWidget* HTMLAppletElement::renderWidgetForJSBindings()
 {
     if (!canEmbedJava())
         return 0;
index 84ce618..d813ee3 100644 (file)
@@ -42,7 +42,7 @@ private:
 
     virtual void defaultEventHandler(Event*);
 
-    virtual RenderWidget* renderWidgetForJSBindings() const;
+    virtual RenderWidget* renderWidgetForJSBindings();
 
     void setupApplet() const;
     bool canEmbedJava() const;
index 70cc1cb..fbacea5 100644 (file)
@@ -179,7 +179,7 @@ Path HTMLAreaElement::getRegion(const IntSize& size) const
     return path;
 }
 
-HTMLImageElement* HTMLAreaElement::imageElement() const
+HTMLImageElement* HTMLAreaElement::imageElement()
 {
     Node* mapElement = parentNode();
     if (!mapElement || !mapElement->hasTagName(mapTag))
index 1956453..8ad1cbc 100644 (file)
@@ -45,7 +45,7 @@ public:
     Path computePath(RenderObject*) const;
 
     // The parent map's image.
-    HTMLImageElement* imageElement() const;
+    HTMLImageElement* imageElement();
     
 private:
     HTMLAreaElement(const QualifiedName&, Document*);
index 64a6bf9..057fc66 100644 (file)
@@ -302,7 +302,7 @@ static int adjustForZoom(int value, Document* document)
     return static_cast<int>(value / zoomFactor);
 }
 
-int HTMLBodyElement::scrollLeft() const
+int HTMLBodyElement::scrollLeft()
 {
     // Update the document's layout.
     Document* document = this->document();
@@ -324,7 +324,7 @@ void HTMLBodyElement::setScrollLeft(int scrollLeft)
     view->setScrollPosition(IntPoint(static_cast<int>(scrollLeft * frame->pageZoomFactor() * frame->pageScaleFactor()), view->scrollY()));
 }
 
-int HTMLBodyElement::scrollTop() const
+int HTMLBodyElement::scrollTop()
 {
     // Update the document's layout.
     Document* document = this->document();
@@ -346,7 +346,7 @@ void HTMLBodyElement::setScrollTop(int scrollTop)
     view->setScrollPosition(IntPoint(view->scrollX(), static_cast<int>(scrollTop * frame->pageZoomFactor() * frame->pageScaleFactor())));
 }
 
-int HTMLBodyElement::scrollHeight() const
+int HTMLBodyElement::scrollHeight()
 {
     // Update the document's layout.
     Document* document = this->document();
@@ -355,7 +355,7 @@ int HTMLBodyElement::scrollHeight() const
     return view ? adjustForZoom(view->contentsHeight(), document) : 0;    
 }
 
-int HTMLBodyElement::scrollWidth() const
+int HTMLBodyElement::scrollWidth()
 {
     // Update the document's layout.
     Document* document = this->document();
index 884522c..4477453 100644 (file)
@@ -83,14 +83,14 @@ private:
     
     virtual bool supportsFocus() const;
 
-    virtual int scrollLeft() const;
+    virtual int scrollLeft();
     virtual void setScrollLeft(int scrollLeft);
     
-    virtual int scrollTop() const;
+    virtual int scrollTop();
     virtual void setScrollTop(int scrollTop);
     
-    virtual int scrollHeight() const;
-    virtual int scrollWidth() const;
+    virtual int scrollHeight();
+    virtual int scrollWidth();
     
     virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
 
index 38b2416..cd728b5 100644 (file)
@@ -68,7 +68,7 @@ static inline RenderWidget* findWidgetRenderer(const Node* n)
     return 0;
 }
 
-RenderWidget* HTMLEmbedElement::renderWidgetForJSBindings() const
+RenderWidget* HTMLEmbedElement::renderWidgetForJSBindings()
 {
     document()->updateLayoutIgnorePendingStylesheets();
     return findWidgetRenderer(this);
index b9dc798..076981b 100644 (file)
@@ -45,7 +45,7 @@ private:
     virtual bool isURLAttribute(Attribute*) const;
     virtual const QualifiedName& imageSourceAttributeName() const;
 
-    virtual RenderWidget* renderWidgetForJSBindings() const;
+    virtual RenderWidget* renderWidgetForJSBindings();
 
     virtual void updateWidget(PluginCreationOption);
 
index 7fa083e..234ec02 100644 (file)
@@ -235,7 +235,7 @@ bool HTMLFrameElementBase::isURLAttribute(Attribute *attr) const
     return attr->name() == srcAttr;
 }
 
-int HTMLFrameElementBase::width() const
+int HTMLFrameElementBase::width()
 {
     document()->updateLayoutIgnorePendingStylesheets();
     if (!renderBox())
@@ -243,7 +243,7 @@ int HTMLFrameElementBase::width() const
     return renderBox()->width();
 }
 
-int HTMLFrameElementBase::height() const
+int HTMLFrameElementBase::height()
 {
     document()->updateLayoutIgnorePendingStylesheets();
     if (!renderBox())
index 65826cb..a0f57cc 100644 (file)
@@ -39,8 +39,8 @@ public:
     int marginWidth() const { return m_marginWidth; }
     int marginHeight() const { return m_marginHeight; }
 
-    int width() const;
-    int height() const;
+    int width();
+    int height();
 
     void setRemainsAliveOnRemovalFromTree(bool);
 #if ENABLE(FULLSCREEN_API)
index a831f0f..8b9fa27 100644 (file)
@@ -257,7 +257,7 @@ void HTMLImageElement::removedFromTree(bool deep)
     HTMLElement::removedFromTree(deep);
 }
 
-int HTMLImageElement::width(bool ignorePendingStylesheets) const
+int HTMLImageElement::width(bool ignorePendingStylesheets)
 {
     if (!renderer()) {
         // check the attribute first for an explicit pixel value
@@ -280,7 +280,7 @@ int HTMLImageElement::width(bool ignorePendingStylesheets) const
     return box ? adjustForAbsoluteZoom(box->contentWidth(), box) : 0;
 }
 
-int HTMLImageElement::height(bool ignorePendingStylesheets) const
+int HTMLImageElement::height(bool ignorePendingStylesheets)
 {
     if (!renderer()) {
         // check the attribute first for an explicit pixel value
index 29f42a1..93c87b9 100644 (file)
@@ -41,8 +41,8 @@ public:
 
     virtual ~HTMLImageElement();
 
-    int width(bool ignorePendingStylesheets = false) const;
-    int height(bool ignorePendingStylesheets = false) const;
+    int width(bool ignorePendingStylesheets = false);
+    int height(bool ignorePendingStylesheets = false);
 
     int naturalWidth() const;
     int naturalHeight() const;
index 60640b5..508efea 100644 (file)
@@ -495,7 +495,7 @@ void HTMLInputElement::aboutToUnload()
     frame->editor()->textFieldDidEndEditing(this);
 }
 
-bool HTMLInputElement::shouldUseInputMethod() const
+bool HTMLInputElement::shouldUseInputMethod()
 {
     return m_inputType->shouldUseInputMethod();
 }
index 1b9d4bc..07137be 100644 (file)
@@ -249,7 +249,7 @@ private:
     virtual bool isEnumeratable() const;
     virtual void updateFocusAppearance(bool restorePreviousSelection);
     virtual void aboutToUnload();
-    virtual bool shouldUseInputMethod() const;
+    virtual bool shouldUseInputMethod();
 
     virtual const AtomicString& formControlName() const;
 
index eb68c50..d60b105 100644 (file)
@@ -62,7 +62,7 @@ protected:
     }
 
 private:
-    virtual PassRefPtr<Element> cloneElementWithoutAttributesAndChildren() const
+    virtual PassRefPtr<Element> cloneElementWithoutAttributesAndChildren()
     {
         return create(document());
     }
index 1318807..f83bdc4 100644 (file)
@@ -80,7 +80,7 @@ bool HTMLMapElement::mapMouseEvent(int x, int y, const IntSize& size, HitTestRes
     return defaultArea;
 }
 
-HTMLImageElement* HTMLMapElement::imageElement() const
+HTMLImageElement* HTMLMapElement::imageElement()
 {
     RefPtr<HTMLCollection> coll = document()->images();
     for (Node* curr = coll->firstItem(); curr; curr = coll->nextItem()) {
index c0c1395..8e356b4 100644 (file)
@@ -41,7 +41,7 @@ public:
 
     bool mapMouseEvent(int x, int y, const IntSize&, HitTestResult&);
     
-    HTMLImageElement* imageElement() const;
+    HTMLImageElement* imageElement();
     PassRefPtr<HTMLCollection> areas();
 
 private:
index 8276c45..6a1226b 100644 (file)
@@ -77,7 +77,7 @@ PassRefPtr<HTMLObjectElement> HTMLObjectElement::create(const QualifiedName& tag
     return adoptRef(new HTMLObjectElement(tagName, document, form, createdByParser));
 }
 
-RenderWidget* HTMLObjectElement::renderWidgetForJSBindings() const
+RenderWidget* HTMLObjectElement::renderWidgetForJSBindings()
 {
     document()->updateLayoutIgnorePendingStylesheets();
     return renderPart(); // This will return 0 if the renderer is not a RenderPart.
index a90c1b2..6c0e05e 100644 (file)
@@ -82,7 +82,7 @@ private:
     virtual bool isURLAttribute(Attribute*) const;
     virtual const QualifiedName& imageSourceAttributeName() const;
 
-    virtual RenderWidget* renderWidgetForJSBindings() const;
+    virtual RenderWidget* renderWidgetForJSBindings();
 
     virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
 
index 24166de..5f4e0a0 100644 (file)
@@ -161,7 +161,7 @@ void HTMLOptionElement::setValue(const String& value)
     setAttribute(valueAttr, value);
 }
 
-bool HTMLOptionElement::selected() const
+bool HTMLOptionElement::selected()
 {
     if (HTMLSelectElement* select = ownerSelectElement())
         select->recalcListItemsIfNeeded();
index 22ce1b6..8685ded 100644 (file)
@@ -49,7 +49,7 @@ public:
     virtual String value() const;
     void setValue(const String&);
 
-    virtual bool selected() const;
+    virtual bool selected();
     void setSelected(bool);
 
     HTMLSelectElement* ownerSelectElement() const;
index cff1e37..ea90cfe 100644 (file)
@@ -81,7 +81,7 @@ void HTMLPlugInElement::detach()
     HTMLFrameOwnerElement::detach();
 }
 
-PassScriptInstance HTMLPlugInElement::getInstance() const
+PassScriptInstance HTMLPlugInElement::getInstance()
 {
     Frame* frame = document()->frame();
     if (!frame)
@@ -98,7 +98,7 @@ PassScriptInstance HTMLPlugInElement::getInstance() const
     return m_instance;
 }
 
-Widget* HTMLPlugInElement::pluginWidget() const
+Widget* HTMLPlugInElement::pluginWidget()
 {
     if (m_inBeforeLoadEventHandler) {
         // The plug-in hasn't loaded yet, and it makes no sense to try to load if beforeload handler happened to touch the plug-in element.
index 6f4a6ea..fd99caa 100644 (file)
@@ -40,9 +40,9 @@ class HTMLPlugInElement : public HTMLFrameOwnerElement {
 public:
     virtual ~HTMLPlugInElement();
 
-    PassScriptInstance getInstance() const;
+    PassScriptInstance getInstance();
 
-    Widget* pluginWidget() const;
+    Widget* pluginWidget();
 
 #if ENABLE(NETSCAPE_PLUGIN_API)
     NPObject* getNPObject();
@@ -66,7 +66,7 @@ protected:
 private:
     virtual void defaultEventHandler(Event*);
 
-    virtual RenderWidget* renderWidgetForJSBindings() const = 0;
+    virtual RenderWidget* renderWidgetForJSBindings() = 0;
 
 protected:
     AtomicString m_name;
index 14bcf8c..4be121d 100644 (file)
@@ -211,7 +211,7 @@ void HTMLScriptElement::dispatchLoadEvent()
     dispatchEvent(Event::create(eventNames().loadEvent, false, false));
 }
 
-PassRefPtr<Element> HTMLScriptElement::cloneElementWithoutAttributesAndChildren() const
+PassRefPtr<Element> HTMLScriptElement::cloneElementWithoutAttributesAndChildren()
 {
     return adoptRef(new HTMLScriptElement(tagQName(), document(), false, alreadyStarted()));
 }
index 85dad60..ee6f7b5 100644 (file)
@@ -66,7 +66,7 @@ private:
 
     virtual void dispatchLoadEvent();
 
-    PassRefPtr<Element> cloneElementWithoutAttributesAndChildren() const;
+    virtual PassRefPtr<Element> cloneElementWithoutAttributesAndChildren();
 };
 
 } //namespace
index 6d8c84b..3555f77 100644 (file)
@@ -452,7 +452,7 @@ bool HTMLTextAreaElement::lastChangeWasUserEdit() const
     return toRenderTextControl(renderer())->lastChangeWasUserEdit();
 }
 
-bool HTMLTextAreaElement::shouldUseInputMethod() const
+bool HTMLTextAreaElement::shouldUseInputMethod()
 {
     return true;
 }
index aea6d08..b93a005 100644 (file)
@@ -105,7 +105,7 @@ private:
 
     virtual void accessKeyAction(bool sendToAnyElement);
 
-    virtual bool shouldUseInputMethod() const;
+    virtual bool shouldUseInputMethod();
 
     int m_rows;
     int m_cols;
index ad03547..5815ca8 100644 (file)
@@ -60,7 +60,7 @@ public:
 private:
     SliderThumbElement(Document*);
     virtual RenderObject* createRenderer(RenderArena*, RenderStyle*);
-    virtual PassRefPtr<Element> cloneElementWithoutAttributesAndChildren() const;
+    virtual PassRefPtr<Element> cloneElementWithoutAttributesAndChildren();
     virtual bool isEnabledFormControl() const;
     virtual bool isReadOnlyFormControl() const;
     virtual Node* focusDelegate();
@@ -82,7 +82,7 @@ inline PassRefPtr<SliderThumbElement> SliderThumbElement::create(Document* docum
     return adoptRef(new SliderThumbElement(document));
 }
 
-inline PassRefPtr<Element> SliderThumbElement::cloneElementWithoutAttributesAndChildren() const
+inline PassRefPtr<Element> SliderThumbElement::cloneElementWithoutAttributesAndChildren()
 {
     return create(document());
 }
index ac4a63e..698d183 100644 (file)
@@ -47,7 +47,7 @@ FloatSize SVGShadowTreeContainerElement::containerTranslation() const
     return FloatSize(m_xOffset.value(this), m_yOffset.value(this));
 }
 
-PassRefPtr<Element> SVGShadowTreeContainerElement::cloneElementWithoutAttributesAndChildren() const
+PassRefPtr<Element> SVGShadowTreeContainerElement::cloneElementWithoutAttributesAndChildren()
 {
     return adoptRef(new SVGShadowTreeContainerElement(document()));
 }
index 8cbd4b7..02defee 100644 (file)
@@ -44,7 +44,7 @@ protected:
     SVGShadowTreeContainerElement(Document*);
 
 private:
-    virtual PassRefPtr<Element> cloneElementWithoutAttributesAndChildren() const;
+    virtual PassRefPtr<Element> cloneElementWithoutAttributesAndChildren();
     virtual bool isShadowTreeContainerElement() const { return true; }
 
     SVGLength m_xOffset;
index d1ce3f6..0fe7f82 100644 (file)
@@ -103,7 +103,7 @@ void SVGAnimateElement::adjustForInheritance(SVGElement* targetElement, const Qu
         getPropertyValue(svgParent, attributeName, value);
 }
 
-bool SVGAnimateElement::hasValidAttributeType() const
+bool SVGAnimateElement::hasValidAttributeType()
 {
     SVGElement* targetElement = this->targetElement();
     if (!targetElement)
index f524232..cd57b41 100644 (file)
@@ -68,7 +68,7 @@ protected:
 private:
     SVGAnimatedTypeAnimator* ensureAnimator();
     
-    virtual bool hasValidAttributeType() const;
+    virtual bool hasValidAttributeType();
     AnimatedPropertyType m_animatedPropertyType;
 
     AnimatedPropertyValueType m_fromPropertyValueType;
index 5b2c0b5..6ce9cfe 100644 (file)
@@ -54,7 +54,7 @@ PassRefPtr<SVGAnimateMotionElement> SVGAnimateMotionElement::create(const Qualif
     return adoptRef(new SVGAnimateMotionElement(tagName, document));
 }
 
-bool SVGAnimateMotionElement::hasValidAttributeType() const
+bool SVGAnimateMotionElement::hasValidAttributeType()
 {
     SVGElement* targetElement = this->targetElement();
     if (!targetElement)
index 5789726..b899412 100644 (file)
@@ -34,7 +34,7 @@ public:
 private:
     SVGAnimateMotionElement(const QualifiedName&, Document*);
 
-    virtual bool hasValidAttributeType() const;
+    virtual bool hasValidAttributeType();
 
     bool isSupportedAttribute(const QualifiedName&);
     virtual void parseMappedAttribute(Attribute*);
index 4817121..74eeaae 100644 (file)
@@ -60,7 +60,7 @@ PassRefPtr<SVGAnimateTransformElement> SVGAnimateTransformElement::create(const
     return adoptRef(new SVGAnimateTransformElement(tagName, document));
 }
 
-bool SVGAnimateTransformElement::hasValidAttributeType() const
+bool SVGAnimateTransformElement::hasValidAttributeType()
 {
     SVGElement* targetElement = this->targetElement();
     if (!targetElement)
index 0e09065..f61208a 100644 (file)
@@ -39,7 +39,7 @@ public:
 private:
     SVGAnimateTransformElement(const QualifiedName&, Document*);
     
-    virtual bool hasValidAttributeType() const;
+    virtual bool hasValidAttributeType();
     AnimatedPropertyType determineAnimatedPropertyType(SVGElement*) const;
 
     bool isSupportedAttribute(const QualifiedName&);
index 93eac60..7507b55 100644 (file)
@@ -470,7 +470,7 @@ void SVGAnimationElement::currentValuesFromKeyPoints(float percent, float& effec
     to = m_values[index + 1];
 }
     
-void SVGAnimationElement::currentValuesForValuesAnimation(float percent, float& effectivePercent, String& from, String& to) const
+void SVGAnimationElement::currentValuesForValuesAnimation(float percent, float& effectivePercent, String& from, String& to)
 {
     unsigned valuesCount = m_values.size();
     ASSERT(m_animationValid);
@@ -485,7 +485,7 @@ void SVGAnimationElement::currentValuesForValuesAnimation(float percent, float&
 
     CalcMode calcMode = this->calcMode();
     if (hasTagName(SVGNames::animateTag) || hasTagName(SVGNames::animateColorTag)) {
-        const SVGAnimateElement* animateElement = static_cast<const SVGAnimateElement*>(this);
+        SVGAnimateElement* animateElement = static_cast<SVGAnimateElement*>(this);
         AnimatedPropertyType attributeType = animateElement->determineAnimatedPropertyType(targetElement());
         // Fall back to discrete animations for Strings.
         if (attributeType == AnimatedBoolean
index 64603ff..cc4a04c 100644 (file)
@@ -115,7 +115,7 @@ private:
     virtual float calculateDistance(const String& /*fromString*/, const String& /*toString*/) { return -1.f; }
     virtual Path animationPath() const { return Path(); }
 
-    void currentValuesForValuesAnimation(float percent, float& effectivePercent, String& from, String& to) const;
+    void currentValuesForValuesAnimation(float percent, float& effectivePercent, String& from, String& to);
     void calculateKeyTimesForCalcModePaced();
     float calculatePercentFromKeyPoints(float percent) const;
     void currentValuesFromKeyPoints(float percent, float& effectivePercent, String& from, String& to) const;
index ba47abb..07ac31c 100644 (file)
@@ -174,7 +174,7 @@ SVGElement* SVGElement::viewportElement() const
     return 0;
 }
 
-SVGDocumentExtensions* SVGElement::accessDocumentSVGExtensions() const
+SVGDocumentExtensions* SVGElement::accessDocumentSVGExtensions()
 {
     // This function is provided for use by SVGAnimatedProperty to avoid
     // global inclusion of Document.h in SVG code.
@@ -211,14 +211,14 @@ const HashSet<SVGElementInstance*>& SVGElement::instancesForElement() const
     return rareSVGData()->elementInstances();
 }
 
-bool SVGElement::boundingBox(FloatRect& rect, SVGLocatable::StyleUpdateStrategy styleUpdateStrategy) const
+bool SVGElement::boundingBox(FloatRect& rect, SVGLocatable::StyleUpdateStrategy styleUpdateStrategy)
 {
     if (isStyledLocatable()) {
-        rect = static_cast<const SVGStyledLocatableElement*>(this)->getBBox(styleUpdateStrategy);
+        rect = static_cast<SVGStyledLocatableElement*>(this)->getBBox(styleUpdateStrategy);
         return true;
     }
     if (hasTagName(SVGNames::textTag)) {
-        rect = static_cast<const SVGTextElement*>(this)->getBBox(styleUpdateStrategy);
+        rect = static_cast<SVGTextElement*>(this)->getBBox(styleUpdateStrategy);
         return true;
     }
     return false;
index 2f9c9f1..f079e71 100644 (file)
@@ -57,7 +57,7 @@ public:
     SVGSVGElement* ownerSVGElement() const;
     SVGElement* viewportElement() const;
 
-    SVGDocumentExtensions* accessDocumentSVGExtensions() const;
+    SVGDocumentExtensions* accessDocumentSVGExtensions();
 
     virtual bool isStyled() const { return false; }
     virtual bool isStyledTransformable() const { return false; }
@@ -82,7 +82,7 @@ public:
 
     const HashSet<SVGElementInstance*>& instancesForElement() const;
 
-    bool boundingBox(FloatRect&, SVGLocatable::StyleUpdateStrategy = SVGLocatable::AllowStyleUpdate) const;
+    bool boundingBox(FloatRect&, SVGLocatable::StyleUpdateStrategy = SVGLocatable::AllowStyleUpdate);
 
     void setCursorElement(SVGCursorElement*);
     void cursorElementRemoved();
index 2a7c9be..0cb4c59 100644 (file)
@@ -64,7 +64,7 @@ SVGElement* SVGLocatable::farthestViewportElement(const SVGElement* element)
     return farthest;
 }
 
-FloatRect SVGLocatable::getBBox(const SVGElement* element, StyleUpdateStrategy styleUpdateStrategy)
+FloatRect SVGLocatable::getBBox(SVGElement* element, StyleUpdateStrategy styleUpdateStrategy)
 {
     ASSERT(element);
     if (styleUpdateStrategy == AllowStyleUpdate)
@@ -77,7 +77,7 @@ FloatRect SVGLocatable::getBBox(const SVGElement* element, StyleUpdateStrategy s
     return element->renderer()->objectBoundingBox();
 }
 
-AffineTransform SVGLocatable::computeCTM(const SVGElement* element, CTMScope mode, StyleUpdateStrategy styleUpdateStrategy)
+AffineTransform SVGLocatable::computeCTM(SVGElement* element, CTMScope mode, StyleUpdateStrategy styleUpdateStrategy)
 {
     ASSERT(element);
     if (styleUpdateStrategy == AllowStyleUpdate)
@@ -104,7 +104,7 @@ AffineTransform SVGLocatable::computeCTM(const SVGElement* element, CTMScope mod
     return ctm;
 }
 
-AffineTransform SVGLocatable::getTransformToElement(SVGElement* target, ExceptionCode& ec, StyleUpdateStrategy styleUpdateStrategy) const
+AffineTransform SVGLocatable::getTransformToElement(SVGElement* target, ExceptionCode& ec, StyleUpdateStrategy styleUpdateStrategy)
 {
     AffineTransform ctm = getCTM(styleUpdateStrategy);
 
index b19f5cd..ef0cf8a 100644 (file)
@@ -41,10 +41,10 @@ public:
 
     enum StyleUpdateStrategy { AllowStyleUpdate, DisallowStyleUpdate };
     
-    virtual FloatRect getBBox(StyleUpdateStrategy) const = 0;
-    virtual AffineTransform getCTM(StyleUpdateStrategy) const = 0;
-    virtual AffineTransform getScreenCTM(StyleUpdateStrategy) const = 0;
-    AffineTransform getTransformToElement(SVGElement*, ExceptionCode&, StyleUpdateStrategy = AllowStyleUpdate) const;
+    virtual FloatRect getBBox(StyleUpdateStrategy) = 0;
+    virtual AffineTransform getCTM(StyleUpdateStrategy) = 0;
+    virtual AffineTransform getScreenCTM(StyleUpdateStrategy) = 0;
+    AffineTransform getTransformToElement(SVGElement*, ExceptionCode&, StyleUpdateStrategy = AllowStyleUpdate);
 
     static SVGElement* nearestViewportElement(const SVGElement*);
     static SVGElement* farthestViewportElement(const SVGElement*);
@@ -57,8 +57,8 @@ public:
 protected:
     virtual AffineTransform localCoordinateSpaceTransform(SVGLocatable::CTMScope) const { return AffineTransform(); }
 
-    static FloatRect getBBox(const SVGElement*, StyleUpdateStrategy);
-    static AffineTransform computeCTM(const SVGElement*, CTMScope, StyleUpdateStrategy);
+    static FloatRect getBBox(SVGElement*, StyleUpdateStrategy);
+    static AffineTransform computeCTM(SVGElement*, CTMScope, StyleUpdateStrategy);
 };
 
 } // namespace WebCore
index 2870a64..7b9bcc8 100644 (file)
@@ -263,7 +263,7 @@ void SVGScriptElement::dispatchLoadEvent()
     }
 }
 
-PassRefPtr<Element> SVGScriptElement::cloneElementWithoutAttributesAndChildren() const
+PassRefPtr<Element> SVGScriptElement::cloneElementWithoutAttributesAndChildren()
 {
     return adoptRef(new SVGScriptElement(tagQName(), document(), false, alreadyStarted()));
 }
index 116a321..fa3fab7 100644 (file)
@@ -70,7 +70,7 @@ private:
 
     virtual void dispatchLoadEvent();
 
-    PassRefPtr<Element> cloneElementWithoutAttributesAndChildren() const;
+    virtual PassRefPtr<Element> cloneElementWithoutAttributesAndChildren();
 
     BEGIN_DECLARE_ANIMATED_PROPERTIES(SVGScriptElement)
         DECLARE_ANIMATED_STRING(Href, href)
index b06a9ac..88de72e 100644 (file)
@@ -371,7 +371,7 @@ void SVGStyledElement::insertedIntoDocument()
     buildPendingResourcesIfNeeded();
 }
 
-void SVGStyledElement::buildPendingResourcesIfNeeded() const
+void SVGStyledElement::buildPendingResourcesIfNeeded()
 {
     Document* document = this->document();
     if (!needsPendingResourceHandling() || !document)
index e60d792..a4df4c0 100644 (file)
@@ -86,7 +86,7 @@ private:
     virtual bool isKeyboardFocusable(KeyboardEvent*) const;
     virtual bool isMouseFocusable() const;
 
-    void buildPendingResourcesIfNeeded() const;
+    void buildPendingResourcesIfNeeded();
 
     HashSet<SVGStyledElement*> m_elementsWithRelativeLengths;
 
index f29121b..9153842 100644 (file)
@@ -44,17 +44,17 @@ SVGElement* SVGStyledLocatableElement::farthestViewportElement() const
     return SVGLocatable::farthestViewportElement(this);
 }
 
-FloatRect SVGStyledLocatableElement::getBBox(StyleUpdateStrategy styleUpdateStrategy) const
+FloatRect SVGStyledLocatableElement::getBBox(StyleUpdateStrategy styleUpdateStrategy)
 {
     return SVGLocatable::getBBox(this, styleUpdateStrategy);
 }
 
-AffineTransform SVGStyledLocatableElement::getCTM(StyleUpdateStrategy styleUpdateStrategy) const
+AffineTransform SVGStyledLocatableElement::getCTM(StyleUpdateStrategy styleUpdateStrategy)
 {
     return SVGLocatable::computeCTM(this, SVGLocatable::NearestViewportScope, styleUpdateStrategy);
 }
 
-AffineTransform SVGStyledLocatableElement::getScreenCTM(StyleUpdateStrategy styleUpdateStrategy) const
+AffineTransform SVGStyledLocatableElement::getScreenCTM(StyleUpdateStrategy styleUpdateStrategy)
 {
     return SVGLocatable::computeCTM(this, SVGLocatable::ScreenScope, styleUpdateStrategy);
 }
index 114ae83..5eebd05 100644 (file)
@@ -35,9 +35,9 @@ public:
     virtual SVGElement* nearestViewportElement() const;
     virtual SVGElement* farthestViewportElement() const;
 
-    virtual FloatRect getBBox(StyleUpdateStrategy = AllowStyleUpdate) const;
-    virtual AffineTransform getCTM(StyleUpdateStrategy = AllowStyleUpdate) const;
-    virtual AffineTransform getScreenCTM(StyleUpdateStrategy = AllowStyleUpdate) const;
+    virtual FloatRect getBBox(StyleUpdateStrategy = AllowStyleUpdate);
+    virtual AffineTransform getCTM(StyleUpdateStrategy = AllowStyleUpdate);
+    virtual AffineTransform getScreenCTM(StyleUpdateStrategy = AllowStyleUpdate);
 
     virtual AffineTransform localCoordinateSpaceTransform(SVGLocatable::CTMScope mode) const { return SVGLocatable::localCoordinateSpaceTransform(mode); }
 
index 37de33b..4a6bd63 100644 (file)
@@ -50,12 +50,12 @@ SVGStyledTransformableElement::~SVGStyledTransformableElement()
 {
 }
 
-AffineTransform SVGStyledTransformableElement::getCTM(StyleUpdateStrategy styleUpdateStrategy) const
+AffineTransform SVGStyledTransformableElement::getCTM(StyleUpdateStrategy styleUpdateStrategy)
 {
     return SVGLocatable::computeCTM(this, SVGLocatable::NearestViewportScope, styleUpdateStrategy);
 }
 
-AffineTransform SVGStyledTransformableElement::getScreenCTM(StyleUpdateStrategy styleUpdateStrategy) const
+AffineTransform SVGStyledTransformableElement::getScreenCTM(StyleUpdateStrategy styleUpdateStrategy)
 {
     return SVGLocatable::computeCTM(this, SVGLocatable::ScreenScope, styleUpdateStrategy);
 }
@@ -135,7 +135,7 @@ SVGElement* SVGStyledTransformableElement::farthestViewportElement() const
     return SVGTransformable::farthestViewportElement(this);
 }
 
-FloatRect SVGStyledTransformableElement::getBBox(StyleUpdateStrategy styleUpdateStrategy) const
+FloatRect SVGStyledTransformableElement::getBBox(StyleUpdateStrategy styleUpdateStrategy)
 {
     return SVGTransformable::getBBox(this, styleUpdateStrategy);
 }
@@ -146,7 +146,7 @@ RenderObject* SVGStyledTransformableElement::createRenderer(RenderArena* arena,
     return new (arena) RenderSVGPath(this);
 }
 
-void SVGStyledTransformableElement::toClipPath(Path& path) const
+void SVGStyledTransformableElement::toClipPath(Path& path)
 {
     toPathData(path);
     // FIXME: How do we know the element has done a layout?
index 09e2fa7..227d25b 100644 (file)
@@ -36,8 +36,8 @@ class SVGStyledTransformableElement : public SVGStyledLocatableElement,
 public:
     virtual ~SVGStyledTransformableElement();
 
-    virtual AffineTransform getCTM(StyleUpdateStrategy = AllowStyleUpdate) const;
-    virtual AffineTransform getScreenCTM(StyleUpdateStrategy = AllowStyleUpdate) const;
+    virtual AffineTransform getCTM(StyleUpdateStrategy = AllowStyleUpdate);
+    virtual AffineTransform getScreenCTM(StyleUpdateStrategy = AllowStyleUpdate);
     virtual SVGElement* nearestViewportElement() const;
     virtual SVGElement* farthestViewportElement() const;
 
@@ -45,11 +45,11 @@ public:
     virtual AffineTransform animatedLocalTransform() const;
     virtual AffineTransform* supplementalTransform();
 
-    virtual FloatRect getBBox(StyleUpdateStrategy = AllowStyleUpdate) const;
+    virtual FloatRect getBBox(StyleUpdateStrategy = AllowStyleUpdate);
 
     // "base class" methods for all the elements which render as paths
     virtual void toPathData(Path&) const { }
-    virtual void toClipPath(Path&) const;
+    virtual void toClipPath(Path&);
     virtual RenderObject* createRenderer(RenderArena*, RenderStyle*);
 
 protected:
index 80e2f5e..854bc86 100644 (file)
@@ -105,19 +105,19 @@ PassRefPtr<SVGAnimatedLength> SVGTextContentElement::textLengthAnimated()
 
 }
 
-unsigned SVGTextContentElement::getNumberOfChars() const
+unsigned SVGTextContentElement::getNumberOfChars()
 {
     document()->updateLayoutIgnorePendingStylesheets();
     return SVGTextQuery(renderer()).numberOfCharacters();
 }
 
-float SVGTextContentElement::getComputedTextLength() const
+float SVGTextContentElement::getComputedTextLength()
 {
     document()->updateLayoutIgnorePendingStylesheets();
     return SVGTextQuery(renderer()).textLength();
 }
 
-float SVGTextContentElement::getSubStringLength(unsigned charnum, unsigned nchars, ExceptionCode& ec) const
+float SVGTextContentElement::getSubStringLength(unsigned charnum, unsigned nchars, ExceptionCode& ec)
 {
     document()->updateLayoutIgnorePendingStylesheets();
 
@@ -130,7 +130,7 @@ float SVGTextContentElement::getSubStringLength(unsigned charnum, unsigned nchar
     return SVGTextQuery(renderer()).subStringLength(charnum, nchars);
 }
 
-FloatPoint SVGTextContentElement::getStartPositionOfChar(unsigned charnum, ExceptionCode& ec) const
+FloatPoint SVGTextContentElement::getStartPositionOfChar(unsigned charnum, ExceptionCode& ec)
 {
     document()->updateLayoutIgnorePendingStylesheets();
 
@@ -142,7 +142,7 @@ FloatPoint SVGTextContentElement::getStartPositionOfChar(unsigned charnum, Excep
     return SVGTextQuery(renderer()).startPositionOfCharacter(charnum);
 }
 
-FloatPoint SVGTextContentElement::getEndPositionOfChar(unsigned charnum, ExceptionCode& ec) const
+FloatPoint SVGTextContentElement::getEndPositionOfChar(unsigned charnum, ExceptionCode& ec)
 {
     document()->updateLayoutIgnorePendingStylesheets();
 
@@ -154,7 +154,7 @@ FloatPoint SVGTextContentElement::getEndPositionOfChar(unsigned charnum, Excepti
     return SVGTextQuery(renderer()).endPositionOfCharacter(charnum);
 }
 
-FloatRect SVGTextContentElement::getExtentOfChar(unsigned charnum, ExceptionCode& ec) const
+FloatRect SVGTextContentElement::getExtentOfChar(unsigned charnum, ExceptionCode& ec)
 {
     document()->updateLayoutIgnorePendingStylesheets();
 
@@ -166,7 +166,7 @@ FloatRect SVGTextContentElement::getExtentOfChar(unsigned charnum, ExceptionCode
     return SVGTextQuery(renderer()).extentOfCharacter(charnum);
 }
 
-float SVGTextContentElement::getRotationOfChar(unsigned charnum, ExceptionCode& ec) const
+float SVGTextContentElement::getRotationOfChar(unsigned charnum, ExceptionCode& ec)
 {
     document()->updateLayoutIgnorePendingStylesheets();
 
@@ -178,13 +178,13 @@ float SVGTextContentElement::getRotationOfChar(unsigned charnum, ExceptionCode&
     return SVGTextQuery(renderer()).rotationOfCharacter(charnum);
 }
 
-int SVGTextContentElement::getCharNumAtPosition(const FloatPoint& point) const
+int SVGTextContentElement::getCharNumAtPosition(const FloatPoint& point)
 {
     document()->updateLayoutIgnorePendingStylesheets();
     return SVGTextQuery(renderer()).characterNumberAtPosition(point);
 }
 
-void SVGTextContentElement::selectSubString(unsigned charnum, unsigned nchars, ExceptionCode& ec) const
+void SVGTextContentElement::selectSubString(unsigned charnum, unsigned nchars, ExceptionCode& ec)
 {
     unsigned numberOfChars = getNumberOfChars();
     if (charnum >= numberOfChars) {
index a844e29..86d7754 100644 (file)
@@ -79,15 +79,15 @@ public:
         LENGTHADJUST_SPACINGANDGLYPHS = SVGLengthAdjustSpacingAndGlyphs
     };
 
-    unsigned getNumberOfChars() const;
-    float getComputedTextLength() const;
-    float getSubStringLength(unsigned charnum, unsigned nchars, ExceptionCode&) const;
-    FloatPoint getStartPositionOfChar(unsigned charnum, ExceptionCode&) const;
-    FloatPoint getEndPositionOfChar(unsigned charnum, ExceptionCode&) const;
-    FloatRect getExtentOfChar(unsigned charnum, ExceptionCode&) const;
-    float getRotationOfChar(unsigned charnum, ExceptionCode&) const;
-    int getCharNumAtPosition(const FloatPoint&) const;
-    void selectSubString(unsigned charnum, unsigned nchars, ExceptionCode&) const;
+    unsigned getNumberOfChars();
+    float getComputedTextLength();
+    float getSubStringLength(unsigned charnum, unsigned nchars, ExceptionCode&);
+    FloatPoint getStartPositionOfChar(unsigned charnum, ExceptionCode&);
+    FloatPoint getEndPositionOfChar(unsigned charnum, ExceptionCode&);
+    FloatRect getExtentOfChar(unsigned charnum, ExceptionCode&);
+    float getRotationOfChar(unsigned charnum, ExceptionCode&);
+    int getCharNumAtPosition(const FloatPoint&);
+    void selectSubString(unsigned charnum, unsigned nchars, ExceptionCode&);
 
     static SVGTextContentElement* elementFromRenderer(RenderObject*);
 
index 9717878..bba10c3 100644 (file)
@@ -93,17 +93,17 @@ SVGElement* SVGTextElement::farthestViewportElement() const
     return SVGTransformable::farthestViewportElement(this);
 }
 
-FloatRect SVGTextElement::getBBox(StyleUpdateStrategy styleUpdateStrategy) const
+FloatRect SVGTextElement::getBBox(StyleUpdateStrategy styleUpdateStrategy)
 {
     return SVGTransformable::getBBox(this, styleUpdateStrategy);
 }
 
-AffineTransform SVGTextElement::getCTM(StyleUpdateStrategy styleUpdateStrategy) const
+AffineTransform SVGTextElement::getCTM(StyleUpdateStrategy styleUpdateStrategy)
 {
     return SVGLocatable::computeCTM(this, SVGLocatable::NearestViewportScope, styleUpdateStrategy);
 }
 
-AffineTransform SVGTextElement::getScreenCTM(StyleUpdateStrategy styleUpdateStrategy) const
+AffineTransform SVGTextElement::getScreenCTM(StyleUpdateStrategy styleUpdateStrategy)
 {
     return SVGLocatable::computeCTM(this, SVGLocatable::ScreenScope, styleUpdateStrategy);
 }
index 73df5f5..f06f208 100644 (file)
@@ -36,9 +36,9 @@ public:
     virtual SVGElement* nearestViewportElement() const;
     virtual SVGElement* farthestViewportElement() const;
 
-    virtual FloatRect getBBox(StyleUpdateStrategy = AllowStyleUpdate) const;
-    virtual AffineTransform getCTM(StyleUpdateStrategy = AllowStyleUpdate) const;
-    virtual AffineTransform getScreenCTM(StyleUpdateStrategy = AllowStyleUpdate) const;
+    virtual FloatRect getBBox(StyleUpdateStrategy = AllowStyleUpdate);
+    virtual AffineTransform getCTM(StyleUpdateStrategy = AllowStyleUpdate);
+    virtual AffineTransform getScreenCTM(StyleUpdateStrategy = AllowStyleUpdate);
     virtual AffineTransform animatedLocalTransform() const;
 
 private:
index 96712a9..748887f 100644 (file)
@@ -92,7 +92,7 @@ PassRefPtr<SVGUseElement> SVGUseElement::create(const QualifiedName& tagName, Do
     return adoptRef(new SVGUseElement(tagName, document));
 }
 
-SVGElementInstance* SVGUseElement::instanceRoot() const
+SVGElementInstance* SVGUseElement::instanceRoot()
 {
     // If there is no element instance tree, force immediate SVGElementInstance tree
     // creation by asking the document to invoke our recalcStyle function - as we can't
@@ -675,7 +675,7 @@ void SVGUseElement::detach()
     detachInstance();
 }
 
-static bool isDirectReference(Node* node)
+static bool isDirectReference(const Node* node)
 {
     return node->hasTagName(SVGNames::pathTag)
            || node->hasTagName(SVGNames::rectTag)
@@ -686,7 +686,7 @@ static bool isDirectReference(Node* node)
            || node->hasTagName(SVGNames::textTag);
 }
 
-void SVGUseElement::toClipPath(Path& path) const
+void SVGUseElement::toClipPath(Path& path)
 {
     ASSERT(path.isEmpty());
 
index b8f528d..89a79a6 100644 (file)
@@ -43,7 +43,7 @@ class SVGUseElement : public SVGStyledTransformableElement,
 public:
     static PassRefPtr<SVGUseElement> create(const QualifiedName&, Document*);
 
-    SVGElementInstance* instanceRoot() const;
+    SVGElementInstance* instanceRoot();
     SVGElementInstance* animatedInstanceRoot() const;
     SVGElementInstance* instanceForShadowTreeElement(Node*) const;
     void invalidateShadowTree();
@@ -69,7 +69,7 @@ private:
     virtual void attach();
     virtual void detach();
 
-    virtual void toClipPath(Path&) const;
+    virtual void toClipPath(Path&);
 
     static void removeDisallowedElementsFromSubtree(Node* element);
 
index 4b520c5..48f9c51 100644 (file)
@@ -419,7 +419,7 @@ void SVGSMILElement::attributeChanged(Attribute* attr, bool preserveDecls)
     }
 }
 
-inline Element* SVGSMILElement::eventBaseFor(const Condition& condition) const
+inline Element* SVGSMILElement::eventBaseFor(const Condition& condition)
 {
     return condition.m_baseID.isEmpty() ? targetElement() : treeScope()->getElementById(condition.m_baseID);
 }
@@ -489,7 +489,7 @@ void SVGSMILElement::reschedule()
         m_timeContainer->schedule(this);
 }
 
-SVGElement* SVGSMILElement::targetElement() const
+SVGElement* SVGSMILElement::targetElement()
 {
     if (m_targetElement)
         return m_targetElement;
@@ -500,7 +500,7 @@ SVGElement* SVGSMILElement::targetElement() const
         return 0;
     
     m_targetElement = static_cast<SVGElement*>(target);
-    document()->accessSVGExtensions()->addAnimationElementToTarget(const_cast<SVGSMILElement*>(this), m_targetElement);
+    document()->accessSVGExtensions()->addAnimationElementToTarget(this, m_targetElement);
     return m_targetElement;
 }
 
index d921e54..7b6847a 100644 (file)
@@ -49,11 +49,11 @@ public:
     virtual void insertedIntoDocument();
     virtual void removedFromDocument();
     
-    virtual bool hasValidAttributeType() const = 0;
+    virtual bool hasValidAttributeType() = 0;
 
     SMILTimeContainer* timeContainer() const { return m_timeContainer.get(); }
 
-    SVGElement* targetElement() const;
+    SVGElement* targetElement();
     void resetTargetElement() { m_targetElement = 0; }
     const QualifiedName& attributeName() const { return m_attributeName; }
 
@@ -154,7 +154,7 @@ private:
     };
     bool parseCondition(const String&, BeginOrEnd beginOrEnd);
     void parseBeginOrEnd(const String&, BeginOrEnd beginOrEnd);
-    Element* eventBaseFor(const Condition&) const;
+    Element* eventBaseFor(const Condition&);
 
     void connectConditions();
     void disconnectConditions();
index 0d5c778..bd2429b 100644 (file)
@@ -1,3 +1,21 @@
+2011-07-20  Peter Kasting  <pkasting@google.com>
+
+        Remove "const" from obviously-non-const accessors
+        https://bugs.webkit.org/show_bug.cgi?id=62302
+
+        Reviewed by Darin Adler.
+
+        * public/WebElement.h:
+        * src/WebElement.cpp:
+        (WebKit::WebElement::innerText): Not const due to call to newly-non-const Element::innerText().
+        * src/WebSearchableFormData.cpp: Use non-const Node*s for code that calls newly non-const accessors.
+        (HTMLNames::IsSelectInDefaultState):
+        (HTMLNames::IsInDefaultState):
+        * src/WebViewImpl.cpp: Use non-const Node*s for code that calls newly non-const accessors.
+        (WebKit::WebViewImpl::setComposition):
+        (WebKit::WebViewImpl::confirmComposition):
+        (WebKit::WebViewImpl::textInputType):
+
 2011-07-20  Ilya Sherman  <isherman@chromium.org>
 
         Remove TODO in WebNode.h from a two-sided patch that has now landed
index 8d51e98..271b4f0 100644 (file)
@@ -57,7 +57,7 @@ class WebNamedNodeMap;
         WEBKIT_API WebString getAttribute(const WebString&) const;
         WEBKIT_API bool setAttribute(const WebString& name, const WebString& value);
         WEBKIT_API WebNamedNodeMap attributes() const;
-        WEBKIT_API WebString innerText() const;
+        WEBKIT_API WebString innerText();
 
         // Returns the language code specified for this element.  This attribute
         // is inherited, so the returned value is drawn from the closest parent
index 34daa34..27c0b1f 100644 (file)
@@ -85,9 +85,9 @@ WebNamedNodeMap WebElement::attributes() const
     return WebNamedNodeMap(m_private->attributes());
 }
 
-WebString WebElement::innerText() const
+WebString WebElement::innerText()
 {
-    return constUnwrap<Element>()->innerText();
+    return unwrap<Element>()->innerText();
 }
 
 WebString WebElement::computeInheritedLanguage() const
index 2366304..611ceed 100644 (file)
@@ -97,14 +97,14 @@ HTMLFormControlElement* GetButtonToActivate(HTMLFormElement* form)
 
 // Returns true if the selected state of all the options matches the default
 // selected state.
-bool IsSelectInDefaultState(const HTMLSelectElement* select)
+bool IsSelectInDefaultState(HTMLSelectElement* select)
 {
     const Vector<Element*>& listItems = select->listItems();
     if (select->multiple() || select->size() > 1) {
         for (Vector<Element*>::const_iterator i(listItems.begin()); i != listItems.end(); ++i) {
             if (!(*i)->hasLocalName(HTMLNames::optionTag))
                 continue;
-            const HTMLOptionElement* optionElement = static_cast<const HTMLOptionElement*>(*i);
+            HTMLOptionElement* optionElement = static_cast<HTMLOptionElement*>(*i);
             if (optionElement->selected() != optionElement->defaultSelected())
                 return false;
         }
@@ -113,11 +113,11 @@ bool IsSelectInDefaultState(const HTMLSelectElement* select)
 
     // The select is rendered as a combobox (called menulist in WebKit). At
     // least one item is selected, determine which one.
-    const HTMLOptionElement* initialSelected = 0;
+    HTMLOptionElement* initialSelected = 0;
     for (Vector<Element*>::const_iterator i(listItems.begin()); i != listItems.end(); ++i) {
         if (!(*i)->hasLocalName(HTMLNames::optionTag))
             continue;
-        const HTMLOptionElement* optionElement = static_cast<const HTMLOptionElement*>(*i);
+        HTMLOptionElement* optionElement = static_cast<HTMLOptionElement*>(*i);
         if (optionElement->defaultSelected()) {
             // The page specified the option to select.
             initialSelected = optionElement;
@@ -126,21 +126,21 @@ bool IsSelectInDefaultState(const HTMLSelectElement* select)
         if (!initialSelected)
             initialSelected = optionElement;
     }
-    return initialSelected ? initialSelected->selected() : true;
+    return !initialSelected || initialSelected->selected();
 }
 
 // Returns true if the form element is in its default state, false otherwise.
 // The default state is the state of the form element on initial load of the
 // page, and varies depending upon the form element. For example, a checkbox is
 // in its default state if the checked state matches the state of the checked attribute.
-bool IsInDefaultState(const HTMLFormControlElement* formElement)
+bool IsInDefaultState(HTMLFormControlElement* formElement)
 {
     if (formElement->hasTagName(HTMLNames::inputTag)) {
-        const HTMLInputElement* inputElement = static_cast<const HTMLInputElement*>(formElement);
+        const HTMLInputElement* inputElement = static_cast<HTMLInputElement*>(formElement);
         if (inputElement->isCheckbox() || inputElement->isRadioButton())
             return inputElement->checked() == inputElement->hasAttribute(checkedAttr);
     } else if (formElement->hasTagName(HTMLNames::selectTag))
-        return IsSelectInDefaultState(static_cast<const HTMLSelectElement*>(formElement));
+        return IsSelectInDefaultState(static_cast<HTMLSelectElement*>(formElement));
     return true;
 }
 
index a5ac80d..05aab43 100644 (file)
@@ -1385,7 +1385,7 @@ bool WebViewImpl::setComposition(
     // node, which doesn't exist any longer.
     PassRefPtr<Range> range = editor->compositionRange();
     if (range) {
-        const Node* node = range->startContainer();
+        Node* node = range->startContainer();
         if (!node || !node->isContentEditable())
             return false;
     }
@@ -1434,7 +1434,7 @@ bool WebViewImpl::confirmComposition(const WebString& text)
     // node, which doesn't exist any longer.
     PassRefPtr<Range> range = editor->compositionRange();
     if (range) {
-        const Node* node = range->startContainer();
+        Node* node = range->startContainer();
         if (!node || !node->isContentEditable())
             return false;
     }
@@ -1483,7 +1483,7 @@ WebTextInputType WebViewImpl::textInputType()
     if (!selection)
         return type;
 
-    const Node* node = selection->start().containerNode();
+    Node* node = selection->start().containerNode();
     if (!node)
         return type;