2006-03-20 Eric Seidel <eseidel@apple.com>
authoreseidel <eseidel@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 20 Mar 2006 23:25:37 +0000 (23:25 +0000)
committereseidel <eseidel@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 20 Mar 2006 23:25:37 +0000 (23:25 +0000)
        Reviewed by darin.

        Remove many, many uses of DeprecatedString.
        Fix various style issues throughout the touched code.

        * bindings/objc/DOM.mm:
        (-[DOMRange _text]):
        * bridge/mac/BrowserExtensionMac.mm:
        (WebCore::BrowserExtensionMac::createNewWindow):
        * bridge/mac/FrameMac.h:
        (WebCore::FrameMac::bridge):
        (WebCore::FrameMac::markedTextRange):
        (WebCore::Mac):
        * bridge/mac/FrameMac.mm:
        (WebCore::FrameMac::openURLRequest):
        (WebCore::FrameMac::searchForLabelsAboveCell):
        (WebCore::FrameMac::searchForLabelsBeforeElement):
        (WebCore::FrameMac::matchLabelsAgainstElement):
        (WebCore::FrameMac::findString):
        (WebCore::FrameMac::submitForm):
        (WebCore::FrameMac::urlSelected):
        (WebCore::FrameMac::objectContentType):
        (WebCore::FrameMac::createPlugin):
        (WebCore::FrameMac::createFrame):
        (WebCore::FrameMac::setTitle):
        (WebCore::FrameMac::setStatusBarText):
        (WebCore::FrameMac::advanceToNextMisspelling):
        (WebCore::FrameMac::userAgent):
        (WebCore::FrameMac::mimeTypeForFileName):
        (WebCore::FrameMac::openURLFromPageCache):
        (WebCore::FrameMac::incomingReferrer):
        (WebCore::FrameMac::attributedString):
        (WebCore::FrameMac::overrideMediaType):
        (WebCore::FrameMac::shouldClose):
        * bridge/mac/WebCoreFrameBridge.mm:
        (-[WebCoreFrameBridge _documentTypeString]):
        (-[WebCoreFrameBridge selectedString]):
        (-[WebCoreFrameBridge stringForRange:]):
        (-[WebCoreFrameBridge advanceToNextMisspelling]):
        (-[WebCoreFrameBridge advanceToNextMisspellingStartingJustBeforeSelection]):
        (-[WebCoreFrameBridge domain]):
        * bridge/mac/WebCoreSettings.mm:
        (-[WebCoreSettings setStandardFontFamily:]):
        (-[WebCoreSettings setFixedFontFamily:]):
        (-[WebCoreSettings setSerifFontFamily:]):
        (-[WebCoreSettings setSansSerifFontFamily:]):
        (-[WebCoreSettings setCursiveFontFamily:]):
        (-[WebCoreSettings setFantasyFontFamily:]):
        (-[WebCoreSettings setUserStyleSheetLocation:]):
        (-[WebCoreSettings setDefaultTextEncoding:]):
        * css/css_valueimpl.cpp:
        (WebCore::quoteStringIfNeeded):
        * css/cssstyleselector.cpp:
        (WebCore::CSSStyleSelector::CSSStyleSelector):
        (WebCore::CSSStyleSelector::setEncodedURL):
        (WebCore::cleanpath):
        (WebCore::checkPseudoState):
        (WebCore::CSSStyleSelector::styleForElement):
        (WebCore::CSSStyleSelector::styleRulesForElement):
        (WebCore::convertToLength):
        (WebCore::colorForCSSValue):
        (WebCore::CSSStyleSelector::applyProperty):
        (WebCore::CSSStyleSelector::getColorFromPrimitiveValue):
        * css/cssstyleselector.h:
        * dom/CDATASection.cpp:
        (WebCore::CDATASection::toString):
        * dom/Document.cpp:
        (WebCore::Document::recalcStyle):
        * dom/Document.h:
        (WebCore::Document::baseTarget):
        (WebCore::Document::setBaseTarget):
        * dom/dom_elementimpl.cpp:
        (WebCore::Element::openTagStartToString):
        * dom/xml_tokenizer.cpp:
        (WebCore::handleElementAttributes):
        (WebCore::XMLTokenizer::startElementNs):
        * editing/SelectionController.cpp:
        (WebCore::SelectionController::type):
        * editing/TextIterator.cpp:
        (WebCore::TextIterator::advance):
        (WebCore::TextIterator::handleTextBox):
        (WebCore::TextIterator::handleNonTextNode):
        (WebCore::TextIterator::exitNode):
        (WebCore::SimplifiedBackwardsTextIterator::advance):
        (WebCore::SimplifiedBackwardsTextIterator::handleTextNode):
        (WebCore::CharacterIterator::advance):
        (WebCore::WordAwareIterator::advance):
        (WebCore::WordAwareIterator::length):
        (WebCore::WordAwareIterator::characters):
        (WebCore::CircularSearchBuffer::CircularSearchBuffer):
        (WebCore::CircularSearchBuffer::append):
        (WebCore::TextIterator::rangeFromLocationAndLength):
        (WebCore::findPlainText):
        * editing/TextIterator.h:
        * html/HTMLFormElement.cpp:
        (WebCore::HTMLFormElement::formData):
        * khtml/ecma/kjs_navigator.cpp:
        (KJS::Navigator::getValueProperty):
        * khtml/ecma/kjs_proxy.cpp:
        (WebCore::KJSProxy::initScriptIfNeeded):
        * khtml/ecma/kjs_window.cpp:
        (KJS::Location::getValueProperty):
        * ksvg2/misc/KCanvasRenderingStyle.cpp:
        (WebCore::KSVGPainterFactory::fillPaintServer):
        (WebCore::KSVGPainterFactory::strokePaintServer):
        * ksvg2/svg/SVGAngle.cpp:
        (SVGAngle::calculate):
        (SVGAngle::setValueAsString):
        (SVGAngle::valueAsString):
        (SVGAngle::convertToSpecifiedUnits):
        (SVGAngle::shortestArcBisector):
        * ksvg2/svg/SVGAngle.h:
        * ksvg2/svg/SVGAnimationElement.cpp:
        (SVGAnimationElement::targetElement):
        (SVGAnimationElement::parseMappedAttribute):
        (SVGAnimationElement::parseClockValue):
        (SVGAnimationElement::targetAttribute):
        (SVGAnimationElement::setTargetAttribute):
        (SVGAnimationElement::detectAnimationMode):
        (SVGAnimationElement::calculateCurrentValueItem):
        (SVGAnimationElement::calculateRelativeTimePercentage):
        * ksvg2/svg/SVGColor.cpp:
        (SVGColor::setRGBColor):
        * ksvg2/svg/SVGDOMImplementation.cpp:
        (svgFeatureSet):
        (SVGDOMImplementation::self):
        (SVGDOMImplementation::hasFeature):
        (SVGDOMImplementation::createDocumentType):
        (SVGDOMImplementation::createDocument):
        * ksvg2/svg/SVGDOMImplementation.h:
        * ksvg2/svg/SVGPaint.cpp:
        (SVGPaint::cssText):
        * ksvg2/svg/SVGPathSegClosePath.h:
        (WebCore::SVGPathSegClosePath::toString):
        * ksvg2/svg/SVGPreserveAspectRatio.cpp:
        (SVGPreserveAspectRatio::parsePreserveAspectRatio):
        (SVGPreserveAspectRatio::getCTM):
        * ksvg2/svg/SVGStopElement.cpp:
        (SVGStopElement::parseMappedAttribute):
        * ksvg2/svg/SVGStyleElement.cpp:
        (SVGStyleElement::childrenChanged):
        * ksvg2/svg/SVGURIReference.cpp:
        (SVGURIReference::getTarget):
        * kwq/ClipboardMac.mm:
        (WebCore::ClipboardMac::setData):
        (WebCore::ClipboardMac::types):
        (WebCore::ClipboardMac::setEffectAllowed):
        (WebCore::cocoaOpFromIEOp):
        (WebCore::IEOpFromCocoaOp):
        (WebCore::ClipboardMac::sourceOperation):
        (WebCore::ClipboardMac::destinationOperation):
        * kwq/KWQCString.cpp:
        (DeprecatedCString::append):
        * kwq/KWQKHTMLSettings.h:
        (KHTMLSettings::stdFontName):
        (KHTMLSettings::fixedFontName):
        (KHTMLSettings::serifFontName):
        (KHTMLSettings::sansSerifFontName):
        (KHTMLSettings::cursiveFontName):
        (KHTMLSettings::fantasyFontName):
        (KHTMLSettings::minFontSize):
        (KHTMLSettings::minLogicalFontSize):
        (KHTMLSettings::mediumFontSize):
        (KHTMLSettings::mediumFixedFontSize):
        (KHTMLSettings::autoLoadImages):
        (KHTMLSettings::isJavaScriptEnabled):
        (KHTMLSettings::JavaScriptCanOpenWindowsAutomatically):
        (KHTMLSettings::isJavaEnabled):
        (KHTMLSettings::isPluginsEnabled):
        (KHTMLSettings::encoding):
        (KHTMLSettings::userStyleSheet):
        (KHTMLSettings::shouldPrintBackgrounds):
        (KHTMLSettings::textAreasAreResizable):
        (KHTMLSettings::setStdFontName):
        (KHTMLSettings::setFixedFontName):
        (KHTMLSettings::setSerifFontName):
        (KHTMLSettings::setSansSerifFontName):
        (KHTMLSettings::setCursiveFontName):
        (KHTMLSettings::setFantasyFontName):
        (KHTMLSettings::setMinFontSize):
        (KHTMLSettings::setMinLogicalFontSize):
        (KHTMLSettings::setMediumFontSize):
        (KHTMLSettings::setMediumFixedFontSize):
        (KHTMLSettings::setAutoLoadImages):
        (KHTMLSettings::setIsJavaScriptEnabled):
        (KHTMLSettings::setIsJavaEnabled):
        (KHTMLSettings::setArePluginsEnabled):
        (KHTMLSettings::setJavaScriptCanOpenWindowsAutomatically):
        (KHTMLSettings::setEncoding):
        (KHTMLSettings::setUserStyleSheet):
        (KHTMLSettings::setShouldPrintBackgrounds):
        (KHTMLSettings::setTextAreasAreResizable):
        * kwq/KWQLoader.mm:
        (KWQServeSynchronousRequest):
        * kwq/KWQTextStream.cpp:
        (QTextStream::QTextStream):
        (QTextStream::operator<<):
        (QTextStream::precision):
        * kwq/KWQTextStream.h:
        * kwq/RenderTreeAsText.cpp:
        (operator<<):
        (quoteAndEscapeNonPrintables):
        (writeTextRun):
        (write):
        (writeSelection):
        * loader/Cache.cpp:
        (WebCore::Cache::init):
        (WebCore::Cache::requestImage):
        (WebCore::Cache::requestStyleSheet):
        (WebCore::Cache::preloadStyleSheet):
        (WebCore::Cache::requestScript):
        (WebCore::Cache::preloadScript):
        (WebCore::Cache::requestXSLStyleSheet):
        (WebCore::Cache::requestXBLDocument):
        (WebCore::Cache::remove):
        * loader/Cache.h:
        * loader/CachedCSSStyleSheet.cpp:
        (WebCore::CachedCSSStyleSheet::error):
        * loader/CachedCSSStyleSheet.h:
        * loader/CachedImage.cpp:
        (WebCore::CachedImage::error):
        * loader/CachedImage.h:
        * loader/CachedObject.h:
        * loader/CachedScript.cpp:
        (WebCore::CachedScript::checkNotify):
        (WebCore::CachedScript::error):
        * loader/CachedScript.h:
        * loader/CachedXBLDocument.cpp:
        (WebCore::CachedXBLDocument::error):
        * loader/CachedXBLDocument.h:
        * loader/CachedXSLStyleSheet.cpp:
        (WebCore::CachedXSLStyleSheet::error):
        * loader/CachedXSLStyleSheet.h:
        * loader/DocLoader.cpp:
        (WebCore::DocLoader::DocLoader):
        (WebCore::DocLoader::~DocLoader):
        (WebCore::DocLoader::needReload):
        (WebCore::DocLoader::requestImage):
        (WebCore::DocLoader::requestStyleSheet):
        (WebCore::DocLoader::requestScript):
        (WebCore::DocLoader::requestXSLStyleSheet):
        (WebCore::DocLoader::requestXBLDocument):
        (WebCore::DocLoader::setAutoloadImages):
        (WebCore::DocLoader::setShowAnimations):
        (WebCore::DocLoader::removeCachedObject):
        * loader/loader.cpp:
        (WebCore::Loader::receivedAllData):
        (WebCore::Loader::receivedResponse):
        * page/Frame.cpp:
        (WebCore::UserStyleSheetLoader::setStyleSheet):
        (WebCore::Frame::didOpenURL):
        (WebCore::Frame::setMetaRefreshEnabled):
        (WebCore::Frame::setAutoloadImages):
        (WebCore::Frame::autoloadImages):
        (WebCore::Frame::clear):
        (WebCore::Frame::receivedFirstData):
        (WebCore::Frame::begin):
        (WebCore::Frame::write):
        (WebCore::Frame::baseTarget):
        (WebCore::Frame::completeURL):
        (WebCore::Frame::scheduleRedirection):
        (WebCore::Frame::scheduleLocationChange):
        (WebCore::Frame::scheduleHistoryNavigation):
        (WebCore::Frame::changeLocation):
        (WebCore::Frame::redirectionTimerFired):
        (WebCore::Frame::encoding):
        (WebCore::Frame::gotoAnchor):
        (WebCore::Frame::setStandardFont):
        (WebCore::Frame::setFixedFont):
        (WebCore::Frame::selectedText):
        (WebCore::Frame::selection):
        (WebCore::Frame::dragCaret):
        (WebCore::Frame::setSelection):
        (WebCore::Frame::setDragCaret):
        (WebCore::Frame::paintCaret):
        (WebCore::Frame::paintDragCaret):
        (WebCore::Frame::urlSelected):
        (WebCore::Frame::requestFrame):
        (WebCore::Frame::requestObject):
        (WebCore::Frame::shouldUsePlugin):
        (WebCore::Frame::loadPlugin):
        (WebCore::Frame::loadSubframe):
        (WebCore::Frame::submitForm):
        (WebCore::Frame::lastModified):
        (WebCore::Frame::reparseConfiguration):
        (WebCore::Frame::shouldChangeSelection):
        (WebCore::Frame::appliedEditing):
        (WebCore::Frame::unappliedEditing):
        (WebCore::Frame::reappliedEditing):
        (WebCore::Frame::executeScript):
        (WebCore::updateState):
        (WebCore::Frame::isCharacterSmartReplaceExempt):
        (WebCore::Frame::openURL):
        (WebCore::Frame::didNotOpenURL):
        (WebCore::Frame::setEncoding):
        (WebCore::Frame::saveInterpreterBuiltins):
        (WebCore::Frame::restoreInterpreterBuiltins):
        (WebCore::Frame::mutableInstances):
        (WebCore::Frame::setPolicyBaseURL):
        (WebCore::Frame::addMetaData):
        (WebCore::Frame::scrollToAnchor):
        (WebCore::Frame::setMediaType):
        (WebCore::Frame::highlightAllMatchesForString):
        (WebCore::Frame::tree):
        * page/Frame.h:
        (WebCore::Frame::instances):
        * page/FramePrivate.h:
        * page/FrameView.cpp:
        (WebCore::FrameViewPrivate::FrameViewPrivate):
        (WebCore::FrameView::FrameView):
        (WebCore::FrameView::~FrameView):
        (WebCore::FrameView::init):
        (WebCore::FrameView::setMarginWidth):
        (WebCore::FrameView::setMarginHeight):
        (WebCore::FrameView::adjustViewSize):
        (WebCore::FrameView::layout):
        (WebCore::FrameView::dispatchDragEvent):
        (WebCore::FrameView::scrollTo):
        (WebCore::FrameView::setMediaType):
        (WebCore::FrameView::mediaType):
        (WebCore::FrameView::setResizingFrameSet):
        (WebCore::FrameView::setIgnoreWheelEvents):
        * page/FrameView.h:
        (WebCore::FrameView::frameWidth):
        (WebCore::FrameView::marginWidth):
        (WebCore::FrameView::marginHeight):
        (WebCore::FrameView::ref):
        (WebCore::FrameView::deref):
        * page/ResourceRequest.h:
        (WebCore::ResourceRequest::contentType):
        (WebCore::ResourceRequest::setContentType):
        * platform/FontFamily.h:
        * platform/Image.h:
        * platform/KURL.cpp:
        (KURL::KURL):
        (KURL::parse):
        (urlcmp):
        (KURL::encode_string):
        (encodeHostname):
        * platform/KURL.h:
        * platform/PlatformString.h:
        (WebCore::String::left):
        (WebCore::String::right):
        * platform/String.cpp:
        (WebCore::String::toInt):
        * platform/StringImpl.cpp:
        (WebCore::StringImpl::toInt):
        * platform/TransferJob.cpp:
        (WebCore::TransferJob::queryMetaData):
        (WebCore::TransferJob::addMetaData):
        * platform/TransferJob.h:
        * platform/mac/ImageMac.mm:
        (WebCore::Image::supportsType):
        * platform/mac/TransferJobMac.mm:
        (WebCore::TransferJob::start):
        * rendering/RenderObject.cpp:
        (WebCore::RenderObject::information):
        * rendering/RenderTextField.cpp:
        (WebCore::RenderTextField::updateFromElement):
        * xml/xmlhttprequest.cpp:
        (WebCore::getMIMEType):
        (WebCore::getCharset):
        (WebCore::XMLHttpRequest::getReadyState):
        (WebCore::XMLHttpRequest::getResponseText):
        (WebCore::XMLHttpRequest::getResponseXML):
        (WebCore::XMLHttpRequest::XMLHttpRequest):
        (WebCore::XMLHttpRequest::~XMLHttpRequest):
        (WebCore::XMLHttpRequest::changeState):
        (WebCore::XMLHttpRequest::callReadyStateChangeListener):
        (WebCore::XMLHttpRequest::urlMatchesDocumentDomain):
        (WebCore::XMLHttpRequest::open):
        (WebCore::XMLHttpRequest::send):
        (WebCore::XMLHttpRequest::abort):
        (WebCore::XMLHttpRequest::overrideMIMEType):
        (WebCore::XMLHttpRequest::setRequestHeader):
        (WebCore::XMLHttpRequest::getRequestHeader):
        (WebCore::XMLHttpRequest::getAllResponseHeaders):
        (WebCore::XMLHttpRequest::getResponseHeader):
        (WebCore::XMLHttpRequest::responseIsXML):
        (WebCore::XMLHttpRequest::getStatus):
        (WebCore::XMLHttpRequest::getStatusText):
        (WebCore::XMLHttpRequest::processSyncLoadResults):
        (WebCore::XMLHttpRequest::receivedAllData):
        (WebCore::XMLHttpRequest::receivedRedirect):
        (WebCore::XMLHttpRequest::receivedData):
        (WebCore::XMLHttpRequest::cancelRequests):
        (WebCore::XMLHttpRequest::detachRequests):
        * xml/xmlhttprequest.h:

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

78 files changed:
WebCore/ChangeLog
WebCore/bindings/objc/DOM.mm
WebCore/bridge/mac/BrowserExtensionMac.mm
WebCore/bridge/mac/FrameMac.h
WebCore/bridge/mac/FrameMac.mm
WebCore/bridge/mac/WebCoreFrameBridge.mm
WebCore/bridge/mac/WebCoreSettings.mm
WebCore/css/css_valueimpl.cpp
WebCore/css/cssstyleselector.cpp
WebCore/css/cssstyleselector.h
WebCore/dom/CDATASection.cpp
WebCore/dom/Document.cpp
WebCore/dom/Document.h
WebCore/dom/dom_elementimpl.cpp
WebCore/dom/xml_tokenizer.cpp
WebCore/editing/SelectionController.cpp
WebCore/editing/TextIterator.cpp
WebCore/editing/TextIterator.h
WebCore/html/HTMLFormElement.cpp
WebCore/khtml/ecma/kjs_navigator.cpp
WebCore/khtml/ecma/kjs_proxy.cpp
WebCore/khtml/ecma/kjs_window.cpp
WebCore/ksvg2/misc/KCanvasRenderingStyle.cpp
WebCore/ksvg2/svg/SVGAngle.cpp
WebCore/ksvg2/svg/SVGAngle.h
WebCore/ksvg2/svg/SVGAnimationElement.cpp
WebCore/ksvg2/svg/SVGColor.cpp
WebCore/ksvg2/svg/SVGDOMImplementation.cpp
WebCore/ksvg2/svg/SVGDOMImplementation.h
WebCore/ksvg2/svg/SVGPaint.cpp
WebCore/ksvg2/svg/SVGPathSegClosePath.h
WebCore/ksvg2/svg/SVGPreserveAspectRatio.cpp
WebCore/ksvg2/svg/SVGStopElement.cpp
WebCore/ksvg2/svg/SVGStyleElement.cpp
WebCore/ksvg2/svg/SVGURIReference.cpp
WebCore/kwq/ClipboardMac.mm
WebCore/kwq/KWQCString.cpp
WebCore/kwq/KWQKHTMLSettings.h
WebCore/kwq/KWQLoader.mm
WebCore/kwq/KWQTextStream.cpp
WebCore/kwq/KWQTextStream.h
WebCore/kwq/RenderTreeAsText.cpp
WebCore/loader/Cache.cpp
WebCore/loader/Cache.h
WebCore/loader/CachedCSSStyleSheet.cpp
WebCore/loader/CachedCSSStyleSheet.h
WebCore/loader/CachedImage.cpp
WebCore/loader/CachedImage.h
WebCore/loader/CachedObject.h
WebCore/loader/CachedScript.cpp
WebCore/loader/CachedScript.h
WebCore/loader/CachedXBLDocument.cpp
WebCore/loader/CachedXBLDocument.h
WebCore/loader/CachedXSLStyleSheet.cpp
WebCore/loader/CachedXSLStyleSheet.h
WebCore/loader/DocLoader.cpp
WebCore/loader/loader.cpp
WebCore/page/Frame.cpp
WebCore/page/Frame.h
WebCore/page/FramePrivate.h
WebCore/page/FrameView.cpp
WebCore/page/FrameView.h
WebCore/page/ResourceRequest.h
WebCore/platform/FontFamily.h
WebCore/platform/Image.h
WebCore/platform/KURL.cpp
WebCore/platform/KURL.h
WebCore/platform/PlatformString.h
WebCore/platform/String.cpp
WebCore/platform/StringImpl.cpp
WebCore/platform/TransferJob.cpp
WebCore/platform/TransferJob.h
WebCore/platform/mac/ImageMac.mm
WebCore/platform/mac/TransferJobMac.mm
WebCore/rendering/RenderObject.cpp
WebCore/rendering/RenderTextField.cpp
WebCore/xml/xmlhttprequest.cpp
WebCore/xml/xmlhttprequest.h

index eda1835..829b40c 100644 (file)
@@ -1,3 +1,394 @@
+2006-03-20  Eric Seidel  <eseidel@apple.com>
+
+        Reviewed by darin.
+
+        Remove many, many uses of DeprecatedString.
+        Fix various style issues throughout the touched code.
+
+        * bindings/objc/DOM.mm:
+        (-[DOMRange _text]):
+        * bridge/mac/BrowserExtensionMac.mm:
+        (WebCore::BrowserExtensionMac::createNewWindow):
+        * bridge/mac/FrameMac.h:
+        (WebCore::FrameMac::bridge):
+        (WebCore::FrameMac::markedTextRange):
+        (WebCore::Mac):
+        * bridge/mac/FrameMac.mm:
+        (WebCore::FrameMac::openURLRequest):
+        (WebCore::FrameMac::searchForLabelsAboveCell):
+        (WebCore::FrameMac::searchForLabelsBeforeElement):
+        (WebCore::FrameMac::matchLabelsAgainstElement):
+        (WebCore::FrameMac::findString):
+        (WebCore::FrameMac::submitForm):
+        (WebCore::FrameMac::urlSelected):
+        (WebCore::FrameMac::objectContentType):
+        (WebCore::FrameMac::createPlugin):
+        (WebCore::FrameMac::createFrame):
+        (WebCore::FrameMac::setTitle):
+        (WebCore::FrameMac::setStatusBarText):
+        (WebCore::FrameMac::advanceToNextMisspelling):
+        (WebCore::FrameMac::userAgent):
+        (WebCore::FrameMac::mimeTypeForFileName):
+        (WebCore::FrameMac::openURLFromPageCache):
+        (WebCore::FrameMac::incomingReferrer):
+        (WebCore::FrameMac::attributedString):
+        (WebCore::FrameMac::overrideMediaType):
+        (WebCore::FrameMac::shouldClose):
+        * bridge/mac/WebCoreFrameBridge.mm:
+        (-[WebCoreFrameBridge _documentTypeString]):
+        (-[WebCoreFrameBridge selectedString]):
+        (-[WebCoreFrameBridge stringForRange:]):
+        (-[WebCoreFrameBridge advanceToNextMisspelling]):
+        (-[WebCoreFrameBridge advanceToNextMisspellingStartingJustBeforeSelection]):
+        (-[WebCoreFrameBridge domain]):
+        * bridge/mac/WebCoreSettings.mm:
+        (-[WebCoreSettings setStandardFontFamily:]):
+        (-[WebCoreSettings setFixedFontFamily:]):
+        (-[WebCoreSettings setSerifFontFamily:]):
+        (-[WebCoreSettings setSansSerifFontFamily:]):
+        (-[WebCoreSettings setCursiveFontFamily:]):
+        (-[WebCoreSettings setFantasyFontFamily:]):
+        (-[WebCoreSettings setUserStyleSheetLocation:]):
+        (-[WebCoreSettings setDefaultTextEncoding:]):
+        * css/css_valueimpl.cpp:
+        (WebCore::quoteStringIfNeeded):
+        * css/cssstyleselector.cpp:
+        (WebCore::CSSStyleSelector::CSSStyleSelector):
+        (WebCore::CSSStyleSelector::setEncodedURL):
+        (WebCore::cleanpath):
+        (WebCore::checkPseudoState):
+        (WebCore::CSSStyleSelector::styleForElement):
+        (WebCore::CSSStyleSelector::styleRulesForElement):
+        (WebCore::convertToLength):
+        (WebCore::colorForCSSValue):
+        (WebCore::CSSStyleSelector::applyProperty):
+        (WebCore::CSSStyleSelector::getColorFromPrimitiveValue):
+        * css/cssstyleselector.h:
+        * dom/CDATASection.cpp:
+        (WebCore::CDATASection::toString):
+        * dom/Document.cpp:
+        (WebCore::Document::recalcStyle):
+        * dom/Document.h:
+        (WebCore::Document::baseTarget):
+        (WebCore::Document::setBaseTarget):
+        * dom/dom_elementimpl.cpp:
+        (WebCore::Element::openTagStartToString):
+        * dom/xml_tokenizer.cpp:
+        (WebCore::handleElementAttributes):
+        (WebCore::XMLTokenizer::startElementNs):
+        * editing/SelectionController.cpp:
+        (WebCore::SelectionController::type):
+        * editing/TextIterator.cpp:
+        (WebCore::TextIterator::advance):
+        (WebCore::TextIterator::handleTextBox):
+        (WebCore::TextIterator::handleNonTextNode):
+        (WebCore::TextIterator::exitNode):
+        (WebCore::SimplifiedBackwardsTextIterator::advance):
+        (WebCore::SimplifiedBackwardsTextIterator::handleTextNode):
+        (WebCore::CharacterIterator::advance):
+        (WebCore::WordAwareIterator::advance):
+        (WebCore::WordAwareIterator::length):
+        (WebCore::WordAwareIterator::characters):
+        (WebCore::CircularSearchBuffer::CircularSearchBuffer):
+        (WebCore::CircularSearchBuffer::append):
+        (WebCore::TextIterator::rangeFromLocationAndLength):
+        (WebCore::findPlainText):
+        * editing/TextIterator.h:
+        * html/HTMLFormElement.cpp:
+        (WebCore::HTMLFormElement::formData):
+        * khtml/ecma/kjs_navigator.cpp:
+        (KJS::Navigator::getValueProperty):
+        * khtml/ecma/kjs_proxy.cpp:
+        (WebCore::KJSProxy::initScriptIfNeeded):
+        * khtml/ecma/kjs_window.cpp:
+        (KJS::Location::getValueProperty):
+        * ksvg2/misc/KCanvasRenderingStyle.cpp:
+        (WebCore::KSVGPainterFactory::fillPaintServer):
+        (WebCore::KSVGPainterFactory::strokePaintServer):
+        * ksvg2/svg/SVGAngle.cpp:
+        (SVGAngle::calculate):
+        (SVGAngle::setValueAsString):
+        (SVGAngle::valueAsString):
+        (SVGAngle::convertToSpecifiedUnits):
+        (SVGAngle::shortestArcBisector):
+        * ksvg2/svg/SVGAngle.h:
+        * ksvg2/svg/SVGAnimationElement.cpp:
+        (SVGAnimationElement::targetElement):
+        (SVGAnimationElement::parseMappedAttribute):
+        (SVGAnimationElement::parseClockValue):
+        (SVGAnimationElement::targetAttribute):
+        (SVGAnimationElement::setTargetAttribute):
+        (SVGAnimationElement::detectAnimationMode):
+        (SVGAnimationElement::calculateCurrentValueItem):
+        (SVGAnimationElement::calculateRelativeTimePercentage):
+        * ksvg2/svg/SVGColor.cpp:
+        (SVGColor::setRGBColor):
+        * ksvg2/svg/SVGDOMImplementation.cpp:
+        (svgFeatureSet):
+        (SVGDOMImplementation::self):
+        (SVGDOMImplementation::hasFeature):
+        (SVGDOMImplementation::createDocumentType):
+        (SVGDOMImplementation::createDocument):
+        * ksvg2/svg/SVGDOMImplementation.h:
+        * ksvg2/svg/SVGPaint.cpp:
+        (SVGPaint::cssText):
+        * ksvg2/svg/SVGPathSegClosePath.h:
+        (WebCore::SVGPathSegClosePath::toString):
+        * ksvg2/svg/SVGPreserveAspectRatio.cpp:
+        (SVGPreserveAspectRatio::parsePreserveAspectRatio):
+        (SVGPreserveAspectRatio::getCTM):
+        * ksvg2/svg/SVGStopElement.cpp:
+        (SVGStopElement::parseMappedAttribute):
+        * ksvg2/svg/SVGStyleElement.cpp:
+        (SVGStyleElement::childrenChanged):
+        * ksvg2/svg/SVGURIReference.cpp:
+        (SVGURIReference::getTarget):
+        * kwq/ClipboardMac.mm:
+        (WebCore::ClipboardMac::setData):
+        (WebCore::ClipboardMac::types):
+        (WebCore::ClipboardMac::setEffectAllowed):
+        (WebCore::cocoaOpFromIEOp):
+        (WebCore::IEOpFromCocoaOp):
+        (WebCore::ClipboardMac::sourceOperation):
+        (WebCore::ClipboardMac::destinationOperation):
+        * kwq/KWQCString.cpp:
+        (DeprecatedCString::append):
+        * kwq/KWQKHTMLSettings.h:
+        (KHTMLSettings::stdFontName):
+        (KHTMLSettings::fixedFontName):
+        (KHTMLSettings::serifFontName):
+        (KHTMLSettings::sansSerifFontName):
+        (KHTMLSettings::cursiveFontName):
+        (KHTMLSettings::fantasyFontName):
+        (KHTMLSettings::minFontSize):
+        (KHTMLSettings::minLogicalFontSize):
+        (KHTMLSettings::mediumFontSize):
+        (KHTMLSettings::mediumFixedFontSize):
+        (KHTMLSettings::autoLoadImages):
+        (KHTMLSettings::isJavaScriptEnabled):
+        (KHTMLSettings::JavaScriptCanOpenWindowsAutomatically):
+        (KHTMLSettings::isJavaEnabled):
+        (KHTMLSettings::isPluginsEnabled):
+        (KHTMLSettings::encoding):
+        (KHTMLSettings::userStyleSheet):
+        (KHTMLSettings::shouldPrintBackgrounds):
+        (KHTMLSettings::textAreasAreResizable):
+        (KHTMLSettings::setStdFontName):
+        (KHTMLSettings::setFixedFontName):
+        (KHTMLSettings::setSerifFontName):
+        (KHTMLSettings::setSansSerifFontName):
+        (KHTMLSettings::setCursiveFontName):
+        (KHTMLSettings::setFantasyFontName):
+        (KHTMLSettings::setMinFontSize):
+        (KHTMLSettings::setMinLogicalFontSize):
+        (KHTMLSettings::setMediumFontSize):
+        (KHTMLSettings::setMediumFixedFontSize):
+        (KHTMLSettings::setAutoLoadImages):
+        (KHTMLSettings::setIsJavaScriptEnabled):
+        (KHTMLSettings::setIsJavaEnabled):
+        (KHTMLSettings::setArePluginsEnabled):
+        (KHTMLSettings::setJavaScriptCanOpenWindowsAutomatically):
+        (KHTMLSettings::setEncoding):
+        (KHTMLSettings::setUserStyleSheet):
+        (KHTMLSettings::setShouldPrintBackgrounds):
+        (KHTMLSettings::setTextAreasAreResizable):
+        * kwq/KWQLoader.mm:
+        (KWQServeSynchronousRequest):
+        * kwq/KWQTextStream.cpp:
+        (QTextStream::QTextStream):
+        (QTextStream::operator<<):
+        (QTextStream::precision):
+        * kwq/KWQTextStream.h:
+        * kwq/RenderTreeAsText.cpp:
+        (operator<<):
+        (quoteAndEscapeNonPrintables):
+        (writeTextRun):
+        (write):
+        (writeSelection):
+        * loader/Cache.cpp:
+        (WebCore::Cache::init):
+        (WebCore::Cache::requestImage):
+        (WebCore::Cache::requestStyleSheet):
+        (WebCore::Cache::preloadStyleSheet):
+        (WebCore::Cache::requestScript):
+        (WebCore::Cache::preloadScript):
+        (WebCore::Cache::requestXSLStyleSheet):
+        (WebCore::Cache::requestXBLDocument):
+        (WebCore::Cache::remove):
+        * loader/Cache.h:
+        * loader/CachedCSSStyleSheet.cpp:
+        (WebCore::CachedCSSStyleSheet::error):
+        * loader/CachedCSSStyleSheet.h:
+        * loader/CachedImage.cpp:
+        (WebCore::CachedImage::error):
+        * loader/CachedImage.h:
+        * loader/CachedObject.h:
+        * loader/CachedScript.cpp:
+        (WebCore::CachedScript::checkNotify):
+        (WebCore::CachedScript::error):
+        * loader/CachedScript.h:
+        * loader/CachedXBLDocument.cpp:
+        (WebCore::CachedXBLDocument::error):
+        * loader/CachedXBLDocument.h:
+        * loader/CachedXSLStyleSheet.cpp:
+        (WebCore::CachedXSLStyleSheet::error):
+        * loader/CachedXSLStyleSheet.h:
+        * loader/DocLoader.cpp:
+        (WebCore::DocLoader::DocLoader):
+        (WebCore::DocLoader::~DocLoader):
+        (WebCore::DocLoader::needReload):
+        (WebCore::DocLoader::requestImage):
+        (WebCore::DocLoader::requestStyleSheet):
+        (WebCore::DocLoader::requestScript):
+        (WebCore::DocLoader::requestXSLStyleSheet):
+        (WebCore::DocLoader::requestXBLDocument):
+        (WebCore::DocLoader::setAutoloadImages):
+        (WebCore::DocLoader::setShowAnimations):
+        (WebCore::DocLoader::removeCachedObject):
+        * loader/loader.cpp:
+        (WebCore::Loader::receivedAllData):
+        (WebCore::Loader::receivedResponse):
+        * page/Frame.cpp:
+        (WebCore::UserStyleSheetLoader::setStyleSheet):
+        (WebCore::Frame::didOpenURL):
+        (WebCore::Frame::setMetaRefreshEnabled):
+        (WebCore::Frame::setAutoloadImages):
+        (WebCore::Frame::autoloadImages):
+        (WebCore::Frame::clear):
+        (WebCore::Frame::receivedFirstData):
+        (WebCore::Frame::begin):
+        (WebCore::Frame::write):
+        (WebCore::Frame::baseTarget):
+        (WebCore::Frame::completeURL):
+        (WebCore::Frame::scheduleRedirection):
+        (WebCore::Frame::scheduleLocationChange):
+        (WebCore::Frame::scheduleHistoryNavigation):
+        (WebCore::Frame::changeLocation):
+        (WebCore::Frame::redirectionTimerFired):
+        (WebCore::Frame::encoding):
+        (WebCore::Frame::gotoAnchor):
+        (WebCore::Frame::setStandardFont):
+        (WebCore::Frame::setFixedFont):
+        (WebCore::Frame::selectedText):
+        (WebCore::Frame::selection):
+        (WebCore::Frame::dragCaret):
+        (WebCore::Frame::setSelection):
+        (WebCore::Frame::setDragCaret):
+        (WebCore::Frame::paintCaret):
+        (WebCore::Frame::paintDragCaret):
+        (WebCore::Frame::urlSelected):
+        (WebCore::Frame::requestFrame):
+        (WebCore::Frame::requestObject):
+        (WebCore::Frame::shouldUsePlugin):
+        (WebCore::Frame::loadPlugin):
+        (WebCore::Frame::loadSubframe):
+        (WebCore::Frame::submitForm):
+        (WebCore::Frame::lastModified):
+        (WebCore::Frame::reparseConfiguration):
+        (WebCore::Frame::shouldChangeSelection):
+        (WebCore::Frame::appliedEditing):
+        (WebCore::Frame::unappliedEditing):
+        (WebCore::Frame::reappliedEditing):
+        (WebCore::Frame::executeScript):
+        (WebCore::updateState):
+        (WebCore::Frame::isCharacterSmartReplaceExempt):
+        (WebCore::Frame::openURL):
+        (WebCore::Frame::didNotOpenURL):
+        (WebCore::Frame::setEncoding):
+        (WebCore::Frame::saveInterpreterBuiltins):
+        (WebCore::Frame::restoreInterpreterBuiltins):
+        (WebCore::Frame::mutableInstances):
+        (WebCore::Frame::setPolicyBaseURL):
+        (WebCore::Frame::addMetaData):
+        (WebCore::Frame::scrollToAnchor):
+        (WebCore::Frame::setMediaType):
+        (WebCore::Frame::highlightAllMatchesForString):
+        (WebCore::Frame::tree):
+        * page/Frame.h:
+        (WebCore::Frame::instances):
+        * page/FramePrivate.h:
+        * page/FrameView.cpp:
+        (WebCore::FrameViewPrivate::FrameViewPrivate):
+        (WebCore::FrameView::FrameView):
+        (WebCore::FrameView::~FrameView):
+        (WebCore::FrameView::init):
+        (WebCore::FrameView::setMarginWidth):
+        (WebCore::FrameView::setMarginHeight):
+        (WebCore::FrameView::adjustViewSize):
+        (WebCore::FrameView::layout):
+        (WebCore::FrameView::dispatchDragEvent):
+        (WebCore::FrameView::scrollTo):
+        (WebCore::FrameView::setMediaType):
+        (WebCore::FrameView::mediaType):
+        (WebCore::FrameView::setResizingFrameSet):
+        (WebCore::FrameView::setIgnoreWheelEvents):
+        * page/FrameView.h:
+        (WebCore::FrameView::frameWidth):
+        (WebCore::FrameView::marginWidth):
+        (WebCore::FrameView::marginHeight):
+        (WebCore::FrameView::ref):
+        (WebCore::FrameView::deref):
+        * page/ResourceRequest.h:
+        (WebCore::ResourceRequest::contentType):
+        (WebCore::ResourceRequest::setContentType):
+        * platform/FontFamily.h:
+        * platform/Image.h:
+        * platform/KURL.cpp:
+        (KURL::KURL):
+        (KURL::parse):
+        (urlcmp):
+        (KURL::encode_string):
+        (encodeHostname):
+        * platform/KURL.h:
+        * platform/PlatformString.h:
+        (WebCore::String::left):
+        (WebCore::String::right):
+        * platform/String.cpp:
+        (WebCore::String::toInt):
+        * platform/StringImpl.cpp:
+        (WebCore::StringImpl::toInt):
+        * platform/TransferJob.cpp:
+        (WebCore::TransferJob::queryMetaData):
+        (WebCore::TransferJob::addMetaData):
+        * platform/TransferJob.h:
+        * platform/mac/ImageMac.mm:
+        (WebCore::Image::supportsType):
+        * platform/mac/TransferJobMac.mm:
+        (WebCore::TransferJob::start):
+        * rendering/RenderObject.cpp:
+        (WebCore::RenderObject::information):
+        * rendering/RenderTextField.cpp:
+        (WebCore::RenderTextField::updateFromElement):
+        * xml/xmlhttprequest.cpp:
+        (WebCore::getMIMEType):
+        (WebCore::getCharset):
+        (WebCore::XMLHttpRequest::getReadyState):
+        (WebCore::XMLHttpRequest::getResponseText):
+        (WebCore::XMLHttpRequest::getResponseXML):
+        (WebCore::XMLHttpRequest::XMLHttpRequest):
+        (WebCore::XMLHttpRequest::~XMLHttpRequest):
+        (WebCore::XMLHttpRequest::changeState):
+        (WebCore::XMLHttpRequest::callReadyStateChangeListener):
+        (WebCore::XMLHttpRequest::urlMatchesDocumentDomain):
+        (WebCore::XMLHttpRequest::open):
+        (WebCore::XMLHttpRequest::send):
+        (WebCore::XMLHttpRequest::abort):
+        (WebCore::XMLHttpRequest::overrideMIMEType):
+        (WebCore::XMLHttpRequest::setRequestHeader):
+        (WebCore::XMLHttpRequest::getRequestHeader):
+        (WebCore::XMLHttpRequest::getAllResponseHeaders):
+        (WebCore::XMLHttpRequest::getResponseHeader):
+        (WebCore::XMLHttpRequest::responseIsXML):
+        (WebCore::XMLHttpRequest::getStatus):
+        (WebCore::XMLHttpRequest::getStatusText):
+        (WebCore::XMLHttpRequest::processSyncLoadResults):
+        (WebCore::XMLHttpRequest::receivedAllData):
+        (WebCore::XMLHttpRequest::receivedRedirect):
+        (WebCore::XMLHttpRequest::receivedData):
+        (WebCore::XMLHttpRequest::cancelRequests):
+        (WebCore::XMLHttpRequest::detachRequests):
+        * xml/xmlhttprequest.h:
+
 2006-03-20  Alexey Proskuryakov  <ap@nypop.com>
 
         Reviewed by Darin.
index 80cb1a0..1649b0e 100644 (file)
@@ -1968,7 +1968,7 @@ static ListenerMap *listenerMap;
 
 - (NSString *)_text
 {
-    return [self _range]->text().deprecatedString().getNSString();
+    return [self _range]->text();
 }
 
 @end
index c3241aa..0cb1f78 100644 (file)
@@ -65,7 +65,7 @@ void BrowserExtensionMac::createNewWindow(const ResourceRequest& request,
     
     const KURL& url = request.url();
 
-    NSString *frameName = request.frameName.length() == 0 ? nil : request.frameName.getNSString();
+    NSString *frameName = request.frameName.isEmpty() ? nil : (NSString*)request.frameName;
     if (frameName) {
         // FIXME: Can't we just use m_frame->findFrame?
         if (WebCoreFrameBridge *bridge = [m_frame->bridge() findFrameNamed:frameName]) {
@@ -106,7 +106,7 @@ void BrowserExtensionMac::createNewWindow(const ResourceRequest& request,
     
     WebCoreFrameBridge *bridge = [page mainFrame];
     if ([bridge impl])
-       [bridge impl]->tree()->setName(request.frameName);
+       [bridge impl]->tree()->setName(AtomicString(request.frameName));
     
     if (partResult)
        *partResult = [bridge impl];
index 0ff36fc..4c46e54 100644 (file)
@@ -122,32 +122,32 @@ public:
     
     void clear();
 
-    void setBridge(WebCoreFrameBridge *p);
-    WebCoreFrameBridge *bridge() const { return _bridge; }
+    void setBridge(WebCoreFrameBridgep);
+    WebCoreFrameBridgebridge() const { return _bridge; }
     virtual void setView(FrameView*);
     virtual void frameDetached();
 
-    virtual bool openURL(const KURL &);
+    virtual bool openURL(const KURL&);
     
     virtual void openURLRequest(const ResourceRequest&);
     virtual void submitForm(const ResourceRequest&);
 
-    DeprecatedString advanceToNextMisspelling(bool startBeforeSelection = false);
+    String advanceToNextMisspelling(bool startBeforeSelection = false);
     
-    virtual void setTitle(const String &);
+    virtual void setTitle(const String&);
     virtual void setStatusBarText(const String&);
 
     virtual void urlSelected(const ResourceRequest&);
 
-    virtual ObjectContentType objectContentType(const KURL& url, const DeprecatedString& mimeType);
-    virtual Plugin* createPlugin(const KURL& url, const DeprecatedStringList& paramNames, const DeprecatedStringList& paramValues, const DeprecatedString& mimeType);
-    virtual Frame* createFrame(const KURL& url, const DeprecatedString& name, RenderPart* renderer, const String& referrer);
+    virtual ObjectContentType objectContentType(const KURL& url, const String& mimeType);
+    virtual Plugin* createPlugin(const KURL& url, const DeprecatedStringList& paramNames, const DeprecatedStringList& paramValues, const String& mimeType);
+    virtual Frame* createFrame(const KURL& url, const String& name, RenderPart* renderer, const String& referrer);
 
     virtual void scheduleClose();
 
     virtual void unfocusWindow();
     
-    void openURLFromPageCache(KWQPageState *state);
+    void openURLFromPageCache(KWQPageStatestate);
 
     virtual void saveDocumentState();
     virtual void restoreDocumentState();
@@ -155,16 +155,16 @@ public:
     virtual void addMessageToConsole(const String& message,  unsigned int lineNumber, const String& sourceID);
     virtual void setDisplaysWithFocusAttributes(bool flag);
     
-    NSView *nextKeyView(Node *startingPoint, KWQSelectionDirection);
-    NSView *nextKeyViewInFrameHierarchy(Node *startingPoint, KWQSelectionDirection);
-    static NSView *nextKeyViewForWidget(Widget *startingPoint, KWQSelectionDirection);
+    NSView* nextKeyView(Node* startingPoint, KWQSelectionDirection);
+    NSView* nextKeyViewInFrameHierarchy(Node* startingPoint, KWQSelectionDirection);
+    static NSView* nextKeyViewForWidget(Widget* startingPoint, KWQSelectionDirection);
     static bool currentEventIsKeyboardOptionTab();
-    static bool handleKeyboardOptionTabInView(NSView *view);
+    static bool handleKeyboardOptionTabInView(NSViewview);
     
     virtual bool tabsToLinks() const;
     virtual bool tabsToAllControls() const;
     
-    static bool currentEventIsMouseDownInWidget(Widget *candidate);
+    static bool currentEventIsMouseDownInWidget(Widgetcandidate);
     
     virtual void runJavaScriptAlert(const String& message);
     virtual bool runJavaScriptConfirm(const String& message);
@@ -179,43 +179,43 @@ public:
 
     virtual void createEmptyDocument();
 
-    static WebCoreFrameBridge *bridgeForWidget(const Widget *);
+    static WebCoreFrameBridge* bridgeForWidget(const Widget*);
     
-    virtual DeprecatedString incomingReferrer() const;
-    virtual DeprecatedString userAgent() const;
+    virtual String incomingReferrer() const;
+    virtual String userAgent() const;
 
-    virtual DeprecatedString mimeTypeForFileName(const DeprecatedString &) const;
+    virtual String mimeTypeForFileName(const String&) const;
 
-    NSImage *selectionImage() const;
-    NSImage *snapshotDragImage(Node *node, NSRect *imageRect, NSRect *elementRect) const;
+    NSImageselectionImage() const;
+    NSImage* snapshotDragImage(Node* node, NSRect* imageRect, NSRect* elementRect) const;
 
     bool dispatchDragSrcEvent(const AtomicString &eventType, const PlatformMouseEvent&) const;
 
-    NSFont *fontForSelection(bool *hasMultipleFonts) const;
-    NSDictionary *fontAttributesForSelectionStart() const;
+    NSFont* fontForSelection(bool* hasMultipleFonts) const;
+    NSDictionaryfontAttributesForSelectionStart() const;
     
     NSWritingDirection baseWritingDirectionForSelectionStart() const;
 
-    virtual void markMisspellingsInAdjacentWords(const VisiblePosition &);
-    virtual void markMisspellings(const SelectionController &);
+    virtual void markMisspellingsInAdjacentWords(const VisiblePosition&);
+    virtual void markMisspellings(const SelectionController&);
 
-    NSFileWrapper *fileWrapperForElement(Element *);
-    NSAttributedString *attributedString(Node *startNode, int startOffset, Node *endNode, int endOffset);
+    NSFileWrapper* fileWrapperForElement(Element*);
+    NSAttributedString* attributedString(Node* startNode, int startOffset, Node* endNode, int endOffset);
 
-    void mouseDown(NSEvent *);
-    void mouseDragged(NSEvent *);
-    void mouseUp(NSEvent *);
-    void mouseMoved(NSEvent *);
-    bool keyEvent(NSEvent *);
-    bool wheelEvent(NSEvent *);
+    void mouseDown(NSEvent*);
+    void mouseDragged(NSEvent*);
+    void mouseUp(NSEvent*);
+    void mouseMoved(NSEvent*);
+    bool keyEvent(NSEvent*);
+    bool wheelEvent(NSEvent*);
 
-    void sendFakeEventsAfterWidgetTracking(NSEvent *initiatingEvent);
+    void sendFakeEventsAfterWidgetTracking(NSEventinitiatingEvent);
 
     virtual bool lastEventIsMouseUp() const;
     void setActivationEventNumber(int num) { _activationEventNumber = num; }
 
     bool dragHysteresisExceeded(float dragLocationX, float dragLocationY) const;
-    bool eventMayStartDrag(NSEvent *) const;
+    bool eventMayStartDrag(NSEvent*) const;
     void dragSourceMovedTo(const PlatformMouseEvent&);
     void dragSourceEndedAt(const PlatformMouseEvent&, NSDragOperation);
 
@@ -226,37 +226,37 @@ public:
     bool tryCopy();
     bool tryPaste();
     
-    bool sendContextMenuEvent(NSEvent *);
+    bool sendContextMenuEvent(NSEvent*);
 
-    virtual bool passMouseDownEventToWidget(Widget *);
+    virtual bool passMouseDownEventToWidget(Widget*);
     virtual bool passSubframeEventToSubframe(MouseEventWithHitTestResults&, Frame* subframePart);
-    virtual bool passWheelEventToChildWidget(Node *);
+    virtual bool passWheelEventToChildWidget(Node*);
     
-    NSString *searchForLabelsAboveCell(RegularExpression *regExp, HTMLTableCellElement *cell);
-    NSString *searchForLabelsBeforeElement(NSArray *labels, Element *element);
-    NSString *matchLabelsAgainstElement(NSArray *labels, Element *element);
+    NSString* searchForLabelsAboveCell(RegularExpression* regExp, HTMLTableCellElement* cell);
+    NSString* searchForLabelsBeforeElement(NSArray* labels, Element* element);
+    NSString* matchLabelsAgainstElement(NSArray* labels, Element* element);
 
-    bool findString(NSString *str, bool forward, bool caseFlag, bool wrapFlag);
+    bool findString(NSStringstr, bool forward, bool caseFlag, bool wrapFlag);
 
     virtual void tokenizerProcessedData();
 
-    virtual DeprecatedString overrideMediaType() const;
+    virtual String overrideMediaType() const;
     
-    NSColor *bodyBackgroundColor() const;
+    NSColorbodyBackgroundColor() const;
     
     WebCoreKeyboardUIMode keyboardUIMode() const;
 
     void didTellBridgeAboutLoad(const String& URL);
     bool haveToldBridgeAboutLoad(const String& URL);
 
-    virtual KJS::Bindings::Instance *getEmbedInstanceForWidget(Widget*);
-    virtual KJS::Bindings::Instance *getObjectInstanceForWidget(Widget*);
-    virtual KJS::Bindings::Instance *getAppletInstanceForWidget(Widget*);
-    void addPluginRootObject(const KJS::Bindings::RootObject *root);
+    virtual KJS::Bindings::InstancegetEmbedInstanceForWidget(Widget*);
+    virtual KJS::Bindings::InstancegetObjectInstanceForWidget(Widget*);
+    virtual KJS::Bindings::InstancegetAppletInstanceForWidget(Widget*);
+    void addPluginRootObject(const KJS::Bindings::RootObjectroot);
     void cleanupPluginRootObjects();
     
-    virtual void registerCommandForUndo(const EditCommandPtr &);
-    virtual void registerCommandForRedo(const EditCommandPtr &);
+    virtual void registerCommandForUndo(const EditCommandPtr&);
+    virtual void registerCommandForRedo(const EditCommandPtr&);
     virtual void clearUndoRedoOperations();
     virtual void issueUndoCommand();
     virtual void issueRedoCommand();
@@ -269,30 +269,30 @@ public:
     virtual void respondToChangedContents();
     virtual bool isContentEditable() const;
     virtual bool shouldChangeSelection(const SelectionController &oldSelection, const SelectionController &newSelection, EAffinity affinity, bool stillSelecting) const;
-    virtual bool shouldBeginEditing(const Range *) const;
-    virtual bool shouldEndEditing(const Range *) const;
+    virtual bool shouldBeginEditing(const Range*) const;
+    virtual bool shouldEndEditing(const Range*) const;
     virtual void didBeginEditing() const;
     virtual void didEndEditing() const;
 
-    KJS::Bindings::RootObject *executionContextForDOM();
-    KJS::Bindings::RootObject *bindingRootObject();
+    KJS::Bindings::RootObjectexecutionContextForDOM();
+    KJS::Bindings::RootObjectbindingRootObject();
     
-    WebScriptObject *windowScriptObject();
-    NPObject *windowScriptNPObject();
+    WebScriptObjectwindowScriptObject();
+    NPObjectwindowScriptNPObject();
     
     virtual void partClearedInBegin();
     
     // Implementation of CSS property -khtml-user-drag == auto
     virtual bool shouldDragAutoNode(Node*, int x, int y) const;
 
-    void setMarkedTextRange(const Range *, NSArray *attributes, NSArray *ranges);
-    virtual Range *markedTextRange() const { return m_markedTextRange.get(); }
+    void setMarkedTextRange(const Range* , NSArray* attributes, NSArray* ranges);
+    virtual RangemarkedTextRange() const { return m_markedTextRange.get(); }
 
     virtual bool canGoBackOrForward(int distance) const;
 
     virtual void didFirstLayout();
     
-    NSMutableDictionary *dashboardRegionsDictionary();
+    NSMutableDictionarydashboardRegionsDictionary();
     void dashboardRegionsChanged();
     
     virtual bool isCharacterSmartReplaceExempt(const QChar &, bool);
@@ -316,14 +316,14 @@ private:
     virtual void handleMouseMoveEvent(const MouseEventWithHitTestResults&);
     virtual void handleMouseReleaseEvent(const MouseEventWithHitTestResults&);
     
-    NSView *mouseDownViewIfStillGood();
+    NSViewmouseDownViewIfStillGood();
 
-    NSView *nextKeyViewInFrame(Node *startingPoint, KWQSelectionDirection);
-    static NSView *documentViewForNode(Node *);
+    NSView* nextKeyViewInFrame(Node* startingPoint, KWQSelectionDirection);
+    static NSView* documentViewForNode(Node*);
     
     bool dispatchCPPEvent(const AtomicString &eventType, ClipboardMac::AccessPolicy policy);
 
-    NSImage *imageFromRect(NSRect rect) const;
+    NSImageimageFromRect(NSRect rect) const;
 
     void freeClipboard();
 
@@ -331,9 +331,9 @@ private:
 
     virtual void detachFromView();
 
-    WebCoreFrameBridge *_bridge;
+    WebCoreFrameBridge_bridge;
     
-    NSView *_mouseDownView;
+    NSView_mouseDownView;
     bool _mouseDownWasInSubframe;
     bool _sendingEventToSubview;
     bool _mouseDownMayStartDrag;
@@ -344,10 +344,10 @@ private:
     float _mouseDownTimestamp;
     int _activationEventNumber;
     
-    static NSEvent *_currentEvent;
+    static NSEvent_currentEvent;
 
-    NSMutableDictionary *_formValuesAboutToBeSubmitted;
-    ObjCDOMElement *_formAboutToBeSubmitted;
+    NSMutableDictionary_formValuesAboutToBeSubmitted;
+    ObjCDOMElement_formAboutToBeSubmitted;
 
     bool _haveUndoRedoOperations;
     
@@ -355,11 +355,11 @@ private:
 
     friend class Frame;
 
-    KJS::Bindings::RootObject *_bindingRoot;  // The root object used for objects
+    KJS::Bindings::RootObject_bindingRoot;  // The root object used for objects
                                             // bound outside the context of a plugin.
     DeprecatedPtrList<KJS::Bindings::RootObject> rootObjects;
-    WebScriptObject *_windowScriptObject;
-    NPObject *_windowScriptNPObject;
+    WebScriptObject_windowScriptObject;
+    NPObject_windowScriptNPObject;
     
     RefPtr<Node> _dragSrc;     // element that may be a drag source, for the current mouse gesture
     bool _dragSrcIsLink;
@@ -372,8 +372,8 @@ private:
     RefPtr<Range> m_markedTextRange;
 };
 
-inline FrameMac *Mac(Frame *frame) { return static_cast<FrameMac *>(frame); }
-inline const FrameMac *Mac(const Frame *frame) { return static_cast<const FrameMac *>(frame); }
+inline FrameMac* Mac(Frame* frame) { return static_cast<FrameMac*>(frame); }
+inline const FrameMac* Mac(const Frame* frame) { return static_cast<const FrameMac*>(frame); }
 
 }
 
index f6f0996..cbb09b0 100644 (file)
@@ -185,7 +185,7 @@ void FrameMac::openURLRequest(const ResourceRequest& request)
             referrer:referrer
               reload:request.reload
          userGesture:userGestureHint()
-              target:request.frameName.getNSString()
+              target:request.frameName
      triggeringEvent:nil
                 form:nil
           formValues:nil];
@@ -283,14 +283,12 @@ NSString *FrameMac::searchForLabelsAboveCell(RegularExpression *regExp, HTMLTabl
             if (aboveCell) {
                 // search within the above cell we found for a match
                 for (Node *n = aboveCell->firstChild(); n; n = n->traverseNextNode(aboveCell)) {
-                    if (n->isTextNode() && n->renderer() && n->renderer()->style()->visibility() == VISIBLE)
-                    {
+                    if (n->isTextNode() && n->renderer() && n->renderer()->style()->visibility() == VISIBLE) {
                         // For each text chunk, run the regexp
                         DeprecatedString nodeString = n->nodeValue().deprecatedString();
                         int pos = regExp->searchRev(nodeString);
-                        if (pos >= 0) {
+                        if (pos >= 0)
                             return nodeString.mid(pos, regExp->matchedLength()).getNSString();
-                        }
                     }
                 }
             }
@@ -342,11 +340,10 @@ NSString *FrameMac::searchForLabelsBeforeElement(NSArray *labels, Element *eleme
                 nodeString = nodeString.right(charsSearchedThreshold - lengthSearched);
             }
             int pos = regExp->searchRev(nodeString);
-            if (pos >= 0) {
+            if (pos >= 0)
                 return nodeString.mid(pos, regExp->matchedLength()).getNSString();
-            } else {
+            else
                 lengthSearched += nodeString.length();
-            }
         }
     }
 
@@ -385,39 +382,35 @@ NSString *FrameMac::matchLabelsAgainstElement(NSArray *labels, Element *element)
         }
     } while (pos != -1);
 
-    if (bestPos != -1) {
+    if (bestPos != -1)
         return name.mid(bestPos, bestLength).getNSString();
-    }
     return nil;
 }
 
 // Searches from the beginning of the document if nothing is selected.
 bool FrameMac::findString(NSString *string, bool forward, bool caseFlag, bool wrapFlag)
 {
-    DeprecatedString target = DeprecatedString::fromNSString(string);
-    if (target.isEmpty()) {
+    String target = string;
+    if (target.isEmpty())
         return false;
-    }
     
     // Initially search from the start (if forward) or end (if backward) of the selection, and search to edge of document.
     RefPtr<Range> searchRange(rangeOfContents(document()));
     if (selection().start().node()) {
-        if (forward) {
+        if (forward)
             setStart(searchRange.get(), VisiblePosition(selection().start(), selection().affinity()));
-        } else {
+        else
             setEnd(searchRange.get(), VisiblePosition(selection().end(), selection().affinity()));
-        }
     }
     RefPtr<Range> resultRange(findPlainText(searchRange.get(), target, forward, caseFlag));
     
     // If we re-found the (non-empty) selected range, then search again starting just past the selected range.
     if (selection().start().node() && *resultRange == *selection().toRange()) {
         searchRange = rangeOfContents(document());
-        if (forward) {
+        if (forward)
             setStart(searchRange.get(), VisiblePosition(selection().end(), selection().affinity()));
-        } else {
+        else
             setEnd(searchRange.get(), VisiblePosition(selection().start(), selection().affinity()));
-        }
         resultRange = findPlainText(searchRange.get(), target, forward, caseFlag);
     }
     
@@ -433,9 +426,8 @@ bool FrameMac::findString(NSString *string, bool forward, bool caseFlag, bool wr
         // this should be a success case instead, so we'll just fall through in that case.
     }
 
-    if (resultRange->collapsed(exception)) {
+    if (resultRange->collapsed(exception))
         return false;
-    }
 
     setSelection(SelectionController(resultRange.get(), DOWNSTREAM));
     revealSelection();
@@ -456,7 +448,7 @@ void FrameMac::submitForm(const ResourceRequest& request)
     // FIXME: Frame targeting is only one of the ways the submission could end up doing something other
     // than replacing this frame's content, so this check is flawed. On the other hand, the check is hardly
     // needed any more now that we reset d->m_submittedFormURL on each mouse or key down event.
-    WebCoreFrameBridge *target = request.frameName.isEmpty() ? _bridge : [_bridge findFrameNamed:request.frameName.getNSString()];
+    WebCoreFrameBridge *target = request.frameName.isEmpty() ? _bridge : [_bridge findFrameNamed:request.frameName];
     Frame *targetPart = [target impl];
     bool willReplaceThisFrame = false;
     for (Frame *p = this; p; p = p->tree()->parent()) {
@@ -476,7 +468,7 @@ void FrameMac::submitForm(const ResourceRequest& request)
                 referrer:[_bridge referrer] 
                   reload:request.reload
              userGesture:true
-                  target:request.frameName.getNSString()
+                  target:request.frameName
          triggeringEvent:_currentEvent
                     form:_formAboutToBeSubmitted
               formValues:_formValuesAboutToBeSubmitted];
@@ -484,9 +476,9 @@ void FrameMac::submitForm(const ResourceRequest& request)
         ASSERT(request.contentType().startsWith("Content-Type: "));
         [_bridge postWithURL:request.url().getNSURL()
                     referrer:[_bridge referrer] 
-                      target:request.frameName.getNSString()
+                      target:request.frameName
                         data:arrayFromFormData(request.postData)
-                 contentType:request.contentType().mid(14).getNSString()
+                 contentType:request.contentType().substring(14)
              triggeringEvent:_currentEvent
                         form:_formAboutToBeSubmitted
                   formValues:_formValuesAboutToBeSubmitted];
@@ -520,7 +512,7 @@ void FrameMac::urlSelected(const ResourceRequest& request)
             referrer:referrer
               reload:request.reload
          userGesture:true
-              target:request.frameName.getNSString()
+              target:request.frameName
      triggeringEvent:_currentEvent
                 form:nil
           formValues:nil];
@@ -528,27 +520,27 @@ void FrameMac::urlSelected(const ResourceRequest& request)
     END_BLOCK_OBJC_EXCEPTIONS;
 }
 
-ObjectContentType FrameMac::objectContentType(const KURL& url, const DeprecatedString& mimeType)
+ObjectContentType FrameMac::objectContentType(const KURL& url, const String& mimeType)
 {
-    return (ObjectContentType)[_bridge determineObjectFromMIMEType:mimeType.getNSString() URL:url.getNSURL()];
+    return (ObjectContentType)[_bridge determineObjectFromMIMEType:mimeType URL:url.getNSURL()];
 }
 
 
-Plugin* FrameMac::createPlugin(const KURL& url, const DeprecatedStringList& paramNames, const DeprecatedStringList& paramValues, const DeprecatedString& mimeType)
+Plugin* FrameMac::createPlugin(const KURL& url, const DeprecatedStringList& paramNames, const DeprecatedStringList& paramValues, const String& mimeType)
 {
     BEGIN_BLOCK_OBJC_EXCEPTIONS;
 
     return new Plugin(new Widget([_bridge viewForPluginWithURL:url.getNSURL()
                                   attributeNames:paramNames.getNSArray()
                                   attributeValues:paramValues.getNSArray()
-                                  MIMEType:mimeType.getNSString()]));
+                                  MIMEType:mimeType]));
 
     END_BLOCK_OBJC_EXCEPTIONS;
     return 0;
 }
 
 
-Frame* FrameMac::createFrame(const KURL& url, const DeprecatedString& name, RenderPart* renderer, const String& referrer)
+Frame* FrameMac::createFrame(const KURL& url, const String& name, RenderPart* renderer, const String& referrer)
 {
     BEGIN_BLOCK_OBJC_EXCEPTIONS;
     
@@ -562,7 +554,7 @@ Frame* FrameMac::createFrame(const KURL& url, const DeprecatedString& name, Rend
         marginHeight = o->getMarginHeight();
     }
 
-    WebCoreFrameBridge *childBridge = [_bridge createChildFrameNamed:name.getNSString()
+    WebCoreFrameBridge *childBridge = [_bridge createChildFrameNamed:name
                                                              withURL:url.getNSURL()
                                                             referrer:referrer 
                                                           renderPart:renderer
@@ -607,21 +599,21 @@ void FrameMac::setView(FrameView *view)
 
 void FrameMac::setTitle(const String &title)
 {
-    DeprecatedString text = title.deprecatedString();
+    String text = title;
     text.replace(QChar('\\'), backslashAsCurrencySymbol());
 
     BEGIN_BLOCK_OBJC_EXCEPTIONS;
-    [_bridge setTitle:text.getNSString()];
+    [_bridge setTitle:text];
     END_BLOCK_OBJC_EXCEPTIONS;
 }
 
 void FrameMac::setStatusBarText(const String& status)
 {
-    DeprecatedString text = status.deprecatedString();
+    String text = status;
     text.replace(QChar('\\'), backslashAsCurrencySymbol());
-
+    
     BEGIN_BLOCK_OBJC_EXCEPTIONS;
-    [_bridge setStatusText:text.getNSString()];
+    [_bridge setStatusText:text];
     END_BLOCK_OBJC_EXCEPTIONS;
 }
 
@@ -641,7 +633,7 @@ void FrameMac::unfocusWindow()
     END_BLOCK_OBJC_EXCEPTIONS;
 }
 
-DeprecatedString FrameMac::advanceToNextMisspelling(bool startBeforeSelection)
+String FrameMac::advanceToNextMisspelling(bool startBeforeSelection)
 {
     int exception = 0;
 
@@ -668,7 +660,7 @@ DeprecatedString FrameMac::advanceToNextMisspelling(bool startBeforeSelection)
     if (!editableNode->isContentEditable()) {
         editableNode = editableNode->nextEditable();
         if (!editableNode) {
-            return DeprecatedString();
+            return String();
         }
         searchRange->setStartBefore(editableNode, exception);
         startedWithSelection = false;   // won't need to wrap
@@ -688,12 +680,12 @@ DeprecatedString FrameMac::advanceToNextMisspelling(bool startBeforeSelection)
     }
     
     if (searchRange->collapsed(exception))
-        return DeprecatedString();       // nothing to search in
+        return String();       // nothing to search in
     
     // Get the spell checker if it is available
     NSSpellChecker *checker = [NSSpellChecker sharedSpellChecker];
     if (checker == nil)
-        return DeprecatedString();
+        return String();
         
     WordAwareIterator it(searchRange.get());
     bool wrapped = false;
@@ -746,7 +738,7 @@ DeprecatedString FrameMac::advanceToNextMisspelling(bool startBeforeSelection)
         }   
     }
     
-    return DeprecatedString();
+    return String();
 }
 
 bool FrameMac::wheelEvent(NSEvent *event)
@@ -853,24 +845,22 @@ void FrameMac::redirectionTimerFired(Timer<Frame>* timer)
     Frame::redirectionTimerFired(timer);
 }
 
-DeprecatedString FrameMac::userAgent() const
+String FrameMac::userAgent() const
 {
     BEGIN_BLOCK_OBJC_EXCEPTIONS;
-    return DeprecatedString::fromNSString([_bridge userAgentForURL:url().getNSURL()]);
+    return [_bridge userAgentForURL:url().getNSURL()];
     END_BLOCK_OBJC_EXCEPTIONS;
          
-    return DeprecatedString();
+    return String();
 }
 
-DeprecatedString FrameMac::mimeTypeForFileName(const DeprecatedString &fileName) const
+String FrameMac::mimeTypeForFileName(const String& fileName) const
 {
-    NSString *ns = fileName.getNSString();
-
     BEGIN_BLOCK_OBJC_EXCEPTIONS;
-    return DeprecatedString::fromNSString([_bridge MIMETypeForPath:ns]);
+    return [_bridge MIMETypeForPath:fileName];
     END_BLOCK_OBJC_EXCEPTIONS;
 
-    return DeprecatedString();
+    return String();
 }
 
 NSView *FrameMac::nextKeyViewInFrame(Node *node, KWQSelectionDirection direction)
@@ -1125,8 +1115,8 @@ void FrameMac::openURLFromPageCache(KWQPageState *state)
     
     // delete old status bar msg's from kjs (if it _was_ activated on last URL)
     if (d->m_bJScriptEnabled) {
-        d->m_kjsStatusBarText = DeprecatedString::null;
-        d->m_kjsDefaultStatusBarText = DeprecatedString::null;
+        d->m_kjsStatusBarText = String();
+        d->m_kjsDefaultStatusBarText = String();
     }
 
     ASSERT(kurl);
@@ -1206,13 +1196,13 @@ void FrameMac::restoreDocumentState()
     END_BLOCK_OBJC_EXCEPTIONS;
 }
 
-DeprecatedString FrameMac::incomingReferrer() const
+String FrameMac::incomingReferrer() const
 {
     BEGIN_BLOCK_OBJC_EXCEPTIONS;
-    return DeprecatedString::fromNSString([_bridge incomingReferrer]);
+    return [_bridge incomingReferrer];
     END_BLOCK_OBJC_EXCEPTIONS;
 
-    return DeprecatedString();
+    return String();
 }
 
 void FrameMac::runJavaScriptAlert(const String& message)
@@ -2219,9 +2209,8 @@ NSAttributedString *FrameMac::attributedString(Node *_start, int startOffset, No
 
     Node * _startNode = _start;
 
-    if (_startNode == nil) {
+    if (!_startNode)
         return nil;
-    }
 
     result = [[[NSMutableAttributedString alloc] init] autorelease];
 
@@ -2268,9 +2257,8 @@ NSAttributedString *FrameMac::attributedString(Node *_start, int startOffset, No
                 if (renderer->isText()) {
                     if (!style->collapseWhiteSpace()) {
                         if (needSpace && !addedSpace) {
-                            if (text.isEmpty() && linkStartLocation == [result length]) {
+                            if (text.isEmpty() && linkStartLocation == [result length])
                                 ++linkStartLocation;
-                            }
                             [result appendAttributedString:pendingStyledSpace];
                         }
                         int runStart = (start == -1) ? 0 : start;
@@ -2296,18 +2284,15 @@ NSAttributedString *FrameMac::attributedString(Node *_start, int startOffset, No
                                 runEnd = kMin(runEnd, box->m_start + box->m_len);
                                 if (runStart >= box->m_start &&
                                     runStart < box->m_start + box->m_len) {
-                                    if (box == textObj->firstTextBox() && box->m_start == runStart && runStart > 0) {
+                                    if (box == textObj->firstTextBox() && box->m_start == runStart && runStart > 0)
                                         needSpace = true; // collapsed space at the start
-                                    }
                                     if (needSpace && !addedSpace) {
                                         if (pendingStyledSpace != nil) {
-                                            if (text.isEmpty() && linkStartLocation == [result length]) {
+                                            if (text.isEmpty() && linkStartLocation == [result length])
                                                 ++linkStartLocation;
-                                            }
                                             [result appendAttributedString:pendingStyledSpace];
-                                        } else {
+                                        } else
                                             text += ' ';
-                                        }
                                     }
                                     DeprecatedString runText = str.mid(runStart, runEnd - runStart);
                                     runText.replace('\n', ' ');
@@ -2854,12 +2839,12 @@ void FrameMac::setBridge(WebCoreFrameBridge *bridge)
     _bridge = bridge;
 }
 
-DeprecatedString FrameMac::overrideMediaType() const
+String FrameMac::overrideMediaType() const
 {
     NSString *overrideType = [_bridge overrideMediaType];
     if (overrideType)
-        return DeprecatedString::fromNSString(overrideType);
-    return DeprecatedString();
+        return overrideType;
+    return String();
 }
 
 void FrameMac::setDisplaysWithFocusAttributes(bool flag)
@@ -3378,10 +3363,10 @@ bool FrameMac::shouldClose()
     if (event->result().isNull())
         return true;
 
-    DeprecatedString text = event->result().deprecatedString();
+    String text = event->result();
     text.replace(QChar('\\'), backslashAsCurrencySymbol());
 
-    return [_bridge runBeforeUnloadConfirmPanelWithMessage:text.getNSString()];
+    return [_bridge runBeforeUnloadConfirmPanelWithMessage:text];
 
     END_BLOCK_OBJC_EXCEPTIONS;
 
index bc836d8..a38c60a 100644 (file)
@@ -766,10 +766,8 @@ static inline WebCoreFrameBridge *bridge(Frame *frame)
     NSString *documentTypeString = nil;
     Document *doc = m_frame->document();
     if (doc) {
-        DocumentType *doctype = doc->realDocType();
-        if (doctype) {
-            documentTypeString = doctype->toString().deprecatedString().getNSString();
-        }
+        if (DocumentType *doctype = doc->realDocType())
+            documentTypeString = doctype->toString();
     }
     return documentTypeString;
 }
@@ -821,16 +819,16 @@ static inline WebCoreFrameBridge *bridge(Frame *frame)
 
 - (NSString *)selectedString
 {
-    DeprecatedString text = m_frame->selectedText();
+    String text = m_frame->selectedText();
     text.replace(QChar('\\'), m_frame->backslashAsCurrencySymbol());
-    return [[text.getNSString() copy] autorelease];
+    return [[(NSString*)text copy] autorelease];
 }
 
 - (NSString *)stringForRange:(DOMRange *)range
 {
-    DeprecatedString text = plainText([range _range]);
+    String text = plainText([range _range]);
     text.replace(QChar('\\'), m_frame->backslashAsCurrencySymbol());
-    return [[text.getNSString() copy] autorelease];
+    return [[(NSString*)text copy] autorelease];
 }
 
 - (void)selectAll
@@ -1228,12 +1226,12 @@ static HTMLFormElement *formElementFromDOMElement(DOMElement *element)
 
 - (NSString *)advanceToNextMisspelling
 {
-    return m_frame->advanceToNextMisspelling().getNSString();
+    return m_frame->advanceToNextMisspelling();
 }
 
 - (NSString *)advanceToNextMisspellingStartingJustBeforeSelection
 {
-    return m_frame->advanceToNextMisspelling(true).getNSString();
+    return m_frame->advanceToNextMisspelling(true);
 }
 
 - (void)unmarkAllMisspellings
@@ -1420,9 +1418,8 @@ static HTMLFormElement *formElementFromDOMElement(DOMElement *element)
 - (NSString *)domain
 {
     Document *doc = m_frame->document();
-    if (doc && doc->isHTMLDocument()) {
-        return doc->domain().deprecatedString().getNSString();
-    }
+    if (doc && doc->isHTMLDocument())
+        return doc->domain();
     return nil;
 }
 
index 1f2636a..df303db 100644 (file)
@@ -80,7 +80,7 @@ using namespace WebCore;
     }
     [standardFontFamily release];
     standardFontFamily = [s copy];
-    settings->setStdFontName(DeprecatedString::fromNSString(s));
+    settings->setStdFontName(s);
     [self _updateAllViews];
 }
 
@@ -91,12 +91,11 @@ using namespace WebCore;
 
 - (void)setFixedFontFamily:(NSString *)s
 {
-    if ([fixedFontFamily isEqualToString:s]) {
+    if ([fixedFontFamily isEqualToString:s])
         return;
-    }
     [fixedFontFamily release];
     fixedFontFamily = [s copy];
-    settings->setFixedFontName(DeprecatedString::fromNSString(s));
+    settings->setFixedFontName(s);
     [self _updateAllViews];
 }
 
@@ -107,12 +106,11 @@ using namespace WebCore;
 
 - (void)setSerifFontFamily:(NSString *)s
 {
-    if ([serifFontFamily isEqualToString:s]) {
+    if ([serifFontFamily isEqualToString:s])
         return;
-    }
     [serifFontFamily release];
     serifFontFamily = [s copy];
-    settings->setSerifFontName(DeprecatedString::fromNSString(s));
+    settings->setSerifFontName(s);
     [self _updateAllViews];
 }
 
@@ -123,12 +121,11 @@ using namespace WebCore;
 
 - (void)setSansSerifFontFamily:(NSString *)s
 {
-    if ([sansSerifFontFamily isEqualToString:s]) {
+    if ([sansSerifFontFamily isEqualToString:s])
         return;
-    }
     [sansSerifFontFamily release];
     sansSerifFontFamily = [s copy];
-    settings->setSansSerifFontName(DeprecatedString::fromNSString(s));
+    settings->setSansSerifFontName(s);
     [self _updateAllViews];
 }
 
@@ -139,12 +136,11 @@ using namespace WebCore;
 
 - (void)setCursiveFontFamily:(NSString *)s
 {
-    if ([cursiveFontFamily isEqualToString:s]) {
+    if ([cursiveFontFamily isEqualToString:s])
         return;
-    }
     [cursiveFontFamily release];
     cursiveFontFamily = [s copy];
-    settings->setCursiveFontName(DeprecatedString::fromNSString(s));
+    settings->setCursiveFontName(s);
     [self _updateAllViews];
 }
 
@@ -155,12 +151,11 @@ using namespace WebCore;
 
 - (void)setFantasyFontFamily:(NSString *)s
 {
-    if ([fantasyFontFamily isEqualToString:s]) {
+    if ([fantasyFontFamily isEqualToString:s])
         return;
-    }
     [fantasyFontFamily release];
     fantasyFontFamily = [s copy];
-    settings->setFantasyFontName(DeprecatedString::fromNSString(s));
+    settings->setFantasyFontName(s);
     [self _updateAllViews];
 }
 
@@ -286,9 +281,8 @@ using namespace WebCore;
 
 - (void)setUserStyleSheetLocation:(NSString *)s
 {
-    if ([userStyleSheetLocation isEqualToString:s]) {
+    if ([userStyleSheetLocation isEqualToString:s])
         return;
-    }
     [userStyleSheetLocation release];
     userStyleSheetLocation = [s copy];
     settings->setUserStyleSheet(DeprecatedString::fromNSString(s));
@@ -324,9 +318,8 @@ using namespace WebCore;
 
 - (void)setDefaultTextEncoding:(NSString *)s
 {
-    if ([defaultTextEncoding isEqualToString:s]) {
+    if ([defaultTextEncoding isEqualToString:s])
         return;
-    }
     [defaultTextEncoding release];
     defaultTextEncoding = [s copy];
     settings->setEncoding(DeprecatedString::fromNSString(s));
index d008ad3..1babd9c 100644 (file)
@@ -83,9 +83,8 @@ static String quoteStringIfNeeded(const String &string)
 {
     // For now, just do this for strings that start with "#" to fix Korean font names that start with "#".
     // Post-Tiger, we should isLegalIdentifier instead after working out all the ancillary issues.
-    if (string[0] != '#') {
+    if (string[0] != '#')
         return string;
-    }
 
     // FIXME: Also need to transform control characters into \ sequences.
     DeprecatedString s = string.deprecatedString();
index c73d713..09ab6a0 100644 (file)
@@ -197,8 +197,7 @@ CSSStyleSheet *CSSStyleSelector::svgSheet = 0;
 static CSSStyleSelector::Encodedurl *currentEncodedURL = 0;
 static PseudoState pseudoState;
 
-CSSStyleSelector::CSSStyleSelector( Document* doc, DeprecatedString userStyleSheet, StyleSheetList *styleSheets,
-                                    bool _strictParsing )
+CSSStyleSelector::CSSStyleSelector(Document* doc, DeprecatedString userStyleSheet, StyleSheetList *styleSheets, bool _strictParsing)
 {
     init();
 
@@ -207,7 +206,7 @@ CSSStyleSelector::CSSStyleSelector( Document* doc, DeprecatedString userStyleShe
     settings = view ? view->frame()->settings() : 0;
     if (!defaultStyle)
         loadDefaultStyle();
-    m_medium = view ? view->mediaType() : DeprecatedString("all");
+    m_mediaType = view ? view->mediaType() : String("all");
 
     m_userStyle = 0;
     m_userSheet = 0;
@@ -218,7 +217,7 @@ CSSStyleSelector::CSSStyleSelector( Document* doc, DeprecatedString userStyleShe
         m_userSheet->parseString(String(userStyleSheet), strictParsing);
 
         m_userStyle = new CSSRuleSet();
-        m_userStyle->addRulesFromSheet( m_userSheet, m_medium );
+        m_userStyle->addRulesFromSheet(m_userSheet, m_mediaType);
     }
 
     // add stylesheets from document
@@ -227,22 +226,23 @@ CSSStyleSelector::CSSStyleSelector( Document* doc, DeprecatedString userStyleShe
     DeprecatedPtrListIterator<StyleSheet> it(styleSheets->styleSheets);
     for (; it.current(); ++it)
         if (it.current()->isCSSStyleSheet() && !it.current()->disabled())
-            m_authorStyle->addRulesFromSheet(static_cast<CSSStyleSheet*>(it.current()), m_medium);
+            m_authorStyle->addRulesFromSheet(static_cast<CSSStyleSheet*>(it.current()), m_mediaType);
 
     m_ruleList = 0;
     m_collectRulesOnly = false;
 }
 
-CSSStyleSelector::CSSStyleSelector( CSSStyleSheet *sheet )
+CSSStyleSelector::CSSStyleSelector(CSSStyleSheet *sheet)
 {
     init();
 
-    if(!defaultStyle) loadDefaultStyle();
+    if (!defaultStyle)
+        loadDefaultStyle();
     FrameView *view = sheet->doc()->view();
-    m_medium =  view ? view->mediaType() : DeprecatedString("all");
+    m_mediaType =  view ? view->mediaType() : String("all");
 
     m_authorStyle = new CSSRuleSet();
-    m_authorStyle->addRulesFromSheet( sheet, m_medium );
+    m_authorStyle->addRulesFromSheet(sheet, m_mediaType);
 }
 
 void CSSStyleSelector::init()
@@ -256,16 +256,16 @@ void CSSStyleSelector::setEncodedURL(const KURL& url)
 {
     KURL u = url;
 
-    u.setQuery( DeprecatedString::null );
-    u.setRef( DeprecatedString::null );
+    u.setQuery(DeprecatedString::null);
+    u.setRef(DeprecatedString::null);
     encodedurl.file = u.url();
     int pos = encodedurl.file.findRev('/');
     encodedurl.path = encodedurl.file;
-    if ( pos > 0 ) {
-        encodedurl.path.truncate( pos );
+    if (pos > 0) {
+        encodedurl.path.truncate(pos);
         encodedurl.path += '/';
     }
-    u.setPath( DeprecatedString::null );
+    u.setPath(DeprecatedString::null);
     encodedurl.host = u.url();
 }
 
@@ -514,16 +514,16 @@ void CSSStyleSelector::initForStyleResolve(Element* e, RenderStyle* defaultParen
 static void cleanpath(DeprecatedString &path)
 {
     int pos;
-    while ( (pos = path.find( "/../" )) != -1 ) {
+    while ((pos = path.find("/../")) != -1) {
         int prev = 0;
-        if ( pos > 0 )
-            prev = path.findRev( "/", pos -1 );
+        if (pos > 0)
+            prev = path.findRev("/", pos -1);
         // don't remove the host, i.e. http://foo.org/../foo.html
         if (prev < 0 || (prev > 3 && path.findRev("://", prev-1) == prev-2))
-            path.remove( pos, 3);
+            path.remove(pos, 3);
         else
             // matching directory found ?
-            path.remove( prev, pos- prev + 3 );
+            path.remove(prev, pos- prev + 3);
     }
     pos = 0;
     
@@ -532,22 +532,22 @@ static void cleanpath(DeprecatedString &path)
     // We don't want to waste a function call on the search for the the anchor
     // in the vast majority of cases where there is no "//" in the path.
     int refPos = -2;
-    while ( (pos = path.find( "//", pos )) != -1) {
+    while ((pos = path.find("//", pos)) != -1) {
         if (refPos == -2)
             refPos = path.find("#");
         if (refPos > 0 && pos >= refPos)
             break;
         
-        if ( pos == 0 || path[pos-1] != ':' )
-            path.remove( pos, 1 );
+        if (pos == 0 || path[pos-1] != ':')
+            path.remove(pos, 1);
         else
             pos += 2;
     }
-    while ( (pos = path.find( "/./" )) != -1)
-        path.remove( pos, 2 );
+    while ((pos = path.find("/./")) != -1)
+        path.remove(pos, 2);
 }
 
-static void checkPseudoState( Element *e, bool checkVisited = true )
+static void checkPseudoState(Element *e, bool checkVisited = true)
 {
     if (!e->isLink()) {
         pseudoState = PseudoNone;
@@ -567,14 +567,14 @@ static void checkPseudoState( Element *e, bool checkVisited = true )
     
     QConstString cu(attr.unicode(), attr.length());
     DeprecatedString u = cu.string();
-    if ( !u.contains("://") ) {
-        if ( u[0] == '/' )
+    if (!u.contains("://")) {
+        if (u[0] == '/')
             u.prepend(currentEncodedURL->host);
-        else if ( u[0] == '#' )
+        else if (u[0] == '#')
             u.prepend(currentEncodedURL->file);
         else
             u.prepend(currentEncodedURL->path);
-        cleanpath( u );
+        cleanpath(u);
     }
     pseudoState = historyContains(u) ? PseudoVisited : PseudoLink;
 }
@@ -732,7 +732,7 @@ RenderStyle* CSSStyleSelector::styleForElement(Element* e, RenderStyle* defaultP
         matchRules(defaultQuirksStyle, firstUARule, lastUARule);
     
     // 3. If our medium is print, then we match rules from the print sheet.
-    if (m_medium == "print")
+    if (m_mediaType == "print")
         matchRules(defaultPrintStyle, firstUARule, lastUARule);
 
     // 4. Now we check user sheet rules.
@@ -1046,7 +1046,7 @@ RefPtr<CSSRuleList> CSSStyleSelector::styleRulesForElement(Element* e, bool auth
             matchRules(defaultQuirksStyle, firstUARule, lastUARule);
         
         // If our medium is print, then we match rules from the print sheet.
-        if (m_medium == "print")
+        if (m_mediaType == "print")
             matchRules(defaultPrintStyle, firstUARule, lastUARule);
 
         // Now we check user sheet rules.
@@ -1618,11 +1618,11 @@ void CSSRuleSet::addRulesFromSheet(CSSStyleSheet *sheet, const String &medium)
 // -------------------------------------------------------------------------------------
 // this is mostly boring stuff on how to apply a certain rule to the renderstyle...
 
-static Length convertToLength( CSSPrimitiveValue *primitiveValue, RenderStyle *style, bool *ok = 0 )
+static Length convertToLength(CSSPrimitiveValue *primitiveValue, RenderStyle *style, bool *ok = 0)
 {
     Length l;
-    if ( !primitiveValue ) {
-        if ( ok )
+    if (!primitiveValue) {
+        if (ok)
             *ok = false;
     } else {
         int type = primitiveValue->primitiveType();
@@ -1632,7 +1632,7 @@ static Length convertToLength( CSSPrimitiveValue *primitiveValue, RenderStyle *s
             l = Length(int(primitiveValue->getFloatValue(CSSPrimitiveValue::CSS_PERCENTAGE)), Percent);
         else if(type == CSSPrimitiveValue::CSS_NUMBER)
             l = Length(int(primitiveValue->getFloatValue(CSSPrimitiveValue::CSS_NUMBER)*100), Percent);
-        else if ( ok )
+        else if (ok)
             *ok = false;
     }
     return l;
@@ -1697,13 +1697,13 @@ static const colorMap cmap[] = {
 };
 
 
-static Color colorForCSSValue( int css_value )
+static Color colorForCSSValue(int css_value)
 {
     // try the regular ones first
     const colorMap *col = cmap;
-    while ( col->css_value && col->css_value != css_value )
+    while (col->css_value && col->css_value != css_value)
         ++col;
-    if ( col->css_value )
+    if (col->css_value)
         return col->color;
 
     return Color();
@@ -1748,7 +1748,7 @@ void CSSStyleSelector::applyDeclarations(bool applyFirst, bool isImportant,
     }
 }
 
-void CSSStyleSelector::applyProperty( int id, CSSValue *value )
+void CSSStyleSelector::applyProperty(int id, CSSValue *value)
 {
     CSSPrimitiveValue *primitiveValue = 0;
     if(value->isPrimitiveValue()) primitiveValue = static_cast<CSSPrimitiveValue *>(value);
@@ -2027,7 +2027,7 @@ void CSSStyleSelector::applyProperty( int id, CSSValue *value )
         HANDLE_INHERIT_AND_INITIAL(listStylePosition, ListStylePosition)
         if (!primitiveValue) return;
         if (primitiveValue->getIdent())
-            style->setListStylePosition( (EListStylePosition) (primitiveValue->getIdent() - CSS_VAL_OUTSIDE) );
+            style->setListStylePosition((EListStylePosition) (primitiveValue->getIdent() - CSS_VAL_OUTSIDE));
         return;
     }
 
@@ -2039,7 +2039,7 @@ void CSSStyleSelector::applyProperty( int id, CSSValue *value )
         {
             EListStyleType t;
             int id = primitiveValue->getIdent();
-            if ( id == CSS_VAL_NONE) { // important!!
+            if (id == CSS_VAL_NONE) { // important!!
               t = LNONE;
             } else {
               t = EListStyleType(id - CSS_VAL_DISC);
@@ -2143,7 +2143,7 @@ void CSSStyleSelector::applyProperty( int id, CSSValue *value )
             p = AbsolutePosition; break;
         case CSS_VAL_FIXED:
             {
-                if ( view )
+                if (view)
                     view->useSlowRepaints();
                 p = FixedPosition;
                 break;
@@ -2158,16 +2158,16 @@ void CSSStyleSelector::applyProperty( int id, CSSValue *value )
     case CSS_PROP_TABLE_LAYOUT: {
         HANDLE_INHERIT_AND_INITIAL(tableLayout, TableLayout)
 
-        if ( !primitiveValue->getIdent() )
+        if (!primitiveValue->getIdent())
             return;
 
         ETableLayout l = RenderStyle::initialTableLayout();
-        switch( primitiveValue->getIdent() ) {
+        switch(primitiveValue->getIdent()) {
             case CSS_VAL_FIXED:
                 l = TFIXED;
                 // fall through
             case CSS_VAL_AUTO:
-                style->setTableLayout( l );
+                style->setTableLayout(l);
             default:
                 break;
         }
@@ -2212,15 +2212,15 @@ void CSSStyleSelector::applyProperty( int id, CSSValue *value )
     {
         HANDLE_INHERIT_AND_INITIAL(visibility, Visibility)
 
-        switch( primitiveValue->getIdent() ) {
+        switch(primitiveValue->getIdent()) {
         case CSS_VAL_HIDDEN:
-            style->setVisibility( HIDDEN );
+            style->setVisibility(HIDDEN);
             break;
         case CSS_VAL_VISIBLE:
-            style->setVisibility( VISIBLE );
+            style->setVisibility(VISIBLE);
             break;
         case CSS_VAL_COLLAPSE:
-            style->setVisibility( COLLAPSE );
+            style->setVisibility(COLLAPSE);
         default:
             break;
         }
@@ -2795,12 +2795,12 @@ void CSSStyleSelector::applyProperty( int id, CSSValue *value )
           int type = primitiveValue->primitiveType();
           Length l;
           if(type > CSSPrimitiveValue::CSS_PERCENTAGE && type < CSSPrimitiveValue::CSS_DEG)
-            l = Length(primitiveValue->computeLength(style), Fixed );
+            l = Length(primitiveValue->computeLength(style), Fixed);
           else if(type == CSSPrimitiveValue::CSS_PERCENTAGE)
-            l = Length( int( primitiveValue->getFloatValue(CSSPrimitiveValue::CSS_PERCENTAGE) ), Percent );
+            l = Length(int(primitiveValue->getFloatValue(CSSPrimitiveValue::CSS_PERCENTAGE)), Percent);
 
-          style->setVerticalAlign( LENGTH );
-          style->setVerticalAlignLength( l );
+          style->setVerticalAlign(LENGTH);
+          style->setVerticalAlignLength(l);
         }
         break;
 
@@ -2920,7 +2920,7 @@ void CSSStyleSelector::applyProperty( int id, CSSValue *value )
         Length lineHeight;
         int type = primitiveValue->primitiveType();
         if (primitiveValue->getIdent() == CSS_VAL_NORMAL)
-            lineHeight = Length( -100, Percent );
+            lineHeight = Length(-100, Percent);
         else if (type > CSSPrimitiveValue::CSS_PERCENTAGE && type < CSSPrimitiveValue::CSS_DEG) {
             double multiplier = 1.0;
             // Scale for the font zoom factor only for types other than "em" and "ex", since those are
@@ -2930,7 +2930,7 @@ void CSSStyleSelector::applyProperty( int id, CSSValue *value )
             }
             lineHeight = Length(primitiveValue->computeLength(style, multiplier), Fixed);
         } else if (type == CSSPrimitiveValue::CSS_PERCENTAGE)
-            lineHeight = Length( (style->fontSize() * int(primitiveValue->getFloatValue(CSSPrimitiveValue::CSS_PERCENTAGE)) ) / 100, Fixed );
+            lineHeight = Length((style->fontSize() * int(primitiveValue->getFloatValue(CSSPrimitiveValue::CSS_PERCENTAGE))) / 100, Fixed);
         else if (type == CSSPrimitiveValue::CSS_NUMBER)
             lineHeight = Length(int(primitiveValue->getFloatValue(CSSPrimitiveValue::CSS_NUMBER)*100), Percent);
         else
@@ -2945,7 +2945,7 @@ void CSSStyleSelector::applyProperty( int id, CSSValue *value )
         HANDLE_INHERIT_AND_INITIAL(textAlign, TextAlign)
         if (!primitiveValue) return;
         if (primitiveValue->getIdent())
-            style->setTextAlign( (ETextAlign) (primitiveValue->getIdent() - CSS_VAL__KHTML_AUTO) );
+            style->setTextAlign((ETextAlign) (primitiveValue->getIdent() - CSS_VAL__KHTML_AUTO));
         return;
     }
 
@@ -2971,18 +2971,18 @@ void CSSStyleSelector::applyProperty( int id, CSSValue *value )
         } else if (isInitial) {
             hasClip = false;
             top = right = bottom = left = Length();
-        } else if ( !primitiveValue ) {
+        } else if (!primitiveValue) {
             break;
-        } else if ( primitiveValue->primitiveType() == CSSPrimitiveValue::CSS_RECT ) {
+        } else if (primitiveValue->primitiveType() == CSSPrimitiveValue::CSS_RECT) {
             RectImpl *rect = primitiveValue->getRectValue();
-            if ( !rect )
+            if (!rect)
                 break;
             top = convertToLength(rect->top(), style);
             right = convertToLength(rect->right(), style);
             bottom = convertToLength(rect->bottom(), style);
             left = convertToLength(rect->left(), style);
 
-        } else if ( primitiveValue->getIdent() != CSS_VAL_AUTO ) {
+        } else if (primitiveValue->getIdent() != CSS_VAL_AUTO) {
             break;
         }
         style->setClip(top, right, bottom, left);
@@ -3068,8 +3068,8 @@ void CSSStyleSelector::applyProperty( int id, CSSValue *value )
             CSSValue *item = list->item(i);
             if(!item->isPrimitiveValue()) continue;
             CSSPrimitiveValue *val = static_cast<CSSPrimitiveValue *>(item);
-            DeprecatedString face;
-            if( val->primitiveType() == CSSPrimitiveValue::CSS_STRING )
+            AtomicString face;
+            if(val->primitiveType() == CSSPrimitiveValue::CSS_STRING)
                 face = static_cast<FontFamilyValue *>(val)->fontName();
             else if (val->primitiveType() == CSSPrimitiveValue::CSS_IDENT) {
                 switch (val->getIdent()) {
@@ -3099,7 +3099,7 @@ void CSSStyleSelector::applyProperty( int id, CSSValue *value )
                 }
             }
     
-            if ( !face.isEmpty() ) {
+            if (!face.isEmpty()) {
                 if (!currFamily) {
                     // Filling in the first family.
                     firstFamily.setFamily(face);
@@ -3291,10 +3291,10 @@ void CSSStyleSelector::applyProperty( int id, CSSValue *value )
             style->setLineHeight(RenderStyle::initialLineHeight());
             if (style->setFontDescription(fontDescription))
                 fontDirty = true;
-        } else if ( value->isFontValue() ) {
+        } else if (value->isFontValue()) {
             FontValue *font = static_cast<FontValue *>(value);
-            if ( !font->style || !font->variant || !font->weight ||
-                 !font->size || !font->lineHeight || !font->family )
+            if (!font->style || !font->variant || !font->weight ||
+                 !font->size || !font->lineHeight || !font->family)
                 return;
             applyProperty(CSS_PROP_FONT_STYLE, font->style.get());
             applyProperty(CSS_PROP_FONT_VARIANT, font->variant.get());
@@ -3549,7 +3549,7 @@ void CSSStyleSelector::applyProperty( int id, CSSValue *value )
             if (item->color) {
                 int ident = item->color->getIdent();
                 if (ident)
-                    col = colorForCSSValue( ident );
+                    col = colorForCSSValue(ident);
                 else if (item->color->primitiveType() == CSSPrimitiveValue::CSS_RGBCOLOR)
                     col.setRgb(item->color->getRGBColorValue());
             }
@@ -4265,8 +4265,8 @@ Color CSSStyleSelector::getColorFromPrimitiveValue(CSSPrimitiveValue* primitiveV
         else if (ident == CSS_VAL__KHTML_ACTIVELINK)
             col = element->getDocument()->activeLinkColor();
         else
-            col = colorForCSSValue( ident );
-    } else if ( primitiveValue->primitiveType() == CSSPrimitiveValue::CSS_RGBCOLOR )
+            col = colorForCSSValue(ident);
+    } else if (primitiveValue->primitiveType() == CSSPrimitiveValue::CSS_RGBCOLOR)
         col.setRgb(primitiveValue->getRGBColorValue());
     return col;    
 }
index 2b2cf40..c293b6e 100644 (file)
@@ -199,7 +199,7 @@ public:
         CSSRuleList* m_ruleList;
         bool m_collectRulesOnly;
 
-        DeprecatedString m_medium;
+        String m_mediaType;
 
         RenderStyle::PseudoId dynamicPseudo;
         
index 29f884b..3678ee1 100644 (file)
@@ -70,7 +70,7 @@ Text *CDATASection::createNew(StringImpl *_str)
 String CDATASection::toString() const
 {
     // FIXME: substitute entity references as needed!
-    return String("<![CDATA[") + nodeValue() + "]]>";
+    return "<![CDATA[" + nodeValue() + "]]>";
 }
 
 } // namespace WebCore
index 9779082..9153794 100644 (file)
@@ -805,7 +805,7 @@ void Document::recalcStyle(StyleChange change)
             const KHTMLSettings *settings = m_view->frame()->settings();
             if (printing() && !settings->shouldPrintBackgrounds())
                 _style->setForceBackgroundsToWhite(true);
-            DeprecatedString stdfont = settings->stdFontName();
+            const AtomicString& stdfont = settings->stdFontName();
             if (!stdfont.isEmpty()) {
                 fontDescription.firstFamily().setFamily(stdfont);
                 fontDescription.firstFamily().appendFamily(0);
index 6448dda..43d01ce 100644 (file)
@@ -272,8 +272,8 @@ public:
     DeprecatedString baseURL() const { return m_baseURL.isEmpty() ? URL() : m_baseURL; }
     void setBaseURL(const DeprecatedString& baseURL) { m_baseURL = baseURL; }
 
-    DeprecatedString baseTarget() const { return m_baseTarget; }
-    void setBaseTarget(const DeprecatedString& baseTarget) { m_baseTarget = baseTarget; }
+    String baseTarget() const { return m_baseTarget; }
+    void setBaseTarget(const String& baseTarget) { m_baseTarget = baseTarget; }
 
     DeprecatedString completeURL(const DeprecatedString &);
     String completeURL(const String&);
@@ -541,7 +541,7 @@ protected:
     Tokenizer *m_tokenizer;
     DeprecatedString m_url;
     DeprecatedString m_baseURL;
-    DeprecatedString m_baseTarget;
+    String m_baseTarget;
 
     RefPtr<DocumentType> m_docType;
     RefPtr<DOMImplementation> m_implementation;
index 11ee991..68afc77 100644 (file)
@@ -643,7 +643,7 @@ void Element::dispatchAttrAdditionEvent(Attribute *attr)
 
 String Element::openTagStartToString() const
 {
-    String result = String("<") + nodeName();
+    String result = "<" + nodeName();
 
     NamedAttrMap *attrMap = attributes(true);
 
index 8b01712..1996f5d 100644 (file)
@@ -405,7 +405,7 @@ static inline void handleElementAttributes(Element *newElement, const xmlChar **
         String attrValue = toQString(attributes[i].value, valueLength);
         String attrPrefix = toQString(attributes[i].prefix);
         String attrURI = attrPrefix.isEmpty() ? String() : toQString(attributes[i].uri);
-        String attrQName = attrPrefix.isEmpty() ? attrLocalName : attrPrefix + String(":") + attrLocalName;
+        String attrQName = attrPrefix.isEmpty() ? attrLocalName : attrPrefix + ":" + attrLocalName;
         
         newElement->setAttributeNS(attrURI, attrQName, attrValue, ec);
         if (ec) // exception setting attributes
@@ -425,7 +425,7 @@ void XMLTokenizer::startElementNs(const xmlChar *xmlLocalName, const xmlChar *xm
     String localName = toQString(xmlLocalName);
     String uri = toQString(xmlURI);
     String prefix = toQString(xmlPrefix);
-    String qName = prefix.isEmpty() ? localName : prefix + DeprecatedString::fromLatin1(":") + localName;
+    String qName = prefix.isEmpty() ? localName : prefix + ":" + localName;
     
     if (m_parsingFragment && uri.isEmpty()) {
         if (!prefix.isEmpty())
index 8547706..5a5c316 100644 (file)
@@ -674,11 +674,11 @@ void SelectionController::setNeedsLayout(bool flag)
 String SelectionController::type() const
 {
     if (isNone())
-        return String("None");
+        return "None";
     else if (isCaret())
-        return String("Caret");
+        return "Caret";
     else
-        return String("Range");
+        return "Range";
 }
 
 String SelectionController::toString() const
index cd7c0ef..b494727 100644 (file)
@@ -50,19 +50,19 @@ const unsigned short nonBreakingSpace = 0xA0;
 // but no more.
 class CircularSearchBuffer {
 public:
-    CircularSearchBuffer(const DeprecatedString &target, bool isCaseSensitive);
+    CircularSearchBuffer(const String& target, bool isCaseSensitive);
     ~CircularSearchBuffer() { fastFree(m_buffer); }
 
     void clear() { m_cursor = m_buffer; m_bufferFull = false; }
     void append(int length, const QChar *characters);
-    void append(const QChar &);
+    void append(const QChar&);
 
     int neededCharacters() const;
     bool isMatch() const;
     int length() const { return m_target.length(); }
 
 private:
-    DeprecatedString m_target;
+    String m_target;
     bool m_isCaseSensitive;
 
     QChar *m_buffer;
@@ -145,9 +145,8 @@ void TextIterator::advance()
     // handle remembered text box
     if (m_textBox) {
         handleTextBox();
-        if (m_positionNode) {
+        if (m_positionNode)
             return;
-        }
     }
 
     while (m_node && m_node != m_pastEndNode) {
@@ -156,19 +155,15 @@ void TextIterator::advance()
             RenderObject *renderer = m_node->renderer();
             if (renderer && renderer->isText() && m_node->nodeType() == Node::TEXT_NODE) {
                 // FIXME: What about CDATA_SECTION_NODE?
-                if (renderer->style()->visibility() == VISIBLE) {
+                if (renderer->style()->visibility() == VISIBLE)
                     m_handledNode = handleTextNode();
-                }
             } else if (renderer && (renderer->isImage() || renderer->isWidget())) {
-                if (renderer->style()->visibility() == VISIBLE) {
+                if (renderer->style()->visibility() == VISIBLE)
                     m_handledNode = handleReplacedElement();
-                }
-            } else {
+            } else
                 m_handledNode = handleNonTextNode();
-            }
-            if (m_positionNode) {
+            if (m_positionNode)
                 return;
-            }
         }
 
         // find a new current node to handle in depth-first manner,
@@ -199,9 +194,8 @@ void TextIterator::advance()
         m_handledChildren = false;
 
         // how would this ever be?
-        if (m_positionNode) {
+        if (m_positionNode)
             return;
-        }
     }
 }
 
@@ -301,9 +295,8 @@ void TextIterator::handleTextBox()
                 m_offset = runStart + 1;
             } else {
                 int subrunEnd = str.find('\n', runStart);
-                if (subrunEnd == -1 || subrunEnd > runEnd) {
+                if (subrunEnd == -1 || subrunEnd > runEnd)
                     subrunEnd = runEnd;
-                }
 
                 m_offset = subrunEnd;
 
@@ -320,15 +313,13 @@ void TextIterator::handleTextBox()
 
             // If we are doing a subrun that doesn't go to the end of the text box,
             // come back again to finish handling this text box; don't advance to the next one.
-            if (m_positionEndOffset < textBoxEnd) {
+            if (m_positionEndOffset < textBoxEnd)
                 return;
-            }
 
             // Advance and return
             int nextRunStart = nextTextBox ? nextTextBox->m_start : str.length();
-            if (nextRunStart > runEnd) {
+            if (nextRunStart > runEnd)
                 m_lastTextNodeEndedWithCollapsedSpace = true; // collapsed space between runs or at the end
-            }
             m_textBox = nextTextBox;
             if (renderer->containsReversedText())
                 ++m_sortedTextBoxesPosition;
@@ -366,9 +357,8 @@ bool TextIterator::handleNonTextNode()
     if (m_node->hasTagName(brTag)) {
         emitCharacter('\n', m_node->parentNode(), m_node, 0, 1);
     } else if (m_node->hasTagName(tdTag) || m_node->hasTagName(thTag)) {
-        if (m_lastCharacter != '\n' && m_lastTextNode) {
+        if (m_lastCharacter != '\n' && m_lastTextNode)
             emitCharacter('\t', m_lastTextNode->parentNode(), m_lastTextNode, 0, 1);
-        }
     } else if (m_node->hasTagName(blockquoteTag) || m_node->hasTagName(ddTag) ||
                m_node->hasTagName(divTag) ||
                m_node->hasTagName(dlTag) || m_node->hasTagName(dtTag) || 
@@ -379,9 +369,8 @@ bool TextIterator::handleNonTextNode()
                m_node->hasTagName(olTag) || m_node->hasTagName(pTag) ||
                m_node->hasTagName(preTag) || m_node->hasTagName(trTag) ||
                m_node->hasTagName(ulTag)) {
-        if (m_lastCharacter != '\n' && m_lastTextNode) {
+        if (m_lastCharacter != '\n' && m_lastTextNode)
             emitCharacter('\n', m_lastTextNode->parentNode(), m_lastTextNode, 0, 1);
-        }
     }
 
     return true;
@@ -419,9 +408,8 @@ void TextIterator::exitNode()
             if (style) {
                 int bottomMargin = renderer->collapsedMarginBottom();
                 int fontSize = style->fontDescription().computedPixelSize();
-                if (bottomMargin * 2 >= fontSize) {
+                if (bottomMargin * 2 >= fontSize)
                     addNewline = true;
-                }
             }
         }
     }
@@ -550,19 +538,15 @@ void SimplifiedBackwardsTextIterator::advance()
             RenderObject *renderer = m_node->renderer();
             if (renderer && renderer->isText() && m_node->nodeType() == Node::TEXT_NODE) {
                 // FIXME: What about CDATA_SECTION_NODE?
-                if (renderer->style()->visibility() == VISIBLE && m_offset > 0) {
+                if (renderer->style()->visibility() == VISIBLE && m_offset > 0)
                     m_handledNode = handleTextNode();
-                }
             } else if (renderer && (renderer->isImage() || renderer->isWidget())) {
-                if (renderer->style()->visibility() == VISIBLE && m_offset > 0) {
+                if (renderer->style()->visibility() == VISIBLE && m_offset > 0)
                     m_handledNode = handleReplacedElement();
-                }
-            } else {
+            } else
                 m_handledNode = handleNonTextNode();
-            }
-            if (m_positionNode) {
+            if (m_positionNode)
                 return;
-            }
         }
 
         if (m_node == m_startNode)
@@ -609,9 +593,8 @@ void SimplifiedBackwardsTextIterator::advance()
             m_offset = 0;
         m_handledNode = false;
         
-        if (m_positionNode) {
+        if (m_positionNode)
             return;
-        }
     }
 }
 
@@ -622,9 +605,8 @@ bool SimplifiedBackwardsTextIterator::handleTextNode()
     RenderText *renderer = static_cast<RenderText *>(m_node->renderer());
     String str = m_node->nodeValue();
 
-    if (!renderer->firstTextBox() && str.length() > 0) {
+    if (!renderer->firstTextBox() && str.length() > 0)
         return true;
-    }
 
     m_positionEndOffset = m_offset;
 
@@ -770,9 +752,9 @@ void CharacterIterator::advance(int count)
     // move to a subsequent m_textIterator run
     for (m_textIterator.advance(); !atEnd(); m_textIterator.advance()) {
         int runLength = m_textIterator.length();
-        if (runLength == 0) {
+        if (runLength == 0)
             m_atBreak = true;
-        else {
+        else {
             // see whether this is m_textIterator to use
             if (count < runLength) {
                 m_runOffset = count;
@@ -843,15 +825,13 @@ void WordAwareIterator::advance()
     }
     m_range = m_textIterator.range();
 
-    if (m_textIterator.atEnd()) {
+    if (m_textIterator.atEnd())
         return;
-    }
     
     while (1) {
         // If this chunk ends in whitespace we can just use it as our chunk.
-        if (m_textIterator.characters()[m_textIterator.length()-1].isSpace()) {
+        if (m_textIterator.characters()[m_textIterator.length()-1].isSpace())
             return;
-        }
 
         // If this is the first chunk that failed, save it in previousText before look ahead
         if (m_buffer.isEmpty()) {
@@ -873,41 +853,37 @@ void WordAwareIterator::advance()
         }
         m_buffer.append(m_textIterator.characters(), m_textIterator.length());
         int exception = 0;
-        m_range->setEnd(m_textIterator.range()->endContainer(exception),
-            m_textIterator.range()->endOffset(exception), exception);
+        m_range->setEnd(m_textIterator.range()->endContainer(exception), m_textIterator.range()->endOffset(exception), exception);
     }
 }
 
 int WordAwareIterator::length() const
 {
-    if (!m_buffer.isEmpty()) {
+    if (!m_buffer.isEmpty())
         return m_buffer.length();
-    } else if (m_previousText) {
+    else if (m_previousText)
         return m_previousLength;
-    } else {
+    else
         return m_textIterator.length();
-    }
 }
 
 const QChar *WordAwareIterator::characters() const
 {
-    if (!m_buffer.isEmpty()) {
+    if (!m_buffer.isEmpty())
         return m_buffer.unicode();
-    } else if (m_previousText) {
+    else if (m_previousText)
         return m_previousText;
-    } else {
+    else
         return m_textIterator.characters();
-    }
 }
 
-CircularSearchBuffer::CircularSearchBuffer(const DeprecatedString &s, bool isCaseSensitive)
+CircularSearchBuffer::CircularSearchBuffer(const String& s, bool isCaseSensitive)
     : m_target(s)
 {
     assert(!s.isEmpty());
 
-    if (!isCaseSensitive) {
+    if (!isCaseSensitive)
         m_target = s.lower();
-    }
     m_target.replace(nonBreakingSpace, ' ');
     m_isCaseSensitive = isCaseSensitive;
 
@@ -918,11 +894,10 @@ CircularSearchBuffer::CircularSearchBuffer(const DeprecatedString &s, bool isCas
 
 void CircularSearchBuffer::append(const QChar &c)
 {
-    if (m_isCaseSensitive) {
+    if (m_isCaseSensitive)
         *m_cursor++ = c.unicode() == nonBreakingSpace ? ' ' : c.unicode();
-    } else {
+    else
         *m_cursor++ = c.unicode() == nonBreakingSpace ? ' ' : c.lower().unicode();
-    }
     if (m_cursor == m_buffer + length()) {
         m_cursor = m_buffer;
         m_bufferFull = true;
@@ -950,9 +925,9 @@ void CircularSearchBuffer::append(int count, const QChar *characters)
             m_cursor[i] = c.unicode() == nonBreakingSpace ? ' ' : c.lower().unicode();
         }
     }
-    if (count < tailSpace) {
+    if (count < tailSpace)
         m_cursor += count;
-    else {
+    else {
         m_bufferFull = true;
         m_cursor = m_buffer;
     }
@@ -1018,11 +993,10 @@ PassRefPtr<Range> TextIterator::rangeFromLocationAndLength(Document *doc, int ra
                 int offset = rangeLocation - docTextPosition;
                 resultRange->setStart(textRunRange->startContainer(exception), offset + textRunRange->startOffset(exception), exception);
             } else {
-                if (rangeLocation == docTextPosition) {
+                if (rangeLocation == docTextPosition)
                     resultRange->setStart(textRunRange->startContainer(exception), textRunRange->startOffset(exception), exception);
-                } else {
+                else
                     resultRange->setStart(textRunRange->endContainer(exception), textRunRange->endOffset(exception), exception);
-                }
             }
         }
 
@@ -1032,11 +1006,10 @@ PassRefPtr<Range> TextIterator::rangeFromLocationAndLength(Document *doc, int ra
                 int offset = rangeEnd - docTextPosition;
                 resultRange->setEnd(textRunRange->startContainer(exception), offset + textRunRange->startOffset(exception), exception);
             } else {
-                if (rangeEnd == docTextPosition) {
+                if (rangeEnd == docTextPosition)
                     resultRange->setEnd(textRunRange->startContainer(exception), textRunRange->startOffset(exception), exception);
-                } else {
+                else
                     resultRange->setEnd(textRunRange->endContainer(exception), textRunRange->endOffset(exception), exception);
-                }
             }
             if (!(rangeLength == 0 && rangeEnd == docTextPosition + len)) {
                 docTextPosition += len;
@@ -1066,7 +1039,7 @@ DeprecatedString plainText(const Range *r)
     return result;
 }
 
-PassRefPtr<Range> findPlainText(const Range *r, const DeprecatedString &s, bool forward, bool caseSensitive)
+PassRefPtr<Range> findPlainText(const Range *r, const String& s, bool forward, bool caseSensitive)
 {
     // FIXME: Can we do Boyer-Moore or equivalent instead for speed?
 
@@ -1090,9 +1063,8 @@ PassRefPtr<Range> findPlainText(const Range *r, const DeprecatedString &s, bool
             // Fill the buffer.
             while (int needed = buffer.neededCharacters()) {
                 if (it.atBreak()) {
-                    if (it.atEnd()) {
+                    if (it.atEnd())
                         goto done;
-                    }
                     buffer.clear();
                 }
                 int available = it.length();
@@ -1109,9 +1081,8 @@ PassRefPtr<Range> findPlainText(const Range *r, const DeprecatedString &s, bool
                     rangeEnd = it;
                     // If searching forward, stop on the first match.
                     // If searching backward, don't stop, so we end up with the last match.
-                    if (forward) {
+                    if (forward)
                         goto done;
-                    }
                 }
                 if (it.atBreak())
                     break;
@@ -1125,9 +1096,9 @@ PassRefPtr<Range> findPlainText(const Range *r, const DeprecatedString &s, bool
 done:
     int exception = 0;
     RefPtr<Range> result = r->cloneRange(exception);
-    if (!found) {
+    if (!found)
         result->collapse(!forward, exception);
-    else {
+    else {
         CharacterIterator it(r);
         it.advance(rangeEnd.characterOffset() - buffer.length());
         result->setStart(it.range()->startContainer(exception), it.range()->startOffset(exception), exception);
index 3f27a8d..95b4709 100644 (file)
@@ -47,8 +47,8 @@ inline bool isCollapsibleWhitespace(const QChar& c)
     }
 }
 
-DeprecatedString plainText(const Range *);
-PassRefPtr<Range> findPlainText(const Range *, const DeprecatedString &, bool forward, bool caseSensitive);
+DeprecatedString plainText(const Range*);
+PassRefPtr<Range> findPlainText(const Range*, const String&, bool forward, bool caseSensitive);
 
 // Iterates through the DOM range, returning all the text, and 0-length boundaries
 // at points where replaced elements break up the text flow.  The text comes back in
index 63f2c2a..5eebd2e 100644 (file)
@@ -239,8 +239,7 @@ bool HTMLFormElement::formData(FormData &form_data) const
                     // if the current type is FILE, then we also need to
                     // include the filename
                     if (current->hasLocalName(inputTag) &&
-                        static_cast<HTMLInputElement*>(current)->inputType() == HTMLInputElement::FILE)
-                    {
+                        static_cast<HTMLInputElement*>(current)->inputType() == HTMLInputElement::FILE) {
                         DeprecatedString path = static_cast<HTMLInputElement*>(current)->value().deprecatedString();
 
                         // FIXME: This won't work if the filename includes a " mark,
@@ -251,9 +250,8 @@ bool HTMLFormElement::formData(FormData &form_data) const
                         hstr += encoding.fromUnicode(path.mid(path.findRev('/') + 1), true);
                         hstr += "\"";
 
-                        if(!static_cast<HTMLInputElement*>(current)->value().isEmpty())
-                        {
-                            DeprecatedString mimeType = frame ? frame->mimeTypeForFileName(path) : DeprecatedString();
+                        if (!static_cast<HTMLInputElement*>(current)->value().isEmpty()) {
+                            DeprecatedString mimeType = frame ? frame->mimeTypeForFileName(path).deprecatedString() : DeprecatedString();
                             if (!mimeType.isEmpty()) {
                                 hstr += "\r\nContent-Type: ";
                                 hstr += mimeType.ascii();
index f1735b1..6770af4 100644 (file)
@@ -148,7 +148,7 @@ bool Navigator::getOwnPropertySlot(ExecState *exec, const Identifier& propertyNa
 
 JSValue *Navigator::getValueProperty(ExecState *exec, int token) const
 {
-  DeprecatedString userAgent = m_frame->userAgent();
+  String userAgent = m_frame->userAgent();
   switch (token) {
   case AppCodeName:
     return jsString("Mozilla");
@@ -161,7 +161,7 @@ JSValue *Navigator::getValueProperty(ExecState *exec, int token) const
     return jsUndefined();
   case AppVersion:
     // We assume the string is something like Mozilla/version (properties)
-    return jsString(userAgent.mid(userAgent.find('/') + 1));
+    return jsString(userAgent.substring(userAgent.find('/') + 1));
   case Product:
     // When acting normal, we pretend to be "Gecko".
     if (userAgent.find("Mozilla/5.0") >= 0 && userAgent.find("compatible") == -1)
index 6b0d572..66c4f6d 100644 (file)
@@ -151,7 +151,7 @@ void KJSProxy::initScriptIfNeeded()
   m_script = new ScriptInterpreter(globalObject, m_frame);
   globalObject->put(m_script->globalExec(), "debug", new TestFunctionImp(), Internal);
 
-  DeprecatedString userAgent = m_frame->userAgent();
+  String userAgent = m_frame->userAgent();
   if (userAgent.find("Microsoft") >= 0 || userAgent.find("MSIE") >= 0)
     m_script->setCompatMode(Interpreter::IECompat);
   else
index 56165e9..d4b733b 100644 (file)
@@ -2072,7 +2072,7 @@ JSValue *Location::getValueProperty(ExecState *exec, int token) const
   KURL url = m_frame->url();
   switch (token) {
   case Hash:
-    return jsString(url.ref().isNull() ? DeprecatedString("") : "#" + url.ref());
+    return jsString(url.ref().isNull() ? "" : "#" + url.ref());
   case Host: {
     // Note: this is the IE spec. The NS spec swaps the two, it says
     // "The hostname property is the concatenation of the host and port properties, separated by a colon."
@@ -2090,11 +2090,11 @@ JSValue *Location::getValueProperty(ExecState *exec, int token) const
     else
       return jsString(url.prettyURL());
   case Pathname:
-    return jsString(url.path().isEmpty() ? DeprecatedString("/") : url.path());
+    return jsString(url.path().isEmpty() ? "/" : url.path());
   case Port:
-    return jsString(url.port() ? DeprecatedString::number((int)url.port()) : DeprecatedString::fromLatin1(""));
+    return jsString(url.port() ? DeprecatedString::number((int)url.port()) : "");
   case Protocol:
-    return jsString(url.protocol()+":");
+    return jsString(url.protocol() + ":");
   case Search:
     return jsString(url.query());
   default:
index 076ef51..ad357f1 100644 (file)
@@ -78,7 +78,7 @@ KRenderingPaintServer *KSVGPainterFactory::fillPaintServer(const RenderStyle* st
         static_cast<KRenderingPaintServerSolid *>(fillPaintServer)->setColor(Color::black);
     } else if (fill->paintType() == SVG_PAINTTYPE_URI) {
         String id(fill->uri());
-        fillPaintServer = getPaintServerById(item->document(), id.deprecatedString().mid(1));
+        fillPaintServer = getPaintServerById(item->document(), AtomicString(id.substring(1)));
         if (item && fillPaintServer && item->isRenderPath())
             fillPaintServer->addClient(static_cast<const RenderPath*>(item));
     } else {
@@ -112,7 +112,7 @@ KRenderingPaintServer *KSVGPainterFactory::strokePaintServer(const RenderStyle*
     KRenderingPaintServer *strokePaintServer;
     if (stroke && stroke->paintType() == SVG_PAINTTYPE_URI) {
         String id(stroke->uri());
-        strokePaintServer = getPaintServerById(item->document(), id.deprecatedString().mid(1));
+        strokePaintServer = getPaintServerById(item->document(), AtomicString(id.substring(1)));
         if(item && strokePaintServer && item->isRenderPath())
             strokePaintServer->addClient(static_cast<const RenderPath*>(item));
     } else {
index 5bdacee..91b8d60 100644 (file)
@@ -67,11 +67,11 @@ float SVGAngle::value() const
 // calc m_value
 void SVGAngle::calculate()
 {
-    if(m_unitType == SVG_ANGLETYPE_GRAD)
+    if (m_unitType == SVG_ANGLETYPE_GRAD)
         m_value = m_valueInSpecifiedUnits / deg2grad;
-    else if(m_unitType == SVG_ANGLETYPE_RAD)
+    else if (m_unitType == SVG_ANGLETYPE_RAD)
         m_value = m_valueInSpecifiedUnits / deg2rad;
-    else if(m_unitType == SVG_ANGLETYPE_UNSPECIFIED || m_unitType == SVG_ANGLETYPE_DEG)
+    else if (m_unitType == SVG_ANGLETYPE_UNSPECIFIED || m_unitType == SVG_ANGLETYPE_DEG)
         m_value = m_valueInSpecifiedUnits;
 }
 
@@ -90,19 +90,16 @@ void SVGAngle::setValueAsString(StringImpl *valueAsString)
 {
     m_valueAsString = String(valueAsString);
 
-    DeprecatedString s = m_valueAsString.deprecatedString();
-
     bool bOK;
-    m_valueInSpecifiedUnits = s.toDouble(&bOK);
+    m_valueInSpecifiedUnits = m_valueAsString.deprecatedString().toDouble(&bOK);
     m_unitType = SVG_ANGLETYPE_UNSPECIFIED;
 
-    if(!bOK)
-    {
-        if(s.endsWith(DeprecatedString::fromLatin1("deg")))
+    if (!bOK) {
+        if (m_valueAsString.endsWith("deg"))
             m_unitType = SVG_ANGLETYPE_DEG;
-        else if(s.endsWith(DeprecatedString::fromLatin1("grad")))
+        else if (m_valueAsString.endsWith("grad"))
             m_unitType = SVG_ANGLETYPE_GRAD;
-        else if(s.endsWith(DeprecatedString::fromLatin1("rad")))
+        else if (m_valueAsString.endsWith("rad"))
             m_unitType = SVG_ANGLETYPE_RAD;
     }
     
@@ -111,19 +108,18 @@ void SVGAngle::setValueAsString(StringImpl *valueAsString)
 
 StringImpl *SVGAngle::valueAsString() const
 {
-    m_valueAsString.deprecatedString().setNum(m_valueInSpecifiedUnits);
+    m_valueAsString = DeprecatedString::number(m_valueInSpecifiedUnits);
 
-    switch(m_unitType)
-    {
+    switch(m_unitType) {
         case SVG_ANGLETYPE_UNSPECIFIED:
         case SVG_ANGLETYPE_DEG:
-            m_valueAsString.deprecatedString() += DeprecatedString::fromLatin1("deg");
+            m_valueAsString += "deg";
             break;
         case SVG_ANGLETYPE_RAD:
-            m_valueAsString.deprecatedString() += DeprecatedString::fromLatin1("rad");
+            m_valueAsString += "rad";
             break;
         case SVG_ANGLETYPE_GRAD:
-            m_valueAsString.deprecatedString() += DeprecatedString::fromLatin1("grad");
+            m_valueAsString += "grad";
             break;
     }
     
@@ -139,20 +135,20 @@ void SVGAngle::newValueSpecifiedUnits(unsigned short unitType, float valueInSpec
 
 void SVGAngle::convertToSpecifiedUnits(unsigned short unitType)
 {
-    if(m_unitType == unitType)
+    if (m_unitType == unitType)
         return;
 
-    if(m_unitType == SVG_ANGLETYPE_DEG && unitType == SVG_ANGLETYPE_RAD)
+    if (m_unitType == SVG_ANGLETYPE_DEG && unitType == SVG_ANGLETYPE_RAD)
         m_valueInSpecifiedUnits *= deg2rad;
-    else if(m_unitType == SVG_ANGLETYPE_GRAD && unitType == SVG_ANGLETYPE_RAD)
+    else if (m_unitType == SVG_ANGLETYPE_GRAD && unitType == SVG_ANGLETYPE_RAD)
         m_valueInSpecifiedUnits /= rad2grad;
-    else if(m_unitType == SVG_ANGLETYPE_DEG && unitType == SVG_ANGLETYPE_GRAD)
+    else if (m_unitType == SVG_ANGLETYPE_DEG && unitType == SVG_ANGLETYPE_GRAD)
         m_valueInSpecifiedUnits *= deg2grad;
-    else if(m_unitType == SVG_ANGLETYPE_RAD && unitType == SVG_ANGLETYPE_GRAD)
+    else if (m_unitType == SVG_ANGLETYPE_RAD && unitType == SVG_ANGLETYPE_GRAD)
         m_valueInSpecifiedUnits *= rad2grad;
-    else if(m_unitType == SVG_ANGLETYPE_RAD && unitType == SVG_ANGLETYPE_DEG)
+    else if (m_unitType == SVG_ANGLETYPE_RAD && unitType == SVG_ANGLETYPE_DEG)
         m_valueInSpecifiedUnits /= deg2rad;
-    else if(m_unitType == SVG_ANGLETYPE_GRAD && unitType == SVG_ANGLETYPE_DEG)
+    else if (m_unitType == SVG_ANGLETYPE_GRAD && unitType == SVG_ANGLETYPE_DEG)
         m_valueInSpecifiedUnits /= deg2grad;
 
     m_unitType = unitType;
@@ -173,7 +169,7 @@ double SVGAngle::shortestArcBisector(double angle1, double angle2)
 {
     double bisector = (angle1 + angle2) / 2;
 
-    if(fabs(angle1 - angle2) > 180)
+    if (fabs(angle1 - angle2) > 180)
         bisector += 180;
 
     return bisector;
index 4f2f1e7..ee9c885 100644 (file)
@@ -65,7 +65,7 @@ namespace WebCore
         unsigned short m_unitType;
         float m_value;
         float m_valueInSpecifiedUnits;
-        String m_valueAsString;
+        mutable String m_valueAsString;
         const SVGStyledElement *m_context;
 
         void calculate();
index dd9a620..4b1cc2c 100644 (file)
@@ -81,20 +81,20 @@ SVGAnimationElement::~SVGAnimationElement()
 
 SVGElement *SVGAnimationElement::targetElement() const
 {
-    if(!m_targetElement)
+    if (!m_targetElement)
     {
-        if(!m_href.isEmpty())
+        if (!m_href.isEmpty())
         {
             String targetId = SVGURIReference::getTarget(m_href);
             Element *element = ownerDocument()->getElementById(targetId.impl());
             m_targetElement = svg_dynamic_cast(element);
         }
-        else if(parentNode())
+        else if (parentNode())
         {
             Node *target = parentNode();
             while(target != 0)
             {
-                if(target->nodeType() != ELEMENT_NODE)
+                if (target->nodeType() != ELEMENT_NODE)
                     target = target->parentNode();
                 else
                     break;
@@ -135,11 +135,11 @@ void SVGAnimationElement::parseMappedAttribute(MappedAttribute *attr)
             m_attributeName = value.deprecatedString();
     else if (attr->name() == SVGNames::attributeTypeAttr)
     {
-        if(value == "CSS")
+        if (value == "CSS")
             m_attributeType = ATTRIBUTETYPE_CSS;
-        else if(value == "XML")
+        else if (value == "XML")
             m_attributeType = ATTRIBUTETYPE_XML;
-        else if(value == "auto")
+        else if (value == "auto")
             m_attributeType = ATTRIBUTETYPE_AUTO;
     }
     else if (attr->name() == SVGNames::beginAttr || attr->name() == SVGNames::endAttr)
@@ -151,44 +151,40 @@ void SVGAnimationElement::parseMappedAttribute(MappedAttribute *attr)
         SVGHelper::ParseSeperatedList(temp.get(), value.deprecatedString(), ';');
 
         // Parse data
-        for(unsigned int i = 0; i < temp->numberOfItems(); i++)
-        {
+        for (unsigned int i = 0; i < temp->numberOfItems(); i++) {
             DeprecatedString current = String(temp->getItem(i)).deprecatedString();
 
-            if(current.startsWith(DeprecatedString::fromLatin1("accessKey")))
-            {
+            if (current.startsWith("accessKey")) {
                 // Register keyDownEventListener for the character
                 DeprecatedString character = current.mid(current.length() - 2, 1);
 
                 //kdDebug() << k_funcinfo << " Supposed to register accessKey Character: " << character << " UNSUPPORTED!" << endl;
-            }
-            else if(current.startsWith(DeprecatedString::fromLatin1("wallclock")))
-            {
+            } else if (current.startsWith("wallclock")) {
                 int firstBrace = current.find('(');
                 int secondBrace = current.find(')');
 
                 DeprecatedString wallclockValue = current.mid(firstBrace + 1, secondBrace - firstBrace - 2);
                 //kdDebug() << k_funcinfo << " Supposed to use wallClock value: " << wallclockValue << " UNSUPPORTED!" << endl;
             }
-            else if(current.contains('.'))
+            else if (current.contains('.'))
             {
                 int dotPosition = current.find('.');
 
                 DeprecatedString element = current.mid(0, dotPosition);
                 DeprecatedString clockValue;
-                if(current.contains(DeprecatedString::fromLatin1("begin")))
+                if (current.contains("begin"))
                     clockValue = current.mid(dotPosition + 6);
-                else if(current.contains(DeprecatedString::fromLatin1("end")))
+                else if (current.contains("end"))
                     clockValue = current.mid(dotPosition + 4);
-                else if(current.contains(DeprecatedString::fromLatin1("repeat")))
+                else if (current.contains("repeat"))
                     clockValue = current.mid(dotPosition + 7);
                 else // DOM2 Event Reference
                 {
                     int plusMinusPosition = -1;
 
-                    if(current.contains('+'))
+                    if (current.contains('+'))
                         plusMinusPosition = current.find('+');
-                    else if(current.contains('-'))
+                    else if (current.contains('-'))
                         plusMinusPosition = current.find('-');
 
                     DeprecatedString event = current.mid(dotPosition + 1, plusMinusPosition - dotPosition - 1);
@@ -199,10 +195,10 @@ void SVGAnimationElement::parseMappedAttribute(MappedAttribute *attr)
             }
             else
             {
-                if(attr->name() == SVGNames::beginAttr)
+                if (attr->name() == SVGNames::beginAttr)
                 {
                     m_begin = parseClockValue(current);
-                    if(!isIndefinite(m_begin))
+                    if (!isIndefinite(m_begin))
                         m_begin *= 1000.0;
 
                     //kdDebug() << k_funcinfo << " Setting begin time to " << m_begin << " ms!" << endl;
@@ -210,7 +206,7 @@ void SVGAnimationElement::parseMappedAttribute(MappedAttribute *attr)
                 else
                 {
                     m_end = parseClockValue(current);
-                    if(!isIndefinite(m_end))
+                    if (!isIndefinite(m_end))
                         m_end *= 1000.0;
 
                     //kdDebug() << k_funcinfo << " Setting end time to " << m_end << " ms!" << endl;
@@ -221,33 +217,33 @@ void SVGAnimationElement::parseMappedAttribute(MappedAttribute *attr)
     else if (attr->name() == SVGNames::durAttr)
     {
         m_simpleDuration = parseClockValue(value.deprecatedString());
-        if(!isIndefinite(m_simpleDuration))
+        if (!isIndefinite(m_simpleDuration))
             m_simpleDuration *= 1000.0;
     }
     else if (attr->name() == SVGNames::minAttr)
     {
         m_min = parseClockValue(value.deprecatedString());
-        if(!isIndefinite(m_min))
+        if (!isIndefinite(m_min))
             m_min *= 1000.0;
     }
     else if (attr->name() == SVGNames::maxAttr)
     {
         m_max = parseClockValue(value.deprecatedString());
-        if(!isIndefinite(m_max))
+        if (!isIndefinite(m_max))
             m_max *= 1000.0;
     }
     else if (attr->name() == SVGNames::restartAttr)
     {
-        if(value == "whenNotActive")
+        if (value == "whenNotActive")
             m_restart = RESTART_WHENNOTACTIVE;
-        else if(value == "never")
+        else if (value == "never")
             m_restart = RESTART_NEVER;
-        else if(value == "always")
+        else if (value == "always")
             m_restart = RESTART_ALWAYS;
     }
     else if (attr->name() == SVGNames::repeatCountAttr)
     {
-        if(value == "indefinite")
+        if (value == "indefinite")
             m_repeatCount = DBL_MAX;
         else
             m_repeatCount = value.deprecatedString().toDouble();
@@ -256,20 +252,20 @@ void SVGAnimationElement::parseMappedAttribute(MappedAttribute *attr)
         m_repeatDur = value.deprecatedString();
     else if (attr->name() == SVGNames::fillAttr)
     {
-        if(value == "freeze")
+        if (value == "freeze")
             m_fill = FILL_FREEZE;
-        else if(value == "remove")
+        else if (value == "remove")
             m_fill = FILL_REMOVE;
     }
     else if (attr->name() == SVGNames::calcModeAttr)
     {
-        if(value == "discrete")
+        if (value == "discrete")
             m_calcMode = CALCMODE_DISCRETE;
-        else if(value == "linear")
+        else if (value == "linear")
             m_calcMode = CALCMODE_LINEAR;
-        else if(value == "spline")
+        else if (value == "spline")
             m_calcMode = CALCMODE_SPLINE;
-        else if(value == "paced")
+        else if (value == "paced")
             m_calcMode = CALCMODE_PACED;
     }
     else if (attr->name() == SVGNames::valuesAttr)
@@ -295,22 +291,22 @@ void SVGAnimationElement::parseMappedAttribute(MappedAttribute *attr)
         m_by = value.deprecatedString();
     else if (attr->name() == SVGNames::additiveAttr)
     {
-        if(value == "sum")
+        if (value == "sum")
             m_additive = ADDITIVE_SUM;
-        else if(value == "replace")
+        else if (value == "replace")
             m_additive = ADDITIVE_REPLACE;
     }
     else if (attr->name() == SVGNames::accumulateAttr)
     {
-        if(value == "sum")
+        if (value == "sum")
             m_accumulate = ACCUMULATE_SUM;
-        else if(value == "none")
+        else if (value == "none")
             m_accumulate = ACCUMULATE_NONE;
     }
     else
     {
-        if(SVGTests::parseMappedAttribute(attr)) return;
-        if(SVGExternalResourcesRequired::parseMappedAttribute(attr)) return;
+        if (SVGTests::parseMappedAttribute(attr)) return;
+        if (SVGExternalResourcesRequired::parseMappedAttribute(attr)) return;
         
         SVGElement::parseMappedAttribute(attr);
     }
@@ -320,7 +316,7 @@ double SVGAnimationElement::parseClockValue(const DeprecatedString &data) const
 {
     DeprecatedString parse = data.stripWhiteSpace();
     
-    if(parse == DeprecatedString::fromLatin1("indefinite")) // Saves some time...
+    if (parse == "indefinite") // Saves some time...
         return DBL_MAX;
 
     double result;
@@ -328,7 +324,7 @@ double SVGAnimationElement::parseClockValue(const DeprecatedString &data) const
     int doublePointOne = parse.find(':');
     int doublePointTwo = parse.find(':', doublePointOne + 1);
 
-    if(doublePointOne != -1 && doublePointTwo != -1) // Spec: "Full clock values"
+    if (doublePointOne != -1 && doublePointTwo != -1) // Spec: "Full clock values"
     {
         unsigned int hours = parse.mid(0, 2).toUInt();
         unsigned int minutes = parse.mid(3, 2).toUInt();
@@ -337,14 +333,14 @@ double SVGAnimationElement::parseClockValue(const DeprecatedString &data) const
 
         result = (3600 * hours) + (60 * minutes) + seconds;
 
-        if(parse.find('.') != -1)
+        if (parse.find('.') != -1)
         {
             DeprecatedString temp = parse.mid(9, 2);
             milliseconds = temp.toUInt();
             result += (milliseconds * (1 / pow(10.0, int(temp.length()))));
         }
     }
-    else if(doublePointOne != -1 && doublePointTwo == -1) // Spec: "Partial clock values"
+    else if (doublePointOne != -1 && doublePointTwo == -1) // Spec: "Partial clock values"
     {
         unsigned int minutes = parse.mid(0, 2).toUInt();
         unsigned int seconds = parse.mid(3, 2).toUInt();
@@ -352,7 +348,7 @@ double SVGAnimationElement::parseClockValue(const DeprecatedString &data) const
 
         result = (60 * minutes) + seconds;
 
-        if(parse.find('.') != -1)
+        if (parse.find('.') != -1)
         {
             DeprecatedString temp = parse.mid(6, 2);
             milliseconds = temp.toUInt();
@@ -363,9 +359,9 @@ double SVGAnimationElement::parseClockValue(const DeprecatedString &data) const
     {
         int dotPosition = parse.find('.');
 
-        if(parse.endsWith(DeprecatedString::fromLatin1("h")))
+        if (parse.endsWith("h"))
         {
-            if(dotPosition == -1)
+            if (dotPosition == -1)
                 result = parse.mid(0, parse.length() - 1).toUInt() * 3600;
             else
             {
@@ -374,9 +370,9 @@ double SVGAnimationElement::parseClockValue(const DeprecatedString &data) const
                 result += (3600.0 * temp.toUInt()) * (1 / pow(10.0, int(temp.length())));
             }
         }
-        else if(parse.endsWith(DeprecatedString::fromLatin1("min")))
+        else if (parse.endsWith("min"))
         {
-            if(dotPosition == -1)
+            if (dotPosition == -1)
                 result = parse.mid(0, parse.length() - 3).toUInt() * 60;
             else
             {
@@ -385,9 +381,9 @@ double SVGAnimationElement::parseClockValue(const DeprecatedString &data) const
                 result += (60.0 * temp.toUInt()) * (1 / pow(10.0, int(temp.length())));
             }
         }
-        else if(parse.endsWith(DeprecatedString::fromLatin1("ms")))
+        else if (parse.endsWith("ms"))
         {
-            if(dotPosition == -1)
+            if (dotPosition == -1)
                 result = parse.mid(0, parse.length() - 2).toUInt() / 1000.0;
             else
             {
@@ -396,9 +392,9 @@ double SVGAnimationElement::parseClockValue(const DeprecatedString &data) const
                 result += (temp.toUInt() / 1000.0) * (1 / pow(10.0, int(temp.length())));
             }
         }
-        else if(parse.endsWith(DeprecatedString::fromLatin1("s")))
+        else if (parse.endsWith("s"))
         {
-            if(dotPosition == -1)
+            if (dotPosition == -1)
                 result = parse.mid(0, parse.length() - 1).toUInt();
             else
             {
@@ -421,7 +417,7 @@ void SVGAnimationElement::closeRenderer()
 
 String SVGAnimationElement::targetAttribute() const
 {
-    if(!targetElement())
+    if (!targetElement())
         return String();
     
     SVGElement *target = targetElement();
@@ -450,7 +446,7 @@ String SVGAnimationElement::targetAttribute() const
             ret = styled->style()->getPropertyValue(m_attributeName);
     }
 
-    if(attributeType == ATTRIBUTETYPE_XML || ret.isEmpty())
+    if (attributeType == ATTRIBUTETYPE_XML || ret.isEmpty())
         ret = targetElement()->getAttribute(String(m_attributeName).impl());
 
     return ret;
@@ -463,7 +459,7 @@ void SVGAnimationElement::setTargetAttribute(StringImpl *value)
 
 void SVGAnimationElement::setTargetAttribute(SVGElement *target, StringImpl *nameImpl, StringImpl *value, EAttributeType type)
 {
-    if(!target || !nameImpl || !value)
+    if (!target || !nameImpl || !value)
         return;
     String name(nameImpl);
     
@@ -472,7 +468,7 @@ void SVGAnimationElement::setTargetAttribute(SVGElement *target, StringImpl *nam
         styled = static_cast<SVGStyledElement *>(target);
 
     EAttributeType attributeType = type;
-    if(type == ATTRIBUTETYPE_AUTO)
+    if (type == ATTRIBUTETYPE_AUTO)
     {
         attributeType = ATTRIBUTETYPE_XML;
 
@@ -521,22 +517,22 @@ bool SVGAnimationElement::isAccumulated() const
 
 EAnimationMode SVGAnimationElement::detectAnimationMode() const
 {
-    if((!m_from.isEmpty() && !m_to.isEmpty()) || (!m_to.isEmpty())) // to/from-to animation
+    if ((!m_from.isEmpty() && !m_to.isEmpty()) || (!m_to.isEmpty())) // to/from-to animation
     {
-        if(!m_from.isEmpty()) // from-to animation
+        if (!m_from.isEmpty()) // from-to animation
             return FROM_TO_ANIMATION;
         else
             return TO_ANIMATION;
     }
-    else if((m_from.isEmpty() && m_to.isEmpty() && !m_by.isEmpty()) ||
+    else if ((m_from.isEmpty() && m_to.isEmpty() && !m_by.isEmpty()) ||
             (!m_from.isEmpty() && !m_by.isEmpty())) // by/from-by animation
     {
-        if(!m_from.isEmpty()) // from-by animation
+        if (!m_from.isEmpty()) // from-by animation
             return FROM_BY_ANIMATION;
         else
             return BY_ANIMATION;
     }
-    else if(m_values)
+    else if (m_values)
         return VALUES_ANIMATION;
 
     return NO_ANIMATION;
@@ -544,20 +540,20 @@ EAnimationMode SVGAnimationElement::detectAnimationMode() const
 
 int SVGAnimationElement::calculateCurrentValueItem(double timePercentage)
 {
-    if(!m_values)
+    if (!m_values)
         return -1;
     
     unsigned long items = m_values->numberOfItems();
 
     // Calculate the relative time percentages for each 'fade'.
     double startTimes[items]; startTimes[0] = 0.0;
-    for(unsigned int i = 1; i < items; ++i)
+    for (unsigned int i = 1; i < items; ++i)
         startTimes[i] = (((2.0 * i)) / (items - 1)) / 2.0;
 
     int itemByPercentage = -1;
-    for(unsigned int i = 0; i < items - 1; ++i)
+    for (unsigned int i = 0; i < items - 1; ++i)
     {
-        if(timePercentage >= startTimes[i] && timePercentage <= startTimes[i + 1])
+        if (timePercentage >= startTimes[i] && timePercentage <= startTimes[i + 1])
         {
             itemByPercentage = i;
             break;
@@ -569,20 +565,20 @@ int SVGAnimationElement::calculateCurrentValueItem(double timePercentage)
 
 double SVGAnimationElement::calculateRelativeTimePercentage(double timePercentage, int currentItem)
 {
-    if(currentItem == -1 || !m_values)
+    if (currentItem == -1 || !m_values)
         return 0.0;
 
     unsigned long items = m_values->numberOfItems();
 
     // Calculate the relative time percentages for each 'fade'.
     double startTimes[items]; startTimes[0] = 0.0;
-    for(unsigned int i = 1; i < items; ++i)
+    for (unsigned int i = 1; i < items; ++i)
         startTimes[i] = (((2.0 * i)) / (items - 1)) / 2.0;
 
     double beginTimePercentage = startTimes[currentItem];
     double endTimePercentage = startTimes[currentItem + 1];
 
-    if((endTimePercentage - beginTimePercentage) == 0.0)
+    if ((endTimePercentage - beginTimePercentage) == 0.0)
         return 0.0;
 
     return ((timePercentage - beginTimePercentage) /
index c98cf6b..4fc9726 100644 (file)
@@ -222,26 +222,25 @@ void SVGColor::setRGBColor(StringImpl *rgbColor)
         return;
 
     DeprecatedString parse = m_rgbColor.deprecatedString().stripWhiteSpace();
-    if(parse.startsWith(DeprecatedString::fromLatin1("rgb(")))
+    if(parse.startsWith("rgb("))
     {
         DeprecatedStringList colors = DeprecatedStringList::split(',', parse);
         DeprecatedString r = colors[0].right((colors[0].length() - 4));
         DeprecatedString g = colors[1];
         DeprecatedString b = colors[2].left((colors[2].length() - 1));
     
-        if(r.contains(DeprecatedString::fromLatin1("%")))
-        {
+        if(r.contains("%")) {
             r = r.left(r.length() - 1);
             r = DeprecatedString::number(int((double(255 * r.toDouble()) / 100.0)));
         }
 
-        if(g.contains(DeprecatedString::fromLatin1("%")))
+        if(g.contains("%"))
         {
             g = g.left(g.length() - 1);
             g = DeprecatedString::number(int((double(255 * g.toDouble()) / 100.0)));
         }
     
-        if(b.contains(DeprecatedString::fromLatin1("%")))
+        if(b.contains("%"))
         {
             b = b.left(b.length() - 1);
             b = DeprecatedString::number(int((double(255 * b.toDouble()) / 100.0)));
index 6a27f53..fa78aa5 100644 (file)
@@ -28,8 +28,8 @@
 #include <kdom/Helper.h>
 #include <kdom/Namespace.h>
 #include <kdom/core/DOMException.h>
-#include <kdom/core/Element.h>
 #include <kdom/kdom.h>
+#include <kxmlcore/HashSet.h>
 
 #include "Document.h"
 #include "DocumentType.h"
 using namespace WebCore;
 
 SVGDOMImplementation *SVGDOMImplementation::s_instance = 0;
-DeprecatedStringList SVGDOMImplementation::s_features;
+
+static const HashSet<String>& svgFeatureSet()
+{
+    static HashSet<String>* svgFeatures = 0;
+    if (!svgFeatures) {
+        svgFeatures = new HashSet<String>;
+        
+        // 1.1 features
+        svgFeatures->add("SVG");
+        svgFeatures->add("SVGDOM");
+        svgFeatures->add("SVG-STATIC");
+        svgFeatures->add("SVGDOM-STATIC");
+        svgFeatures->add("SVG-ANIMATION");
+        svgFeatures->add("SVGDOM-ANIMATION");
+        svgFeatures->add("SVG-DYNAMIC");
+        svgFeatures->add("COREATTRIBUTE");
+        svgFeatures->add("STRUCTURE");
+        svgFeatures->add("CONTAINERATTRIBUTE");
+        svgFeatures->add("CONDITIOANLPROCESSING");
+        svgFeatures->add("IMAGE");
+        svgFeatures->add("STYLE");
+        svgFeatures->add("VIEWPORTATTRIBUTE");
+        svgFeatures->add("SHAPE");
+        svgFeatures->add("TEXT");
+        svgFeatures->add("PAINTATTRIBUTE");
+        svgFeatures->add("OPACITYATTRIBUTE");
+        svgFeatures->add("GRAPHICSATTRIBUTE");
+        svgFeatures->add("MARKER");
+        svgFeatures->add("COLORPROFILE");
+        svgFeatures->add("GRADIENT");
+        svgFeatures->add("PATTERN");
+        svgFeatures->add("CLIP");
+        svgFeatures->add("MASK");
+        svgFeatures->add("FILTER");
+        svgFeatures->add("XLINKATTRIBUTE");
+        svgFeatures->add("FONT");
+        svgFeatures->add("EXTENSIBILITY");
+
+        // 1.0 features
+        //svgFeatures->add("SVG");
+        svgFeatures->add("SVG.STATIC");
+        svgFeatures->add("SVG.ANIMATION");
+        svgFeatures->add("SVG.DYNAMIC");
+        svgFeatures->add("DOM");
+        svgFeatures->add("DOM.SVG");
+        svgFeatures->add("DOM.SVG.STATIC");
+        svgFeatures->add("DOM.SVG.ANIMATION");
+        svgFeatures->add("DOM.SVG.DYNAMIC");
+        svgFeatures->add("SVG.ALL");
+        svgFeatures->add("DOM.SVG.ALL");
+    }
+    return *svgFeatures;
+}
 
 SVGDOMImplementation::SVGDOMImplementation() : DOMImplementation()
 {
@@ -59,74 +111,24 @@ SVGDOMImplementation::~SVGDOMImplementation()
 
 SVGDOMImplementation *SVGDOMImplementation::self()
 {
-    if(!s_instance)
-    {
+    if (!s_instance)
         s_instance = new SVGDOMImplementation();
-        
-        // 1.1 features
-        s_features.append(DeprecatedString::fromLatin1("SVG"));
-        s_features.append(DeprecatedString::fromLatin1("SVGDOM"));
-        s_features.append(DeprecatedString::fromLatin1("SVG-STATIC"));
-        s_features.append(DeprecatedString::fromLatin1("SVGDOM-STATIC"));
-        s_features.append(DeprecatedString::fromLatin1("SVG-ANIMATION"));
-        s_features.append(DeprecatedString::fromLatin1("SVGDOM-ANIMATION"));
-        s_features.append(DeprecatedString::fromLatin1("SVG-DYNAMIC"));
-        s_features.append(DeprecatedString::fromLatin1("COREATTRIBUTE"));
-        s_features.append(DeprecatedString::fromLatin1("STRUCTURE"));
-        s_features.append(DeprecatedString::fromLatin1("CONTAINERATTRIBUTE"));
-        s_features.append(DeprecatedString::fromLatin1("CONDITIOANLPROCESSING"));
-        s_features.append(DeprecatedString::fromLatin1("IMAGE"));
-        s_features.append(DeprecatedString::fromLatin1("STYLE"));
-        s_features.append(DeprecatedString::fromLatin1("VIEWPORTATTRIBUTE"));
-        s_features.append(DeprecatedString::fromLatin1("SHAPE"));
-        s_features.append(DeprecatedString::fromLatin1("TEXT"));
-        s_features.append(DeprecatedString::fromLatin1("PAINTATTRIBUTE"));
-        s_features.append(DeprecatedString::fromLatin1("OPACITYATTRIBUTE"));
-        s_features.append(DeprecatedString::fromLatin1("GRAPHICSATTRIBUTE"));
-        s_features.append(DeprecatedString::fromLatin1("MARKER"));
-        s_features.append(DeprecatedString::fromLatin1("COLORPROFILE"));
-        s_features.append(DeprecatedString::fromLatin1("GRADIENT"));
-        s_features.append(DeprecatedString::fromLatin1("PATTERN"));
-        s_features.append(DeprecatedString::fromLatin1("CLIP"));
-        s_features.append(DeprecatedString::fromLatin1("MASK"));
-        s_features.append(DeprecatedString::fromLatin1("FILTER"));
-        s_features.append(DeprecatedString::fromLatin1("XLINKATTRIBUTE"));
-        s_features.append(DeprecatedString::fromLatin1("FONT"));
-        s_features.append(DeprecatedString::fromLatin1("EXTENSIBILITY"));
-
-        // 1.0 features
-        //s_features.append(DeprecatedString::fromLatin1("SVG"));
-        s_features.append(DeprecatedString::fromLatin1("SVG.STATIC"));
-        s_features.append(DeprecatedString::fromLatin1("SVG.ANIMATION"));
-        s_features.append(DeprecatedString::fromLatin1("SVG.DYNAMIC"));
-        s_features.append(DeprecatedString::fromLatin1("DOM"));
-        s_features.append(DeprecatedString::fromLatin1("DOM.SVG"));
-        s_features.append(DeprecatedString::fromLatin1("DOM.SVG.STATIC"));
-        s_features.append(DeprecatedString::fromLatin1("DOM.SVG.ANIMATION"));
-        s_features.append(DeprecatedString::fromLatin1("DOM.SVG.DYNAMIC"));
-        s_features.append(DeprecatedString::fromLatin1("SVG.ALL"));
-        s_features.append(DeprecatedString::fromLatin1("DOM.SVG.ALL"));
-    }
 
     return s_instance;
 }
 
 bool SVGDOMImplementation::hasFeature(StringImpl *featureImpl, StringImpl *versionImpl) const
 {
-    DeprecatedString feature = (featureImpl ? String(featureImpl).upper().deprecatedString() : DeprecatedString::null);
+    String feature = (featureImpl ? featureImpl->upper(): String());
     String version(versionImpl);
 
-    if((version.isEmpty() || version == "1.1") &&
-       feature.startsWith(DeprecatedString::fromLatin1("HTTP://WWW.W3.ORG/TR/SVG11/FEATURE#")))
-    {
-        if(s_features.contains(feature.right(feature.length() - 35)))
+    if ((version.isEmpty() || version == "1.1") && feature.startsWith("HTTP://WWW.W3.ORG/TR/SVG11/FEATURE#")) {
+        if (svgFeatureSet().contains(feature.right(feature.length() - 35)))
             return true;
     }
 
-    if((version.isEmpty() || version == "1.0") &&
-       feature.startsWith(DeprecatedString::fromLatin1("ORG.W3C.")))
-    {
-        if(s_features.contains(feature.right(feature.length() - 8)))
+    if ((version.isEmpty() || version == "1.0") && feature.startsWith("ORG.W3C.")) {
+        if (svgFeatureSet().contains(feature.right(feature.length() - 8)))
             return true;
     }
 
@@ -138,13 +140,13 @@ PassRefPtr<DocumentType> SVGDOMImplementation::createDocumentType(StringImpl *qu
     String qualifiedName(qualifiedNameImpl);
 #if 0
     // INVALID_CHARACTER_ERR: Raised if the specified qualified name contains an illegal character.
-    if(!qualifiedName.isEmpty() && !Helper::ValidateAttributeName(qualifiedNameImpl)) {
+    if (!qualifiedName.isEmpty() && !Helper::ValidateAttributeName(qualifiedNameImpl)) {
         ec = INVALID_CHARACTER_ERR;
         return 0;
     }
 
     // NAMESPACE_ERR: Raised if no qualifiedName supplied (not mentioned in the spec!)
-    if(qualifiedName.isEmpty()) {
+    if (qualifiedName.isEmpty()) {
         ec = NAMESPACE_ERR;
         return 0;
     }
@@ -165,7 +167,7 @@ PassRefPtr<Document> SVGDOMImplementation::createDocument(StringImpl *namespaceU
 {
     String namespaceURI(namespaceURIImpl);
     String qualifiedName(qualifiedNameImpl);
-    if((namespaceURI != SVGNames::svgNamespaceURI) || (qualifiedName != "svg" && qualifiedName != "svg:svg"))
+    if ((namespaceURI != SVGNames::svgNamespaceURI) || (qualifiedName != "svg" && qualifiedName != "svg:svg"))
         return DOMImplementation::instance()->createDocument(namespaceURIImpl, qualifiedNameImpl, doctype, ec);
 
 #if 0
@@ -175,7 +177,7 @@ PassRefPtr<Document> SVGDOMImplementation::createDocument(StringImpl *namespaceU
 
     // WRONG_DOCUMENT_ERR: Raised if docType has already been used with a different
     //                     document or was created from a different implementation.
-    if(doctype != 0 && doctype->ownerDocument() != 0) {
+    if (doctype != 0 && doctype->ownerDocument() != 0) {
         ec = WRONG_DOCUMENT_ERR;
         return 0;
     }
index 46a25ef..dc71eeb 100644 (file)
@@ -61,7 +61,6 @@ namespace WebCore {
         bool m_animationContext : 1;
 
         static SVGDOMImplementation *s_instance;
-        static DeprecatedStringList s_features;
     };
 }
 
index 9910ce4..0b6cae4 100644 (file)
@@ -85,7 +85,7 @@ String SVGPaint::cssText() const
     else if(m_paintType == SVG_PAINTTYPE_CURRENTCOLOR)
         return "currentColor";
     else if(m_paintType == SVG_PAINTTYPE_URI)
-        return String(DeprecatedString::fromLatin1("url(") + m_uri.deprecatedString() + DeprecatedString::fromLatin1(")"));
+        return "url(" + m_uri + ")";
 
     return SVGColor::cssText();
 }
index ecc12d7..1def102 100644 (file)
@@ -36,7 +36,7 @@ namespace WebCore
 
         virtual unsigned short pathSegType() const { return PATHSEG_CLOSEPATH; }
         virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("Z"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("Z"); }
+        virtual DeprecatedString toString() const { return "Z"; }
     };
 };
 
index df57c6d..bf05de4 100644 (file)
@@ -75,36 +75,35 @@ void SVGPreserveAspectRatio::parsePreserveAspectRatio(StringImpl *strImpl)
     DeprecatedString str = s.deprecatedString();
     DeprecatedStringList params = DeprecatedStringList::split(' ', str.simplifyWhiteSpace());
 
-    if(params[0].compare(DeprecatedString::fromLatin1("none")) == 0)
+    if (params[0] == "none")
         m_align = SVG_PRESERVEASPECTRATIO_NONE;
-    else if(params[0].compare(DeprecatedString::fromLatin1("xMinYMin")) == 0)
+    else if (params[0] == "xMinYMin")
         m_align = SVG_PRESERVEASPECTRATIO_XMINYMIN;
-    else if(params[0].compare(DeprecatedString::fromLatin1("xMidYMin")) == 0)
+    else if (params[0] == "xMidYMin")
         m_align = SVG_PRESERVEASPECTRATIO_XMIDYMIN;
-    else if(params[0].compare(DeprecatedString::fromLatin1("xMaxYMin")) == 0)
+    else if (params[0] == "xMaxYMin")
         m_align = SVG_PRESERVEASPECTRATIO_XMAXYMIN;
-    else if(params[0].compare(DeprecatedString::fromLatin1("xMinYMid")) == 0)
+    else if (params[0] == "xMinYMid")
         m_align = SVG_PRESERVEASPECTRATIO_XMINYMID;
-    else if(params[0].compare(DeprecatedString::fromLatin1("xMidYMid")) == 0)
+    else if (params[0] == "xMidYMid")
         m_align = SVG_PRESERVEASPECTRATIO_XMIDYMID;
-    else if(params[0].compare(DeprecatedString::fromLatin1("xMaxYMid")) == 0)
+    else if (params[0] == "xMaxYMid")
         m_align = SVG_PRESERVEASPECTRATIO_XMAXYMID;
-    else if(params[0].compare(DeprecatedString::fromLatin1("xMinYMax")) == 0)
+    else if (params[0] == "xMinYMax")
         m_align = SVG_PRESERVEASPECTRATIO_XMINYMAX;
-    else if(params[0].compare(DeprecatedString::fromLatin1("xMidYMax")) == 0)
+    else if (params[0] == "xMidYMax")
         m_align = SVG_PRESERVEASPECTRATIO_XMIDYMAX;
-    else if(params[0].compare(DeprecatedString::fromLatin1("xMaxYMax")) == 0)
+    else if (params[0] == "xMaxYMax")
         m_align = SVG_PRESERVEASPECTRATIO_XMAXYMAX;
 
-    if(m_align != SVG_PRESERVEASPECTRATIO_NONE)
-    {
-        if((params.count() > 1) && params[1].compare(DeprecatedString::fromLatin1("slice")) == 0)
+    if (m_align != SVG_PRESERVEASPECTRATIO_NONE) {
+        if ((params.count() > 1) && (params[1] == "slice"))
             m_meetOrSlice = SVG_MEETORSLICE_SLICE;
         else
             m_meetOrSlice = SVG_MEETORSLICE_MEET;
     }
 
-    if(m_context)
+    if (m_context)
         m_context->notifyAttributeChange();
 }
 
@@ -113,24 +112,24 @@ SVGMatrix *SVGPreserveAspectRatio::getCTM(float logicX, float logicY, float logi
 {
     SVGMatrix *temp = SVGSVGElement::createSVGMatrix();
 
-    if(align() == SVG_PRESERVEASPECTRATIO_UNKNOWN)
+    if (align() == SVG_PRESERVEASPECTRATIO_UNKNOWN)
         return temp;
 
     float vpar = logicWidth / logicHeight;
     float svgar = physWidth / physHeight;
 
-    if(align() == SVG_PRESERVEASPECTRATIO_NONE)
+    if (align() == SVG_PRESERVEASPECTRATIO_NONE)
     {
         temp->scaleNonUniform(physWidth / logicWidth, physHeight / logicHeight);
         temp->translate(-logicX, -logicY);
     }
-    else if(vpar < svgar && (meetOrSlice() == SVG_MEETORSLICE_MEET) || vpar >= svgar && (meetOrSlice() == SVG_MEETORSLICE_SLICE))
+    else if (vpar < svgar && (meetOrSlice() == SVG_MEETORSLICE_MEET) || vpar >= svgar && (meetOrSlice() == SVG_MEETORSLICE_SLICE))
     {
         temp->scale(physHeight / logicHeight);
 
-        if(align() == SVG_PRESERVEASPECTRATIO_XMINYMIN || align() == SVG_PRESERVEASPECTRATIO_XMINYMID || align() == SVG_PRESERVEASPECTRATIO_XMINYMAX)
+        if (align() == SVG_PRESERVEASPECTRATIO_XMINYMIN || align() == SVG_PRESERVEASPECTRATIO_XMINYMID || align() == SVG_PRESERVEASPECTRATIO_XMINYMAX)
             temp->translate(-logicX, -logicY);
-        else if(align() == SVG_PRESERVEASPECTRATIO_XMIDYMIN || align() == SVG_PRESERVEASPECTRATIO_XMIDYMID || align() == SVG_PRESERVEASPECTRATIO_XMIDYMAX)
+        else if (align() == SVG_PRESERVEASPECTRATIO_XMIDYMIN || align() == SVG_PRESERVEASPECTRATIO_XMIDYMID || align() == SVG_PRESERVEASPECTRATIO_XMIDYMAX)
             temp->translate(-logicX - (logicWidth - physWidth * logicHeight / physHeight) / 2, -logicY);
         else
             temp->translate(-logicX - (logicWidth - physWidth * logicHeight / physHeight), -logicY);
@@ -139,9 +138,9 @@ SVGMatrix *SVGPreserveAspectRatio::getCTM(float logicX, float logicY, float logi
     {
         temp->scale(physWidth / logicWidth);
 
-        if(align() == SVG_PRESERVEASPECTRATIO_XMINYMIN || align() == SVG_PRESERVEASPECTRATIO_XMIDYMIN || align() == SVG_PRESERVEASPECTRATIO_XMAXYMIN)
+        if (align() == SVG_PRESERVEASPECTRATIO_XMINYMIN || align() == SVG_PRESERVEASPECTRATIO_XMIDYMIN || align() == SVG_PRESERVEASPECTRATIO_XMAXYMIN)
             temp->translate(-logicX, -logicY);
-        else if(align() == SVG_PRESERVEASPECTRATIO_XMINYMID || align() == SVG_PRESERVEASPECTRATIO_XMIDYMID || align() == SVG_PRESERVEASPECTRATIO_XMAXYMID)
+        else if (align() == SVG_PRESERVEASPECTRATIO_XMINYMID || align() == SVG_PRESERVEASPECTRATIO_XMIDYMID || align() == SVG_PRESERVEASPECTRATIO_XMAXYMID)
             temp->translate(-logicX, -logicY - (logicHeight - physHeight * logicWidth / physWidth) / 2);
         else
             temp->translate(-logicX, -logicY - (logicHeight - physHeight * logicWidth / physWidth));
index 98cfeed..8165fff 100644 (file)
@@ -54,7 +54,7 @@ void SVGStopElement::parseMappedAttribute(MappedAttribute *attr)
 {
     String value(attr->value());
     if (attr->name() == SVGNames::offsetAttr) {
-        if(value.deprecatedString().endsWith(DeprecatedString::fromLatin1("%")))
+        if(value.deprecatedString().endsWith("%"))
             offset()->setBaseVal(value.deprecatedString().left(value.length() - 1).toDouble() / 100.);
         else
             offset()->setBaseVal(value.deprecatedString().toDouble());
index bca7b51..3370ab2 100644 (file)
@@ -96,13 +96,8 @@ void SVGStyleElement::childrenChanged()
         m_sheet = 0;
 
     m_loading = false;
-    const AtomicString &mediaDomString = media();
-    DeprecatedString _media = mediaDomString.deprecatedString();
-    if((type().isEmpty() || type() == "text/css") && (_media.isNull() ||
-        _media.contains(DeprecatedString::fromLatin1("screen")) ||
-        _media.contains(DeprecatedString::fromLatin1("all")) |
-        _media.contains(DeprecatedString::fromLatin1("print"))))
-    {
+    const AtomicString& _media = media();
+    if ((type().isEmpty() || type() == "text/css") && (_media.isNull() || _media.contains("screen") || _media.contains("all") || _media.contains("print"))) {
         ownerDocument()->addPendingSheet();
 
         m_loading = true;
@@ -110,7 +105,7 @@ void SVGStyleElement::childrenChanged()
         m_sheet = new CSSStyleSheet(this);
         m_sheet->parseString(textContent(), false);//!getDocument()->inCompatMode());
 
-        MediaList *mediaList = new MediaList(m_sheet.get(), mediaDomString);
+        MediaList *mediaList = new MediaList(m_sheet.get(), _media);
         m_sheet->setMedia(mediaList);
         m_loading = false;
     }
index 7aa8d64..256d314 100644 (file)
@@ -60,7 +60,7 @@ bool SVGURIReference::parseMappedAttribute(MappedAttribute *attr)
 
 DeprecatedString SVGURIReference::getTarget(const DeprecatedString &url)
 {
-    if(url.startsWith(DeprecatedString::fromLatin1("url("))) // URI References, ie. fill:url(#target)
+    if(url.startsWith("url(")) // URI References, ie. fill:url(#target)
     {
         unsigned int start = url.find('#') + 1;
         unsigned int end = url.findRev(')');
index 569a76c..d2a456c 100644 (file)
@@ -219,13 +219,12 @@ String ClipboardMac::getData(const String &type, bool &success) const
 
 bool ClipboardMac::setData(const String &type, const String &data)
 {
-    if (m_policy != Writable) {
+    if (m_policy != Writable)
         return false;
-    }
     // note NSPasteboard enforces changeCount itself on writing - can't write if not the owner
 
     NSString *cocoaType = cocoaTypeFromMIMEType(type);
-    NSString *cocoaData = data.deprecatedString().getNSString();
+    NSString *cocoaData = data;
 
     if ([cocoaType isEqualToString:NSURLPboardType]) {
         [m_pasteboard addTypes:[NSArray arrayWithObject:NSURLPboardType] owner:nil];
@@ -253,17 +252,15 @@ bool ClipboardMac::setData(const String &type, const String &data)
 
 DeprecatedStringList ClipboardMac::types() const
 {
-    if (m_policy != Readable && m_policy != TypesReadable) {
+    if (m_policy != Readable && m_policy != TypesReadable)
         return DeprecatedStringList();
-    }
 
     NSArray *types = [m_pasteboard types];
 
     // Enforce changeCount ourselves for security.  We check after reading instead of before to be
     // sure it doesn't change between our testing the change count and accessing the data.
-    if (m_changeCount != [m_pasteboard changeCount]) {
+    if (m_changeCount != [m_pasteboard changeCount])
         return DeprecatedStringList();
-    }
 
     DeprecatedStringList result;
     if (types) {
@@ -271,14 +268,12 @@ DeprecatedStringList ClipboardMac::types() const
         unsigned i;
         for (i = 0; i < count; i++) {
             NSString *pbType = [types objectAtIndex:i];
-            if ([pbType isEqualToString:@"NeXT plain ascii pasteboard type"]) {
+            if ([pbType isEqualToString:@"NeXT plain ascii pasteboard type"])
                 continue;   // skip this ancient type that gets auto-supplied by some system conversion
-            }
 
             DeprecatedString qstr = MIMETypeFromCocoaType(pbType);
-            if (!result.contains(qstr)) {
+            if (!result.contains(qstr))
                 result.append(qstr);
-            }
         }
     }
     return result;
@@ -382,9 +377,8 @@ WebCore::String ClipboardMac::effectAllowed() const
 
 void ClipboardMac::setEffectAllowed(const WebCore::String &s)
 {
-    if (m_policy == Writable) {
+    if (m_policy == Writable)
         m_effectAllowed = s;
-    }
 }
 
 // These "conversion" methods are called by the bridge and part, and never make sense to JS, so we don't
@@ -408,9 +402,8 @@ static NSDragOperation cocoaOpFromIEOp(const String &op) {
         return NSDragOperationLink | NSDragOperationGeneric | NSDragOperationMove;
     } else if (op == "all") {
         return NSDragOperationEvery;
-    } else {
+    } else
         return NSDragOperationPrivate;  // really a marker for "no conversion"
-    }
 }
 
 static const String IEOpFromCocoaOp(NSDragOperation op) {
@@ -431,16 +424,15 @@ static const String IEOpFromCocoaOp(NSDragOperation op) {
         return "copy";
     } else if (op & NSDragOperationLink) {
         return "link";
-    } else {
+    } else
         return "none";
-    }
 }
 
 bool ClipboardMac::sourceOperation(NSDragOperation *op) const
 {
-    if (m_effectAllowed.isNull()) {
+    if (m_effectAllowed.isNull())
         return false;
-    else {
+    else {
         *op = cocoaOpFromIEOp(m_effectAllowed);
         return true;
     }
@@ -448,9 +440,9 @@ bool ClipboardMac::sourceOperation(NSDragOperation *op) const
 
 bool ClipboardMac::destinationOperation(NSDragOperation *op) const
 {
-    if (m_dropEffect.isNull()) {
+    if (m_dropEffect.isNull())
         return false;
-    else {
+    else {
         *op = cocoaOpFromIEOp(m_dropEffect);
         return true;
     }
index 5bd0f27..4ef1918 100644 (file)
@@ -237,9 +237,8 @@ DeprecatedCString& DeprecatedCString::append(const char *s)
         if (len2) {
             detach();
             unsigned len1 = length();
-            if (DeprecatedByteArray::resize(len1 + len2 + 1)) {
+            if (DeprecatedByteArray::resize(len1 + len2 + 1))
                 memcpy(data() + len1, s, len2 + 1);
-            }
         }
     }
 
index 7c63190..fd4a879 100644 (file)
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
  */
 
-#ifndef KHTML_SETTINGS_H_
-#define KHTML_SETTINGS_H_
+#ifndef WebCoreSettings_H
+#define WebCoreSettings_H
 
-#include "DeprecatedString.h"
-#include "DeprecatedStringList.h"
+#include "PlatformString.h"
 #include "Font.h"
 
 class KHTMLSettings
@@ -39,88 +38,76 @@ public:
         KAnimationEnabled
     };
     
-    KHTMLSettings() { }
-    
-    const DeprecatedString &stdFontName() const { return _stdFontName; }
-    const DeprecatedString &fixedFontName() const { return _fixedFontName; }
-    const DeprecatedString &serifFontName() const { return _serifFontName; }
-    const DeprecatedString &sansSerifFontName() const { return _sansSerifFontName; }
-    const DeprecatedString &cursiveFontName() const { return _cursiveFontName; }
-    const DeprecatedString &fantasyFontName() const { return _fantasyFontName; }
-
-    int minFontSize() const { return _minimumFontSize; }
-    int minLogicalFontSize() const { return _minimumLogicalFontSize; }
-    int mediumFontSize() const { return _defaultFontSize; }
-    int mediumFixedFontSize() const { return _defaultFixedFontSize; }
+    const WebCore::AtomicString& stdFontName() const { return m_stdFontName; }
+    const WebCore::AtomicString& fixedFontName() const { return m_fixedFontName; }
+    const WebCore::AtomicString& serifFontName() const { return m_serifFontName; }
+    const WebCore::AtomicString& sansSerifFontName() const { return m_sansSerifFontName; }
+    const WebCore::AtomicString& cursiveFontName() const { return m_cursiveFontName; }
+    const WebCore::AtomicString& fantasyFontName() const { return m_fantasyFontName; }
 
-    static bool changeCursor() { return true; }
+    int minFontSize() const { return m_minimumFontSize; }
+    int minLogicalFontSize() const { return m_minimumLogicalFontSize; }
+    int mediumFontSize() const { return m_defaultFontSize; }
+    int mediumFixedFontSize() const { return m_defaultFixedFontSize; }
 
-    static bool isFormCompletionEnabled() { return false; }
-    static int maxFormCompletionItems() { return 0; }
+    bool autoLoadImages() const { return m_willLoadImagesAutomatically; }
 
-    bool autoLoadImages() const { return _willLoadImagesAutomatically; }
-    static KAnimationAdvice showAnimations() { return KAnimationEnabled; }
-
-    bool isJavaScriptEnabled() const { return _JavaScriptEnabled; }
-    bool JavaScriptCanOpenWindowsAutomatically() const { return _JavaScriptCanOpenWindowsAutomatically; }
-    bool isJavaScriptEnabled(const DeprecatedString &host) const { return _JavaScriptEnabled; }
-    bool isJavaEnabled() const { return _JavaEnabled; }
-    bool isJavaEnabled(const DeprecatedString &host) const { return _JavaEnabled; }
-    bool isPluginsEnabled() const { return _pluginsEnabled; }
-    bool isPluginsEnabled(const DeprecatedString &host) const { return _pluginsEnabled; }
+    bool isJavaScriptEnabled() const { return m_javaScriptEnabled; }
+    bool JavaScriptCanOpenWindowsAutomatically() const { return m_javaScriptCanOpenWindowsAutomatically; }
+    bool isJavaEnabled() const { return m_javaEnabled; }
+    bool isPluginsEnabled() const { return m_pluginsEnabled; }
     
-    const DeprecatedString &encoding() const { return _encoding; }
+    const DeprecatedString& encoding() const { return m_encoding; }
 
-    const DeprecatedString &userStyleSheet() const { return _userStyleSheetLocation; }
-    bool shouldPrintBackgrounds() const { return _shouldPrintBackgrounds; }
-    bool textAreasAreResizable() const { return _textAreasAreResizable; }
+    DeprecatedString userStyleSheet() const { return m_userStyleSheetLocation; }
+    bool shouldPrintBackgrounds() const { return m_shouldPrintBackgrounds; }
+    bool textAreasAreResizable() const { return m_textAreasAreResizable; }
 
-    void setStdFontName(const DeprecatedString &s) { _stdFontName = s; }
-    void setFixedFontName(const DeprecatedString &s) { _fixedFontName = s; }
-    void setSerifFontName(const DeprecatedString &s) { _serifFontName = s; }
-    void setSansSerifFontName(const DeprecatedString &s) { _sansSerifFontName = s; }
-    void setCursiveFontName(const DeprecatedString &s) { _cursiveFontName = s; }
-    void setFantasyFontName(const DeprecatedString &s) { _fantasyFontName = s; }
+    void setStdFontName(const WebCore::AtomicString& s) { m_stdFontName = s; }
+    void setFixedFontName(const WebCore::AtomicString& s) { m_fixedFontName = s; }
+    void setSerifFontName(const WebCore::AtomicString& s) { m_serifFontName = s; }
+    void setSansSerifFontName(const WebCore::AtomicString& s) { m_sansSerifFontName = s; }
+    void setCursiveFontName(const WebCore::AtomicString& s) { m_cursiveFontName = s; }
+    void setFantasyFontName(const WebCore::AtomicString& s) { m_fantasyFontName = s; }
     
-    void setMinFontSize(int s) { _minimumFontSize = s; }
-    void setMinLogicalFontSize(int s) { _minimumLogicalFontSize = s; }
-    void setMediumFontSize(int s) { _defaultFontSize = s; }
-    void setMediumFixedFontSize(int s) { _defaultFixedFontSize = s; }
+    void setMinFontSize(int s) { m_minimumFontSize = s; }
+    void setMinLogicalFontSize(int s) { m_minimumLogicalFontSize = s; }
+    void setMediumFontSize(int s) { m_defaultFontSize = s; }
+    void setMediumFixedFontSize(int s) { m_defaultFixedFontSize = s; }
     
-    void setAutoLoadImages(bool f) { _willLoadImagesAutomatically = f; }
-    void setIsJavaScriptEnabled(bool f) { _JavaScriptEnabled = f; }
-    void setIsJavaEnabled(bool f) { _JavaEnabled = f; }
-    void setArePluginsEnabled(bool f) { _pluginsEnabled = f; }
-    void setJavaScriptCanOpenWindowsAutomatically(bool f) { _JavaScriptCanOpenWindowsAutomatically = f; }
+    void setAutoLoadImages(bool f) { m_willLoadImagesAutomatically = f; }
+    void setIsJavaScriptEnabled(bool f) { m_javaScriptEnabled = f; }
+    void setIsJavaEnabled(bool f) { m_javaEnabled = f; }
+    void setArePluginsEnabled(bool f) { m_pluginsEnabled = f; }
+    void setJavaScriptCanOpenWindowsAutomatically(bool f) { m_javaScriptCanOpenWindowsAutomatically = f; }
 
-    void setEncoding(const DeprecatedString &s) { _encoding = s; }
+    void setEncoding(const DeprecatedString& s) { m_encoding = s; }
 
-    void setUserStyleSheet(const DeprecatedString &s) { _userStyleSheetLocation = s; }
-    void setShouldPrintBackgrounds(bool f) { _shouldPrintBackgrounds = f; }
-    void setTextAreasAreResizable(bool f) { _textAreasAreResizable = f; }
+    void setUserStyleSheet(const DeprecatedString& s) { m_userStyleSheetLocation = s; }
+    void setShouldPrintBackgrounds(bool f) { m_shouldPrintBackgrounds = f; }
+    void setTextAreasAreResizable(bool f) { m_textAreasAreResizable = f; }
     
 private:
-    DeprecatedString _stdFontName;
-    DeprecatedString _fixedFontName;
-    DeprecatedString _serifFontName;
-    DeprecatedString _sansSerifFontName;
-    DeprecatedString _cursiveFontName;
-    DeprecatedString _fantasyFontName;
-    DeprecatedString _encoding;
-    DeprecatedString _userStyleSheetLocation;
-    
-    int _minimumFontSize;
-    int _minimumLogicalFontSize;
-    int _defaultFontSize;
-    int _defaultFixedFontSize;
-    bool _JavaEnabled : 1;
-    bool _willLoadImagesAutomatically : 1;
-    bool _pluginsEnabled : 1;
-    bool _JavaScriptEnabled : 1;
-    bool _JavaScriptCanOpenWindowsAutomatically : 1;
-    bool _shouldPrintBackgrounds : 1;
-    bool _textAreasAreResizable : 1;
+    WebCore::AtomicString m_stdFontName;
+    WebCore::AtomicString m_fixedFontName;
+    WebCore::AtomicString m_serifFontName;
+    WebCore::AtomicString m_sansSerifFontName;
+    WebCore::AtomicString m_cursiveFontName;
+    WebCore::AtomicString m_fantasyFontName;
+    DeprecatedString m_encoding; // FIXME: TextEncoding takes a latin1 string, which String & AtomicString don't easily produce
+    DeprecatedString m_userStyleSheetLocation; // FIXME: KURLs still use DeprecatedString
     
+    int m_minimumFontSize;
+    int m_minimumLogicalFontSize;
+    int m_defaultFontSize;
+    int m_defaultFixedFontSize;
+    bool m_javaEnabled : 1;
+    bool m_willLoadImagesAutomatically : 1;
+    bool m_pluginsEnabled : 1;
+    bool m_javaScriptEnabled : 1;
+    bool m_javaScriptCanOpenWindowsAutomatically : 1;
+    bool m_shouldPrintBackgrounds : 1;
+    bool m_textAreasAreResizable : 1;
 };
 
 #endif
index d510514..0f5ce8c 100644 (file)
@@ -135,11 +135,10 @@ DeprecatedByteArray KWQServeSynchronousRequest(Loader *loader, DocLoader *docLoa
     BEGIN_BLOCK_OBJC_EXCEPTIONS;
 
     NSDictionary *headerDict = nil;
-    DeprecatedString headerString = job->queryMetaData("customHTTPHeader");
+    String headerString = job->queryMetaData("customHTTPHeader");
 
-    if (!headerString.isEmpty()) {
-        headerDict = [[NSDictionary _webcore_dictionaryWithHeaderString:headerString.getNSString()] retain];
-    }
+    if (!headerString.isEmpty())
+        headerDict = [[NSDictionary _webcore_dictionaryWithHeaderString:headerString] retain];
 
     NSArray *postData = nil;
     if (job->postData().count() > 0)
index 7a3f523..e07bb11 100644 (file)
@@ -26,7 +26,7 @@
 #include "config.h"
 #include "KWQTextStream.h"
 
-#include "DeprecatedString.h"
+#include "PlatformString.h"
 #include "Logging.h"
 
 const size_t integerOrPointerAsStringBufferSize = 100; // large enough for any integer or pointer in string format, including trailing null character
@@ -34,137 +34,139 @@ const char *precisionFormats[] = { "%.0f", "%.1f", "%.2f", "%.3f", "%.4f", "%.5f
 const int maxPrecision = 6; // must match to precisionFormats
 const int defaultPrecision = 6; // matches qt and sprintf(.., "%f", ...) behaviour
 
-QTextStream::QTextStream(const DeprecatedByteArray &ba)
-    : _hasByteArray(true), _byteArray(ba), _string(0), _precision(defaultPrecision)
+QTextStream::QTextStream(const DeprecatedByteArrayba)
+    : m_hasByteArray(true), m_byteArray(ba), m_string(0), m_precision(defaultPrecision)
 {
 }
 
-QTextStream::QTextStream(DeprecatedString *s)
-    : _hasByteArray(false), _string(s), _precision(defaultPrecision)
+QTextStream::QTextStream(DeprecatedStrings)
+    : m_hasByteArray(false), m_string(s), m_precision(defaultPrecision)
 {
 }
 
-QTextStream &QTextStream::operator<<(char c)
+QTextStreamQTextStream::operator<<(char c)
 {
-    if (_hasByteArray) {
-        unsigned oldSize = _byteArray.size();
-        _byteArray.resize(oldSize + 1);
-        _byteArray[oldSize] = c;
-    }
-    if (_string) {
-        _string->append(QChar(c));
+    if (m_hasByteArray) {
+        unsigned oldSize = m_byteArray.size();
+        m_byteArray.resize(oldSize + 1);
+        m_byteArray[oldSize] = c;
     }
+    if (m_string)
+        m_string->append(QChar(c));
     return *this;
 }
 
-QTextStream &QTextStream::operator<<(short i)
+QTextStreamQTextStream::operator<<(short i)
 {
     char buffer[integerOrPointerAsStringBufferSize];
     sprintf(buffer, "%d", i);
     return *this << buffer;
 }
 
-QTextStream &QTextStream::operator<<(unsigned short i)
+QTextStreamQTextStream::operator<<(unsigned short i)
 {
     char buffer[integerOrPointerAsStringBufferSize];
     sprintf(buffer, "%u", i);
     return *this << buffer;
 }
 
-QTextStream &QTextStream::operator<<(int i)
+QTextStreamQTextStream::operator<<(int i)
 {
     char buffer[integerOrPointerAsStringBufferSize];
     sprintf(buffer, "%d", i);
     return *this << buffer;
 }
 
-QTextStream &QTextStream::operator<<(unsigned i)
+QTextStreamQTextStream::operator<<(unsigned i)
 {
     char buffer[integerOrPointerAsStringBufferSize];
     sprintf(buffer, "%u", i);
     return *this << buffer;
 }
 
-QTextStream &QTextStream::operator<<(long i)
+QTextStreamQTextStream::operator<<(long i)
 {
     char buffer[integerOrPointerAsStringBufferSize];
     sprintf(buffer, "%ld", i);
     return *this << buffer;
 }
 
-QTextStream &QTextStream::operator<<(unsigned long i)
+QTextStreamQTextStream::operator<<(unsigned long i)
 {
     char buffer[integerOrPointerAsStringBufferSize];
     sprintf(buffer, "%lu", i);
     return *this << buffer;
 }
 
-QTextStream &QTextStream::operator<<(float f)
+QTextStreamQTextStream::operator<<(float f)
 {
     char buffer[integerOrPointerAsStringBufferSize];
-    sprintf(buffer, precisionFormats[_precision], f);
+    sprintf(buffer, precisionFormats[m_precision], f);
     return *this << buffer;
 }
 
-QTextStream &QTextStream::operator<<(double d)
+QTextStreamQTextStream::operator<<(double d)
 {
     char buffer[integerOrPointerAsStringBufferSize];
-    sprintf(buffer, precisionFormats[_precision], d);
+    sprintf(buffer, precisionFormats[m_precision], d);
     return *this << buffer;
 }
 
-QTextStream &QTextStream::operator<<(const char *s)
+QTextStream& QTextStream::operator<<(const char* s)
 {
-    if (_hasByteArray) {
+    if (m_hasByteArray) {
         unsigned length = strlen(s);
-        unsigned oldSize = _byteArray.size();
-        _byteArray.resize(oldSize + length);
-        memcpy(_byteArray.data() + oldSize, s, length);
-    }
-    if (_string) {
-        _string->append(s);
+        unsigned oldSize = m_byteArray.size();
+        m_byteArray.resize(oldSize + length);
+        memcpy(m_byteArray.data() + oldSize, s, length);
     }
+    if (m_string)
+        m_string->append(s);
     return *this;
 }
 
-QTextStream &QTextStream::operator<<(const DeprecatedCString &qcs)
+QTextStream& QTextStream::operator<<(const DeprecatedCString& qcs)
 {
     const char *s = qcs;
     return *this << s;
 }
 
-QTextStream &QTextStream::operator<<(const DeprecatedString &s)
+QTextStream& QTextStream::operator<<(const DeprecatedString& s)
 {
-    if (_hasByteArray) {
+    if (m_hasByteArray) {
         unsigned length = s.length();
-        unsigned oldSize = _byteArray.size();
-        _byteArray.resize(oldSize + length);
-        memcpy(_byteArray.data() + oldSize, s.latin1(), length);
-    }
-    if (_string) {
-        _string->append(s);
+        unsigned oldSize = m_byteArray.size();
+        m_byteArray.resize(oldSize + length);
+        memcpy(m_byteArray.data() + oldSize, s.latin1(), length);
     }
+    if (m_string)
+        m_string->append(s);
     return *this;
 }
 
-QTextStream &QTextStream::operator<<(void *p)
+QTextStream& QTextStream::operator<<(const WebCore::String& s)
+{
+    return (*this) << s.deprecatedString();
+}
+
+QTextStream& QTextStream::operator<<(void* p)
 {
     char buffer[integerOrPointerAsStringBufferSize];
     sprintf(buffer, "%p", p);
     return *this << buffer;
 }
 
-QTextStream &QTextStream::operator<<(const QTextStreamManipulator &m) 
+QTextStream& QTextStream::operator<<(const QTextStreamManipulator& m) 
 {
     return m(*this);
 }
 
 int QTextStream::precision(int p) 
 {
-    int oldPrecision = _precision;
+    int oldPrecision = m_precision;
     
     if (p >= 0 && p <= maxPrecision)
-        _precision = p;
+        m_precision = p;
 
     return oldPrecision;
 }
index 5e4f1b6..3a14f10 100644 (file)
@@ -23,8 +23,8 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
  */
 
-#ifndef QTEXTSTREAM_H_
-#define QTEXTSTREAM_H_
+#ifndef QTextStream_H
+#define QTextStream_H
 
 #include "DeprecatedArray.h"
 
@@ -33,40 +33,45 @@ class QChar;
 class DeprecatedString;
 class QTextStream;
 
-typedef QTextStream &(*QTextStreamManipulator)(QTextStream &);
+namespace WebCore {
+    class String;
+}
+
+typedef QTextStream &(*QTextStreamManipulator)(QTextStream&);
 
 QTextStream &endl(QTextStream& stream);
 
 class QTextStream {
 public:
-    QTextStream(const DeprecatedByteArray &);
+    QTextStream(const DeprecatedByteArray&);
     QTextStream(DeprecatedString*);
 
-    QTextStream &operator<<(char);
-    QTextStream &operator<<(const QChar &);
-    QTextStream &operator<<(short);
-    QTextStream &operator<<(unsigned short);
-    QTextStream &operator<<(int);
-    QTextStream &operator<<(unsigned);
-    QTextStream &operator<<(long);
-    QTextStream &operator<<(unsigned long);
-    QTextStream &operator<<(float);
-    QTextStream &operator<<(double);
-    QTextStream &operator<<(const char *);
-    QTextStream &operator<<(const DeprecatedString &);
-    QTextStream &operator<<(const DeprecatedCString &);
-    QTextStream &operator<<(void *);
+    QTextStream& operator<<(char);
+    QTextStream& operator<<(const QChar&);
+    QTextStream& operator<<(short);
+    QTextStream& operator<<(unsigned short);
+    QTextStream& operator<<(int);
+    QTextStream& operator<<(unsigned);
+    QTextStream& operator<<(long);
+    QTextStream& operator<<(unsigned long);
+    QTextStream& operator<<(float);
+    QTextStream& operator<<(double);
+    QTextStream& operator<<(const char*);
+    QTextStream& operator<<(const WebCore::String&);
+    QTextStream& operator<<(const DeprecatedString&);
+    QTextStream& operator<<(const DeprecatedCString&);
+    QTextStream& operator<<(void*);
 
-    QTextStream &operator<<(const QTextStreamManipulator &);
-    int precision(int p);
+    QTextStream& operator<<(const QTextStreamManipulator&);
+    int precision(int);
 private:
-    QTextStream(const QTextStream &);
-    QTextStream &operator=(const QTextStream &);
+    QTextStream(const QTextStream&);
+    QTextStream& operator=(const QTextStream&);
 
-    bool _hasByteArray;
-    DeprecatedByteArray _byteArray;
-    DeprecatedString *_string;
-    int _precision;
+    bool m_hasByteArray;
+    DeprecatedByteArray m_byteArray;
+    DeprecatedString* m_string;
+    int m_precision;
 };
 
 #endif
index ac6defe..f2afacc 100644 (file)
@@ -218,30 +218,30 @@ static QTextStream &operator<<(QTextStream &ts, const RenderObject &o)
     }
     
     if (o.isTableCell()) {
-        const RenderTableCell &c = static_cast<const RenderTableCell &>(o);
+        const RenderTableCell& c = static_cast<const RenderTableCell&>(o);
         ts << " [r=" << c.row() << " c=" << c.col() << " rs=" << c.rowSpan() << " cs=" << c.colSpan() << "]";
     }
 
     return ts;
 }
 
-static DeprecatedString quoteAndEscapeNonPrintables(const DeprecatedString &s)
+static String quoteAndEscapeNonPrintables(const String& s)
 {
     DeprecatedString result;
     result += '"';
     for (unsigned i = 0; i != s.length(); ++i) {
-        QChar c = s.at(i);
-        if (c == '\\') {
+        QChar c = s[i];
+        if (c == '\\')
             result += "\\\\";
-        } else if (c == '"') {
+        else if (c == '"')
             result += "\\\"";
-        } else if (c == '\n' || c.unicode() == 0x00A0) {
+        else if (c == '\n' || c.unicode() == 0x00A0)
             result += ' ';
-        else {
+        else {
             unsigned short u = c.unicode();
-            if (u >= 0x20 && u < 0x7F) {
+            if (u >= 0x20 && u < 0x7F)
                 result += c;
-            else {
+            else {
                 DeprecatedString hex;
                 hex.sprintf("\\x{%X}", u);
                 result += hex;
@@ -252,7 +252,7 @@ static DeprecatedString quoteAndEscapeNonPrintables(const DeprecatedString &s)
     return result;
 }
 
-static void writeTextRun(QTextStream &ts, const RenderText &o, const InlineTextBox &run)
+static void writeTextRun(QTextStream& ts, const RenderText& o, const InlineTextBox& run)
 {
     ts << "text run at (" << run.m_x << "," << run.m_y << ") width " << run.m_width;
     if (run.m_reversed || run.m_dirOverride) {
@@ -261,7 +261,7 @@ static void writeTextRun(QTextStream &ts, const RenderText &o, const InlineTextB
             ts << " override";
     }
     ts << ": "
-        << quoteAndEscapeNonPrintables(o.data().deprecatedString().mid(run.m_start, run.m_len))
+        << quoteAndEscapeNonPrintables(o.data().substring(run.m_start, run.m_len))
         << "\n"; 
 }
 
@@ -270,11 +270,11 @@ void write(QTextStream &ts, const RenderObject &o, int indent)
 #if SVG_SUPPORT
     // FIXME:  A hackish way to doing our own "virtual" dispatch
     if (o.isRenderPath()) {
-        write(ts, static_cast<const RenderPath &>(o), indent);
+        write(ts, static_cast<const RenderPath&>(o), indent);
         return;
     }
     if (o.isKCanvasContainer()) {
-        write(ts, static_cast<const KCanvasContainer &>(o), indent);
+        write(ts, static_cast<const KCanvasContainer&>(o), indent);
         return;
     }
 #endif
@@ -283,25 +283,24 @@ void write(QTextStream &ts, const RenderObject &o, int indent)
     ts << o << "\n";
     
     if (o.isText() && !o.isBR()) {
-        const RenderText &text = static_cast<const RenderText &>(o);
+        const RenderText& text = static_cast<const RenderText&>(o);
         for (InlineTextBox* box = text.firstTextBox(); box; box = box->nextTextBox()) {
             writeIndent(ts, indent+1);
             writeTextRun(ts, text, *box);
         }
     }
 
-    for (RenderObject *child = o.firstChild(); child; child = child->nextSibling()) {
-        if (child->layer()) {
+    for (RenderObjectchild = o.firstChild(); child; child = child->nextSibling()) {
+        if (child->layer())
             continue;
-        }
         write(ts, *child, indent + 1);
     }
     
     if (o.isWidget()) {
-        Widget *widget = static_cast<const RenderWidget &>(o).widget();
+        Widget* widget = static_cast<const RenderWidget&>(o).widget();
         if (widget && widget->isFrameView()) {
-            FrameView *view = static_cast<FrameView *>(widget);
-            RenderObject *root = view->frame()->renderer();
+            FrameView* view = static_cast<FrameView*>(widget);
+            RenderObjectroot = view->frame()->renderer();
             if (root) {
                 view->layout();
                 RenderLayer* l = root->layer();
@@ -406,7 +405,7 @@ static void writeSelection(QTextStream &ts, const RenderObject *o)
     if (!n || !n->isDocumentNode())
         return;
 
-    Document *doc = static_cast<Document *>(n);
+    Document *doc = static_cast<Document*>(n);
     Frame *frame = doc->frame();
     if (!frame)
         return;
index 0099966..0e9bc36 100644 (file)
@@ -83,16 +83,16 @@ void Cache::init()
     if (!cache)
         cache = new CacheMap;
 
-    if ( !docloader )
+    if (!docloader)
         docloader = new DeprecatedPtrList<DocLoader>;
 
-    if ( !nullImage )
+    if (!nullImage)
         nullImage = new Image;
 
-    if ( !brokenImage )
+    if (!brokenImage)
         brokenImage = Image::loadResource("missing_image");
 
-    if ( !m_loader )
+    if (!m_loader)
         m_loader = new Loader();
 }
 
@@ -110,18 +110,18 @@ void Cache::clear()
     delete docloader; docloader = 0;
 }
 
-CachedImage *Cache::requestImage( DocLoader* dl, const String & url, bool reload, time_t _expireDate )
+CachedImage *Cache::requestImage(DocLoader* dl, const String& url, bool reload, time_t expireDate)
 {
     // this brings the _url to a standard form...
     KURL kurl;
     if (dl)
-        kurl = dl->m_doc->completeURL( url.deprecatedString() );
+        kurl = dl->m_doc->completeURL(url.deprecatedString());
     else
         kurl = url.deprecatedString();
-    return requestImage(dl, kurl, reload, _expireDate);
+    return requestImage(dl, kurl, reload, expireDate);
 }
 
-CachedImage *Cache::requestImage( DocLoader* dl, const KURL & url, bool reload, time_t _expireDate )
+CachedImage *Cache::requestImage(DocLoader* dl, const KURL& url, bool reload, time_t expireDate)
 {
     KIO::CacheControl cachePolicy;
     if (dl)
@@ -131,20 +131,18 @@ CachedImage *Cache::requestImage( DocLoader* dl, const KURL & url, bool reload,
 
     // Checking if the URL is malformed is lots of extra work for little benefit.
 
-    if (!dl->doc()->shouldCreateRenderers()){
+    if (!dl->doc()->shouldCreateRenderers())
         return 0;
-    }
 
     CachedObject *o = 0;
     if (!reload)
         o = cache->get(String(url.url()).impl());
-    if(!o)
-    {
+    if (!o) {
 #ifdef CACHE_DEBUG
-        kdDebug( 6060 ) << "Cache: new: " << url.url() << endl;
+        kdDebug(6060) << "Cache: new: " << url.url() << endl;
 #endif
-        CachedImage *im = new CachedImage(dl, url.url(), cachePolicy, _expireDate);
-        if ( dl && dl->autoloadImages() ) Cache::loader()->load(dl, im, true);
+        CachedImage *im = new CachedImage(dl, url.url(), cachePolicy, expireDate);
+        if (dl && dl->autoloadImages()) Cache::loader()->load(dl, im, true);
         if (cacheDisabled)
             im->setFree(true);
         else {
@@ -159,33 +157,30 @@ CachedImage *Cache::requestImage( DocLoader* dl, const KURL & url, bool reload,
         return 0;
 
 #ifdef CACHE_DEBUG
-    if( o->status() == CachedObject::Pending )
-        kdDebug( 6060 ) << "Cache: loading in progress: " << kurl.url() << endl;
+    if (o->status() == CachedObject::Pending)
+        kdDebug(6060) << "Cache: loading in progress: " << kurl.url() << endl;
     else
-        kdDebug( 6060 ) << "Cache: using cached: " << kurl.url() << ", status " << o->status() << endl;
+        kdDebug(6060) << "Cache: using cached: " << kurl.url() << ", status " << o->status() << endl;
 #endif
 
     moveToHeadOfLRUList(o);
-    if ( dl ) {
-        dl->m_docObjects.remove( o );
+    if (dl) {
+        dl->m_docObjects.remove(o);
         if (!cacheDisabled)
-        dl->m_docObjects.append( o );
+        dl->m_docObjects.append(o);
     }
     return static_cast<CachedImage *>(o);
 }
 
-CachedCSSStyleSheet *Cache::requestStyleSheet( DocLoader* dl, const String & url, bool reload, time_t _expireDate, const DeprecatedString& charset)
+CachedCSSStyleSheet *Cache::requestStyleSheet(DocLoader* dl, const String& url, bool reload, time_t expireDate, const DeprecatedString& charset)
 {
     // this brings the _url to a standard form...
     KURL kurl;
     KIO::CacheControl cachePolicy;
-    if ( dl )
-    {
-        kurl = dl->m_doc->completeURL( url.deprecatedString() );
+    if (dl) {
+        kurl = dl->m_doc->completeURL(url.deprecatedString());
         cachePolicy = dl->cachePolicy();
-    }
-    else
-    {
+    } else {
         kurl = url.deprecatedString();
         cachePolicy = KIO::CC_Verify;
     }
@@ -193,12 +188,11 @@ CachedCSSStyleSheet *Cache::requestStyleSheet( DocLoader* dl, const String & url
     // Checking if the URL is malformed is lots of extra work for little benefit.
 
     CachedObject *o = cache->get(String(kurl.url()).impl());
-    if(!o)
-    {
+    if (!o) {
 #ifdef CACHE_DEBUG
-        kdDebug( 6060 ) << "Cache: new: " << kurl.url() << endl;
+        kdDebug(6060) << "Cache: new: " << kurl.url() << endl;
 #endif
-        CachedCSSStyleSheet *sheet = new CachedCSSStyleSheet(dl, kurl.url(), cachePolicy, _expireDate, charset);
+        CachedCSSStyleSheet *sheet = new CachedCSSStyleSheet(dl, kurl.url(), cachePolicy, expireDate, charset);
         if (cacheDisabled)
             sheet->setFree(true);
         else {
@@ -209,31 +203,31 @@ CachedCSSStyleSheet *Cache::requestStyleSheet( DocLoader* dl, const String & url
     }
 
     
-    if(o->type() != CachedObject::CSSStyleSheet)
+    if (o->type() != CachedObject::CSSStyleSheet)
     {
 #ifdef CACHE_DEBUG
-        kdDebug( 6060 ) << "Cache::Internal Error in requestStyleSheet url=" << kurl.url() << "!" << endl;
+        kdDebug(6060) << "Cache::Internal Error in requestStyleSheet url=" << kurl.url() << "!" << endl;
 #endif
         return 0;
     }
 
 #ifdef CACHE_DEBUG
-    if( o->status() == CachedObject::Pending )
-        kdDebug( 6060 ) << "Cache: loading in progress: " << kurl.url() << endl;
+    if (o->status() == CachedObject::Pending)
+        kdDebug(6060) << "Cache: loading in progress: " << kurl.url() << endl;
     else
-        kdDebug( 6060 ) << "Cache: using cached: " << kurl.url() << endl;
+        kdDebug(6060) << "Cache: using cached: " << kurl.url() << endl;
 #endif
 
     moveToHeadOfLRUList(o);
-    if ( dl ) {
-        dl->m_docObjects.remove( o );
+    if (dl) {
+        dl->m_docObjects.remove(o);
         if (!cacheDisabled)
-        dl->m_docObjects.append( o );
+        dl->m_docObjects.append(o);
     }
     return static_cast<CachedCSSStyleSheet *>(o);
 }
 
-void Cache::preloadStyleSheet(const DeprecatedString &url, const DeprecatedString &stylesheet_data)
+void Cache::preloadStyleSheet(const DeprecatedString& url, const DeprecatedString& stylesheet_data)
 {
     CachedObject *o = cache->get(String(url).impl());
     if (o)
@@ -241,18 +235,15 @@ void Cache::preloadStyleSheet(const DeprecatedString &url, const DeprecatedStrin
     cache->set(String(url).impl(), new CachedCSSStyleSheet(url, stylesheet_data));
 }
 
-CachedScript *Cache::requestScript( DocLoader* dl, const WebCore::String &url, bool reload, time_t _expireDate, const DeprecatedString& charset)
+CachedScript *Cache::requestScript(DocLoader* dl, const String& url, bool reload, time_t expireDate, const DeprecatedString& charset)
 {
     // this brings the _url to a standard form...
     KURL kurl;
     KIO::CacheControl cachePolicy;
-    if ( dl )
-    {
-        kurl = dl->m_doc->completeURL( url.deprecatedString() );
+    if (dl) {
+        kurl = dl->m_doc->completeURL(url.deprecatedString());
         cachePolicy = dl->cachePolicy();
-    }
-    else
-    {
+    } else {
         kurl = url.deprecatedString();
         cachePolicy = KIO::CC_Verify;
     }
@@ -260,58 +251,56 @@ CachedScript *Cache::requestScript( DocLoader* dl, const WebCore::String &url, b
     // Checking if the URL is malformed is lots of extra work for little benefit.
 
     CachedObject *o = cache->get(String(kurl.url()).impl());
-    if(!o)
+    if (!o)
     {
 #ifdef CACHE_DEBUG
-        kdDebug( 6060 ) << "Cache: new: " << kurl.url() << endl;
+        kdDebug(6060) << "Cache: new: " << kurl.url() << endl;
 #endif
-        CachedScript *script = new CachedScript(dl, kurl.url(), cachePolicy, _expireDate, charset);
+        CachedScript *script = new CachedScript(dl, kurl.url(), cachePolicy, expireDate, charset);
         if (cacheDisabled)
             script->setFree(true);
         else {
-            cache->set(String(kurl.url()).impl(), script );
+            cache->set(String(kurl.url()).impl(), script);
             moveToHeadOfLRUList(script);
         }
         o = script;
     }
 
     
-    if(!(o->type() == CachedObject::Script))
-    {
+    if (!(o->type() == CachedObject::Script)) {
 #ifdef CACHE_DEBUG
-        kdDebug( 6060 ) << "Cache::Internal Error in requestScript url=" << kurl.url() << "!" << endl;
+        kdDebug(6060) << "Cache::Internal Error in requestScript url=" << kurl.url() << "!" << endl;
 #endif
         return 0;
     }
     
     
 #ifdef CACHE_DEBUG
-    if( o->status() == CachedObject::Pending )
-        kdDebug( 6060 ) << "Cache: loading in progress: " << kurl.url() << endl;
+    if (o->status() == CachedObject::Pending)
+        kdDebug(6060) << "Cache: loading in progress: " << kurl.url() << endl;
     else
-        kdDebug( 6060 ) << "Cache: using cached: " << kurl.url() << endl;
+        kdDebug(6060) << "Cache: using cached: " << kurl.url() << endl;
 #endif
 
     moveToHeadOfLRUList(o);
-    if ( dl ) {
-        dl->m_docObjects.remove( o );
+    if (dl) {
+        dl->m_docObjects.remove(o);
         if (!cacheDisabled)
-        dl->m_docObjects.append( o );
+            dl->m_docObjects.append(o);
     }
     return static_cast<CachedScript *>(o);
 }
 
-void Cache::preloadScript(const DeprecatedString &url, const DeprecatedString &script_data)
+void Cache::preloadScript(const DeprecatedString& url, const DeprecatedString& script_data)
 {
     CachedObject *o = cache->get(String(url).impl());
-    if(o)
+    if (o)
         remove(o);
     cache->set(String(url).impl(), new CachedScript(url, script_data));
 }
 
 #ifdef KHTML_XSLT
-CachedXSLStyleSheet* Cache::requestXSLStyleSheet(DocLoader* dl, const String & url, bool reload, 
-                                                 time_t _expireDate)
+CachedXSLStyleSheet* Cache::requestXSLStyleSheet(DocLoader* dl, const String& url, bool reload, time_t expireDate)
 {
     // this brings the _url to a standard form...
     KURL kurl;
@@ -330,9 +319,9 @@ CachedXSLStyleSheet* Cache::requestXSLStyleSheet(DocLoader* dl, const String & u
     CachedObject *o = cache->get(String(kurl.url()).impl());
     if (!o) {
 #ifdef CACHE_DEBUG
-        kdDebug( 6060 ) << "Cache: new: " << kurl.url() << endl;
+        kdDebug(6060) << "Cache: new: " << kurl.url() << endl;
 #endif
-        CachedXSLStyleSheet* doc = new CachedXSLStyleSheet(dl, kurl.url(), cachePolicy, _expireDate);
+        CachedXSLStyleSheet* doc = new CachedXSLStyleSheet(dl, kurl.url(), cachePolicy, expireDate);
         if (cacheDisabled)
             doc->setFree(true);
         else {
@@ -345,31 +334,31 @@ CachedXSLStyleSheet* Cache::requestXSLStyleSheet(DocLoader* dl, const String & u
     
     if (o->type() != CachedObject::XSLStyleSheet) {
 #ifdef CACHE_DEBUG
-        kdDebug( 6060 ) << "Cache::Internal Error in requestXSLStyleSheet url=" << kurl.url() << "!" << endl;
+        kdDebug(6060) << "Cache::Internal Error in requestXSLStyleSheet url=" << kurl.url() << "!" << endl;
 #endif
         return 0;
     }
     
 #ifdef CACHE_DEBUG
     if (o->status() == CachedObject::Pending)
-        kdDebug( 6060 ) << "Cache: loading in progress: " << kurl.url() << endl;
+        kdDebug(6060) << "Cache: loading in progress: " << kurl.url() << endl;
     else
-        kdDebug( 6060 ) << "Cache: using cached: " << kurl.url() << endl;
+        kdDebug(6060) << "Cache: using cached: " << kurl.url() << endl;
 #endif
     
     moveToHeadOfLRUList(o);
     if (dl) {
-        dl->m_docObjects.remove( o );
+        dl->m_docObjects.remove(o);
         if (!cacheDisabled)
-            dl->m_docObjects.append( o );
+            dl->m_docObjects.append(o);
     }
     return static_cast<CachedXSLStyleSheet*>(o);
 }
 #endif
 
 #ifndef KHTML_NO_XBL
-CachedXBLDocument* Cache::requestXBLDocument(DocLoader* dl, const String & url, bool reload, 
-                                             time_t _expireDate)
+CachedXBLDocument* Cache::requestXBLDocument(DocLoader* dl, const String& url, bool reload, 
+                                             time_t expireDate)
 {
     // this brings the _url to a standard form...
     KURL kurl;
@@ -377,8 +366,7 @@ CachedXBLDocument* Cache::requestXBLDocument(DocLoader* dl, const String & url,
     if (dl) {
         kurl = dl->m_doc->completeURL(url.deprecatedString());
         cachePolicy = dl->cachePolicy();
-    }
-    else {
+    } else {
         kurl = url.deprecatedString();
         cachePolicy = KIO::CC_Verify;
     }
@@ -386,12 +374,11 @@ CachedXBLDocument* Cache::requestXBLDocument(DocLoader* dl, const String & url,
     // Checking if the URL is malformed is lots of extra work for little benefit.
     
     CachedObject *o = cache->get(String(kurl.url()).impl());
-    if(!o)
-    {
+    if (!o) {
 #ifdef CACHE_DEBUG
-        kdDebug( 6060 ) << "Cache: new: " << kurl.url() << endl;
+        kdDebug(6060) << "Cache: new: " << kurl.url() << endl;
 #endif
-        CachedXBLDocument* doc = new CachedXBLDocument(dl, kurl.url(), cachePolicy, _expireDate);
+        CachedXBLDocument* doc = new CachedXBLDocument(dl, kurl.url(), cachePolicy, expireDate);
         if (cacheDisabled)
             doc->setFree(true);
         else {
@@ -402,26 +389,25 @@ CachedXBLDocument* Cache::requestXBLDocument(DocLoader* dl, const String & url,
     }
     
     
-    if(o->type() != CachedObject::XBL)
-    {
+    if (o->type() != CachedObject::XBL) {
 #ifdef CACHE_DEBUG
-        kdDebug( 6060 ) << "Cache::Internal Error in requestXBLDocument url=" << kurl.url() << "!" << endl;
+        kdDebug(6060) << "Cache::Internal Error in requestXBLDocument url=" << kurl.url() << "!" << endl;
 #endif
         return 0;
     }
     
 #ifdef CACHE_DEBUG
-    if( o->status() == CachedObject::Pending )
-        kdDebug( 6060 ) << "Cache: loading in progress: " << kurl.url() << endl;
+    if (o->status() == CachedObject::Pending)
+        kdDebug(6060) << "Cache: loading in progress: " << kurl.url() << endl;
     else
-        kdDebug( 6060 ) << "Cache: using cached: " << kurl.url() << endl;
+        kdDebug(6060) << "Cache: using cached: " << kurl.url() << endl;
 #endif
     
     moveToHeadOfLRUList(o);
-    if ( dl ) {
-        dl->m_docObjects.remove( o );
+    if (dl) {
+        dl->m_docObjects.remove(o);
         if (!cacheDisabled)
-            dl->m_docObjects.append( o );
+            dl->m_docObjects.append(o);
     }
     return static_cast<CachedXBLDocument*>(o);
 }
@@ -462,7 +448,7 @@ void Cache::setSize(int bytes)
     flush(true);
 }
 
-void Cache::remove( CachedObject *object )
+void Cache::remove(CachedObject *object)
 {
   // this indicates the deref() method of CachedObject to delete itself when the reference counter
   // drops down to zero
@@ -472,10 +458,10 @@ void Cache::remove( CachedObject *object )
   removeFromLRUList(object);
 
   const DocLoader* dl;
-  for ( dl=docloader->first(); dl; dl=docloader->next() )
-      dl->removeCachedObject( object );
+  for (dl=docloader->first(); dl; dl=docloader->next())
+      dl->removeCachedObject(object);
 
-  if ( object->canDelete() )
+  if (object->canDelete())
      delete object;
 }
 
index bb237d4..30669f9 100644 (file)
@@ -66,14 +66,14 @@ namespace WebCore
          * if the DocLoader is zero, the url must be full-qualified.
          * Otherwise, it is automatically base-url expanded
          */
-        static CachedImage* requestImage(DocLoader*, const WebCore::String& URL, bool reload = false, time_t expireDate = 0);
+        static CachedImage* requestImage(DocLoader*, const String& URL, bool reload = false, time_t expireDate = 0);
         static CachedImage* requestImage(DocLoader*, const KURL& url, bool reload = false, time_t expireDate = 0);
 
         /**
          * Ask the cache for some url. Will return a cachedObject, and
          * load the requested data in case it's not cached
          */
-        static CachedCSSStyleSheet* requestStyleSheet(DocLoader*, const WebCore::String& URL, bool reload = false, time_t expireDate = 0, const DeprecatedString& charset = DeprecatedString::null);
+        static CachedCSSStyleSheet* requestStyleSheet(DocLoader*, const String& URL, bool reload = false, time_t expireDate = 0, const DeprecatedString& charset = DeprecatedString::null);
 
         /**
          * Pre-loads a stylesheet into the cache.
@@ -84,7 +84,7 @@ namespace WebCore
          * Ask the cache for some url. Will return a cachedObject, and
          * load the requested data in case it's not cahced
          */
-        static CachedScript* requestScript(DocLoader*, const WebCore::String& URL, bool reload = false, time_t expireDate = 0, const DeprecatedString& charset = DeprecatedString::null);
+        static CachedScript* requestScript(DocLoader*, const String& URL, bool reload = false, time_t expireDate = 0, const DeprecatedString& charset = DeprecatedString::null);
 
         /**
          * Pre-loads a script into the cache.
@@ -93,12 +93,12 @@ namespace WebCore
 
 #ifdef KHTML_XSLT
         // Ask the cache for an XSL stylesheet.
-        static CachedXSLStyleSheet* requestXSLStyleSheet(DocLoader*, const WebCore::String& URL, bool reload = false, time_t expireDate = 0);
+        static CachedXSLStyleSheet* requestXSLStyleSheet(DocLoader*, const String& URL, bool reload = false, time_t expireDate = 0);
 #endif
 
 #ifndef KHTML_NO_XBL
         // Ask the cache for an XBL document.
-        static CachedXBLDocument* requestXBLDocument(DocLoader*, const WebCore::String& URL, bool reload = false, time_t expireDate = 0);
+        static CachedXBLDocument* requestXBLDocument(DocLoader*, const String& URL, bool reload = false, time_t expireDate = 0);
 #endif
 
         /**
@@ -114,7 +114,7 @@ namespace WebCore
         static int maxCacheableObjectSize() { return maxCacheable; }
 
         // Get an existing cache entry by URL.
-        static CachedObject* get(const WebCore::String& URL);
+        static CachedObject* get(const String& URL);
 
         // Remove an existing cache entry.
         static void remove(CachedObject*);
index 27350f5..baf40aa 100644 (file)
@@ -115,7 +115,7 @@ void CachedCSSStyleSheet::checkNotify()
     }
 }
 
-void CachedCSSStyleSheet::error( int /*err*/, const char */*text*/ )
+void CachedCSSStyleSheet::error()
 {
     m_loading = false;
     checkNotify();
index 3355d2e..c5a4405 100644 (file)
@@ -48,7 +48,7 @@ namespace WebCore
 
         virtual void setCharset(const DeprecatedString&);
         virtual void data(DeprecatedByteArray&, bool atEnd);
-        virtual void error(int code, const char* message);
+        virtual void error();
 
         virtual bool schedule() const { return true; }
 
index 541bfa5..8a4c41f 100644 (file)
@@ -154,7 +154,7 @@ void CachedImage::data(DeprecatedByteArray& data, bool eof)
     }
 }
 
-void CachedImage::error( int /*err*/, const char */*text*/ )
+void CachedImage::error()
 {
     clear();
     m_errorOccurred = true;
index edb59bf..52a47c2 100644 (file)
@@ -54,7 +54,7 @@ public:
     virtual void deref(CachedObjectClient*);
 
     virtual void data(DeprecatedByteArray&, bool atEnd);
-    virtual void error(int code, const char* message);
+    virtual void error();
 
     bool isErrorImage() const { return m_errorOccurred; }
 
index 05abe72..e80377c 100644 (file)
@@ -104,7 +104,7 @@ namespace WebCore
 
         virtual void setCharset(const DeprecatedString&) {}
         virtual void data(DeprecatedByteArray&, bool atEnd) = 0;
-        virtual void error(int code, const char* message) = 0;
+        virtual void error() = 0;
 
         const String &url() const { return m_url; }
         Type type() const { return m_type; }
index f005c1d..9d6eab6 100644 (file)
@@ -100,15 +100,15 @@ void CachedScript::data(DeprecatedByteArray& data, bool eof )
 
 void CachedScript::checkNotify()
 {
-    if(m_loading) return;
+    if (m_loading)
+        return;
 
     CachedObjectClientWalker w(m_clients);
     while (CachedObjectClient *c = w.next())
         c->notifyFinished(this);
 }
 
-
-void CachedScript::error( int /*err*/, const char */*text*/ )
+void CachedScript::error()
 {
     m_loading = false;
     m_errorOccurred = true;
index 7bf09db..c97c305 100644 (file)
@@ -46,7 +46,7 @@ namespace WebCore {
 
         virtual void setCharset(const DeprecatedString&);
        virtual void data(DeprecatedByteArray&, bool atEnd);
-       virtual void error(int code, const char* message);
+       virtual void error();
 
         virtual bool schedule() const { return false; }
         
index 1f491a6..7bb007c 100644 (file)
@@ -107,7 +107,7 @@ void CachedXBLDocument::checkNotify()
         c->setXBLDocument(m_url, m_document);
 }
 
-void CachedXBLDocument::error( int /*err*/, const char */*text*/ )
+void CachedXBLDocument::error()
 {
     m_loading = false;
     checkNotify();
index dd16e4c..4893e6c 100644 (file)
@@ -53,7 +53,7 @@ namespace WebCore
         
         virtual void setCharset( const DeprecatedString &chs );
         virtual void data(DeprecatedByteArray&, bool eof );
-        virtual void error( int err, const char *text );
+        virtual void error();
         
         virtual bool schedule() const { return true; }
         
index a160db7..4f29881 100644 (file)
@@ -97,7 +97,7 @@ void CachedXSLStyleSheet::checkNotify()
 }
 
 
-void CachedXSLStyleSheet::error( int /*err*/, const char */*text*/ )
+void CachedXSLStyleSheet::error()
 {
     m_loading = false;
     checkNotify();
index 277c8b7..e1c4ed7 100644 (file)
@@ -47,7 +47,7 @@ namespace WebCore {
         
         virtual void setCharset(const DeprecatedString&);
         virtual void data(DeprecatedByteArray&, bool eof);
-        virtual void error(int err, const char* text);
+        virtual void error();
         
         virtual bool schedule() const { return true; }
         
index e9c635e..cb6b1cf 100644 (file)
@@ -52,12 +52,12 @@ DocLoader::DocLoader(Frame *frame, Document* doc)
     m_loadInProgress = false;
 
     Cache::init();
-    Cache::docloader->append( this );
+    Cache::docloader->append(this);
 }
 
 DocLoader::~DocLoader()
 {
-    Cache::docloader->remove( this );
+    Cache::docloader->remove(this);
 }
 
 void DocLoader::setExpireDate(time_t _expireDate)
@@ -65,26 +65,20 @@ void DocLoader::setExpireDate(time_t _expireDate)
     m_expireDate = _expireDate;
 }
 
-bool DocLoader::needReload(const KURL &fullURL)
+bool DocLoader::needReload(const KURLfullURL)
 {
     bool reload = false;
-    if (m_cachePolicy == KIO::CC_Verify)
-    {
-       if (!m_reloadedURLs.contains(fullURL.url()))
-       {
+    if (m_cachePolicy == KIO::CC_Verify) {
+       if (!m_reloadedURLs.contains(fullURL.url())) {
           CachedObject* existing = Cache::get(fullURL.url());
-          if (existing && existing->isExpired())
-          {
+          if (existing && existing->isExpired()) {
              Cache::remove(existing);
              m_reloadedURLs.append(fullURL.url());
              reload = true;
           }
        }
-    }
-    else if ((m_cachePolicy == KIO::CC_Reload) || (m_cachePolicy == KIO::CC_Refresh))
-    {
-       if (!m_reloadedURLs.contains(fullURL.url()))
-       {
+    } else if ((m_cachePolicy == KIO::CC_Reload) || (m_cachePolicy == KIO::CC_Refresh)) {
+       if (!m_reloadedURLs.contains(fullURL.url())) {
           CachedObject* existing = Cache::get(fullURL.url());
           if (existing)
              Cache::remove(existing);
@@ -95,13 +89,12 @@ bool DocLoader::needReload(const KURL &fullURL)
     return reload;
 }
 
-CachedImage *DocLoader::requestImage(const WebCore::String &url)
+CachedImage *DocLoader::requestImage(const String& url)
 {
     KURL fullURL = m_doc->completeURL(url.deprecatedString());
 
-    if (KWQCheckIfReloading(this)) {
+    if (KWQCheckIfReloading(this))
         setCachePolicy(KIO::CC_Reload);
-    }
 
     bool reload = needReload(fullURL);
 
@@ -110,13 +103,12 @@ CachedImage *DocLoader::requestImage(const WebCore::String &url)
     return cachedObject;
 }
 
-CachedCSSStyleSheet *DocLoader::requestStyleSheet( const WebCore::String &url, const DeprecatedString& charset)
+CachedCSSStyleSheet *DocLoader::requestStyleSheet(const String& url, const DeprecatedString& charset)
 {
     KURL fullURL = m_doc->completeURL(url.deprecatedString());
 
-    if (KWQCheckIfReloading(this)) {
+    if (KWQCheckIfReloading(this))
         setCachePolicy(KIO::CC_Reload);
-    }
 
     bool reload = needReload(fullURL);
 
@@ -125,13 +117,12 @@ CachedCSSStyleSheet *DocLoader::requestStyleSheet( const WebCore::String &url, c
     return cachedObject;
 }
 
-CachedScript *DocLoader::requestScript( const WebCore::String &url, const DeprecatedString& charset)
+CachedScript *DocLoader::requestScript(const String& url, const DeprecatedString& charset)
 {
     KURL fullURL = m_doc->completeURL(url.deprecatedString());
 
-    if (KWQCheckIfReloading(this)) {
+    if (KWQCheckIfReloading(this))
         setCachePolicy(KIO::CC_Reload);
-    }
 
     bool reload = needReload(fullURL);
 
@@ -141,7 +132,7 @@ CachedScript *DocLoader::requestScript( const WebCore::String &url, const Deprec
 }
 
 #ifdef KHTML_XSLT
-CachedXSLStyleSheet* DocLoader::requestXSLStyleSheet(const WebCore::String &url)
+CachedXSLStyleSheet* DocLoader::requestXSLStyleSheet(const String& url)
 {
     KURL fullURL = m_doc->completeURL(url.deprecatedString());
     
@@ -157,16 +148,15 @@ CachedXSLStyleSheet* DocLoader::requestXSLStyleSheet(const WebCore::String &url)
 #endif
 
 #ifndef KHTML_NO_XBL
-CachedXBLDocument* DocLoader::requestXBLDocument(const WebCore::String &url)
+CachedXBLDocument* DocLoader::requestXBLDocument(const String& url)
 {
     KURL fullURL = m_doc->completeURL(url.deprecatedString());
     
     // FIXME: Is this right for XBL?
     if (m_frame && m_frame->onlyLocalReferences() && fullURL.protocol() != "file") return 0;
     
-    if (KWQCheckIfReloading(this)) {
+    if (KWQCheckIfReloading(this))
         setCachePolicy(KIO::CC_Reload);
-    }
     
     bool reload = needReload(fullURL);
     
@@ -176,22 +166,22 @@ CachedXBLDocument* DocLoader::requestXBLDocument(const WebCore::String &url)
 }
 #endif
 
-void DocLoader::setAutoloadImages( bool enable )
+void DocLoader::setAutoloadImages(bool enable)
 {
-    if ( enable == m_bautoloadImages )
+    if (enable == m_bautoloadImages)
         return;
 
     m_bautoloadImages = enable;
 
-    if ( !m_bautoloadImages ) return;
+    if (!m_bautoloadImages)
+        return;
 
-    for ( const CachedObject* co=m_docObjects.first(); co; co=m_docObjects.next() )
-        if ( co->type() == CachedObject::ImageResource )
-        {
-            CachedImage *img = const_cast<CachedImage*>( static_cast<const CachedImage *>( co ) );
+    for (const CachedObject* co=m_docObjects.first(); co; co=m_docObjects.next())
+        if (co->type() == CachedObject::ImageResource) {
+            CachedImage *img = const_cast<CachedImage*>(static_cast<const CachedImage *>(co));
 
             CachedObject::Status status = img->status();
-            if ( status != CachedObject::Unknown )
+            if (status != CachedObject::Unknown)
                 continue;
 
             Cache::loader()->load(this, img, true);
@@ -211,17 +201,16 @@ void DocLoader::setShowAnimations(KHTMLSettings::KAnimationAdvice showAnimations
 
     const CachedObject* co;
     for (co=m_docObjects.first(); co; co = m_docObjects.next())
-        if (co->type() == CachedObject::ImageResource)
-        {
+        if (co->type() == CachedObject::ImageResource) {
             CachedImage *img = const_cast<CachedImage*>(static_cast<const CachedImage*>(co));
 
             img->setShowAnimations(showAnimations);
         }
 }
 
-void DocLoader::removeCachedObject( CachedObject* o ) const
+void DocLoader::removeCachedObject(CachedObject* o) const
 {
-    m_docObjects.removeRef( o );
+    m_docObjects.removeRef(o);
 }
 
 void DocLoader::setLoadInProgress(bool load)
index 0bcbf52..1ccb627 100644 (file)
@@ -126,7 +126,7 @@ void Loader::receivedAllData(TransferJob* job, PlatformData allData)
 
     if (job->error() || job->isErrorPage()) {
         docLoader->setLoadInProgress(true);
-        r->object->error( job->error(), job->errorText().ascii() );
+        r->object->error();
         docLoader->setLoadInProgress(false);
         Cache::remove(object);
     } else {
@@ -153,7 +153,7 @@ void Loader::receivedResponse(TransferJob* job, PlatformResponse response)
     r->object->setResponse(response);
     r->object->setExpireDate(KWQCacheObjectExpiresTime(r->m_docLoader, response), false);
     
-    DeprecatedString chs = job->queryMetaData("charset");
+    DeprecatedString chs = job->queryMetaData("charset").deprecatedString();
     if (!chs.isNull())
         r->object->setCharset(chs);
     
index 7d6a356..230fa95 100644 (file)
@@ -120,9 +120,9 @@ public:
         m_cachedSheet->deref(this);
     }
 private:
-    virtual void setStyleSheet(const String&, const String &sheet)
+    virtual void setStyleSheet(const String&, const Stringsheet)
     {
-        m_frame->setUserStyleSheet(sheet.deprecatedString());
+        m_frame->setUserStyleSheet(sheet);
     }
     Frame* m_frame;
     CachedCSSStyleSheet* m_cachedSheet;
@@ -215,7 +215,7 @@ Frame::~Frame()
     d = 0;
 }
 
-bool Frame::didOpenURL(const KURL &url)
+bool Frame::didOpenURL(const KURLurl)
 {
   if (d->m_scheduledRedirection == locationChangeScheduledDuringLoad) {
     // A redirect was shceduled before the document was created. This can happen
@@ -248,14 +248,14 @@ bool Frame::didOpenURL(const KURL &url)
   d->m_kjsStatusBarText = String();
   d->m_kjsDefaultStatusBarText = String();
 
-  d->m_bJScriptEnabled = d->m_settings->isJavaScriptEnabled(url.host());
-  d->m_bJavaEnabled = d->m_settings->isJavaEnabled(url.host());
-  d->m_bPluginsEnabled = d->m_settings->isPluginsEnabled(url.host());
+  d->m_bJScriptEnabled = d->m_settings->isJavaScriptEnabled();
+  d->m_bJavaEnabled = d->m_settings->isJavaEnabled();
+  d->m_bPluginsEnabled = d->m_settings->isPluginsEnabled();
 
   // initializing d->m_url to the new url breaks relative links when opening such a link after this call and _before_ begin() is called (when the first
   // data arrives) (Simon)
   d->m_url = url;
-  if(d->m_url.protocol().startsWith("http") && !d->m_url.host().isEmpty() && d->m_url.path().isEmpty())
+  if (d->m_url.protocol().startsWith("http") && !d->m_url.host().isEmpty() && d->m_url.path().isEmpty())
     d->m_url.setPath("/");
   d->m_workingURL = d->m_url;
 
@@ -348,7 +348,7 @@ bool Frame::jScriptEnabled() const
   return d->m_bJScriptEnabled;
 }
 
-void Frame::setMetaRefreshEnabled( bool enable )
+void Frame::setMetaRefreshEnabled(bool enable)
 {
   d->m_metaRefreshEnabled = enable;
 }
@@ -428,18 +428,18 @@ bool Frame::pluginsEnabled() const
   return d->m_bPluginsEnabled;
 }
 
-void Frame::setAutoloadImages( bool enable )
+void Frame::setAutoloadImages(bool enable)
 {
-  if ( d->m_doc && d->m_doc->docLoader()->autoloadImages() == enable )
+  if (d->m_doc && d->m_doc->docLoader()->autoloadImages() == enable)
     return;
 
-  if ( d->m_doc )
-    d->m_doc->docLoader()->setAutoloadImages( enable );
+  if (d->m_doc)
+    d->m_doc->docLoader()->setAutoloadImages(enable);
 }
 
 bool Frame::autoloadImages() const
 {
-  if ( d->m_doc )
+  if (d->m_doc)
     return d->m_doc->docLoader()->autoloadImages();
 
   return true;
@@ -447,7 +447,7 @@ bool Frame::autoloadImages() const
 
 void Frame::clear(bool clearWindowProperties)
 {
-  if ( d->m_bCleared )
+  if (d->m_bCleared)
     return;
   d->m_bCleared = true;
   d->m_mousePressNode = 0;
@@ -461,7 +461,7 @@ void Frame::clear(bool clearWindowProperties)
   if (clearWindowProperties && d->m_jscript)
     d->m_jscript->clear();
 
-  if ( d->m_view )
+  if (d->m_view)
     d->m_view->clear();
 
   // do not drop the document before the jscript and view are cleared, as some destructors
@@ -482,7 +482,7 @@ void Frame::clear(bool clearWindowProperties)
 
   d->m_bMousePressed = false;
 
-  if ( !d->m_haveEncoding )
+  if (!d->m_haveEncoding)
     d->m_encoding = DeprecatedString::null;
 }
 
@@ -515,45 +515,39 @@ void Frame::receivedFirstData()
     DeprecatedString qData;
 
     // Support for http-refresh
-    qData = d->m_job->queryMetaData("http-refresh");
-    if(!qData.isEmpty() && d->m_metaRefreshEnabled) {
+    qData = d->m_job->queryMetaData("http-refresh").deprecatedString();
+    if (!qData.isEmpty() && d->m_metaRefreshEnabled) {
       double delay;
-      int pos = qData.find( ';' );
-      if ( pos == -1 )
-        pos = qData.find( ',' );
+      int pos = qData.find(';');
+      if (pos == -1)
+        pos = qData.find(',');
 
-      if( pos == -1 )
-      {
+      if (pos == -1) {
         delay = qData.stripWhiteSpace().toDouble();
         // We want a new history item if the refresh timeout > 1 second
-        scheduleRedirection( delay, d->m_url.url(), delay <= 1);
-      }
-      else
-      {
+        scheduleRedirection(delay, d->m_url.url(), delay <= 1);
+      } else {
         int end_pos = qData.length();
         delay = qData.left(pos).stripWhiteSpace().toDouble();
-        while ( qData[++pos] == ' ' );
-        if ( qData.find( "url", pos, false ) == pos )
-        {
+        while (qData[++pos] == ' ');
+        if (qData.find("url", pos, false) == pos) {
           pos += 3;
-          while (qData[pos] == ' ' || qData[pos] == '=' )
+          while (qData[pos] == ' ' || qData[pos] == '=')
               pos++;
-          if ( qData[pos] == '"' )
-          {
+          if (qData[pos] == '"') {
               pos++;
               int index = end_pos-1;
-              while( index > pos )
-              {
-                if ( qData[index] == '"' )
+              while (index > pos) {
+                if (qData[index] == '"')
                     break;
                 index--;
               }
-              if ( index > pos )
+              if (index > pos)
                 end_pos = index;
           }
         }
         // We want a new history item if the refresh timeout > 1 second
-        scheduleRedirection( delay, d->m_doc->completeURL( qData.mid( pos, end_pos ) ), delay <= 1);
+        scheduleRedirection(delay, d->m_doc->completeURL(qData.mid(pos, end_pos)), delay <= 1);
       }
       d->m_bHTTPRefresh = true;
     }
@@ -595,7 +589,7 @@ const ResourceRequest& Frame::resourceRequest() const
     return d->m_request;
 }
 
-void Frame::begin(const KURL &url)
+void Frame::begin(const KURLurl)
 {
   if (d->m_workingURL.isEmpty())
     createEmptyDocument(); // Creates an empty document if we don't have one already
@@ -626,34 +620,34 @@ void Frame::begin(const KURL &url)
     d->m_doc = DOMImplementation::instance()->createHTMLDocument(d->m_view.get());
 
   if (!d->m_doc->attached())
-    d->m_doc->attach( );
-  d->m_doc->setURL( d->m_url.url() );
+    d->m_doc->attach();
+  d->m_doc->setURL(d->m_url.url());
   // We prefer m_baseURL over d->m_url because d->m_url changes when we are
   // about to load a new page.
-  d->m_doc->setBaseURL( baseurl.url() );
+  d->m_doc->setBaseURL(baseurl.url());
   if (d->m_decoder)
     d->m_doc->setDecoder(d->m_decoder.get());
-  d->m_doc->docLoader()->setShowAnimations( d->m_settings->showAnimations() );
+  d->m_doc->docLoader()->setShowAnimations(KHTMLSettings::KAnimationEnabled);
 
   updatePolicyBaseURL();
 
-  setAutoloadImages( d->m_settings->autoLoadImages() );
-  DeprecatedString userStyleSheet = d->m_settings->userStyleSheet();
+  setAutoloadImages(d->m_settings->autoLoadImages());
+  String userStyleSheet = d->m_settings->userStyleSheet();
 
-  if ( !userStyleSheet.isEmpty() )
-    setUserStyleSheet( KURL( userStyleSheet ) );
+  if (!userStyleSheet.isEmpty())
+    setUserStyleSheet(KURL(userStyleSheet));
 
   restoreDocumentState();
 
   d->m_doc->implicitOpen();
   // clear widget
   if (d->m_view)
-    d->m_view->resizeContents( 0, 0 );
+    d->m_view->resizeContents(0, 0);
 }
 
 void Frame::write(const char* str, int len)
 {
-    if ( !d->m_decoder ) {
+    if (!d->m_decoder) {
         d->m_decoder = new Decoder;
         if (!d->m_encoding.isNull())
             d->m_decoder->setEncodingName(d->m_encoding.latin1(),
@@ -664,43 +658,44 @@ void Frame::write(const char* str, int len)
         if (d->m_doc)
             d->m_doc->setDecoder(d->m_decoder.get());
     }
-  if ( len == 0 )
+  if (len == 0)
     return;
 
-  if ( len == -1 )
-    len = strlen( str );
+  if (len == -1)
+    len = strlen(str);
 
-  DeprecatedString decoded = d->m_decoder->decode( str, len );
+  DeprecatedString decoded = d->m_decoder->decode(str, len);
 
-  if(decoded.isEmpty()) return;
+  if (decoded.isEmpty())
+    return;
 
-  if(d->m_bFirstData) {
+  if (d->m_bFirstData) {
       // determine the parse mode
-      d->m_doc->determineParseMode( decoded );
+      d->m_doc->determineParseMode(decoded);
       d->m_bFirstData = false;
 
       // ### this is still quite hacky, but should work a lot better than the old solution
-      if(d->m_decoder->visuallyOrdered()) d->m_doc->setVisuallyOrdered();
-      d->m_doc->recalcStyle( Node::Force );
+      if (d->m_decoder->visuallyOrdered()) d->m_doc->setVisuallyOrdered();
+      d->m_doc->recalcStyle(Node::Force);
   }
 
   if (Tokenizer* t = d->m_doc->tokenizer())
-      t->write( decoded, true );
+      t->write(decoded, true);
 }
 
-void Frame::write( const DeprecatedString &str )
+void Frame::write(const DeprecatedString& str)
 {
-  if ( str.isNull() )
+  if (str.isNull())
     return;
 
-  if(d->m_bFirstData) {
+  if (d->m_bFirstData) {
       // determine the parse mode
-      d->m_doc->setParseMode( Document::Strict );
+      d->m_doc->setParseMode(Document::Strict);
       d->m_bFirstData = false;
   }
   Tokenizer* t = d->m_doc->tokenizer();
-  if(t)
-    t->write( str, true );
+  if (t)
+    t->write(str, true);
 }
 
 void Frame::end()
@@ -865,14 +860,14 @@ KURL Frame::baseURL() const
     return d->m_doc->baseURL();
 }
 
-DeprecatedString Frame::baseTarget() const
+String Frame::baseTarget() const
 {
     if (!d->m_doc)
         return DeprecatedString();
     return d->m_doc->baseTarget();
 }
 
-KURL Frame::completeURL(const DeprecatedString &url)
+KURL Frame::completeURL(const DeprecatedStringurl)
 {
     if (!d->m_doc)
         return url;
@@ -880,11 +875,11 @@ KURL Frame::completeURL(const DeprecatedString &url)
     return KURL(d->m_doc->completeURL(url));
 }
 
-void Frame::scheduleRedirection( double delay, const DeprecatedString &url, bool doLockHistory)
+void Frame::scheduleRedirection(double delay, const DeprecatedString& url, bool doLockHistory)
 {
     if (delay < 0 || delay > INT_MAX / 1000)
       return;
-    if ( d->m_scheduledRedirection == noRedirectionScheduled || delay <= d->m_delayRedirect )
+    if (d->m_scheduledRedirection == noRedirectionScheduled || delay <= d->m_delayRedirect)
     {
        d->m_scheduledRedirection = redirectionScheduled;
        d->m_delayRedirect = delay;
@@ -899,7 +894,7 @@ void Frame::scheduleRedirection( double delay, const DeprecatedString &url, bool
     }
 }
 
-void Frame::scheduleLocationChange(const DeprecatedString &url, const DeprecatedString &referrer, bool lockHistory, bool userGesture)
+void Frame::scheduleLocationChange(const DeprecatedString& url, const DeprecatedString& referrer, bool lockHistory, bool userGesture)
 {
     // Handle a location change of a page with no document as a special case.
     // This may happen when a frame changes the location of another frame.
@@ -936,7 +931,7 @@ bool Frame::isScheduledLocationChangePending() const
     return false;
 }
 
-void Frame::scheduleHistoryNavigation( int steps )
+void Frame::scheduleHistoryNavigation(int steps)
 {
     // navigation will always be allowed in the 0 steps case, which is OK because
     // that's supposed to force a reload.
@@ -964,7 +959,7 @@ void Frame::cancelRedirection(bool cancelWithLoadInProgress)
     }
 }
 
-void Frame::changeLocation(const DeprecatedString &URL, const DeprecatedString &referrer, bool lockHistory, bool userGesture)
+void Frame::changeLocation(const DeprecatedString& URL, const DeprecatedString& referrer, bool lockHistory, bool userGesture)
 {
     if (URL.find("javascript:", 0, false) == 0) {
         DeprecatedString script = KURL::decode_string(URL.mid(11));
@@ -996,7 +991,7 @@ void Frame::redirectionTimerFired(Timer<Frame>*)
         // in both IE and NS (but not in Mozilla).... we can't easily do that
         // in Konqueror...
         if (d->m_scheduledHistoryNavigationSteps == 0) // add && parent() to get only frames, but doesn't matter
-            openURL( url() ); /// ## need args.reload=true?
+            openURL(url()); /// ## need args.reload=true?
         else {
             if (d->m_extension) {
                 d->m_extension->goBackOrForward(d->m_scheduledHistoryNavigationSteps);
@@ -1029,7 +1024,7 @@ DeprecatedString Frame::encoding() const
         return d->m_encoding;
 
     if (d->m_decoder && d->m_decoder->encoding().isValid())
-        return DeprecatedString(d->m_decoder->encodingName());
+        return d->m_decoder->encodingName();
 
     return settings()->encoding();
 }
@@ -1050,12 +1045,12 @@ void Frame::setUserStyleSheet(const DeprecatedString& styleSheet)
         d->m_doc->setUserStyleSheet(styleSheet);
 }
 
-bool Frame::gotoAnchor( const DeprecatedString &name )
+bool Frame::gotoAnchor(const String& name)
 {
   if (!d->m_doc)
     return false;
 
-  Node *n = d->m_doc->getElementById(name);
+  Node *n = d->m_doc->getElementById(AtomicString(name));
   if (!n) {
     HTMLCollection *anchors =
         new HTMLCollection(d->m_doc.get(), HTMLCollection::DOC_ANCHORS);
@@ -1072,10 +1067,10 @@ bool Frame::gotoAnchor( const DeprecatedString &name )
 
   // We need to update the layout before scrolling, otherwise we could
   // really mess things up if an anchor scroll comes at a bad moment.
-  if ( d->m_doc ) {
+  if (d->m_doc) {
     d->m_doc->updateRendering();
     // Only do a layout if changes have occurred that make it necessary.      
-    if ( d->m_view && d->m_doc->renderer() && d->m_doc->renderer()->needsLayout() ) {
+    if (d->m_view && d->m_doc->renderer() && d->m_doc->renderer()->needsLayout()) {
       d->m_view->layout();
     }
   }
@@ -1100,17 +1095,17 @@ bool Frame::gotoAnchor( const DeprecatedString &name )
   return true;
 }
 
-void Frame::setStandardFont( const DeprecatedString &name )
+void Frame::setStandardFont(const String& name)
 {
-    d->m_settings->setStdFontName(name);
+    d->m_settings->setStdFontName(AtomicString(name));
 }
 
-void Frame::setFixedFont( const DeprecatedString &name )
+void Frame::setFixedFont(const String& name)
 {
-    d->m_settings->setFixedFontName(name);
+    d->m_settings->setFixedFontName(AtomicString(name));
 }
 
-DeprecatedString Frame::selectedText() const
+String Frame::selectedText() const
 {
     return plainText(selection().toRange().get());
 }
@@ -1120,7 +1115,7 @@ bool Frame::hasSelection() const
     return d->m_selection.isCaretOrRange();
 }
 
-SelectionController &Frame::selection() const
+SelectionControllerFrame::selection() const
 {
     return d->m_selection;
 }
@@ -1135,7 +1130,7 @@ void Frame::setSelectionGranularity(TextGranularity granularity) const
     d->m_selectionGranularity = granularity;
 }
 
-const SelectionController &Frame::dragCaret() const
+const SelectionControllerFrame::dragCaret() const
 {
     return d->m_dragCaret;
 }
@@ -1150,7 +1145,7 @@ void Frame::setMark(const Selection& s)
     d->m_mark = s;
 }
 
-void Frame::setSelection(const SelectionController &s, bool closeTyping, bool keepTypingStyle)
+void Frame::setSelection(const SelectionControllers, bool closeTyping, bool keepTypingStyle)
 {
     if (d->m_selection == s) {
         return;
@@ -1179,7 +1174,7 @@ void Frame::setSelection(const SelectionController &s, bool closeTyping, bool ke
     respondToChangedSelection(oldSelection, closeTyping);
 }
 
-void Frame::setDragCaret(const SelectionController &dragCaret)
+void Frame::setDragCaret(const SelectionControllerdragCaret)
 {
     if (d->m_dragCaret != dragCaret) {
         d->m_dragCaret.needsCaretRepaint();
@@ -1303,25 +1298,25 @@ void Frame::caretBlinkTimerFired(Timer<Frame>*)
     d->m_selection.needsCaretRepaint();
 }
 
-void Frame::paintCaret(GraphicsContext* p, const IntRect &rect) const
+void Frame::paintCaret(GraphicsContext* p, const IntRectrect) const
 {
     if (d->m_caretPaint)
         d->m_selection.paintCaret(p, rect);
 }
 
-void Frame::paintDragCaret(GraphicsContext* p, const IntRect &rect) const
+void Frame::paintDragCaret(GraphicsContext* p, const IntRectrect) const
 {
     d->m_dragCaret.paintCaret(p, rect);
 }
 
-void Frame::urlSelected(const DeprecatedString& url, const DeprecatedString& target)
+void Frame::urlSelected(const DeprecatedString& url, const String& target)
 {
     urlSelected(ResourceRequest(completeURL(url)), target);
 }
 
-void Frame::urlSelected(const ResourceRequest& request, const DeprecatedString& _target)
+void Frame::urlSelected(const ResourceRequest& request, const String& _target)
 {
-  DeprecatedString target = _target;
+  String target = _target;
   if (target.isEmpty() && d->m_doc)
     target = d->m_doc->baseTarget();
 
@@ -1348,8 +1343,9 @@ void Frame::urlSelected(const ResourceRequest& request, const DeprecatedString&
   urlSelected(requestCopy);
 }
 
-bool Frame::requestFrame(RenderPart* renderer, const DeprecatedString& _url, const DeprecatedString& frameName)
+bool Frame::requestFrame(RenderPart* renderer, const String& urlParam, const AtomicString& frameName)
 {
+    DeprecatedString _url = urlParam.deprecatedString();
     // Support for <frame src="javascript:string">
     KURL scriptURL;
     KURL url;
@@ -1377,12 +1373,12 @@ bool Frame::requestFrame(RenderPart* renderer, const DeprecatedString& _url, con
     return true;
 }
 
-bool Frame::requestObject(RenderPart* renderer, const DeprecatedString& url, const DeprecatedString& frameName,
-                          const DeprecatedString& mimeType, const DeprecatedStringList& paramNames, const DeprecatedStringList& paramValues)
+bool Frame::requestObject(RenderPart* renderer, const String& url, const AtomicString& frameName,
+                          const String& mimeType, const DeprecatedStringList& paramNames, const DeprecatedStringList& paramValues)
 {
     KURL completedURL;
     if (!url.isEmpty())
-        completedURL = completeURL(url);
+        completedURL = completeURL(url.deprecatedString());
     
     if (url.isEmpty() && mimeType.isEmpty())
         return true;
@@ -1395,7 +1391,7 @@ bool Frame::requestObject(RenderPart* renderer, const DeprecatedString& url, con
     return loadSubframe(renderer, completedURL, frameName, d->m_referrer);
 }
 
-bool Frame::shouldUsePlugin(Node* element, const KURL& url, const DeprecatedString& mimeType, bool hasFallback, bool& useFallback)
+bool Frame::shouldUsePlugin(Node* element, const KURL& url, const String& mimeType, bool hasFallback, bool& useFallback)
 {
     useFallback = false;
     ObjectContentType objectType = objectContentType(url, mimeType);
@@ -1409,7 +1405,7 @@ bool Frame::shouldUsePlugin(Node* element, const KURL& url, const DeprecatedStri
 }
 
 
-bool Frame::loadPlugin(RenderPart *renderer, const KURL &url, const DeprecatedString &mimeType, 
+bool Frame::loadPlugin(RenderPart *renderer, const KURL& url, const String& mimeType, 
                        const DeprecatedStringList& paramNames, const DeprecatedStringList& paramValues, bool useFallback)
 {
     if (useFallback) {
@@ -1432,7 +1428,7 @@ bool Frame::loadPlugin(RenderPart *renderer, const KURL &url, const DeprecatedSt
     return true;
 }
 
-Frame* Frame::loadSubframe(RenderPart* renderer, const KURL& url, const DeprecatedString& name, const String& referrer)
+Frame* Frame::loadSubframe(RenderPart* renderer, const KURL& url, const String& name, const String& referrer)
 {
     Frame* frame = createFrame(url, name, renderer, referrer);
     if (!frame)  {
@@ -1483,9 +1479,9 @@ void Frame::submitFormAgain()
     delete form;
 }
 
-void Frame::submitForm(const char *action, const DeprecatedString &url, const FormData &formData, const DeprecatedString &_target, const DeprecatedString& contentType, const DeprecatedString& boundary)
+void Frame::submitForm(const char *action, const String& url, const FormData& formData, const String& _target, const String& contentType, const String& boundary)
 {
-  KURL u = completeURL( url );
+  KURL u = completeURL(url.deprecatedString());
 
   if (!u.isValid())
     // ### ERROR HANDLING!
@@ -1562,8 +1558,8 @@ void Frame::submitForm(const char *action, const DeprecatedString &url, const Fo
       request.setContentType("Content-Type: " + contentType + "; boundary=" + boundary);
   }
 
-  if ( d->m_doc->parsing() || d->m_runningScripts > 0 ) {
-    if(d->m_submitForm)
+  if (d->m_doc->parsing() || d->m_runningScripts > 0) {
+    if (d->m_submitForm)
         return;
     d->m_submitForm = new FramePrivate::SubmitForm;
     d->m_submitForm->submitAction = action;
@@ -1640,29 +1636,28 @@ DeprecatedString Frame::referrer() const
    return d->m_referrer;
 }
 
-DeprecatedString Frame::lastModified() const
+String Frame::lastModified() const
 {
   return d->m_lastModified;
 }
 
-
 void Frame::reparseConfiguration()
 {
-  setAutoloadImages( d->m_settings->autoLoadImages() );
+  setAutoloadImages(d->m_settings->autoLoadImages());
   if (d->m_doc)
-     d->m_doc->docLoader()->setShowAnimations( d->m_settings->showAnimations() );
+     d->m_doc->docLoader()->setShowAnimations(KHTMLSettings::KAnimationEnabled);
 
-  d->m_bJScriptEnabled = d->m_settings->isJavaScriptEnabled(d->m_url.host());
-  d->m_bJavaEnabled = d->m_settings->isJavaEnabled(d->m_url.host());
-  d->m_bPluginsEnabled = d->m_settings->isPluginsEnabled(d->m_url.host());
+  d->m_bJScriptEnabled = d->m_settings->isJavaScriptEnabled();
+  d->m_bJavaEnabled = d->m_settings->isJavaEnabled();
+  d->m_bPluginsEnabled = d->m_settings->isPluginsEnabled();
 
   DeprecatedString userStyleSheet = d->m_settings->userStyleSheet();
-  if ( !userStyleSheet.isEmpty() )
-    setUserStyleSheet( KURL( userStyleSheet ) );
+  if (!userStyleSheet.isEmpty())
+    setUserStyleSheet(KURL(userStyleSheet));
   else
-    setUserStyleSheet( DeprecatedString() );
+    setUserStyleSheet(DeprecatedString());
 
-  if(d->m_doc) d->m_doc->updateStyleSelector();
+  if (d->m_doc) d->m_doc->updateStyleSelector();
 }
 
 bool Frame::shouldDragAutoNode(Node *node, int x, int y) const
@@ -1939,7 +1934,7 @@ bool Frame::selectContentsOfNode(Node* node)
     return false;
 }
 
-bool Frame::shouldChangeSelection(const SelectionController &newselection) const
+bool Frame::shouldChangeSelection(const SelectionControllernewselection) const
 {
     return shouldChangeSelection(d->m_selection, newselection, newselection.affinity(), false);
 }
@@ -1976,7 +1971,7 @@ void dispatchEditableContentChangedEvent(Node* root)
     EventTargetNodeCast(root)->dispatchEvent(evt, ec, true);
 }
 
-void Frame::appliedEditing(EditCommandPtr &cmd)
+void Frame::appliedEditing(EditCommandPtrcmd)
 {
     SelectionController sel(cmd.endingSelection());
     if (shouldChangeSelection(sel)) {
@@ -2009,7 +2004,7 @@ void Frame::appliedEditing(EditCommandPtr &cmd)
     respondToChangedContents();
 }
 
-void Frame::unappliedEditing(EditCommandPtr &cmd)
+void Frame::unappliedEditing(EditCommandPtrcmd)
 {
     SelectionController sel(cmd.startingSelection());
     if (shouldChangeSelection(sel)) {
@@ -2023,7 +2018,7 @@ void Frame::unappliedEditing(EditCommandPtr &cmd)
     d->m_lastEditCommand = EditCommandPtr::emptyCommand();
 }
 
-void Frame::reappliedEditing(EditCommandPtr &cmd)
+void Frame::reappliedEditing(EditCommandPtrcmd)
 {
     SelectionController sel(cmd.endingSelection());
     if (shouldChangeSelection(sel)) {
@@ -2052,7 +2047,7 @@ void Frame::clearTypingStyle()
     d->m_typingStyle = 0;
 }
 
-JSValue* Frame::executeScript(const DeprecatedString& filename, int baseLine, Node* n, const DeprecatedString &script)
+JSValue* Frame::executeScript(const String& filename, int baseLine, Node* n, const DeprecatedString& script)
 {
   // FIXME: This is missing stuff that the other executeScript has.
   // --> d->m_runningScripts and submitFormAgain.
@@ -2200,7 +2195,7 @@ void Frame::applyParagraphStyle(CSSStyleDeclaration *style, EditAction editingAc
     }
 }
 
-static void updateState(CSSMutableStyleDeclaration *desiredStyle, CSSComputedStyleDeclaration *computedStyle, bool &atStart, Frame::TriState &state)
+static void updateState(CSSMutableStyleDeclaration *desiredStyle, CSSComputedStyleDeclaration *computedStyle, bool& atStart, Frame::TriState& state)
 {
     DeprecatedValueListConstIterator<CSSProperty> end;
     for (DeprecatedValueListConstIterator<CSSProperty> it = desiredStyle->valuesIterator(); it != end; ++it) {
@@ -2389,7 +2384,7 @@ void Frame::removeEditingStyleFromElement(Element *element) const
 }
 
 
-bool Frame::isCharacterSmartReplaceExempt(const QChar &, bool)
+bool Frame::isCharacterSmartReplaceExempt(const QChar&, bool)
 {
     // no smart replace
     return true;
@@ -2568,13 +2563,13 @@ bool Frame::isFrameSet() const
     return body && body->renderer() && body->hasTagName(framesetTag);
 }
 
-bool Frame::openURL(const KURL &URL)
+bool Frame::openURL(const KURLURL)
 {
     ASSERT_NOT_REACHED();
     return true;
 }
 
-void Frame::didNotOpenURL(const KURL &URL)
+void Frame::didNotOpenURL(const KURLURL)
 {
     if (d->m_submittedFormURL == URL)
         d->m_submittedFormURL = KURL();
@@ -2620,7 +2615,7 @@ HTMLFormElement *Frame::currentForm() const
     return start ? scanForForm(start) : 0;
 }
 
-void Frame::setEncoding(const DeprecatedString &name, bool userChosen)
+void Frame::setEncoding(const DeprecatedStringname, bool userChosen)
 {
     if (!d->m_workingURL.isEmpty())
         receivedFirstData();
@@ -2854,13 +2849,13 @@ void Frame::restoreLocationProperties(SavedProperties *locationProperties)
     }
 }
 
-void Frame::saveInterpreterBuiltins(SavedBuiltins &interpreterBuiltins)
+void Frame::saveInterpreterBuiltins(SavedBuiltinsinterpreterBuiltins)
 {
     if (jScript())
         jScript()->interpreter()->saveBuiltins(interpreterBuiltins);
 }
 
-void Frame::restoreInterpreterBuiltins(const SavedBuiltins &interpreterBuiltins)
+void Frame::restoreInterpreterBuiltins(const SavedBuiltinsinterpreterBuiltins)
 {
     if (jScript())
         jScript()->interpreter()->restoreBuiltins(interpreterBuiltins);
@@ -2901,7 +2896,7 @@ void Frame::clearDocumentFocus(Widget *widget)
     node->getDocument()->setFocusNode(0);
 }
 
-DeprecatedPtrList<Frame> &Frame::mutableInstances()
+DeprecatedPtrList<Frame>Frame::mutableInstances()
 {
     static DeprecatedPtrList<Frame> instancesList;
     return instancesList;
@@ -2915,7 +2910,7 @@ void Frame::updatePolicyBaseURL()
         setPolicyBaseURL(d->m_url.url());
 }
 
-void Frame::setPolicyBaseURL(const String &s)
+void Frame::setPolicyBaseURL(const Strings)
 {
     if (document())
         document()->setPolicyBaseURL(s);
@@ -2991,14 +2986,14 @@ bool Frame::scrollbarsVisible()
     return true;
 }
 
-void Frame::addMetaData(const DeprecatedString &key, const DeprecatedString &value)
+void Frame::addMetaData(const String& key, const String& value)
 {
     d->m_job->addMetaData(key, value);
 }
 
 // This does the same kind of work that Frame::openURL does, except it relies on the fact
 // that a higher level already checked that the URLs match and the scrolling is the right thing to do.
-void Frame::scrollToAnchor(const KURL &URL)
+void Frame::scrollToAnchor(const KURLURL)
 {
     d->m_url = URL;
     started();
@@ -3154,7 +3149,7 @@ RenderStyle *Frame::styleForSelectionStart(Node *&nodeToRemove) const
     return styleElement->renderer()->style();
 }
 
-void Frame::setMediaType(const DeprecatedString &type)
+void Frame::setMediaType(const String& type)
 {
     if (d->m_view)
         d->m_view->setMediaType(type);
@@ -3248,11 +3243,10 @@ DeprecatedValueList<MarkedTextUnderline> Frame::markedTextUnderlines() const
     return d->m_markedTextUnderlines;
 }
 
-unsigned Frame::highlightAllMatchesForString(const DeprecatedString &target, bool caseFlag)
+unsigned Frame::highlightAllMatchesForString(const String& target, bool caseFlag)
 {
-    if (target.isEmpty()) {
+    if (target.isEmpty())
         return 0;
-    }
     
     RefPtr<Range> searchRange(rangeOfContents(document()));
     
@@ -3296,7 +3290,7 @@ bool Frame::isLoadingMainResource() const
 
 FrameTree *Frame::tree() const
 {
-    return &d->m_treeNode;
+    returnd->m_treeNode;
 }
 
 void Frame::detachFromView()
index d97ace6..e506699 100644 (file)
@@ -25,8 +25,8 @@
  * Boston, MA 02111-1307, USA.
  */
 
-#ifndef FRAME_H
-#define FRAME_H
+#ifndef Frame_H
+#define Frame_H
 
 #include "BrowserExtension.h"
 #include "Color.h"
@@ -115,12 +115,12 @@ public:
   /**
    * Returns a pointer to the @ref BrowserExtension.
    */
-  BrowserExtension *browserExtension() const;
+  BrowserExtensionbrowserExtension() const;
 
   /**
    * Returns a pointer to the HTML document's view.
    */
-  FrameView *view() const;
+  FrameViewview() const;
 
   virtual void setView(FrameView*);
 
@@ -142,7 +142,7 @@ public:
   /**
    * Enable/disable the automatic forwarding by <meta http-equiv="refresh" ....>
    */
-  void setMetaRefreshEnabled( bool enable );
+  void setMetaRefreshEnabled(bool enable);
 
   /**
    * Returns @p true if automtaic forwarding is enabled.
@@ -165,7 +165,7 @@ public:
    *
    * @note Request will be ignored if called before @ref begin().
    */
-  void setAutoloadImages( bool enable );
+  void setAutoloadImages(bool enable);
   /**
    * Returns whether images contained in the document are loaded automatically
    * or not.
@@ -175,26 +175,26 @@ public:
   bool autoloadImages() const;
 
   KURL baseURL() const;
-  DeprecatedString baseTarget() const;
+  String baseTarget() const;
 
   /**
    * Schedules a redirection after @p delay seconds.
    */
-  void scheduleRedirection(double delay, const DeprecatedString &url, bool lockHistory = true);
+  void scheduleRedirection(double delay, const DeprecatedStringurl, bool lockHistory = true);
 
   /**
    * Make a location change, or schedule one for later.
    * These are used for JavaScript-triggered location changes.
    */
-  void changeLocation(const DeprecatedString &URL, const DeprecatedString &referrer, bool lockHistory = true, bool userGesture = false);
-  void scheduleLocationChange(const DeprecatedString &url, const DeprecatedString &referrer, bool lockHistory = true, bool userGesture = false);
+  void changeLocation(const DeprecatedString& URL, const DeprecatedString& referrer, bool lockHistory = true, bool userGesture = false);
+  void scheduleLocationChange(const DeprecatedString& url, const DeprecatedString& referrer, bool lockHistory = true, bool userGesture = false);
   bool isScheduledLocationChangePending() const;
 
   /**
    * Schedules a history navigation operation (go forward, go back, etc.).
    * This is used for JavaScript-triggered location changes.
    */
-  void scheduleHistoryNavigation( int steps );
+  void scheduleHistoryNavigation(int steps);
 
   /**
    * Clears the widget and prepares it for new content.
@@ -202,7 +202,7 @@ public:
    * If you want @ref url() to return
    * for example "file:/tmp/test.html", you can use the following code:
    * <PRE>
-   * view->begin( KURL("file:/tmp/test.html" ) );
+   * view->begin(KURL("file:/tmp/test.html"));
    * </PRE>
    *
    * @param url is the url of the document to be displayed.  Even if you
@@ -212,7 +212,7 @@ public:
    * All child frames and the old document are removed if you call
    * this method.
    */
-  virtual void begin( const KURL &url = KURL());
+  virtual void begin(const KURL& url = KURL());
 
   /**
    * Writes another part of the HTML code to the widget.
@@ -229,12 +229,12 @@ public:
    * you're using isn't utf-16, you can safely leave out the length
    * parameter.
    *
-   * Attention: Don't mix calls to @ref write( const char *) with calls
-   * to @ref write( const DeprecatedString & ).
+   * Attention: Don't mix calls to @ref write(const char*) with calls
+   * to @ref write(const DeprecatedString&  ).
    *
    * The result might not be what you want.
    */
-  virtual void write( const char *str, int len = -1 );
+  virtual void write(const char* str, int len = -1);
 
   /**
    * Writes another part of the HTML code to the widget.
@@ -243,7 +243,7 @@ public:
    * this function many times in sequence. But remember: The fewer calls
    * you make, the faster the widget will be.
    */
-  virtual void write( const DeprecatedString &str );
+  virtual void write(const DeprecatedString& str);
 
   /**
    * Call this after your last call to @ref write().
@@ -259,7 +259,7 @@ public:
 
   void paint(GraphicsContext*, const IntRect&);
 
-  void setEncoding(const DeprecatedString &encoding, bool userChosen);
+  void setEncoding(const DeprecatedStringencoding, bool userChosen);
 
   /**
    * Returns the encoding the page currently uses.
@@ -275,7 +275,7 @@ public:
    * This gives a wide range of possibilities to
    * change the layout of the page.
    */
-  void setUserStyleSheet(const KURL &url);
+  void setUserStyleSheet(const KURLurl);
 
   /**
    * Sets a user defined style sheet to be used on top of the HTML 4
@@ -284,14 +284,14 @@ public:
    * This gives a wide range of possibilities to
    * change the layout of the page.
    */
-  void setUserStyleSheet(const DeprecatedString &styleSheet);
+  void setUserStyleSheet(const DeprecatedStringstyleSheet);
 
   /**
    * Sets the standard font style.
    *
    * @param name The font name to use for standard text.
    */
-  void setStandardFont( const DeprecatedString &name );
+  void setStandardFont(const String& name);
 
   /**
    * Sets the fixed font style.
@@ -299,7 +299,7 @@ public:
    * @param name The font name to use for fixed text, e.g.
    * the <tt>&lt;pre&gt;</tt> tag.
    */
-  void setFixedFont( const DeprecatedString &name );
+  void setFixedFont(const String& name);
 
   /**
    * Finds the anchor named @p name.
@@ -308,7 +308,7 @@ public:
    * scrolls to the closest position. Returns @p if the anchor has
    * been found.
    */
-  bool gotoAnchor( const DeprecatedString &name );
+  bool gotoAnchor(const String& name);
 
   /**
    * Sets the Zoom factor. The value is given in percent, larger values mean a
@@ -330,7 +330,7 @@ public:
   /**
    * Returns the text the user has marked.
    */
-  virtual DeprecatedString selectedText() const;
+  virtual String selectedText() const;
 
   /**
    * Returns the granularity of the selection (character, word, line, paragraph).
@@ -345,32 +345,32 @@ public:
   /**
    * Returns the drag caret of the HTML.
    */
-  const SelectionController &dragCaret() const;
+  const SelectionControllerdragCaret() const;
 
   /**
    * Sets the current selection.
    */
-  void setSelection(const SelectionController &, bool closeTyping = true, bool keepTypingStyle = false);
+  void setSelection(const SelectionController&, bool closeTyping = true, bool keepTypingStyle = false);
 
   /**
    * Returns whether selection can be changed.
    */
-  bool shouldChangeSelection(const SelectionController &) const;
+  bool shouldChangeSelection(const SelectionController&) const;
 
   /**
    * Returns a mark, to be used as emacs uses it.
    */
-  const Selection &mark() const;
+  const Selectionmark() const;
 
   /**
    * Returns the mark.
    */
-  void setMark(const Selection &);
+  void setMark(const Selection&);
 
   /**
    * Sets the current drag caret.
    */
-  void setDragCaret(const SelectionController &);
+  void setDragCaret(const SelectionController&);
   
   /**
    * Transposes characters either side of caret selection.
@@ -420,12 +420,12 @@ public:
   /**
    * Returns whether editing should end in the given range
    */
-  virtual bool shouldBeginEditing(const Range *) const;
+  virtual bool shouldBeginEditing(const Range*) const;
 
   /**
    * Returns whether editing should end in the given range
    */
-  virtual bool shouldEndEditing(const Range *) const;
+  virtual bool shouldEndEditing(const Range*) const;
 
   /**
    * Called when editing has begun.
@@ -450,27 +450,27 @@ public:
   /**
    * Called when editing has been applied.
    */
-  void appliedEditing(EditCommandPtr &);
+  void appliedEditing(EditCommandPtr&);
 
   /**
    * Called when editing has been unapplied.
    */
-  void unappliedEditing(EditCommandPtr &);
+  void unappliedEditing(EditCommandPtr&);
 
   /**
    * Called when editing has been reapplied.
    */
-  void reappliedEditing(EditCommandPtr &);
+  void reappliedEditing(EditCommandPtr&);
 
   /**
    * Returns the typing style for the document.
    */
-  CSSMutableStyleDeclaration *typingStyle() const;
+  CSSMutableStyleDeclarationtypingStyle() const;
 
   /**
    * Sets the typing style for the document.
    */
-  void setTypingStyle(CSSMutableStyleDeclaration *);
+  void setTypingStyle(CSSMutableStyleDeclaration*);
 
   /**
    * Clears the typing style for the document.
@@ -494,7 +494,7 @@ public:
   /**
    * Last-modified date (in raw string format), if received in the [HTTP] headers.
    */
-  DeprecatedString lastModified() const;
+  String lastModified() const;
 
   bool isPointInsideSelection(int x, int y);
 
@@ -512,18 +512,18 @@ public:
   void undo();
   virtual bool canRedo() const = 0;
   virtual bool canUndo() const = 0;
-  void computeAndSetTypingStyle(CSSStyleDeclaration *, EditAction editingAction=EditActionUnspecified);
-  void applyStyle(CSSStyleDeclaration *, EditAction editingAction=EditActionUnspecified);
-  void applyParagraphStyle(CSSStyleDeclaration *, EditAction editingAction=EditActionUnspecified);
-  TriState selectionHasStyle(CSSStyleDeclaration *) const;
-  bool selectionStartHasStyle(CSSStyleDeclaration *) const;
+  void computeAndSetTypingStyle(CSSStyleDeclaration, EditAction editingAction=EditActionUnspecified);
+  void applyStyle(CSSStyleDeclaration, EditAction editingAction=EditActionUnspecified);
+  void applyParagraphStyle(CSSStyleDeclaration, EditAction editingAction=EditActionUnspecified);
+  TriState selectionHasStyle(CSSStyleDeclaration*) const;
+  bool selectionStartHasStyle(CSSStyleDeclaration*) const;
   String selectionStartStylePropertyValue(int stylePropertyID) const;
   void applyEditingStyleToBodyElement() const;
   void removeEditingStyleFromBodyElement() const;
-  void applyEditingStyleToElement(Element *) const;
-  void removeEditingStyleFromElement(Element *) const;
+  void applyEditingStyleToElement(Element*) const;
+  void removeEditingStyleFromElement(Element*) const;
   virtual void print() = 0;
-  virtual bool isCharacterSmartReplaceExempt(const QChar &, bool);
+  virtual bool isCharacterSmartReplaceExempt(const QChar&, bool);
 
   // Used to keep the part alive when running a script that might destroy it.
   void keepAlive();
@@ -534,7 +534,7 @@ public:
    * returns a KURL object for the given url. Use when
    * you know what you're doing.
    */
-  KURL completeURL( const DeprecatedString &url );
+  KURL completeURL(const DeprecatedString& url);
 
   virtual void handleMouseReleaseDoubleClickEvent(const MouseEventWithHitTestResults&);
   virtual void handleMousePressEvent(const MouseEventWithHitTestResults&);
@@ -543,21 +543,21 @@ public:
   
   void selectClosestWordFromMouseEvent(const PlatformMouseEvent&, Node* innerNode, int x, int y);
 
-  virtual void urlSelected(const DeprecatedString& url, const DeprecatedString& target);
-  virtual void urlSelected(const ResourceRequest&, const DeprecatedString& target);
+  virtual void urlSelected(const DeprecatedString& url, const String& target);
+  virtual void urlSelected(const ResourceRequest&, const String& target);
 
 
   // Methods with platform-specific overrides (and no base class implementation).
-  virtual void setTitle(const String &) = 0;
+  virtual void setTitle(const String&) = 0;
   virtual void handledOnloadEvents() = 0;
-  virtual DeprecatedString userAgent() const = 0;
-  virtual DeprecatedString incomingReferrer() const = 0;
-  virtual DeprecatedString mimeTypeForFileName(const DeprecatedString &) const = 0;
-  virtual KJS::Bindings::Instance *getEmbedInstanceForWidget(Widget*) = 0;
-  virtual KJS::Bindings::Instance *getObjectInstanceForWidget(Widget*) = 0;
-  virtual KJS::Bindings::Instance *getAppletInstanceForWidget(Widget*) = 0;
-  virtual void markMisspellingsInAdjacentWords(const VisiblePosition &) = 0;
-  virtual void markMisspellings(const SelectionController &) = 0;
+  virtual String userAgent() const = 0;
+  virtual String incomingReferrer() const = 0;
+  virtual String mimeTypeForFileName(const String&) const = 0;
+  virtual KJS::Bindings::InstancegetEmbedInstanceForWidget(Widget*) = 0;
+  virtual KJS::Bindings::InstancegetObjectInstanceForWidget(Widget*) = 0;
+  virtual KJS::Bindings::InstancegetAppletInstanceForWidget(Widget*) = 0;
+  virtual void markMisspellingsInAdjacentWords(const VisiblePosition&) = 0;
+  virtual void markMisspellings(const SelectionController&) = 0;
   virtual void addMessageToConsole(const String& message,  unsigned int lineNumber, const String& sourceID) = 0;
   virtual void runJavaScriptAlert(const String& message) = 0;
   virtual bool runJavaScriptConfirm(const String& message) = 0;
@@ -570,9 +570,9 @@ public:
   virtual void scheduleClose() = 0;
   virtual void unfocusWindow() = 0;
   virtual void createEmptyDocument() = 0;
-  virtual Range *markedTextRange() const = 0;
-  virtual void registerCommandForUndo(const EditCommandPtr &) = 0;
-  virtual void registerCommandForRedo(const EditCommandPtr &) = 0;
+  virtual RangemarkedTextRange() const = 0;
+  virtual void registerCommandForUndo(const EditCommandPtr&) = 0;
+  virtual void registerCommandForRedo(const EditCommandPtr&) = 0;
   virtual void clearUndoRedoOperations() = 0;
   virtual void issueUndoCommand() = 0;
   virtual void issueRedoCommand() = 0;
@@ -581,9 +581,9 @@ public:
   virtual void issuePasteCommand() = 0;
   virtual void issuePasteAndMatchStyleCommand() = 0;
   virtual void issueTransposeCommand() = 0;
-  virtual void respondToChangedSelection(const SelectionController &oldSelection, bool closeTyping) = 0;
+  virtual void respondToChangedSelection(const SelectionControlleroldSelection, bool closeTyping) = 0;
   virtual void respondToChangedContents() = 0;
-  virtual bool shouldChangeSelection(const SelectionController &oldSelection, const SelectionController &newSelection, EAffinity affinity, bool stillSelecting) const = 0;
+  virtual bool shouldChangeSelection(const SelectionController& oldSelection, const SelectionController& newSelection, EAffinity affinity, bool stillSelecting) const = 0;
   virtual void partClearedInBegin() = 0; 
   virtual void saveDocumentState() = 0;
   virtual void restoreDocumentState() = 0;
@@ -592,13 +592,13 @@ public:
   virtual void submitForm(const ResourceRequest&) = 0;
   virtual void urlSelected(const ResourceRequest&) = 0;
   virtual bool passSubframeEventToSubframe(MouseEventWithHitTestResults&, Frame* subframePart = 0) = 0;
-  virtual bool passWheelEventToChildWidget(Node *) = 0;
+  virtual bool passWheelEventToChildWidget(Node*) = 0;
   virtual bool lastEventIsMouseUp() const = 0;
-  virtual DeprecatedString overrideMediaType() const = 0;
+  virtual String overrideMediaType() const = 0;
 protected:
-  virtual Plugin* createPlugin(const KURL& url, const DeprecatedStringList& paramNames, const DeprecatedStringList& paramValues, const DeprecatedString& mimeType) = 0;
-  virtual Frame* createFrame(const KURL& url, const DeprecatedString& name, RenderPart* renderer, const String& referrer) = 0;
-  virtual ObjectContentType objectContentType(const KURL& url, const DeprecatedString& mimeType) = 0;
+  virtual Plugin* createPlugin(const KURL& url, const DeprecatedStringList& paramNames, const DeprecatedStringList& paramValues, const String& mimeType) = 0;
+  virtual Frame* createFrame(const KURL& url, const String& name, RenderPart* renderer, const String& referrer) = 0;
+  virtual ObjectContentType objectContentType(const KURL& url, const String& mimeType) = 0;
 
     virtual void redirectionTimerFired(Timer<Frame>*);
 
@@ -640,21 +640,21 @@ private:
   void selectionLayoutChanged();
     void caretBlinkTimerFired(Timer<Frame>*);
 
-  bool shouldUsePlugin(Node* element, const KURL& url, const DeprecatedString& mimeType, bool hasFallback, bool& useFallback);
-  bool loadPlugin(RenderPart* renderer, const KURL &url, const DeprecatedString &mimeType, 
+  bool shouldUsePlugin(Node* element, const KURL& url, const String& mimeType, bool hasFallback, bool& useFallback);
+  bool loadPlugin(RenderPart* renderer, const KURL& url, const String& mimeType, 
                   const DeprecatedStringList& paramNames, const DeprecatedStringList& paramValues, bool useFallback);
-  Frame* loadSubframe(RenderPart* renderer, const KURL& url, const DeprecatedString& name, const String& referrer);
+  Frame* loadSubframe(RenderPart* renderer, const KURL& url, const String& name, const String& referrer);
 
 public:
-  void submitForm(const char *action, const DeprecatedString &url, const FormData &formData,
-                  const DeprecatedString &target, const DeprecatedString& contentType = DeprecatedString::null,
-                  const DeprecatedString& boundary = DeprecatedString::null );
+  void submitForm(const char* action, const String& url, const FormData& formData,
+                  const String& target, const String& contentType = String(),
+                  const String& boundary = String());
   
-  bool requestObject(RenderPart *frame, const DeprecatedString &url, const DeprecatedString &frameName,
-                     const DeprecatedString &serviceType, const DeprecatedStringList &paramNames, const DeprecatedStringList &paramValues);
-  bool requestFrame(RenderPart *frame, const DeprecatedString &url, const DeprecatedString &frameName);
+  bool requestObject(RenderPart* frame, const String& url, const AtomicString& frameName,
+                     const String& serviceType, const DeprecatedStringList& paramNames, const DeprecatedStringList& paramValues);
+  bool requestFrame(RenderPart* frame, const String& url, const AtomicString& frameName);
 
-  Document *document() const;
+  Documentdocument() const;
   void setDocument(Document* newDoc);
 
   // Workaround for the fact that it's hard to delete a frame.
@@ -669,21 +669,21 @@ private:
   void cancelRedirection(bool newLoadInProgress = false);
 
  public:
-  KJS::JSValue* executeScript(const DeprecatedString& filename, int baseLine, Node*, const DeprecatedString& script);
-  KJSProxy *jScript();
-  Frame *opener();
-  void setOpener(Frame *_opener);
+  KJS::JSValue* executeScript(const String& filename, int baseLine, Node*, const DeprecatedString& script);
+  KJSProxyjScript();
+  Frameopener();
+  void setOpener(Frame_opener);
   bool openedByJS();
   void setOpenedByJS(bool _openedByJS);
 
-  void setSettings(KHTMLSettings *);
+  void setSettings(KHTMLSettings*);
 
   void provisionalLoadStarted();
   bool userGestureHint();
-  void didNotOpenURL(const KURL &);
-  void addData(const char *bytes, int length);
-  void addMetaData(const DeprecatedString &key, const DeprecatedString &value);
-  void setMediaType(const DeprecatedString &);
+  void didNotOpenURL(const KURL&);
+  void addData(const charbytes, int length);
+  void addMetaData(const String& key, const String& value);
+  void setMediaType(const String&);
 
   // root renderer for the document contained in this frame
   RenderObject* renderer() const;
@@ -696,9 +696,9 @@ private:
   FloatRect visibleSelectionRect() const;
   bool isFrameSet() const;
 
-  HTMLFormElement *currentForm() const;
+  HTMLFormElementcurrentForm() const;
 
-  RenderStyle *styleForSelectionStart(Node *&nodeToRemove) const;
+  RenderStyle* styleForSelectionStart(Node* &nodeToRemove) const;
 
   // Scrolls as necessary to reveal the selection
   void revealSelection();
@@ -708,26 +708,26 @@ private:
 
   bool scrollOverflow(KWQScrollDirection direction, KWQScrollGranularity granularity);
 
-  void adjustPageHeight(float *newBottom, float oldTop, float oldBottom, float bottomLimit);
+  void adjustPageHeight(floatnewBottom, float oldTop, float oldBottom, float bottomLimit);
 
   bool canCachePage();
-  KJS::PausedTimeouts *pauseTimeouts();
-  void resumeTimeouts(KJS::PausedTimeouts *);
-  void saveWindowProperties(KJS::SavedProperties *windowProperties);
-  void saveLocationProperties(KJS::SavedProperties *locationProperties);
-  void restoreWindowProperties(KJS::SavedProperties *windowProperties);
-  void restoreLocationProperties(KJS::SavedProperties *locationProperties);
-  void saveInterpreterBuiltins(KJS::SavedBuiltins &interpreterBuiltins);
-  void restoreInterpreterBuiltins(const KJS::SavedBuiltins &interpreterBuiltins);
+  KJS::PausedTimeoutspauseTimeouts();
+  void resumeTimeouts(KJS::PausedTimeouts*);
+  void saveWindowProperties(KJS::SavedPropertieswindowProperties);
+  void saveLocationProperties(KJS::SavedPropertieslocationProperties);
+  void restoreWindowProperties(KJS::SavedPropertieswindowProperties);
+  void restoreLocationProperties(KJS::SavedPropertieslocationProperties);
+  void saveInterpreterBuiltins(KJS::SavedBuiltinsinterpreterBuiltins);
+  void restoreInterpreterBuiltins(const KJS::SavedBuiltinsinterpreterBuiltins);
 
-  static Frame *frameForWidget(const Widget *);
-  static Node *nodeForWidget(const Widget *);
-  static Frame *frameForNode(Node *);
+  static Frame* frameForWidget(const Widget*);
+  static Node* nodeForWidget(const Widget*);
+  static Frame* frameForNode(Node*);
 
-  static void clearDocumentFocus(Widget *);
+  static void clearDocumentFocus(Widget*);
 
-  static const DeprecatedPtrList<Frame> &instances() { return mutableInstances(); }    
-  static DeprecatedPtrList<Frame> &mutableInstances();
+  static const DeprecatedPtrList<Frame>instances() { return mutableInstances(); }    
+  static DeprecatedPtrList<Frame>mutableInstances();
 
   void updatePolicyBaseURL();
   void setPolicyBaseURL(const String&);
@@ -738,14 +738,14 @@ private:
   void sendResizeEvent();
   void sendScrollEvent();
   bool scrollbarsVisible();
-  void scrollToAnchor(const KURL &);
+  void scrollToAnchor(const KURL&);
   bool canMouseDownStartSelect(Node*);
   bool passWidgetMouseDownEventToWidget(const MouseEventWithHitTestResults&, bool isDoubleClick);
   bool passWidgetMouseDownEventToWidget(RenderWidget*);
   virtual bool passMouseDownEventToWidget(Widget*) = 0;
 
   void clearTimers();
-  static void clearTimers(FrameView *);
+  static void clearTimers(FrameView*);
 
   bool displaysWithFocusAttributes() const;
   virtual void setDisplaysWithFocusAttributes(bool flag);
@@ -756,12 +756,12 @@ private:
 
   DeprecatedValueList<MarkedTextUnderline> markedTextUnderlines() const;  
   bool markedTextUsesUnderlines() const;
-  unsigned highlightAllMatchesForString(const DeprecatedString &, bool caseFlag);
+  unsigned highlightAllMatchesForString(const String&, bool caseFlag);
 
   // Call this method before handling a new user action, like on a mouse down or key down.
   // Currently, all this does is clear the "don't submit form twice" data member.
   void prepareForUserAction();
-  Node *mousePressNode();
+  NodemousePressNode();
   
   void clearRecordedFormValues();
   void recordFormValue(const String& name, const String& value, PassRefPtr<HTMLFormElement>);
@@ -769,7 +769,7 @@ private:
   bool isComplete() const;
   bool isLoadingMainResource() const;
   
-  void replaceContentsWithScriptResult(const KURL &url);
+  void replaceContentsWithScriptResult(const KURLurl);
 
     void disconnectOwnerRenderer();
 
@@ -799,7 +799,7 @@ protected:
   void handleMousePressEventDoubleClick(const MouseEventWithHitTestResults&);
   void handleMousePressEventTripleClick(const MouseEventWithHitTestResults&);
 
-  CSSComputedStyleDeclaration *selectionComputedStyle(Node *&nodeToRemove) const;
+  CSSComputedStyleDeclaration* selectionComputedStyle(Node* &nodeToRemove) const;
 
     virtual void setStatusBarText(const String&);
     
@@ -810,7 +810,7 @@ public:
   friend class FrameWin;
 
   void checkEmitLoadEvent();
-  bool didOpenURL(const KURL &);
+  bool didOpenURL(const KURL&);
   virtual void didFirstLayout() {}
 
   virtual void frameDetached();
index 42fc282..7a08ad7 100644 (file)
@@ -136,7 +136,7 @@ namespace WebCore {
 
         String m_kjsStatusBarText;
         String m_kjsDefaultStatusBarText;
-        DeprecatedString m_lastModified;
+        String m_lastModified;
 
         bool m_bComplete : 1;
         bool m_bLoadingMainResource : 1;
@@ -166,11 +166,11 @@ namespace WebCore {
 
         struct SubmitForm {
             const char* submitAction;
-            DeprecatedString submitUrl;
+            String submitUrl;
             FormData submitFormData;
-            DeprecatedString target;
-            DeprecatedString submitContentType;
-            DeprecatedString submitBoundary;
+            String target;
+            String submitContentType;
+            String submitBoundary;
         };
         SubmitForm* m_submitForm;
 
index a793b52..46023ed 100644 (file)
@@ -62,6 +62,7 @@ public:
         : m_hasBorder(false)
         , layoutTimer(view, &FrameView::layoutTimerFired)
         , hoverTimer(view, &FrameView::hoverTimerFired)
+        , m_mediaType("screen")
     {
         repaintRects = 0;
         isTransparent = false;
@@ -144,13 +145,14 @@ public:
     
     RefPtr<Node> dragTarget;
     RefPtr<HTMLFrameSetElement> resizingFrameSet;
+    
+    String m_mediaType;
 };
 
 FrameView::FrameView(Frame *frame)
-    : _refCount(1)
+    : m_refCount(1)
     , m_frame(frame)
     , d(new FrameViewPrivate(this))
-    , m_medium("screen")
 {
     init();
 
@@ -161,7 +163,7 @@ FrameView::~FrameView()
 {
     resetScrollBars();
 
-    ASSERT(_refCount == 0);
+    ASSERT(m_refCount == 0);
 
     if (d->hoverTimer.isActive())
         d->hoverTimer.stop();
@@ -194,10 +196,8 @@ void FrameView::resetScrollBars()
 
 void FrameView::init()
 {
-    _marginWidth = -1; // undefined
-    _marginHeight = -1;
-    _width = 0;
-    _height = 0;
+    m_margins = IntSize(-1, -1); // undefined
+    m_size = IntSize();
 }
 
 void FrameView::clear()
@@ -231,13 +231,13 @@ void FrameView::initScrollBars()
 void FrameView::setMarginWidth(int w)
 {
     // make it update the rendering area when set
-    _marginWidth = w;
+    m_margins.setWidth(w);
 }
 
 void FrameView::setMarginHeight(int h)
 {
     // make it update the rendering area when set
-    _marginHeight = h;
+    m_margins.setHeight(h);
 }
 
 void FrameView::adjustViewSize()
@@ -246,7 +246,7 @@ void FrameView::adjustViewSize()
         Document *document = m_frame->document();
 
         RenderCanvas* root = static_cast<RenderCanvas *>(document->renderer());
-        if ( !root )
+        if (!root)
             return;
         
         int docw = root->docWidth();
@@ -312,16 +312,16 @@ void FrameView::layout()
     d->delayedLayout = false;
 
     if (!m_frame) {
-        // FIXME: Do we need to set _width here?
-        // FIXME: Should we set _height here too?
-        _width = visibleWidth();
+        // FIXME: Do we need to set m_size.width here?
+        // FIXME: Should we set m_size.height here too?
+        m_size.setWidth(visibleWidth());
         return;
     }
 
     Document* document = m_frame->document();
     if (!document) {
-        // FIXME: Should we set _height here too?
-        _width = visibleWidth();
+        // FIXME: Should we set m_size.height here too?
+        m_size.setWidth(visibleWidth());
         return;
     }
 
@@ -334,7 +334,7 @@ void FrameView::layout()
 
     RenderCanvas* root = static_cast<RenderCanvas*>(document->renderer());
     if (!root) {
-        // FIXME: Do we need to set _width or _height here?
+        // FIXME: Do we need to set m_size here?
         d->layoutSchedulingEnabled = true;
         return;
     }
@@ -398,13 +398,11 @@ void FrameView::layout()
         suppressScrollBars(false, true);
     }
 
-    int oldHeight = _height;
-    int oldWidth = _width;
-    
-    _height = visibleHeight();
-    _width = visibleWidth();
+    IntSize oldSize = m_size;
 
-    if (oldHeight != _height || oldWidth != _width)
+    m_size = IntSize(visibleWidth(), visibleHeight());
+
+    if (oldSize != m_size)
         d->doFullRepaint = true;
     
     RenderLayer* layer = root->layer();
@@ -435,7 +433,7 @@ void FrameView::layout()
         // if they ever do full repaints.
         RenderObject::RepaintInfo* r;
         DeprecatedPtrListIterator<RenderObject::RepaintInfo> it(*d->repaintRects);
-        for ( ; (r = it.current()); ++it)
+        for (; (r = it.current()); ++it)
             r->m_object->repaintRectangle(r->m_repaintRect);
         d->repaintRects->clear();
     }
@@ -449,7 +447,6 @@ void FrameView::layout()
 #endif
 
     if (root->needsLayout()) {
-        //qDebug("needs layout, delaying repaint");
         scheduleRelayout();
         return;
     }
@@ -682,7 +679,7 @@ void FrameView::handleMouseReleaseEvent(const PlatformMouseEvent& mouseEvent)
     invalidateClick();
 }
 
-bool FrameView::dispatchDragEvent(const AtomicString &eventType, Node *dragTarget, const PlatformMouseEvent& event, Clipboard *clipboard)
+bool FrameView::dispatchDragEvent(const AtomicString& eventType, Node *dragTarget, const PlatformMouseEvent& event, Clipboard* clipboard)
 {
     int clientX, clientY;
     viewportToContents(event.x(), event.y(), clientX, clientY);
@@ -745,7 +742,7 @@ Node* FrameView::nodeUnderMouse() const
     return d->underMouse.get();
 }
 
-bool FrameView::scrollTo(const IntRect &bounds)
+bool FrameView::scrollTo(const IntRectbounds)
 {
     d->scrollingSelf = true; // so scroll events get ignored
 
@@ -768,11 +765,11 @@ bool FrameView::scrollTo(const IntRect &bounds)
         xe = x + curWidth - d->borderX;
 
     // is xpos of target left of the view's border?
-    if (x < contentsX() + d->borderX )
+    if (x < contentsX() + d->borderX)
         deltax = x - contentsX() - d->borderX;
     // is xpos of target right of the view's right border?
     else if (xe + d->borderX > contentsX() + curWidth)
-        deltax = xe + d->borderX - ( contentsX() + curWidth );
+        deltax = xe + d->borderX - (contentsX() + curWidth);
     else
         deltax = 0;
 
@@ -781,7 +778,7 @@ bool FrameView::scrollTo(const IntRect &bounds)
         deltay = y - contentsY() - d->borderY;
     // is ypos of target below lower border?
     else if (ye + d->borderY > contentsY() + curHeight)
-        deltay = ye + d->borderY - ( contentsY() + curHeight );
+        deltay = ye + d->borderY - (contentsY() + curHeight);
     else
         deltay = 0;
 
@@ -888,21 +885,20 @@ void FrameView::focusNextPrevNode(bool next)
     m_frame->document()->setFocusNode(newFocusNode);
 }
 
-void FrameView::setMediaType( const DeprecatedString &medium )
+void FrameView::setMediaType(const String& mediaType)
 {
-    m_medium = medium;
+    d->m_mediaType = mediaType;
 }
 
-DeprecatedString FrameView::mediaType() const
+String FrameView::mediaType() const
 {
     // See if we have an override type.
-    DeprecatedString overrideType = m_frame->overrideMediaType();
+    String overrideType = m_frame->overrideMediaType();
     if (!overrideType.isNull())
         return overrideType;
-    return m_medium;
+    return d->m_mediaType;
 }
 
-
 void FrameView::useSlowRepaints()
 {
     d->useSlowRepaints = true;
@@ -934,7 +930,7 @@ void FrameView::restoreScrollBar()
     suppressScrollBars(false);
 }
 
-void FrameView::setResizingFrameSet(HTMLFrameSetElement *frameSet)
+void FrameView::setResizingFrameSet(HTMLFrameSetElementframeSet)
 {
     d->resizingFrameSet = frameSet;
 }
@@ -1009,7 +1005,7 @@ bool FrameView::dispatchMouseEvent(const AtomicString& eventType, Node* targetNo
     return swallowEvent;
 }
 
-void FrameView::setIgnoreWheelEvents( bool e )
+void FrameView::setIgnoreWheelEvents(bool e)
 {
     d->ignoreWheelEvents = e;
 }
index 88d76f8..31a4bf3 100644 (file)
    Boston, MA 02111-1307, USA.
 */
 
-#ifndef FRAMEVIEW_H
-#define FRAMEVIEW_H
+#ifndef FrameView_H
+#define FrameView_H
 
-#include "DeprecatedString.h"
 #include "ScrollView.h"
+#include "IntSize.h"
+#include "PlatformString.h"
+#include <kxmlcore/RefPtr.h>
 
 class DeprecatedStringList;
 
@@ -65,6 +67,7 @@ class RenderPartObject;
 class RenderStyle;
 class RenderWidget;
 class PlatformWheelEvent;
+class String;
 
 template <typename T> class Timer;
 
@@ -95,25 +98,17 @@ public:
 
     Frame* frame() const { return m_frame.get(); }
 
-    int frameWidth() const { return _width; }
-
-    void setMarginWidth(int x);
-
-    /**
-     * Returns the margin width.
-     *
-     * A return value of -1 means the default value will be used.
-     */
-    int marginWidth() const { return _marginWidth; }
-
-    void setMarginHeight(int y);
+    int frameWidth() const { return m_size.width(); }
 
     /**
-     * Returns the margin height.
+     * Gets/Sets the margin width/height
      *
      * A return value of -1 means the default value will be used.
      */
-    int marginHeight() { return _marginHeight; }
+    int marginWidth() const { return m_margins.width(); }
+    int marginHeight() { return  m_margins.height(); }
+    void setMarginWidth(int);
+    void setMarginHeight(int);
 
     virtual void setVScrollBarMode(ScrollBarMode);
     virtual void setHScrollBarMode(ScrollBarMode);
@@ -128,11 +123,11 @@ public:
 
     bool needsFullRepaint() const;
     
-    void addRepaintInfo(RenderObject* o, const IntRect& r);
+    void addRepaintInfo(RenderObject*, const IntRect&);
 
     void resetScrollBars();
 
-     void clear();
+    void clear();
 
 public:
     void clearPart();
@@ -176,8 +171,8 @@ public:
     void updateDashboardRegions();
 #endif
 
-    void ref() { ++_refCount; }
-    void deref() { if (!--_refCount) delete this; }
+    void ref() { ++m_refCount; }
+    void deref() { if (!--m_refCount) delete this; }
     
 private:
     void cleared();
@@ -196,8 +191,8 @@ private:
      * you only need to enable the media type in the view and if necessary
      * add the media type dependent changes to the renderer.
      */
-    void setMediaType(const DeprecatedString&);
-    DeprecatedString mediaType() const;
+    void setMediaType(const String&);
+    String mediaType() const;
 
     bool scrollTo(const IntRect&);
 
@@ -213,8 +208,8 @@ private:
 
     void restoreScrollBar();
 
-    DeprecatedStringList formCompletionItems(const DeprecatedString &name) const;
-    void addFormCompletionItem(const DeprecatedString &name, const DeprecatedString &value);
+    DeprecatedStringList formCompletionItems(const String& name) const;
+    void addFormCompletionItem(const String& name, const String& value);
 
     MouseEventWithHitTestResults prepareMouseEvent(bool readonly, bool active, bool mouseMove, const PlatformMouseEvent&);
 
@@ -229,18 +224,13 @@ private:
 
     void updateBorder();
 
-    unsigned _refCount;
-
-    int _width;
-    int _height;
-
-    int _marginWidth;
-    int _marginHeight;
-
+    unsigned m_refCount;
+    
+    IntSize m_size;
+    IntSize m_margins;
+    
     RefPtr<Frame> m_frame;
     FrameViewPrivate* d;
-
-    DeprecatedString m_medium; // media type
 };
 
 }
index 290cc74..0e20164 100644 (file)
@@ -41,8 +41,8 @@ namespace WebCore {
         const KURL& url() const { return m_url; }
         void setURL(const KURL& url) { m_url = url; }
 
-        DeprecatedString contentType() const { return m_contentType; }
-        void setContentType(const DeprecatedString &t) { m_contentType = t; }
+        String contentType() const { return m_contentType; }
+        void setContentType(const String& t) { m_contentType = t; }
         
         bool doPost() const { return m_doPost; }
         void setDoPost(bool post) { m_doPost = post; }
@@ -55,7 +55,7 @@ namespace WebCore {
 
         // FIXME: these two parameters are specific to frame opening,
         // should move to FrameRequest once we have that
-        DeprecatedString frameName;
+        String frameName;
     private:
         bool m_lockHistory;
 
@@ -69,7 +69,7 @@ namespace WebCore {
     private:
         KURL m_url;
         String m_referrer;
-        DeprecatedString m_contentType;
+        String m_contentType;
         bool m_doPost;
     };
 
index 4780cb7..917dbb1 100644 (file)
@@ -26,7 +26,6 @@
 #ifndef FONT_FAMILY_H
 #define FONT_FAMILY_H
 
-#include "DeprecatedString.h"
 #include "AtomicString.h"
 
 #if __APPLE__
index 9fda0d6..5e074c2 100644 (file)
@@ -111,7 +111,7 @@ public:
     ~Image();
     
     static Image* loadResource(const char *name);
-    static bool supportsType(const DeprecatedString& type);
+    static bool supportsType(const String& type);
 
     bool isNull() const;
 
index aff57b9..90aae8a 100644 (file)
@@ -27,6 +27,7 @@
 #include "KURL.h"
 
 #include <kxmlcore/Assertions.h>
+#include "PlatformString.h"
 #include "RegularExpression.h"
 #include "TextEncoding.h"
 #include <kxmlcore/Vector.h>
@@ -258,6 +259,23 @@ KURL::KURL(const DeprecatedString &url)
         parse(url.ascii(), &url);
 }
 
+KURL::KURL(const String& stringURL)
+{
+    DeprecatedString url = stringURL.deprecatedString();
+    if (!url.isEmpty() && url[0] == '/') {
+        // 5 for "file:", 1 for terminator
+        Vector<char, 2048> buffer(url.length() + 6);
+        buffer[0] = 'f';
+        buffer[1] = 'i';
+        buffer[2] = 'l';
+        buffer[3] = 'e';
+        buffer[4] = ':';
+        url.copyLatin1(&buffer[5]);
+        parse(buffer, 0);
+    } else
+        parse(url.ascii(), &url);
+}
+
 KURL::KURL(const KURL &base, const DeprecatedString &relative, const WebCore::TextEncoding& encoding)
 {
     // Allow at lest absolute URLs to resolve against an empty URL.
@@ -1203,9 +1221,8 @@ void KURL::parse(const char *url, const DeprecatedString *originalString)
     // allocation.
     if (originalString && strncmp(buffer, url, fragmentEndPos) == 0) {
         urlString = *originalString;
-    } else {
+    } else
         urlString = DeprecatedString(buffer, fragmentEndPos);
-    }
 
     ASSERT(p - buffer <= (int)buffer.size());
 }
@@ -1220,9 +1237,8 @@ bool urlcmp(const DeprecatedString &a, const DeprecatedString &b, bool ignoreTra
     if (ignoreRef) {
         KURL aURL(a);
         KURL bURL(b);
-        if (aURL.m_isValid && bURL.m_isValid) {
+        if (aURL.m_isValid && bURL.m_isValid)
             return aURL.urlString.left(aURL.queryEndPos) == bURL.urlString.left(bURL.queryEndPos);
-        }
     }
     return a == b;
 }
@@ -1242,9 +1258,8 @@ DeprecatedString KURL::encode_string(const DeprecatedString& notEncodedString)
             *p++ = '%';
             *p++ = hexDigits[c >> 4];
             *p++ = hexDigits[c & 0xF];
-        } else {
+        } else
             *p++ = c;
-        }
     }
     
     DeprecatedString result(buffer, p - buffer);
@@ -1260,9 +1275,8 @@ static DeprecatedString encodeHostname(const DeprecatedString &s)
     // For host names bigger than this, we won't do IDN encoding, which is almost certainly OK.
     const unsigned hostnameBufferLength = 2048;
 
-    if (s.isAllASCII() || s.length() > hostnameBufferLength) {
+    if (s.isAllASCII() || s.length() > hostnameBufferLength)
         return s;
-    }
 
     UChar buffer[hostnameBufferLength];    
     UErrorCode error = U_ZERO_ERROR;