Timeline



Sep 5, 2006:

11:02 PM Changeset in webkit [16247] by darin
  • 5 edits in trunk/WebCore
  • some quick build fixes
  • bridge/mac/WebCoreAXObject.mm: Remove use of WebCore::UChar.
  • bridge/mac/WebCoreFrameBridge.mm: Ditto. Also remove include of CharsetNames.h.
  • css/CSSGrammar.y: Remove use of WebCore::UChar.
  • platform/UChar.h: Back out WebCore::UChar for now.
10:23 PM Changeset in webkit [16246] by thatcher
  • 2 edits in trunk/WebKit

Reviewed by Darin.

Drastically simplified the makefile for migrating headers from
WebCore and JavaScriptCore. The old version was always copying the files.

  • MigrateHeaders.make:
10:18 PM Changeset in webkit [22143] by darin
  • 1 edit in branches/WindowsMerge/WebCore/platform/win/TemporaryLinkStubs.cpp

Cleared a couple of stray executable bits.

9:40 PM Changeset in webkit [16245] by darin
  • 93 edits
    14 adds
    3 deletes in trunk

LayoutTests:

Reviewed by Alexey.

  • test for bug where the charset in a link element for a CSS stylesheet is ignored. I noticed this while working on new text encoding machinery.
  • fast/encoding/css-link-charset-expected.txt: Added.
  • fast/encoding/css-link-charset.css: Added.
  • fast/encoding/css-link-charset.html: Added.
  • fast/forms/form-data-encoding-normalization-overrun-expected.txt: Added.
  • fast/forms/form-data-encoding-normalization-overrun.html: Added.
  • and a tweak to an existing test
  • fast/forms/form-data-encoding.html: Changed to dump encoded URL so it's easier to diagnose this when it fails.

WebCore:

Reviewed by Alexey.

  • fixed a bug where link elements would not set the charset properly for CSS stylesheets they loaded
  • converted DeprecatedString code paths that are related to decoding web pages to use String instead, to ensure that conversion back and forth won't hurt performance

Test: fast/encoding/css-link-charset.html
Test: fast/forms/form-data-encoding-normalization-overrun.html

Coverage for encoding issues is pretty good, so we probably don't need more
tests to land this. Our existing tests did find issues with this patch while
it was under development. And I suppose it would be nice to have even more tests.

  • platform/TextEncoding.h:
  • platform/TextEncoding.cpp: (WebCore::addEncodingName): Added. Used to build up the set used by isJapanese. (WebCore::TextEncoding::TextEncoding): Removed boolean "eight bit only" parameter and added an overload for String as well as const char*. Simplified because now the only data member is m_name -- calls the registry's atomicCanonicalTextEncodingName function to make the name canonical (resolve aliases) and atomic (use a single pointer for each encoding name so we can compare and hash efficiently). (WebCore::TextEncoding::decode): Renamed from toUnicode. Just a simple wrapper on top of TextDecoder that can be used when the data to decode is all present at once. (WebCore::TextEncoding::encode): Renamed from fromUnicode. Handles the normalization and then uses the registry to get a codec to handle the rest. (WebCore::TextEncoding::usesVisualOrdering): New implementation that compares with the name of the only encoding that uses visual ordering. We blur the concepts a bit so that we treat the visual ordering and logical ordering variations as two separate encodings. (WebCore::TextEncoding::isJapanese): New implementation that uses a set to efficiently determine if an encoding is Japanese. (WebCore::TextEncoding::backslashAsCurrencySymbol): New implementation that compares with the names of the two encodings that have the strange backslash. (WebCore::TextEncoding::closest8BitEquivalent): Added. Replaces the old "eight bit only" boolean parameter to the constructor. (WebCore::ASCIIEncoding): Added. (WebCore::Latin1Encoding): Added. (WebCore::UTF16BigEndianEncoding): Added. (WebCore::UTF16LittleEndianEncoding): Added. (WebCore::UTF8Encoding): Added. (WebCore::WindowsLatin1Encoding): Added.
  • platform/TextEncodingRegistry.h: Added.
  • platform/TextEncodingRegistry.cpp: Added. Keeps a table of all the character set aliases and names and another of all the codecs and parameters for each name.
  • platform/TextDecoder.h: Added.
  • platform/TextDecoder.cpp: Added. Contains logic to look for a BOM and hand the data to the proper codec, based on code that used to be in both the ICU and Mac codecs.
  • platform/StreamingTextDecoder.h: Renamed class to TextCodec. We'll rename the files in a later check-in. Moved creation functions into TextEncodingRegistry.h. Added typedefs of the registrar function types so classes derived from this one can use them without including the TextEncodingRegistry header. Renamed toUnicode and fromUnicode to decode and encode. Changed the parameter and return types so that the parameters are pointers to the data and the return types are String and CString.
  • platform/StreamingTextDecoder.cpp: (WebCore::TextCodec::appendOmittingBOM): Added. Helper function used by multiple classes derived from this one.
  • platform/TextCodecLatin1.h: Added.
  • platform/TextCodecLatin1.cpp: Added. Contains logic to handle encoding and decoding Windows Latin-1, based on code that used to be in both the ICU and Mac codecs.
  • platform/TextCodecUTF16.h: Added.
  • platform/TextCodecUTF16.cpp: Added. Contains logic to handle encoding and decoding UTF-16, based on code that used to be in both the ICU and Mac codecs.
  • platform/StreamingTextDecoderICU.h: Renamed class to TextCodecICU. We'll rename the files in a later check-in. Removed all the functions having to do with handling BOM, UTF-16, and Latin-1; those are now handled elsewhere. Removed textEncodingSupported because that's superseded by the registry. Added registry hook functions.
  • platform/StreamingTextDecoderICU.cpp: (WebCore::TextCodecICU::registerEncodingNames): Added. Registers all encodings that ICU can handle with the "IANA" standard. Also includes a special case for a particular type of encoding for Hebrew that uses logical ordering. Also includes aliases that are not in ICU but that were historically known to WebKit for encodings that ICU handles. (WebCore::newTextCodecICU): Added. Used by registerCodecs. (WebCore::TextCodecICU::registerCodecs): Added. Registers codecs for the same encodings as above. (WebCore::TextCodecICU::TextCodecICU): Much simplified since this now only handles the actual ICU encoding and decoding. (WebCore::TextCodecICU::~TextCodecICU): Renamed. (WebCore::TextCodecICU::releaseICUConverter): Changed to be a const member function. (WebCore::TextCodecICU::createICUConverter): Changed to be a const member function and to check if the cached converter can be reused in a simpler way. (WebCore::TextCodecICU::decode): Updated for changes to types. (WebCore::TextCodecICU::encode): Updated for changes to types, and removed normalization since this is now handled by the caller.
  • platform/mac/StreamingTextDecoderMac.h: Renamed class to TextCodecMac. We'll rename the files in a later check-in. Removed all the functions having to do with handling BOM, UTF-16, and Latin-1; those are now handled elsewhere. Removed textEncodingSupported because that's superseded by the registry. Added registry hook functions.
  • platform/mac/StreamingTextDecoderMac.cpp: (WebCore::TextCodecMac::registerEncodingNames): Added. Registers encodings based on the charset table generated by the make-charset-table.pl perl script. (WebCore::newTextCodecMac): Added. Used by registerCodecs. (WebCore::TextCodecMac::registerCodecs): Added. Registers codecs for the same encodings as above. (WebCore::TextCodecMac::TextCodecMac): Much simplified since this now only handles the actual TEC/CF encoding and decoding. (WebCore::TextCodecMac::~TextCodecMac): Renamed. (WebCore::TextCodecMac::releaseTECConverter): Changed to be a const member function. (WebCore::TextCodecMac::createTECConverter): Changed to be a const member function. (WebCore::TextCodecMac::decode): Updated for changes to types. (WebCore::TextCodecMac::encode): Updated for changes to types, and removed normalization since this is now handled by the caller.
  • platform/mac/mac-encodings.txt: Removed most of the names in this file. This now only includes encodings where we want to use Mac OS X Carbon Text Encoding Converter, which is only encodings that are not supported by ICU.
  • platform/make-charset-table.pl: Removed flags from output. We don't use them any more.
  • platform/CharsetData.h: Changed from a platform-independent header into a Macintosh-specific one. A later patch should move this and rename it. Also subsumes ExtraCFEncodings.h.
  • WebCore.xcodeproj/project.pbxproj: Added new files. Changed the prefix on the "make character sets" rule to be kTextEncoding instead of kCFStringEncoding.
  • loader/Decoder.h: Change the default encoding parameter to the constructor to be a TextEncoding object. Renamed setEncodingName to setEncoding, and made it take a TextEncoding for the encoding. Removed the encodingName and visuallyOrdered functions, since TextEncoding supports both directly in a straightforward way. Changed both decode and flush functions to return String instead of DeprecatedString. Added a number of private functions to factor this class a bit more so it's easier to read. Got rid of a number of redundant data members. Changed the buffer to a Vector<char>.
  • loader/Decoder.cpp: (WebCore::Decoder::determineContentType): Added. Used by constructor to determine the content type based on the passed-in MIME type. (WebCore::Decoder::defaultEncoding): Added. Used by constructor to determine the default encoding based on the passed in default and the content type. (WebCore::Decoder::Decoder): Changed to use the functions above. Also renamed m_reachedBody to m_checkedForHeadCharset. (WebCore::Decoder::setEncoding): Renamed and changed to take an encoding rather than an encoding name. (WebCore::Decoder::checkForBOM): Factored out of decode. (WebCore::Decoder::checkForCSSCharset): Factored out of decode. (WebCore::Decoder::checkForHeadCharset): Factored out of decode. (WebCore::Decoder::detectJapaneseEncoding): Factored out of decode. (WebCore::Decoder::decode): Refactored so it's no longer one huge function. Changed to use the new Vector<char> and the new API for TextDecoder. (WebCore::Decoder::flush): Added code to empty out the buffer. Not an issue in practice since we don't re-use the decoder after flushing it.
  • platform/UChar.h: Added. Has the type named WebCore::UChar that we'll be switching to. We'll switch away from the ICU ::UChar type, because we don't want to be so closely tied to ICU -- include this instead of <unicode/umachine.h>.
  • platform/PlatformString.h:
  • platform/String.cpp: (WebCore::String::latin1): Updated for changes to TextEncoding. (WebCore::String::utf8): Ditto. (WebCore::String::newUninitialized): Added. Gives a way to create a String and then write directly into its buffer.
  • platform/StringImpl.h: Changed return value for charactersWithNullTermination to be a const UChar*. While it's true that this function changes the underlying StringImpl, the characters still shouldn't be modified with the returned pointer.
  • platform/StringImpl.cpp: (WebCore::StringImpl::charactersWithNullTermination): Updated for change above. (WebCore::StringImpl::newUninitialized): Added. Gives a way to create a StringImpl and then write directly into its buffer.
  • platform/CString.h:
  • platform/CString.cpp: (WebCore::CString::newUninitialized): Added. Gives a way to create a CString and then write directly into its buffer.
  • bridge/mac/WebCoreFrameBridge.h: Removed textEncoding method, and replaced +[WebCoreFrameBridge stringWithData:textEncoding:] with -[WebCoreFrameBridge stringWithData:] to avoid having to pass text encoding IDs around.
  • bridge/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge setEncoding:userChosen:]): Removed now-unneeded conversion to DeprecatedString. (-[WebCoreFrameBridge stringByEvaluatingJavaScriptFromString:forceUserGesture:]): Ditto. (-[WebCoreFrameBridge aeDescByEvaluatingJavaScriptFromString:]): Ditto. (-[WebCoreFrameBridge referrer]): Removed now-unneeded call to getNSString. (-[WebCoreFrameBridge stringWithData:]): Added. Asks the document's decoder what its encoding is, and decodes using that. (+[WebCoreFrameBridge stringWithData:textEncodingName:]): Simplified so it no longer involved a text encoding ID number. (-[WebCoreFrameBridge smartInsertForString:replacingRange:beforeString:afterString:]): Changed to use UChar instead of DeprecatedChar. (-[WebCoreFrameBridge documentFragmentWithMarkupString:baseURLString:]): Removed now-unneeded conversion to DeprecatedString. (-[WebCoreFrameBridge documentFragmentWithText:inContext:]): Ditto.
  • html/HTMLFormElement.cpp: (WebCore::encodeCString): Changed parameter to CString. (WebCore::HTMLFormElement::formData): Updated code for improvements to TextEncoding.
  • loader/CachedCSSStyleSheet.h:
  • loader/CachedCSSStyleSheet.cpp: (WebCore::CachedCSSStyleSheet::CachedCSSStyleSheet): Fixed mistake where the decoder was created without passing in the character set. Also changed from DeprecatedString to String. (WebCore::CachedCSSStyleSheet::setCharset): More of the same.
  • bindings/js/kjs_window.h: (KJS::ScheduledAction::ScheduledAction): Changed to use String instead of DeprecatedString, UChar instead of DeprecatedChar, CString instead of DeprecatedCString, etc.
  • bridge/mac/FormDataMac.mm: (WebCore::arrayFromFormData): Ditto.
  • bridge/mac/FrameMac.h: Ditto.
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::isCharacterSmartReplaceExempt): Ditto.
  • bridge/mac/WebCoreAXObject.mm: (-[WebCoreAXObject helpText]): Ditto. (-[WebCoreAXObject value]): Ditto. (-[WebCoreAXObject accessibilityDescription]): Ditto. (-[WebCoreAXObject doAXStringForTextMarkerRange:]): Ditto.
  • bridge/mac/WebCoreEncodings.mm: (+[WebCoreEncodings decodeData:]): Ditto. Also fixed code that does a deref without a ref to use RefPtr instead.
  • bridge/mac/WebCoreScriptDebugger.mm: (-[WebCoreScriptCallFrame evaluateWebScript:]): Ditto.
  • bridge/mac/WebCoreSettings.mm: (-[WebCoreSettings setDefaultTextEncoding:]): Ditto.
  • css/CSSImportRule.cpp: (WebCore::CSSImportRule::insertedIntoParent): Ditto.
  • css/cssparser.cpp: (WebCore::CSSParser::lex): Ditto.
  • dom/Document.h:
  • dom/Document.cpp: (WebCore::Document::setCharset): Ditto. (WebCore::Document::write): Ditto. (WebCore::Document::determineParseMode): Ditto.
  • dom/ProcessingInstruction.cpp: (WebCore::ProcessingInstruction::checkStyleSheet): Ditto.
  • dom/XMLTokenizer.h:
  • dom/XMLTokenizer.cpp: (WebCore::shouldAllowExternalLoad): Ditto. (WebCore::createStringParser): Ditto. (WebCore::XMLTokenizer::write): Ditto. (WebCore::toString): Ditto. (WebCore::handleElementAttributes): Ditto. (WebCore::XMLTokenizer::startElementNs): Ditto. (WebCore::XMLTokenizer::endElementNs): Ditto. (WebCore::XMLTokenizer::characters): Ditto. (WebCore::XMLTokenizer::processingInstruction): Ditto. (WebCore::XMLTokenizer::cdataBlock): Ditto. (WebCore::XMLTokenizer::comment): Ditto. (WebCore::XMLTokenizer::internalSubset): Ditto. (WebCore::getXHTMLEntity): Ditto. (WebCore::externalSubsetHandler): Ditto. (WebCore::XMLTokenizer::initializeParserContext): Ditto. (WebCore::XMLTokenizer::notifyFinished): Ditto. (WebCore::xmlDocPtrForString): Ditto. (WebCore::parseXMLDocumentFragment): Ditto. (WebCore::attributesStartElementNsHandler): Ditto. (WebCore::parseAttributes): Ditto.
  • html/FormDataList.h:
  • html/FormDataList.cpp: (WebCore::FormDataList::appendString): Ditto. Also changed to call the encoding function by its new name and with new parameters. (WebCore::FormDataList::appendFile): Ditto.
  • html/HTMLDocument.h:
  • html/HTMLDocument.cpp: (WebCore::parseDocTypePart): Ditto. (WebCore::containsString): Ditto. (WebCore::parseDocTypeDeclaration): Ditto. (WebCore::HTMLDocument::determineParseMode): Ditto.
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::appendFormData): Ditto.
  • html/HTMLScriptElement.cpp: (WebCore::HTMLScriptElement::parseMappedAttribute): Ditto.
  • html/HTMLTokenizer.h:
  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::scriptHandler): Ditto. (WebCore::HTMLTokenizer::parseTag): Ditto. (WebCore::HTMLTokenizer::write): Ditto. (WebCore::HTMLTokenizer::finish): Ditto. (WebCore::parseHTMLDocumentFragment): Ditto.
  • loader/Cache.h:
  • loader/Cache.cpp: (WebCore::Cache::requestStyleSheet): Ditto. (WebCore::Cache::requestScript): Ditto.
  • loader/CachedResource.h: Ditto.
  • loader/CachedScript.h:
  • loader/CachedScript.cpp: (WebCore::CachedScript::CachedScript): Ditto. (WebCore::CachedScript::ref): Ditto. (WebCore::CachedScript::deref): Ditto. (WebCore::CachedScript::setCharset): Ditto. (WebCore::CachedScript::data): Ditto. (WebCore::CachedScript::checkNotify): Ditto.
  • loader/CachedXBLDocument.h:
  • loader/CachedXBLDocument.cpp: (WebCore::CachedXBLDocument::setCharset): Ditto.
  • loader/CachedXSLStyleSheet.h:
  • loader/CachedXSLStyleSheet.cpp: (WebCore::CachedXSLStyleSheet::setCharset): Ditto.
  • loader/DocLoader.cpp: (WebCore::DocLoader::requestStyleSheet): Ditto. (WebCore::DocLoader::requestScript): Ditto.
  • loader/DocLoader.h: Ditto.
  • loader/FormData.h:
  • loader/FormData.cpp: (WebCore::FormData::FormData): Ditto. (WebCore::FormData::appendFile): Ditto. (WebCore::FormData::flattenToString): Ditto.
  • page/Frame.h:
  • page/FramePrivate.h:
  • page/Frame.cpp: (WebCore::UserStyleSheetLoader::setStyleSheet): Ditto. (WebCore::getString): Ditto. (WebCore::Frame::replaceContentsWithScriptResult): Ditto. (WebCore::Frame::executeScript): Ditto. (WebCore::Frame::clear): Ditto. (WebCore::Frame::write): Ditto. (WebCore::Frame::endIfNotLoading): Ditto. (WebCore::Frame::baseTarget): Ditto. (WebCore::Frame::scheduleRedirection): Ditto. (WebCore::Frame::scheduleLocationChange): Ditto. (WebCore::Frame::scheduleHistoryNavigation): Ditto. (WebCore::Frame::changeLocation): Ditto. (WebCore::Frame::redirectionTimerFired): Ditto. (WebCore::Frame::encoding): Ditto. (WebCore::Frame::submitForm): Ditto. (WebCore::Frame::referrer): Ditto. (WebCore::Frame::isCharacterSmartReplaceExempt): Ditto. (WebCore::Frame::setEncoding): Ditto.
  • page/Settings.h: Ditto.
  • platform/SegmentedString.h: Ditto.
  • platform/SegmentedString.cpp: Ditto.
  • xml/XSLStyleSheet.cpp: (WebCore::XSLStyleSheet::parseString): Ditto.
  • xml/XSLTProcessor.cpp: (WebCore::transformTextStringToXHTMLDocumentString): Ditto. (WebCore::XSLTProcessor::createDocumentFromSource): Ditto.
  • xml/xmlhttprequest.h:
  • xml/xmlhttprequest.cpp: (WebCore::XMLHttpRequest::open): Ditto. (WebCore::XMLHttpRequest::send): Ditto. (WebCore::XMLHttpRequest::receivedData): Ditto.
  • platform/DeprecatedString.cpp: (WebCore::DeprecatedString::fromUtf8): Updated for changes to TextEncoding. (WebCore::DeprecatedString::utf8): Ditto.
  • platform/KURL.h:
  • platform/KURL.cpp: (WebCore::KURL::KURL): Updated to overload based on presence or absence of TextEncoding rather than having a default. (WebCore::KURL::init): Moved body of constructor in here. Updated to use the new TextEncoding interface. (WebCore::KURL::decode_string): Updated to overload based on presence or absence of TextEncoding rather than having a default. Updated to use the new TextEncoding interface. (WebCore::encodeRelativeString): Updated to use the new TextEncoding interface.
  • platform/Font.cpp: (WebCore::WidthIterator::normalizeVoicingMarks): Fixed code to use U_ZERO_ERROR instead of a typecast.
  • bindings/js/kjs_proxy.h: Removed unneeded declaration of DeprecatedString.
  • platform/GraphicsContext.h: Ditto.
  • platform/GraphicsContext.cpp: Removed unneeded include of "DeprecatedString.h".
  • rendering/break_lines.cpp: Ditto.
  • xml/XMLSerializer.cpp: Ditto.
  • platform/mac/FontDataMac.mm: Removed unneeded include of <unicode/unorm.h>.
  • platform/CharsetNames.h: Emptied out this file. A later patch could remove it.
  • platform/CharsetNames.cpp: Ditto.
  • platform/mac/ExtraCFEncodings.h: Ditto.

WebKit:

Reviewed by Alexey.

  • WebKit side of changes to encoding
  • WebView/WebHTMLRepresentation.m: (-[WebHTMLRepresentation documentSource]): Changed to call new -[WebCoreFrameBridge stringWithData:] instead of the calling the old methods that used a CFStringEncoding: -[WebCoreFrameBridge textEncoding] and +[WebCoreFrameBridge stringWithData:textEncoding:].
  • WebView/WebResource.m: (-[WebResource _stringValue]): Removed special case for nil encoding name. The bridge itself now has the rule that "nil encoding name means Latin-1", so we don't need to check for nil.
  • WebView/WebFrame.m: (-[WebFrame _checkLoadComplete]): Retain the frame until we get the parent frame while walking up parent frames, because it's possible for _checkLoadCompleteForThisFrame to release the last reference to the frame. (Not reviewed; needed to run performance tests successfully.)
9:40 PM Changeset in webkit [22142] by darin
  • 2 edits in branches/WindowsMerge/WebCore
  • WebCore.vcproj/WebCore.vcproj: Update for new headers.
8:36 PM Changeset in webkit [16244] by eseidel
  • 127 edits in trunk/WebCore

2006-09-05 Nikolas Zimmermann <zimmermann@kde.org>

Reviewed by eseidel. Landed by eseidel.

Fixes: http://bugzilla.opendarwin.org/show_bug.cgi?id=10490

-> Remove all SVGAnimated* usage from ksvg2.
This is all done using a set of macros which live in svg/SVGElement.h
(ANIMATED_PROPERTY_DECLARATIONS / ANIMATED_PROPERTY_DEFINITIONS).
Transform all properties to this new concept using Eric ruby script.

-> Replace all usages of "property()->baseVal()" with "propertyBaseValue()".
-> Replace all usages of "property()->setBaseVal()" with "setPropertyBaseValue()".

-> Add "virtual const SVGElement* contextElement() const".
This concept allows to access the document (needed to access the SVGDocumentExtensions)
from lowest-base-classes like SVGURIReference / SVGExternalResourcesRequired etc.

These classes contain this contextElement() function as pure virtual functions.
The topmost classes (like SVGCircleElement) implement this function with "return this".

-> Move default value initialization to constructors.
As the lazy_create usage, which was involved in default property initialization,
has been removed, we need to do the initialization in the constructor.


This forces us to fix the SVGLength class, as calling viewportElement() from the
constructor does not work. This SVGLength cleanup is one of the next things to fix.

  • bindings/scripts/CodeGeneratorJS.pm:
  • kcanvas/RenderForeignObject.cpp: (WebCore::RenderForeignObject::translationForAttributes):
  • kcanvas/RenderSVGImage.cpp: (WebCore::RenderSVGImage::paint): (WebCore::RenderSVGImage::relativeBBox): (WebCore::RenderSVGImage::getAbsoluteRepaintRect): (WebCore::RenderSVGImage::translationForAttributes):
  • kcanvas/RenderSVGText.cpp: (WebCore::RenderSVGText::translationForAttributes):
  • ksvg2/misc/KSVGTimeScheduler.cpp: (WebCore::SVGTimer::notifyAll):
  • ksvg2/misc/SVGDocumentExtensions.cpp:
  • ksvg2/misc/SVGDocumentExtensions.h: (WebCore::SVGDocumentExtensions::baseValueMap): (WebCore::SVGDocumentExtensions::baseValue): (WebCore::SVGDocumentExtensions::setBaseValue): (WebCore::SVGDocumentExtensions::hasBaseValue): (WebCore::String): (WebCore::bool): (WebCore::double):
  • ksvg2/misc/SVGImageLoader.cpp: (WebCore::SVGImageLoader::updateFromElement):
  • ksvg2/svg/SVGAElement.cpp: (WebCore::SVGAElement::parseMappedAttribute): (WebCore::SVGAElement::defaultEventHandler):
  • ksvg2/svg/SVGAElement.h: (WebCore::SVGAElement::contextElement):
  • ksvg2/svg/SVGAnimateColorElement.h: (WebCore::SVGAnimateColorElement::contextElement):
  • ksvg2/svg/SVGAnimateElement.h: (WebCore::SVGAnimateElement::contextElement):
  • ksvg2/svg/SVGAnimateTransformElement.cpp: (WebCore::SVGAnimateTransformElement::handleTimerEvent): (WebCore::SVGAnimateTransformElement::initialMatrix):
  • ksvg2/svg/SVGAnimateTransformElement.h: (WebCore::SVGAnimateTransformElement::contextElement):
  • ksvg2/svg/SVGCircleElement.cpp: (SVGCircleElement::SVGCircleElement): (SVGCircleElement::parseMappedAttribute): (SVGCircleElement::toPathData): (SVGCircleElement::pushAttributeContext):
  • ksvg2/svg/SVGCircleElement.h: (WebCore::SVGCircleElement::contextElement):
  • ksvg2/svg/SVGClipPathElement.cpp: (SVGClipPathElement::SVGClipPathElement): (SVGClipPathElement::parseMappedAttribute): (SVGClipPathElement::canvasResource):
  • ksvg2/svg/SVGClipPathElement.h: (WebCore::SVGClipPathElement::contextElement):
  • ksvg2/svg/SVGComponentTransferFunctionElement.cpp: (SVGComponentTransferFunctionElement::SVGComponentTransferFunctionElement): (SVGComponentTransferFunctionElement::parseMappedAttribute): (SVGComponentTransferFunctionElement::transferFunction):
  • ksvg2/svg/SVGComponentTransferFunctionElement.h:
  • ksvg2/svg/SVGCursorElement.cpp: (SVGCursorElement::SVGCursorElement): (SVGCursorElement::parseMappedAttribute):
  • ksvg2/svg/SVGCursorElement.h: (WebCore::SVGCursorElement::contextElement):
  • ksvg2/svg/SVGDefsElement.h: (WebCore::SVGDefsElement::contextElement):
  • ksvg2/svg/SVGElement.h:
  • ksvg2/svg/SVGEllipseElement.cpp: (WebCore::SVGEllipseElement::SVGEllipseElement): (WebCore::SVGEllipseElement::parseMappedAttribute): (WebCore::SVGEllipseElement::toPathData): (WebCore::SVGEllipseElement::pushAttributeContext):
  • ksvg2/svg/SVGEllipseElement.h: (WebCore::SVGEllipseElement::contextElement):
  • ksvg2/svg/SVGExternalResourcesRequired.cpp: (WebCore::SVGExternalResourcesRequired::SVGExternalResourcesRequired): (WebCore::SVGExternalResourcesRequired::parseMappedAttribute):
  • ksvg2/svg/SVGExternalResourcesRequired.h:
  • ksvg2/svg/SVGFEBlendElement.cpp: (SVGFEBlendElement::SVGFEBlendElement): (SVGFEBlendElement::parseMappedAttribute): (SVGFEBlendElement::filterEffect):
  • ksvg2/svg/SVGFEBlendElement.h: (WebCore::SVGFEBlendElement::contextElement):
  • ksvg2/svg/SVGFEColorMatrixElement.cpp: (SVGFEColorMatrixElement::SVGFEColorMatrixElement): (SVGFEColorMatrixElement::parseMappedAttribute): (SVGFEColorMatrixElement::filterEffect):
  • ksvg2/svg/SVGFEColorMatrixElement.h: (WebCore::SVGFEColorMatrixElement::contextElement):
  • ksvg2/svg/SVGFEComponentTransferElement.cpp: (SVGFEComponentTransferElement::SVGFEComponentTransferElement): (SVGFEComponentTransferElement::parseMappedAttribute): (SVGFEComponentTransferElement::filterEffect):
  • ksvg2/svg/SVGFEComponentTransferElement.h: (WebCore::SVGFEComponentTransferElement::contextElement):
  • ksvg2/svg/SVGFECompositeElement.cpp: (SVGFECompositeElement::SVGFECompositeElement): (SVGFECompositeElement::parseMappedAttribute): (SVGFECompositeElement::filterEffect):
  • ksvg2/svg/SVGFECompositeElement.h: (WebCore::SVGFECompositeElement::contextElement):
  • ksvg2/svg/SVGFEDiffuseLightingElement.cpp: (WebCore::SVGFEDiffuseLightingElement::SVGFEDiffuseLightingElement): (WebCore::SVGFEDiffuseLightingElement::parseMappedAttribute): (WebCore::SVGFEDiffuseLightingElement::filterEffect):
  • ksvg2/svg/SVGFEDiffuseLightingElement.h: (WebCore::SVGFEDiffuseLightingElement::contextElement):
  • ksvg2/svg/SVGFEDisplacementMapElement.cpp: (SVGFEDisplacementMapElement::SVGFEDisplacementMapElement): (SVGFEDisplacementMapElement::parseMappedAttribute): (SVGFEDisplacementMapElement::filterEffect):
  • ksvg2/svg/SVGFEDisplacementMapElement.h: (WebCore::SVGFEDisplacementMapElement::contextElement):
  • ksvg2/svg/SVGFEDistantLightElement.cpp: (WebCore::SVGFEDistantLightElement::lightSource):
  • ksvg2/svg/SVGFEFloodElement.cpp: (SVGFEFloodElement::parseMappedAttribute): (SVGFEFloodElement::filterEffect):
  • ksvg2/svg/SVGFEFloodElement.h: (WebCore::SVGFEFloodElement::contextElement):
  • ksvg2/svg/SVGFEGaussianBlurElement.cpp: (SVGFEGaussianBlurElement::SVGFEGaussianBlurElement): (SVGFEGaussianBlurElement::parseMappedAttribute): (SVGFEGaussianBlurElement::filterEffect):
  • ksvg2/svg/SVGFEGaussianBlurElement.h: (WebCore::SVGFEGaussianBlurElement::contextElement):
  • ksvg2/svg/SVGFEImageElement.cpp: (SVGFEImageElement::SVGFEImageElement): (SVGFEImageElement::parseMappedAttribute):
  • ksvg2/svg/SVGFEImageElement.h: (WebCore::SVGFEImageElement::contextElement):
  • ksvg2/svg/SVGFELightElement.cpp: (SVGFELightElement::SVGFELightElement): (SVGFELightElement::parseMappedAttribute):
  • ksvg2/svg/SVGFELightElement.h:
  • ksvg2/svg/SVGFEMergeElement.cpp: (SVGFEMergeElement::SVGFEMergeElement): (SVGFEMergeElement::filterEffect):
  • ksvg2/svg/SVGFEMergeElement.h: (WebCore::SVGFEMergeElement::contextElement):
  • ksvg2/svg/SVGFEMergeNodeElement.cpp: (SVGFEMergeNodeElement::parseMappedAttribute):
  • ksvg2/svg/SVGFEMergeNodeElement.h: (WebCore::SVGFEMergeNodeElement::contextElement):
  • ksvg2/svg/SVGFEOffsetElement.cpp: (SVGFEOffsetElement::SVGFEOffsetElement): (SVGFEOffsetElement::parseMappedAttribute): (SVGFEOffsetElement::filterEffect):
  • ksvg2/svg/SVGFEOffsetElement.h: (WebCore::SVGFEOffsetElement::contextElement):
  • ksvg2/svg/SVGFEPointLightElement.cpp: (WebCore::SVGFEPointLightElement::lightSource):
  • ksvg2/svg/SVGFESpecularLightingElement.cpp: (SVGFESpecularLightingElement::SVGFESpecularLightingElement): (SVGFESpecularLightingElement::parseMappedAttribute): (SVGFESpecularLightingElement::filterEffect):
  • ksvg2/svg/SVGFESpecularLightingElement.h: (WebCore::SVGFESpecularLightingElement::contextElement):
  • ksvg2/svg/SVGFESpotLightElement.cpp: (WebCore::SVGFESpotLightElement::lightSource):
  • ksvg2/svg/SVGFETileElement.cpp: (SVGFETileElement::parseMappedAttribute): (SVGFETileElement::filterEffect):
  • ksvg2/svg/SVGFETileElement.h: (WebCore::SVGFETileElement::contextElement):
  • ksvg2/svg/SVGFETurbulenceElement.cpp: (SVGFETurbulenceElement::SVGFETurbulenceElement): (SVGFETurbulenceElement::parseMappedAttribute): (SVGFETurbulenceElement::filterEffect):
  • ksvg2/svg/SVGFETurbulenceElement.h: (WebCore::SVGFETurbulenceElement::contextElement):
  • ksvg2/svg/SVGFilterElement.cpp: (SVGFilterElement::SVGFilterElement): (SVGFilterElement::parseMappedAttribute): (SVGFilterElement::canvasResource):
  • ksvg2/svg/SVGFilterElement.h: (WebCore::SVGFilterElement::contextElement):
  • ksvg2/svg/SVGFilterPrimitiveStandardAttributes.cpp: (SVGFilterPrimitiveStandardAttributes::SVGFilterPrimitiveStandardAttributes): (SVGFilterPrimitiveStandardAttributes::~SVGFilterPrimitiveStandardAttributes): (SVGFilterPrimitiveStandardAttributes::parseMappedAttribute): (SVGFilterPrimitiveStandardAttributes::setStandardAttributes):
  • ksvg2/svg/SVGFilterPrimitiveStandardAttributes.h: (WebCore::SVGFilterPrimitiveStandardAttributes::contextElement):
  • ksvg2/svg/SVGFitToViewBox.cpp: (WebCore::SVGFitToViewBox::SVGFitToViewBox): (WebCore::SVGFitToViewBox::parseViewBox): (WebCore::SVGFitToViewBox::viewBoxToViewTransform): (WebCore::SVGFitToViewBox::parseMappedAttribute):
  • ksvg2/svg/SVGFitToViewBox.h:
  • ksvg2/svg/SVGForeignObjectElement.cpp: (WebCore::SVGForeignObjectElement::SVGForeignObjectElement): (WebCore::SVGForeignObjectElement::parseMappedAttribute):
  • ksvg2/svg/SVGForeignObjectElement.h: (WebCore::SVGForeignObjectElement::isValid): (WebCore::SVGForeignObjectElement::contextElement):
  • ksvg2/svg/SVGGElement.h: (WebCore::SVGGElement::contextElement):
  • ksvg2/svg/SVGGradientElement.cpp: (SVGGradientElement::SVGGradientElement): (SVGGradientElement::parseMappedAttribute): (SVGGradientElement::rebuildStops):
  • ksvg2/svg/SVGGradientElement.h:
  • ksvg2/svg/SVGHelper.cpp: (WebCore::SVGHelper::PercentageOfViewport):
  • ksvg2/svg/SVGImageElement.cpp: (SVGImageElement::SVGImageElement): (SVGImageElement::parseMappedAttribute): (SVGImageElement::haveLoadedRequiredResources):
  • ksvg2/svg/SVGImageElement.h: (WebCore::SVGImageElement::contextElement):
  • ksvg2/svg/SVGLangSpace.h:
  • ksvg2/svg/SVGLength.cpp:
  • ksvg2/svg/SVGLineElement.cpp: (SVGLineElement::SVGLineElement): (SVGLineElement::parseMappedAttribute): (SVGLineElement::toPathData): (SVGLineElement::pushAttributeContext):
  • ksvg2/svg/SVGLineElement.h: (WebCore::SVGLineElement::contextElement):
  • ksvg2/svg/SVGLinearGradientElement.cpp: (SVGLinearGradientElement::SVGLinearGradientElement): (SVGLinearGradientElement::parseMappedAttribute): (SVGLinearGradientElement::buildGradient):
  • ksvg2/svg/SVGLinearGradientElement.h: (WebCore::SVGLinearGradientElement::contextElement):
  • ksvg2/svg/SVGMarkerElement.cpp: (WebCore::SVGMarkerElement::SVGMarkerElement): (WebCore::SVGMarkerElement::parseMappedAttribute): (WebCore::SVGMarkerElement::setOrientToAuto): (WebCore::SVGMarkerElement::setOrientToAngle): (WebCore::SVGMarkerElement::canvasResource):
  • ksvg2/svg/SVGMarkerElement.h: (WebCore::SVGMarkerElement::contextElement):
  • ksvg2/svg/SVGMaskElement.cpp: (WebCore::SVGMaskElement::SVGMaskElement): (WebCore::SVGMaskElement::attributeChanged): (WebCore::SVGMaskElement::parseMappedAttribute): (WebCore::SVGMaskElement::drawMaskerContent):
  • ksvg2/svg/SVGMaskElement.h: (WebCore::SVGMaskElement::contextElement):
  • ksvg2/svg/SVGPathElement.cpp: (WebCore::SVGPathElement::SVGPathElement):
  • ksvg2/svg/SVGPathElement.h: (WebCore::SVGPathElement::isValid): (WebCore::SVGPathElement::contextElement):
  • ksvg2/svg/SVGPatternElement.cpp: (WebCore::SVGPatternElement::SVGPatternElement): (WebCore::SVGPatternElement::parseMappedAttribute): (WebCore::SVGPatternElement::pushAttributeContext): (WebCore::SVGPatternElement::fillAttributesFromReferencePattern): (WebCore::SVGPatternElement::drawPatternContentIntoTile): (WebCore::SVGPatternElement::notifyAttributeChange): (WebCore::SVGPatternElement::getCTM):
  • ksvg2/svg/SVGPatternElement.h: (WebCore::SVGPatternElement::contextElement):
  • ksvg2/svg/SVGPolyElement.h: (WebCore::SVGPolyElement::contextElement):
  • ksvg2/svg/SVGRadialGradientElement.cpp: (WebCore::SVGRadialGradientElement::SVGRadialGradientElement): (WebCore::SVGRadialGradientElement::~SVGRadialGradientElement): (WebCore::SVGRadialGradientElement::parseMappedAttribute): (WebCore::SVGRadialGradientElement::buildGradient):
  • ksvg2/svg/SVGRadialGradientElement.h: (WebCore::SVGRadialGradientElement::contextElement):
  • ksvg2/svg/SVGRectElement.cpp: (WebCore::SVGRectElement::SVGRectElement): (WebCore::SVGRectElement::parseMappedAttribute): (WebCore::SVGRectElement::toPathData): (WebCore::SVGRectElement::pushAttributeContext):
  • ksvg2/svg/SVGRectElement.h: (WebCore::SVGRectElement::contextElement):
  • ksvg2/svg/SVGSVGElement.cpp: (WebCore::SVGSVGElement::SVGSVGElement): (WebCore::SVGSVGElement::~SVGSVGElement): (WebCore::SVGSVGElement::viewport): (WebCore::SVGSVGElement::parseMappedAttribute): (WebCore::SVGSVGElement::getCTM): (WebCore::SVGSVGElement::getScreenCTM): (WebCore::SVGSVGElement::createRenderer):
  • ksvg2/svg/SVGSVGElement.h: (WebCore::SVGSVGElement::contextElement):
  • ksvg2/svg/SVGScriptElement.h: (WebCore::SVGScriptElement::contextElement):
  • ksvg2/svg/SVGSetElement.h: (WebCore::SVGSetElement::contextElement):
  • ksvg2/svg/SVGStopElement.cpp: (SVGStopElement::SVGStopElement): (SVGStopElement::parseMappedAttribute):
  • ksvg2/svg/SVGStopElement.h:
  • ksvg2/svg/SVGStylable.h:
  • ksvg2/svg/SVGStyledElement.cpp:
  • ksvg2/svg/SVGStyledElement.h:
  • ksvg2/svg/SVGStyledTransformableElement.cpp: (SVGStyledTransformableElement::SVGStyledTransformableElement): (SVGStyledTransformableElement::parseMappedAttribute):
  • ksvg2/svg/SVGStyledTransformableElement.h:
  • ksvg2/svg/SVGSwitchElement.cpp:
  • ksvg2/svg/SVGSwitchElement.h: (WebCore::SVGSwitchElement::contextElement):
  • ksvg2/svg/SVGSymbolElement.h: (WebCore::SVGSymbolElement::contextElement):
  • ksvg2/svg/SVGTRefElement.cpp: (SVGTRefElement::updateReferencedText):
  • ksvg2/svg/SVGTRefElement.h: (WebCore::SVGTRefElement::contextElement):
  • ksvg2/svg/SVGTSpanElement.h: (WebCore::SVGTSpanElement::contextElement):
  • ksvg2/svg/SVGTextContentElement.cpp: (SVGTextContentElement::SVGTextContentElement): (SVGTextContentElement::parseMappedAttribute):
  • ksvg2/svg/SVGTextContentElement.h:
  • ksvg2/svg/SVGTextElement.cpp: (WebCore::SVGTextElement::SVGTextElement): (WebCore::SVGTextElement::parseMappedAttribute):
  • ksvg2/svg/SVGTextElement.h: (WebCore::SVGTextElement::contextElement):
  • ksvg2/svg/SVGTextPositioningElement.cpp: (SVGTextPositioningElement::SVGTextPositioningElement): (SVGTextPositioningElement::parseMappedAttribute):
  • ksvg2/svg/SVGTextPositioningElement.h:
  • ksvg2/svg/SVGTransformable.cpp:
  • ksvg2/svg/SVGTransformable.h:
  • ksvg2/svg/SVGURIReference.cpp: (SVGURIReference::parseMappedAttribute):
  • ksvg2/svg/SVGURIReference.h:
  • ksvg2/svg/SVGUseElement.cpp: (SVGUseElement::SVGUseElement): (SVGUseElement::parseMappedAttribute): (SVGUseElement::closeRenderer):
  • ksvg2/svg/SVGUseElement.h: (WebCore::SVGUseElement::contextElement):
  • ksvg2/svg/SVGViewElement.h: (WebCore::SVGViewElement::contextElement):
8:26 PM Changeset in webkit [16243] by eseidel
  • 5 edits in trunk/WebCore

2006-09-05 Eric Seidel <eric@eseidel.com>

Reviewed by aroben.

Speed up path parsing by removing string copies.
http://bugzilla.opendarwin.org/show_bug.cgi?id=10722


Tests already cover path parsing.

  • ksvg2/svg/SVGFitToViewBox.cpp: (WebCore::SVGFitToViewBox::parseViewBox):
  • ksvg2/svg/svgpathparser.cpp: (WebCore::skipOptionalSpaces): Added. (WebCore::skipOptionalSpacesOrComma): Added. (WebCore::parseCoord): (WebCore::SVGPolyParser::parsePoints): (WebCore::SVGPathParser::parseSVG):
7:59 PM Changeset in webkit [16242] by weinig
  • 6 edits in trunk/WebCore

Reviewed by Tim H.

Patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10742
More Objective-C DOM auto-generation cleanup


  • Factor out implementation type-getters into their own function and standardize naming as was done for type-makers (ie. HTML and CSS prefixing for method names).
  • bindings/objc/DOMCSS.mm: (-[DOMCSSStyleDeclaration getPropertyShorthand:]): (-[DOMCSSStyleDeclaration isPropertyImplicit:]): (-[DOMCSSStyleDeclaration _CSSStyleDeclaration]):
  • bindings/objc/DOMHTML.mm: (-[DOMHTMLTableCaptionElement _HTMLTableCaptionElement]): (-[DOMHTMLTableSectionElement _HTMLTableSectionElement]): (-[DOMHTMLTableElement _HTMLTableElement]): (-[DOMHTMLTableCellElement _HTMLTableCellElement]): (-[DOMHTMLEmbedElement _HTMLEmbedElement]): (-[DOMHTMLEmbedElement align]): (-[DOMHTMLEmbedElement setAlign:]): (-[DOMHTMLEmbedElement height]): (-[DOMHTMLEmbedElement setHeight:]): (-[DOMHTMLEmbedElement name]): (-[DOMHTMLEmbedElement setName:]): (-[DOMHTMLEmbedElement src]): (-[DOMHTMLEmbedElement setSrc:]): (-[DOMHTMLEmbedElement type]): (-[DOMHTMLEmbedElement setType:]): (-[DOMHTMLEmbedElement width]): (-[DOMHTMLEmbedElement setWidth:]):
  • bindings/objc/DOMInternal.h:
  • bindings/scripts/CodeGeneratorObjC.pm:
  • bridge/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge setTypingStyle:withUndoAction:]): (-[WebCoreFrameBridge applyStyle:withUndoAction:]): (-[WebCoreFrameBridge applyParagraphStyle:withUndoAction:]): (-[WebCoreFrameBridge selectionStartHasStyle:]): (-[WebCoreFrameBridge selectionHasStyle:]):
7:30 PM Changeset in webkit [16241] by thatcher
  • 5 edits
    2 copies in trunk/WebCore

Rolling out my earlier change, which broke the build for Tiger.

5:48 PM Changeset in webkit [16240] by thatcher
  • 5 edits
    2 deletes in trunk/WebCore

Reviewed by John.

Removing the DOMImplementationFront workaround for an older GCC bug.
This changes requires Xcode 2.3 or later.

  • WebCore.xcodeproj/project.pbxproj:
  • bindings/objc/DOM.mm: (-[DOMImplementation _initWithDOMImplementation:WebCore::]): (+[DOMImplementation _DOMImplementationWith:WebCore::]): (-[DOMImplementation WebCore::]):
  • bindings/objc/DOMImplementationFront.cpp: Removed.
  • bindings/objc/DOMImplementationFront.h: Removed.
  • bindings/objc/DOMInternal.h:
  • bindings/scripts/CodeGeneratorObjC.pm: remove DOMImplementationFront and some cleanup to under variables in WriteData
4:55 PM Changeset in webkit [16239] by darin
  • 2 edits in trunk/WebKitTools
  • Scripts/do-webcore-rename: More renaming plans.
4:38 PM Changeset in webkit [16238] by adele
  • 4 edits in trunk/WebCore

Reviewed by Adam.

Removed dead code for password field.

  • platform/mac/TextFieldMac.mm: (-[NSSearchFieldCell _addStringToRecentSearches:]):
  • platform/mac/WebCoreTextField.h:
  • platform/mac/WebCoreTextField.mm:
3:35 PM Changeset in webkit [16237] by justing
  • 273 edits in trunk

LayoutTests:

Reviewed by thatcher


Setup for <rdar://problem/4344550> Misspellings aren't marked after undo delete

Fixed:

  • editing/pasteboard/4641033-expected.checksum:
  • editing/pasteboard/4641033-expected.png:
  • editing/execCommand/outdent-selection-expected.checksum:
  • editing/execCommand/outdent-selection-expected.png:

Added a didChangeSelection notification that was absent before:

  • editing/deleting/collapse-whitespace-3587601-fix-expected.txt:
  • editing/deleting/delete-3608430-fix-expected.txt:
  • editing/deleting/delete-3608462-fix-expected.txt:
  • editing/deleting/delete-3775172-fix-expected.txt:
  • editing/deleting/delete-3800834-fix-expected.txt:
  • editing/deleting/delete-3857753-fix-expected.txt:
  • editing/deleting/delete-3865854-fix-expected.txt:
  • editing/deleting/delete-3928305-fix-expected.txt:
  • editing/deleting/delete-3959464-fix-expected.txt:
  • editing/deleting/delete-4038408-fix-expected.txt:
  • editing/deleting/delete-all-text-in-text-field-assertion-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-001-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-002-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-003-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-004-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-005-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-006-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-007-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-009-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-010-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-011-expected.txt:
  • editing/deleting/delete-block-contents-001-expected.txt:
  • editing/deleting/delete-block-contents-002-expected.txt:
  • editing/deleting/delete-block-contents-003-expected.txt:
  • editing/deleting/delete-block-merge-contents-001-expected.txt:
  • editing/deleting/delete-block-merge-contents-002-expected.txt:
  • editing/deleting/delete-block-merge-contents-003-expected.txt:
  • editing/deleting/delete-block-merge-contents-004-expected.txt:
  • editing/deleting/delete-block-merge-contents-005-expected.txt:
  • editing/deleting/delete-block-merge-contents-006-expected.txt:
  • editing/deleting/delete-block-merge-contents-007-expected.txt:
  • editing/deleting/delete-block-merge-contents-008-expected.txt:
  • editing/deleting/delete-block-merge-contents-009-expected.txt:
  • editing/deleting/delete-block-merge-contents-010-expected.txt:
  • editing/deleting/delete-block-merge-contents-012-expected.txt:
  • editing/deleting/delete-block-merge-contents-013-expected.txt:
  • editing/deleting/delete-block-merge-contents-014-expected.txt:
  • editing/deleting/delete-block-merge-contents-015-expected.txt:
  • editing/deleting/delete-block-merge-contents-016-expected.txt:
  • editing/deleting/delete-block-merge-contents-017-expected.txt:
  • editing/deleting/delete-block-merge-contents-018-expected.txt:
  • editing/deleting/delete-block-merge-contents-019-expected.txt:
  • editing/deleting/delete-block-merge-contents-020-expected.txt:
  • editing/deleting/delete-block-merge-contents-021-expected.txt:
  • editing/deleting/delete-block-merge-contents-022-expected.txt:
  • editing/deleting/delete-block-merge-contents-023-expected.txt:
  • editing/deleting/delete-block-merge-contents-024-expected.txt:
  • editing/deleting/delete-block-table-expected.txt:
  • editing/deleting/delete-br-008-expected.txt:
  • editing/deleting/delete-br-010-expected.txt:
  • editing/deleting/delete-br-011-expected.txt:
  • editing/deleting/delete-first-list-item-expected.txt:
  • editing/deleting/delete-image-001-expected.txt:
  • editing/deleting/delete-image-002-expected.txt:
  • editing/deleting/delete-image-003-expected.txt:
  • editing/deleting/delete-image-004-expected.txt:
  • editing/deleting/delete-line-001-expected.txt:
  • editing/deleting/delete-line-002-expected.txt:
  • editing/deleting/delete-line-003-expected.txt:
  • editing/deleting/delete-line-004-expected.txt:
  • editing/deleting/delete-line-005-expected.txt:
  • editing/deleting/delete-line-006-expected.txt:
  • editing/deleting/delete-line-007-expected.txt:
  • editing/deleting/delete-line-008-expected.txt:
  • editing/deleting/delete-line-009-expected.txt:
  • editing/deleting/delete-line-010-expected.txt:
  • editing/deleting/delete-line-011-expected.txt:
  • editing/deleting/delete-line-012-expected.txt:
  • editing/deleting/delete-line-013-expected.txt:
  • editing/deleting/delete-line-014-expected.txt:
  • editing/deleting/delete-line-015-expected.txt:
  • editing/deleting/delete-line-016-expected.txt:
  • editing/deleting/delete-line-017-expected.txt:
  • editing/deleting/delete-listitem-001-expected.txt:
  • editing/deleting/delete-listitem-002-expected.txt:
  • editing/deleting/delete-mixed-editable-content-001-expected.txt:
  • editing/deleting/delete-select-all-001-expected.txt:
  • editing/deleting/delete-select-all-002-expected.txt:
  • editing/deleting/delete-select-all-003-expected.txt:
  • editing/deleting/merge-different-styles-expected.txt:
  • editing/deleting/merge-endOfParagraph-expected.txt:
  • editing/deleting/merge-no-br-expected.txt:
  • editing/deleting/pruning-after-merge-1-expected.txt:
  • editing/deleting/pruning-after-merge-2-expected.txt:
  • editing/deleting/table-cells-expected.txt:
  • editing/execCommand/boldSelection-expected.txt:
  • editing/execCommand/create-list-from-range-selection-expected.txt:
  • editing/execCommand/createLink-expected.txt:
  • editing/execCommand/format-block-expected.txt:
  • editing/execCommand/format-block-from-range-selection-expected.txt:
  • editing/execCommand/format-block-with-braces-expected.txt:
  • editing/execCommand/format-block-with-trailing-br-expected.txt:
  • editing/execCommand/hilitecolor-expected.txt:
  • editing/execCommand/indent-list-item-expected.txt:
  • editing/execCommand/indent-selection-expected.txt:
  • editing/execCommand/insert-list-and-stitch-expected.txt:
  • editing/execCommand/insert-list-with-id-expected.txt:
  • editing/execCommand/insertHTML-expected.txt:
  • editing/execCommand/insertHorizontalRule-expected.txt:
  • editing/execCommand/remove-formatting-expected.txt:
  • editing/execCommand/remove-list-from-range-selection-expected.txt:
  • editing/execCommand/remove-list-items-expected.txt:
  • editing/execCommand/strikethroughSelection-expected.txt:
  • editing/execCommand/switch-list-type-expected.txt:
  • editing/execCommand/unlink-expected.txt:
  • editing/input/text-input-controller-expected.txt:
  • editing/inserting/insert-3654864-fix-expected.txt:
  • editing/inserting/insert-3659587-fix-expected.txt:
  • editing/inserting/insert-3775316-fix-expected.txt:
  • editing/inserting/insert-3851164-fix-expected.txt:
  • editing/inserting/insert-3907422-fix-expected.txt:
  • editing/inserting/insert-after-delete-001-expected.txt:
  • editing/inserting/insert-div-001-expected.txt:
  • editing/inserting/insert-div-002-expected.txt:
  • editing/inserting/insert-div-003-expected.txt:
  • editing/inserting/insert-div-005-expected.txt:
  • editing/inserting/insert-div-007-expected.txt:
  • editing/inserting/insert-div-009-expected.txt:
  • editing/inserting/insert-div-010-expected.txt:
  • editing/inserting/insert-div-020-expected.txt:
  • editing/inserting/insert-div-023-expected.txt:
  • editing/inserting/insert-div-024-expected.txt:
  • editing/inserting/insert-div-025-expected.txt:
  • editing/inserting/insert-div-026-expected.txt:
  • editing/inserting/multiple-lines-selected-expected.txt:
  • editing/inserting/paragraph-separator-03-expected.txt:
  • editing/inserting/paragraph-separator-in-table-2-expected.txt:
  • editing/inserting/redo-expected.txt:
  • editing/inserting/return-key-with-selection-001-expected.txt:
  • editing/inserting/return-key-with-selection-002-expected.txt:
  • editing/inserting/return-key-with-selection-003-expected.txt:
  • editing/pasteboard/3976872-expected.txt:
  • editing/pasteboard/4076267-3-expected.txt:
  • editing/pasteboard/4076267-expected.txt:
  • editing/pasteboard/4700297-expected.txt:
  • editing/pasteboard/8145-3-expected.txt:
  • editing/pasteboard/block-wrappers-necessary-expected.txt:
  • editing/pasteboard/copy-paste-bidi-expected.txt:
  • editing/pasteboard/displaced-generic-placeholder-expected.txt:
  • editing/pasteboard/displaced-placeholder-expected.txt:
  • editing/pasteboard/display-block-on-spans-expected.txt:
  • editing/pasteboard/drag-drop-modifies-page-expected.txt:
  • editing/pasteboard/interchange-newline-2-expected.txt:
  • editing/pasteboard/merge-after-delete-1-expected.txt:
  • editing/pasteboard/merge-after-delete-2-expected.txt:
  • editing/pasteboard/merge-after-delete-expected.txt:
  • editing/pasteboard/merge-end-3-expected.txt:
  • editing/pasteboard/merge-end-5-expected.txt:
  • editing/pasteboard/merge-end-blockquote-expected.txt:
  • editing/pasteboard/merge-end-list-expected.txt:
  • editing/pasteboard/merge-end-table-expected.txt:
  • editing/pasteboard/paste-4035648-fix-expected.txt:
  • editing/pasteboard/paste-4039777-fix-expected.txt:
  • editing/pasteboard/paste-line-endings-001-expected.txt:
  • editing/pasteboard/paste-line-endings-006-expected.txt:
  • editing/pasteboard/paste-match-style-001-expected.txt:
  • editing/pasteboard/paste-match-style-002-expected.txt:
  • editing/pasteboard/paste-pre-001-expected.txt:
  • editing/pasteboard/paste-pre-002-expected.txt:
  • editing/pasteboard/paste-table-002-expected.txt:
  • editing/pasteboard/paste-table-003-expected.txt:
  • editing/pasteboard/paste-text-006-expected.txt:
  • editing/pasteboard/paste-text-007-expected.txt:
  • editing/pasteboard/paste-text-010-expected.txt:
  • editing/pasteboard/paste-text-011-expected.txt:
  • editing/pasteboard/paste-text-015-expected.txt:
  • editing/pasteboard/paste-text-016-expected.txt:
  • editing/pasteboard/paste-text-017-expected.txt:
  • editing/pasteboard/paste-text-018-expected.txt:
  • editing/pasteboard/prevent-block-nesting-01-expected.txt:
  • editing/pasteboard/smart-paste-003-expected.txt:
  • editing/pasteboard/smart-paste-004-expected.txt:
  • editing/pasteboard/smart-paste-005-expected.txt:
  • editing/pasteboard/smart-paste-006-expected.txt:
  • editing/pasteboard/smart-paste-007-expected.txt:
  • editing/pasteboard/smart-paste-008-expected.txt:
  • editing/pasteboard/testcase-9507-expected.txt:
  • editing/pasteboard/undoable-fragment-removes-expected.txt:
  • editing/selection/expanding-selections-expected.txt:
  • editing/selection/expanding-selections2-expected.txt:
  • editing/selection/node-removal-1-expected.txt:
  • editing/selection/node-removal-2-expected.txt:
  • editing/selection/select-all-iframe-expected.txt:
  • editing/selection/skip-non-editable-1-expected.txt:
  • editing/selection/triple-click-in-pre-expected.txt:
  • editing/style/apple-style-editable-mix-expected.txt:
  • editing/style/block-style-004-expected.txt:
  • editing/style/block-style-005-expected.txt:
  • editing/style/block-style-006-expected.txt:
  • editing/style/create-block-for-style-001-expected.txt:
  • editing/style/create-block-for-style-002-expected.txt:
  • editing/style/create-block-for-style-003-expected.txt:
  • editing/style/create-block-for-style-004-expected.txt:
  • editing/style/create-block-for-style-007-expected.txt:
  • editing/style/create-block-for-style-009-expected.txt:
  • editing/style/create-block-for-style-010-expected.txt:
  • editing/style/create-block-for-style-011-expected.txt:
  • editing/style/create-block-for-style-012-expected.txt:
  • editing/style/create-block-for-style-013-expected.txt:
  • editing/style/designmode-expected.txt:
  • editing/style/relative-font-size-change-001-expected.txt:
  • editing/style/relative-font-size-change-002-expected.txt:
  • editing/style/remove-underline-across-paragraph-expected.txt:
  • editing/style/remove-underline-across-paragraph-in-bold-expected.txt:
  • editing/style/remove-underline-after-paragraph-expected.txt:
  • editing/style/remove-underline-after-paragraph-in-bold-expected.txt:
  • editing/style/remove-underline-from-stylesheet-expected.txt:
  • editing/style/smoosh-styles-001-expected.txt:
  • editing/style/smoosh-styles-002-expected.txt:
  • editing/style/smoosh-styles-003-expected.txt:
  • editing/style/style-3681552-fix-001-expected.txt:
  • editing/style/style-3681552-fix-002-expected.txt:
  • editing/style/style-3690704-fix-expected.txt:
  • editing/style/style-3998892-fix-expected.txt:
  • editing/style/style-boundary-001-expected.txt:
  • editing/style/style-boundary-003-expected.txt:
  • editing/style/style-boundary-004-expected.txt:
  • editing/style/typing-style-001-expected.txt:
  • editing/style/typing-style-003-expected.txt:
  • editing/undo/undo-misspellings-expected.txt:
  • editing/unsupported-content/list-delete-001-expected.txt:
  • editing/unsupported-content/list-delete-002-expected.txt:
  • editing/unsupported-content/list-delete-003-expected.txt:
  • editing/unsupported-content/table-delete-001-expected.txt:
  • editing/unsupported-content/table-delete-002-expected.txt:
  • editing/unsupported-content/table-delete-003-expected.txt:
  • fast/forms/attributed-strings-expected.txt:
  • fast/forms/cursor-position-expected.txt:
  • fast/forms/drag-into-textarea-expected.txt:
  • fast/forms/input-appearance-maxlength-expected.txt:
  • fast/forms/input-paste-undo-expected.txt:
  • fast/forms/input-selection-restore-expected.txt:
  • fast/forms/input-text-paste-maxlength-expected.txt:
  • fast/forms/selection-functions-expected.txt:
  • fast/forms/text-field-setvalue-crash-expected.txt:
  • fast/forms/textarea-paste-newline-expected.txt:

WebCore:

Reviewed by thatcher


Setup for <rdar://problem/4344550> Misspellings aren't marked after undo delete


Fixes:
<http://bugzilla.opendarwin.org/show_bug.cgi?id=6498>
SelectionController's setters should notify delegates of selection changes


Moved setSelection from Frame to SelectionController.
I haven't moved any of the work that's done as a result of a
Selection change to SelectionController yet.
Make fewer uses of SelectionControllers. They're now only used for
the Frame's selection, the Page's drag caret, and functions that
need to do Selection expansion.

  • bindings/js/kjs_window.cpp: (KJS::Selection::getValueProperty): (KJS::Selection::toString): (KJS::SelectionFunc::callAsFunction):
  • bridge/mac/FrameMac.h:
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::advanceToNextMisspelling): (WebCore::FrameMac::dispatchCPPEvent): (WebCore::FrameMac::fontForSelection): (WebCore::FrameMac::baseWritingDirectionForSelectionStart): (WebCore::FrameMac::markMisspellingsInAdjacentWords): (WebCore::FrameMac::markMisspellings): (WebCore::FrameMac::respondToChangedSelection): (WebCore::FrameMac::shouldChangeSelection): (WebCore::FrameMac::shouldDeleteSelection): (WebCore::FrameMac::respondToChangedContents):
  • bridge/mac/PageMac.mm: (WebCore::Page::Page):
  • bridge/mac/WebCoreAXObject.mm: (-[WebCoreAXObject accessibilityAttributeValue:]): (-[WebCoreAXObject doAXTextMarkerRangeForLine:]): (-[WebCoreAXObject doAXBoundsForTextMarkerRange:]): (-[WebCoreAXObject doAXTextMarkerRangeForUnorderedTextMarkers:]): (-[WebCoreAXObject doSetAXSelectedTextMarkerRange:]):
  • bridge/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge isSelectionEditable]): (-[WebCoreFrameBridge isSelectionRichlyEditable]): (-[WebCoreFrameBridge selectionState]): (-[WebCoreFrameBridge deselectText]): (-[WebCoreFrameBridge selectedAttributedString]): (-[WebCoreFrameBridge rangeByExpandingSelectionWithGranularity:]): (-[WebCoreFrameBridge rangeByAlteringCurrentSelection:direction:granularity:]): (-[WebCoreFrameBridge alterCurrentSelection:direction:granularity:]): (-[WebCoreFrameBridge rangeByAlteringCurrentSelection:verticalDistance:]): (-[WebCoreFrameBridge alterCurrentSelection:verticalDistance:]): (-[WebCoreFrameBridge setSelectedDOMRange:affinity:closeTyping:]): (-[WebCoreFrameBridge selectedDOMRange]): (-[WebCoreFrameBridge selectNSRange:]): (-[WebCoreFrameBridge selectedNSRange]): (-[WebCoreFrameBridge selectionAffinity]): (-[WebCoreFrameBridge replaceSelectionWithText:selectReplacement:smartReplace:]): (-[WebCoreFrameBridge setSelectionToDragCaret]): (-[WebCoreFrameBridge moveSelectionToDragCaret:smartMove:]): (-[WebCoreFrameBridge moveDragCaretToPoint:]): (-[WebCoreFrameBridge removeDragCaret]): (-[WebCoreFrameBridge dragCaretDOMRange]): (-[WebCoreFrameBridge isDragCaretRichlyEditable]): (-[WebCoreFrameBridge editableDOMRangeForPoint:]): (-[WebCoreFrameBridge ensureSelectionVisible]): (-[WebCoreFrameBridge rangeOfCharactersAroundCaret]):
  • bridge/win/PageWin.cpp: (WebCore::Page::Page):
  • dom/Document.cpp: (WebCore::Document::updateSelection): (WebCore::Document::clearSelectionIfNeeded): (WebCore::Document::notifyBeforeNodeRemoval):
  • dom/Element.cpp: (WebCore::Element::updateFocusAppearance):
  • editing/EditCommand.cpp: (WebCore::EditCommand::EditCommand):
  • editing/JSEditor.cpp:
  • editing/ModifySelectionListLevel.cpp: (WebCore::IncreaseSelectionListLevelCommand::canIncreaseSelectionListLevel): (WebCore::DecreaseSelectionListLevelCommand::canDecreaseSelectionListLevel):
  • editing/Selection.cpp: (WebCore::Selection::setBase): (WebCore::Selection::setExtent):
  • editing/Selection.h: (WebCore::Selection::extent):
  • editing/SelectionController.cpp: (WebCore::SelectionController::SelectionController): (WebCore::SelectionController::moveTo): (WebCore::SelectionController::setSelection): (WebCore::SelectionController::nodeWillBeRemoved): (WebCore::SelectionController::modify): (WebCore::SelectionController::clear): (WebCore::SelectionController::setBase): (WebCore::SelectionController::setExtent):
  • editing/SelectionController.h:
  • editing/TypingCommand.cpp: (WebCore::TypingCommand::insertText): (WebCore::TypingCommand::deleteKeyPressed): (WebCore::TypingCommand::forwardDeleteKeyPressed):
  • editing/VisiblePosition.cpp: (WebCore::VisiblePosition::caretRect):
  • editing/VisiblePosition.h:
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::defaultEventHandler):
  • page/Frame.cpp: (WebCore::Frame::selectedText): (WebCore::Frame::hasSelection): (WebCore::Frame::selectionController): (WebCore::Frame::dragCaretController): (WebCore::Frame::notifyRendererOfSelectionChange): (WebCore::Frame::invalidateSelection): (WebCore::Frame::clearCaretRectIfNeeded): (WebCore::Frame::setFocusNodeIfNeeded): (WebCore::Frame::selectionLayoutChanged): (WebCore::Frame::caretBlinkTimerFired): (WebCore::Frame::paintCaret): (WebCore::Frame::paintDragCaret): (WebCore::Frame::isPointInsideSelection): (WebCore::Frame::selectClosestWordFromMouseEvent): (WebCore::Frame::handleMousePressEventDoubleClick): (WebCore::Frame::handleMousePressEventTripleClick): (WebCore::Frame::handleMousePressEventSingleClick): (WebCore::Frame::handleMouseMoveEvent): (WebCore::Frame::handleMouseReleaseEvent): (WebCore::Frame::selectAll): (WebCore::Frame::selectContentsOfNode): (WebCore::Frame::shouldChangeSelection): (WebCore::Frame::shouldDeleteSelection): (WebCore::Frame::isSelectionInPasswordField): (WebCore::Frame::appliedEditing): (WebCore::Frame::unappliedEditing): (WebCore::Frame::reappliedEditing): (WebCore::Frame::computeAndSetTypingStyle): (WebCore::Frame::applyStyle): (WebCore::Frame::applyParagraphStyle): (WebCore::Frame::selectionListState): (WebCore::Frame::selectionHasStyle): (WebCore::Frame::selectionComputedStyle): (WebCore::Frame::selectFrameElementInParentIfFullySelected): (WebCore::Frame::currentForm): (WebCore::Frame::revealSelection): (WebCore::Frame::centerSelectionInVisibleArea): (WebCore::Frame::styleForSelectionStart): (WebCore::Frame::setSelectionFromNone): (WebCore::Frame::findString):
  • page/Frame.h:
  • page/FramePrivate.h: (WebCore::FramePrivate::FramePrivate):
  • page/FrameView.cpp: (WebCore::FrameView::clear):
  • page/Page.cpp: (WebCore::Page::dragCaretController):
  • page/Page.h:
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::paintCaret):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::autoscroll):
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::selectionStart): (WebCore::RenderTextControl::selectionEnd): (WebCore::RenderTextControl::setSelectionRange): (WebCore::RenderTextControl::selectionChanged):
  • rendering/RenderTreeAsText.cpp: (WebCore::writeSelection):
2:48 PM Changeset in webkit [16236] by spadma
  • 8 edits in S60/trunk/WebKit

2006-09-05 winship <Stephen.Winship@nokia.com>

Reviewed by Sachin.
DESC: Auto Form Fill usability changes
http://bugzilla.opendarwin.org/show_bug.cgi?id=10642

  • BrowserCore/Formcontrols/inc/FControlInputSkin.h:
  • BrowserCore/Formcontrols/inc/FControlSelectListBox.h: (CControlSelectListBox::IsFocused):
  • BrowserCore/Formcontrols/inc/FControlSelectWidget.h:
  • BrowserCore/Formcontrols/src/FControlInputSkin.cpp: (CFormInputSkin::SetSize): (CFormInputSkin::TextChanged): (CFormInputSkin::OkToExit): (CFormInputSkin::ListBoxKeyEventL): (CFormInputSkin::ActivateL): (CFormInputSkin::DestroyList): (CFormInputSkin::UpdatePopupL):
  • BrowserCore/Formcontrols/src/FControlSelectListBox.cpp: (CControlSelectListBox::ConstructL): (CControlSelectListBox::UpdateContentL): (CControlSelectListBox::SetFocus):
  • BrowserCore/Formcontrols/src/FControlSelectWidget.cpp: (CControlSelectWidget::CControlSelectWidget): (CControlSelectWidget::ConstructL): (CControlSelectWidget::ConstructListBoxL): (CControlSelectWidget::NewL): (CControlSelectWidget::~CControlSelectWidget): (CControlSelectWidget::PositionListBox): (CControlSelectWidget::SetList): (CControlSelectWidget::FilterContentL): (CControlSelectWidget::OfferKeyEventL): (CControlSelectWidget::CountComponentControls): (CControlSelectWidget::SizeChanged): (CControlSelectWidget::HandleListBoxEventL): (CControlSelectWidget::SimulateDoneKeyL):
  • ChangeLog:
  • group/BrowserCore.mmp:
2:14 PM Changeset in webkit [22141] by sfalken
  • 10 edits
    2 adds in branches/WindowsMerge/WebKitWin

2006-09-05 Steve Falkenburg <sfalken@apple.com>

Reviewed by Lou Amadio.


Currently we are saving a copy of the raw data in the FrameView (to
return when requested via IWebDataSource::data). This needs to be
shared with CFNetwork once CFNetwork integration is complete.

Also fixed a problem with autofill where we'd crash on pages with
non-autofillable form elements.

2:10 PM Changeset in webkit [16235] by thatcher
  • 2 edits in trunk/WebKit

Reviewed by Tim O.

<rdar://problem/4682225> conflicting typedefs in Netscape plug-in headers

  • Plugins/npfunctions.h: fix the return type for NPN_IntFromIdentifierProcPtr to be int32_t
1:18 PM Changeset in webkit [16234] by brmorris
  • 2 edits in S60/trunk

2006-09-05 brmorris <bradley.morrison@nokia.com>

Reviewed by me.
DESC: Include test index number in test results
http://bugzilla.opendarwin.org/show_bug.cgi?id=10678

  • runATF.bat:
1:13 PM Changeset in webkit [16233] by thatcher
  • 3 edits in trunk/JavaScriptCore

Reviewed by Tim O.

<rdar://problem/4715840> SEL is not char*

  • bindings/objc/objc_class.mm: (KJS::Bindings::ObjcClass::methodsNamed): use sel_getName instead of a char* cast.
  • bindings/objc/objc_runtime.mm: (ObjcFallbackObjectImp::callAsFunction): ditto
12:04 PM Changeset in webkit [16232] by adele
  • 2 edits in trunk/WebCore

Reviewed by Adam.

Let the engine draw the focus ring for styled menu lists.

  • rendering/RenderTheme.cpp: (WebCore::RenderTheme::supportsFocusRing): Added case for MenulistButtonAppearance.
10:54 AM Changeset in webkit [16231] by adele
  • 3 edits in trunk/WebCore

Reviewed by Adam.

  • Fix for <rdar://problem/4706463> REGRESSION: Must click twice on <select> elements for menu to show
  • rendering/RenderPopupMenuMac.h:
  • rendering/RenderPopupMenuMac.mm: (WebCore::RenderPopupMenuMac::showPopup): Moved call to dismiss popup into hidePopup method. Now calls RenderMenuList::hidePopup which will update the m_popupIsVisible flag on the menu list before hiding the popup. (WebCore::RenderPopupMenuMac::hidePopup): Added.
10:26 AM Changeset in webkit [16230] by thatcher
  • 5 edits in trunk/WebCore/bindings

Build fix for when XPATH_SUPPORT is not defined.

9:57 AM Changeset in webkit [16229] by weinig
  • 24 edits
    5 deletes in trunk/WebCore

Reviewed by Tim H.

Patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10738
Consolidate the Objective-C DOM Bindings Internal files into DOMInternal.h

  • Consolidate DOMHTMLInternal.h, DOMCSSInternal.h, DOMEventsInternal.h, DOMViewsInternal.h and DOMXPathInternal.h into DOMInternal.h.
  • Rename the "type-makers" (used to make a binding from an implementation type) so that all the HTML ones begin with the prefix HTML. Did the same for the CSS.
  • Auto-generate a couple more methods from DOMExtensions.h.
  • Various cleanups of the Objective-C auto-generation code.
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/objc/DOM.mm: (-[DOMDocument createCSSStyleDeclaration]):
  • bindings/objc/DOMCSS.mm: (+[DOMCSSRuleList _CSSRuleListWith:]): (+[DOMCSSRule _CSSRuleWith:]): (+[DOMCSSStyleDeclaration _CSSStyleDeclarationWith:]): (+[DOMCSSValue _CSSValueWith:]): (+[DOMCSSPrimitiveValue _CSSPrimitiveValueWith:]): (-[DOMDocument getComputedStyle::]): (-[DOMDocument getMatchedCSSRules::]):
  • bindings/objc/DOMCSSInternal.h: Removed.
  • bindings/objc/DOMEvents.mm:
  • bindings/objc/DOMEventsInternal.h: Removed.
  • bindings/objc/DOMEventsNonstandard.mm:
  • bindings/objc/DOMExtensions.h:
  • bindings/objc/DOMHTML.mm: (+[DOMHTMLCollection _HTMLCollectionWith:]): (+[DOMHTMLOptionsCollection _HTMLOptionsCollectionWith:]): (+[DOMHTMLElement _HTMLElementWith:]): (+[DOMHTMLFormElement _HTMLFormElementWith:]): (+[DOMHTMLTableCaptionElement _HTMLTableCaptionElementWith:]): (+[DOMHTMLTableSectionElement _HTMLTableSectionElementWith:]): (+[DOMHTMLTableElement _HTMLTableElementWith:]): (+[DOMHTMLTableCellElement _HTMLTableCellElementWith:]):
  • bindings/objc/DOMHTMLAppletElement.mm:
  • bindings/objc/DOMHTMLDocument.mm: (-[DOMHTMLDocument body]): (-[DOMHTMLDocument images]): (-[DOMHTMLDocument applets]): (-[DOMHTMLDocument links]): (-[DOMHTMLDocument forms]): (-[DOMHTMLDocument anchors]):
  • bindings/objc/DOMHTMLInternal.h: Removed.
  • bindings/objc/DOMHTMLOptionElement.mm: (-[DOMHTMLOptionElement form]):
  • bindings/objc/DOMInternal.h:
  • bindings/objc/DOMRGBColor.mm: (setWrapperForRGB): (-[DOMRGBColor red]): (-[DOMRGBColor green]): (-[DOMRGBColor blue]): (-[DOMRGBColor alpha]):
  • bindings/objc/DOMUtility.mm: (KJS::ScriptInterpreter::createObjcInstanceForValue):
  • bindings/objc/DOMViews.mm: (-[DOMAbstractView WebCore::]): (-[DOMAbstractView _initWithAbstractView:WebCore::]): (+[DOMAbstractView _abstractViewWith:WebCore::]):
  • bindings/objc/DOMViewsInternal.h: Removed.
  • bindings/objc/DOMXPath.mm:
  • bindings/objc/DOMXPathInternal.h: Removed.
  • bindings/objc/PublicDOMInterfaces.h:
  • bindings/scripts/CodeGeneratorObjC.pm:
  • bridge/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge typingStyle]):
  • html/HTMLAnchorElement.idl:
  • html/HTMLAreaElement.idl:
  • html/HTMLImageElement.idl:
  • html/HTMLInputElement.idl:
  • html/HTMLLinkElement.idl:
9:24 AM Changeset in webkit [16228] by thatcher
  • 1 edit in trunk/WebCore/DerivedSources.make

Build fix. Remove the now duplicate OS check and add PublicDOMInterfaces.h to the dependancy list for ObjC DOM files.

12:29 AM Changeset in webkit [22140] by sfalken
  • 1 edit in branches/WindowsMerge/WebCore/WebCore.vcproj/WebCore.vcproj

Fix build

12:27 AM Changeset in webkit [16227] by sfalken
  • 2 edits in trunk/WebCore

2006-09-05 Steve Falkenburg <sfalken@apple.com>

Conditionalize new DOM headers for ObjC only

  • DerivedSources.make:

Sep 4, 2006:

10:05 PM Changeset in webkit [16226] by ap
  • 9 edits in trunk

Reviewed by Darin.

http://bugzilla.opendarwin.org/show_bug.cgi?id=10716
XMLHttpRequest.responseText is null if HTTP response is empty

Extended existing tests:

  • http/tests/xmlhttrequest/zero-length-response.html
  • http/tests/xmlhttrequest/zero-length-response-sync.html
  • xml/xmlhttprequest.cpp: (WebCore::XMLHttpRequest::XMLHttpRequest): Initialize responseText to an empty string. (WebCore::XMLHttpRequest::open): Reset responseText to an empty string.
  • bindings/js/JSXMLHttpRequest.cpp: (KJS::JSXMLHttpRequest::getValueProperty): Return null rather than undefined for nil responseXML.
7:45 PM Changeset in webkit [16225] by thatcher
  • 27 edits
    1 add in trunk

WebCore:

Reviewed by Darin.

Bug 10714: ObjC autogeneration needs safe-guards against easily modifying the public API
http://bugzilla.opendarwin.org/show_bug.cgi?id=10714


  • New PublicDOMInterfaces.h file is consulted when generating the ObjC DOM files. All public DOM class interfaces, properties and methods need to be in this file. Anything not in the file will be generated into the appropriate private header file. During generation if something changed or is missing in the public API a build error will occur. New interfaces added to the IDLs files will now not automatically be reflected in the public ObjC API. Methods commented out in PublicDOMInterfaces.h are pending public and will be uncommented once approved.
  • Removed most of the Exclude=ObjC uses from the IDL files. This attribute was mostly used to prevent changes to the public headers.
  • Make a new parameter name if the original conflicts with a property name.
  • Simplified the generation code for dealing with exceptions.
  • Moved file opens to WriteData. Files are unlinked before being opened to work around a Leopard file truncation bug.
  • Fixed #imports for DOMImplementation to be DOMDOMImplementation.h
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/objc/DOMCSS.h:
  • bindings/objc/DOMCore.h:
  • bindings/objc/DOMExtensions.h:
  • bindings/objc/DOMHTMLInternal.h:
  • bindings/objc/DOMPrivate.h:
  • bindings/objc/PublicDOMInterfaces.h: Added.
  • bindings/scripts/CodeGeneratorObjC.pm:
  • dom/Document.idl:
  • dom/Element.idl:
  • dom/NamedNodeMap.idl:
  • dom/ProcessingInstruction.idl:
  • html/HTMLAnchorElement.idl:
  • html/HTMLAreaElement.idl:
  • html/HTMLBodyElement.idl:
  • html/HTMLButtonElement.idl:
  • html/HTMLImageElement.idl:
  • html/HTMLInputElement.idl:
  • html/HTMLLabelElement.idl:
  • html/HTMLLegendElement.idl:
  • html/HTMLLinkElement.idl:
  • html/HTMLOptionsCollection.idl:
  • html/HTMLPreElement.idl:
  • html/HTMLStyleElement.idl:
  • html/HTMLTextAreaElement.idl:

WebKit:

Reviewed by Darin.

Bug 10714: ObjC autogeneration needs safe-guards against easily modifying the public API
http://bugzilla.opendarwin.org/show_bug.cgi?id=10714

  • Added the new private DOM headers.
  • Factored out the common commands into variables.
  • Made WebDashboardRegion.h private again.
  • Rename DOMDOMImplementation.h to DOMImplementation.h when files are migrated. Also fixes up #imports.
  • MigrateHeaders.make:
4:36 PM Changeset in webkit [16224] by darin
  • 2 edits in trunk/WebKitTools
  • Scripts/do-webcore-rename: More renaming plans.
4:16 PM Changeset in webkit [16223] by thatcher
  • 1 edit in trunk/WebKit/WebInspector/WebInspector.m

Another build fix now that the old style ObjC methods are deprecated.

4:10 PM Changeset in webkit [16222] by thatcher
  • 7 edits in trunk

Build fixes now that the old style ObjC methods are deprecated.

3:57 PM Changeset in webkit [16221] by weinig
  • 2 edits in trunk/WebCore

Reviewed by Tim H.

Continue fixing the build, I forgot this in my previous patch!

  • platform/qt/PathQt.cpp: (WebCore::Path::apply):
3:43 PM Changeset in webkit [16220] by weinig
  • 3 edits in trunk/WebCore

Reviewed by Tim H.

Fix build on Qt/Linux.

  • CMakeLists.txt:
  • platform/qt/PathQt.cpp: (WebCore::Path::transform):
3:28 PM Changeset in webkit [16219] by weinig
  • 3 edits
    3 adds
    1 delete in trunk

WebKitQt:

Reviewed by Tim H.

Fixes last part of: http://bugzilla.opendarwin.org/show_bug.cgi?id=10644
Move QtLauncher down to WebKitQt.

  • QtLauncher/CMakeLists.txt: Added.
  • QtLauncher/main.cpp: Added. (main):

WebKitTools:

Reviewed by Tim H.

Fixes last part of: http://bugzilla.opendarwin.org/show_bug.cgi?id=10644
Move QtLauncher down to WebKitQt.

  • QtLauncher/CMakeLists.txt: Removed.
  • QtLauncher/main.cpp: Removed.
12:49 PM Changeset in webkit [16218] by ap
  • 1 edit in trunk/WebCore/WebCore.xcodeproj/project.pbxproj

The project file didn't get committed with my last checking.

11:34 AM Changeset in webkit [16217] by ap
  • 12 edits
    3 adds in trunk

Reviewed by Darin.

http://bugzilla.opendarwin.org/show_bug.cgi?id=10676
@charset rules not accessible via DOM

Test: fast/encoding/css-charset-dom.html

  • WebCore.xcodeproj/project.pbxproj: Added CSSCharsetRule.cpp
  • bindings/js/kjs_css.cpp: (KJS::DOMCSSStyleSheet::getValueProperty): Separated Rules and CssRules, since now they behave differently.
  • css/CSSCharsetRule.h: Make the constructor take an encoding.
  • css/CSSCharsetRule.cpp: Added.
  • css/CSSGrammar.y: Create CSSStylesheetRules as necessary.
  • css/CSSRuleList.cpp: (WebCore::CSSRuleList::CSSRuleList):
  • css/CSSRuleList.h:
  • css/CSSStyleSheet.cpp: (WebCore::CSSStyleSheet::cssRules):
  • css/CSSStyleSheet.h: Skip charset rules in IE compatibility mode.
  • css/StyleBase.h: (WebCore::StyleBase::isCharsetRule): Fixed a typo.
  • css/cssparser.cpp: (WebCore::CSSParser::createCharsetRule):
  • css/cssparser.h: Added createCharsetRule().
10:37 AM Changeset in webkit [16216] by weinig
  • 4 edits
    2 adds in trunk/WebCore

Reviewed by Darin.

Fixes parts of: http://bugzilla.opendarwin.org/show_bug.cgi?id=10644
Let FrameQt operate on a FrameQtClient, just like Win/Mac handle it.

  • CMakeLists.txt:
  • platform/qt/FrameQt.cpp: (WebCore::FrameQt::FrameQt): (WebCore::FrameQt::~FrameQt): (WebCore::FrameQt::openURL): (WebCore::FrameQt::submitForm): (WebCore::FrameQt::urlSelected):
  • platform/qt/FrameQt.h:
  • platform/qt/FrameQtClient.cpp: Added. (WebCore::FrameQtClientDefault::FrameQtClientDefault): (WebCore::FrameQtClientDefault::~FrameQtClientDefault): (WebCore::FrameQtClientDefault::setFrame): (WebCore::FrameQtClientDefault::openURL): (WebCore::FrameQtClientDefault::submitForm): (WebCore::FrameQtClientDefault::receivedResponse): (WebCore::FrameQtClientDefault::receivedData): (WebCore::FrameQtClientDefault::receivedAllData):
  • platform/qt/FrameQtClient.h: Added. (WebCore::FrameQtClient::~FrameQtClient):
10:34 AM Changeset in webkit [16215] by weinig
  • 4 edits
    2 adds in trunk/WebKitTools

Reviewed by Darin.

Fixes parts of: http://bugzilla.opendarwin.org/show_bug.cgi?id=10644
Adjust DumpRenderTree to the FrameQtClient changes.

  • DumpRenderTree/DumpRenderTree.qtproj/CMakeLists.txt:
  • DumpRenderTree/DumpRenderTree.qtproj/DumpRenderTree.cpp: (WebCore::DumpRenderTree::DumpRenderTree): (WebCore::DumpRenderTree::~DumpRenderTree): (WebCore::DumpRenderTree::frame):
  • DumpRenderTree/DumpRenderTree.qtproj/DumpRenderTree.h:
  • DumpRenderTree/DumpRenderTree.qtproj/DumpRenderTreeClient.cpp: Added. (WebCore::DumpRenderTreeClient::DumpRenderTreeClient): (WebCore::DumpRenderTreeClient::~DumpRenderTreeClient):
  • DumpRenderTree/DumpRenderTree.qtproj/DumpRenderTreeClient.h: Added.
10:11 AM Changeset in webkit [16214] by weinig
  • 1 edit
    16 adds in trunk
2:31 AM Changeset in webkit [16213] by eseidel
  • 2 edits in trunk/WebCore

2006-09-04 Eric Seidel <eric@eseidel.com>

Reviewed by andersca.

Clean-up style and spacing.
http://bugzilla.opendarwin.org/show_bug.cgi?id=10724
No tests needed, no functional changes.

  • ksvg2/svg/svgpathparser.cpp: (WebCore::parseCoord): (WebCore::SVGPolyParser::parsePoints): (WebCore::SVGPathParser::parseSVG): (WebCore::SVGPathParser::calculateArc): (WebCore::SVGPathParser::svgLineToHorizontal): (WebCore::SVGPathParser::svgLineToVertical): (WebCore::SVGPathParser::svgCurveToCubicSmooth): (WebCore::SVGPathParser::svgCurveToQuadratic): (WebCore::SVGPathParser::svgCurveToQuadraticSmooth): (WebCore::SVGPathParser::svgArcTo):
2:22 AM Changeset in webkit [16212] by rwlbuis
  • 3 edits
    4 adds in trunk

Reviewed by Eric.

http://bugzilla.opendarwin.org/show_bug.cgi?id=6546
clipPath data does not respect transforms

Get the transform from the dom element and adjust
the clip path with it.

12:57 AM Changeset in webkit [16211] by rwlbuis
  • 2 edits in trunk/WebCore

Reviewed by Eric.

This code should have gone in as part of:
http://bugzilla.opendarwin.org/show_bug.cgi?id=10696
It addresses the comments there, also we use double instead
of float now in calculations.

Sep 3, 2006:

7:11 PM Changeset in webkit [16210] by weinig
  • 13 edits
    2 adds in trunk

WebCore:

Reviewed by Darin and Tim H.

Auto-generates DOMMediaList, DOMStyleSheet, and DOMStyleSheetList.

  • DerivedSources.make:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/objc/DOMCSS.mm: (+[DOMStyleSheet _styleSheetWith:]): (+[DOMCSSStyleSheet _CSSStyleSheetWith:]):
  • bindings/objc/DOMCSSInternal.h:
  • bindings/objc/DOMInternal.h:
  • bindings/objc/DOMObject.mm: (-[DOMObject sheet]):
  • bindings/objc/DOMPrivate.h:
  • bindings/objc/DOMStylesheets.h:
  • bindings/objc/DOMUtility.mm: (KJS::ScriptInterpreter::createObjcInstanceForValue):
  • bindings/scripts/CodeGeneratorObjC.pm:
  • css/StyleSheet.idl: Added.
  • css/StyleSheetList.idl: Added.

WebKit:

Reviewed by Darin and Tim H.

  • MigrateHeaders.make:
6:00 PM Changeset in webkit [16209] by weinig
  • 1 add in trunk/WebCore/platform/Path.cpp

Build fix.

3:19 PM Changeset in webkit [16208] by darin
  • 2 edits in trunk/WebKitTools
  • Scripts/do-webcore-rename: More renaming plans.
2:46 PM Changeset in webkit [16207] by thatcher
  • 1 edit in trunk/WebCore/bindings/objc/DOMHTML.mm

Build fix to avoid warning about property name conflict.

2:31 PM Changeset in webkit [16206] by rwlbuis
  • 38 edits in trunk/WebCore

Reviewed by Darin.

http://bugzilla.opendarwin.org/show_bug.cgi?id=10696
RenderPathQuartz and RenderPathQt should not be needed

Cleanup of code related to RenderPath.

2:20 PM Changeset in webkit [16205] by thatcher
  • 1 edit in trunk/WebCore/ChangeLog

Fix a typo Darin called out, but I forgot to change.

2:11 PM Changeset in webkit [16204] by thatcher
  • 1 edit in trunk/WebCore/bindings/scripts/CodeGeneratorObjC.pm

Build fix.

1:48 PM Changeset in webkit [16203] by thatcher
  • 5 edits in trunk

Changing the ignore rule for *.perspective to *.perspective*. This ignores the new .perspectivev3 file.

1:42 PM Changeset in webkit [16202] by thatcher
  • 1 edit in trunk/WebCore/WebCore.xcodeproj

Changing the ignore rule for *.perspective to *.perspective*. This ignores the new .perspectivev3 file.

1:38 PM Changeset in webkit [16201] by thatcher
  • 30 edits in trunk/WebCore

Reviewed by Darin.

Bug 10685: ObjC DOM should have no unnamed parameters
http://bugzilla.opendarwin.org/show_bug.cgi?id=10685

  • The ObjC code generation script now outputs parameter prefixes for methods that have more than 1 parameter. This prefix is simply the parameter name. Some parameter names have been changed in the IDL files to produce better prefixes. If an extended attribute of "ObjCPrefix" exists on a parameter we will use that for the prefix.
  • A backwards compatible version of the method is placed in a category with a deprecation marco for 10.5 and later. This step only happens if the IDL extended attribute "OldStyleObjC" is defined. All new functions in IDL should not get this attribute.
  • Made the $interfaceMethodSelector regex in IDLStructure.pm allow "=" so parameter extended attributes can have values.
  • Moved RemoveExcludedAttributesAndFunctions to CodeGenerator.pm so it can be shared between the two generators.


  • Removed the die when we encounter a getter that uses exceptions. Sam Weinig added support for this in an earlier change.
  • Check if $ENV{"MACOSX_DEPLOYMENT_TARGET"} is defined before we compare. This caused a perl warning when generating on other platforms.
  • The ObjC generation will not happen on platforms other than Mac OS. This is determined by the "OS" env variable Xcode sets. This check is in the DerivedSources.make.
  • Added a way to skip generation if the constructor of the specific generator returns undef. Not used yet.
  • Many cleanup tweaks in CodeGeneratorObjC.pm.
  • Removed IDL and CSS files from the project's resource copy phase, these do no need to be in WebCore's resources.
  • DerivedSources.make
  • WebCore.xcodeproj/project.pbxproj
  • bindings/scripts/CodeGenerator.pm
  • bindings/scripts/CodeGeneratorJS.pm
  • bindings/scripts/CodeGeneratorObjC.pm
  • bindings/scripts/IDLStructure.pm
  • css/CSSMediaRule.idl
  • css/CSSPrimitiveValue.idl
  • css/CSSStyleDeclaration.idl
  • css/CSSStyleSheet.idl
  • dom/CharacterData.idl
  • dom/DOMImplementation.idl
  • dom/Document.idl
  • dom/Element.idl
  • dom/KeyboardEvent.idl
  • dom/MouseEvent.idl
  • dom/MutationEvent.idl
  • dom/NamedNodeMap.idl
  • dom/UIEvent.idl
  • html/HTMLCanvasElement.idl
  • html/HTMLSelectElement.idl
  • ksvg2/bindings/idl/svg/SVGLengthList.idl
  • ksvg2/bindings/idl/svg/SVGNumberList.idl
  • ksvg2/bindings/idl/svg/SVGPointList.idl
  • ksvg2/bindings/idl/svg/SVGStringList.idl
  • ksvg2/bindings/idl/svg/SVGTextContentElement.idl
  • ksvg2/bindings/idl/svg/SVGTransformList.idl
  • ksvg2/svg/SVGSVGElement.idl
  • page/DOMWindow.idl
12:08 PM Changeset in webkit [16200] by ap
  • 11 edits in trunk

Reviewed by Tim H.

http://bugzilla.opendarwin.org/show_bug.cgi?id=10693
Convert JavaScript arrays to AppleScript lists

JavaScriptCore:

  • JavaScriptCore.exp: Export ArrayInstance::info and ArrayInstance::getItem().
  • kjs/array_instance.h:
  • kjs/array_object.cpp: (ArrayInstance::getItem): Added a method to access array items from C++.

WebCore:

  • bridge/mac/WebCoreFrameBridge.mm: (aeDescFromJSValue): Added a case for ArrayInstance.

LayoutTests:

  • fast/AppleScript/array-expected.txt:
  • fast/AppleScript/array.html: Updated results, added a test for circular dependencies.

WebKitTools:

  • DumpRenderTree/AppleScriptController.m: (convertAEDescToObject): (-[AppleScriptController doJavaScript:]): Support printing AEDescLists.
10:28 AM Changeset in webkit [16199] by ap
  • 1 edit in trunk/LayoutTests/http/tests/incremental/slow-utf8-html.pl

http/tests/incremental/slow-utf8-html-diffs.txt
Set executable bit that was lost when re-applying the patch.

9:59 AM Changeset in webkit [16198] by ap
  • 3 edits
    4 adds in trunk

Reviewed by Eric.

http://bugzilla.opendarwin.org/show_bug.cgi?id=10697
REGRESSION (r16175): Errors in incremental decoding of UTF-8

Tests:

  • http/tests/incremental/slow-utf8-html.pl
  • fast/encoding/charset-invalid.html
  • loader/Decoder.cpp: (Decoder::Decoder): Ensure that we have a valid encoding. Get its name via TextEncoding, to match the logic in setEncodingName(). (Decoder::setEncodingName): Only set m_encodingName if the encoding is valid. Rely on TextEncoding constructor to lowercase it if necessary. (Decoder::decode): Use setEncodingName() to apply encoding from BOM. Don't try to ensure the validity of encoding - it is enforced by class constructor and setEncodingName().
6:45 AM Changeset in webkit [16197] by weinig
  • 1 edit in trunk/WebCore/bindings/objc/DOMRGBColor.h

Fixes build for regression tests.

2:25 AM Changeset in webkit [16196] by ap
  • 4 edits
    2 moves
    1 add
    2 deletes in trunk/LayoutTests

Reviewed by Tim H.

http://bugzilla.opendarwin.org/show_bug.cgi?id=4508
Layout tests access an external server

  • fast/backgrounds/repeat/noRepeatCorrectClip.html: Use a local resource (it was already present, but the test still pointed to a remote one).
  • fast/dom/HTMLImageElement/image-src-absolute-url-expected.txt:
  • fast/dom/HTMLImageElement/image-src-absolute-url.html: Use a non-existent file URL instead of a non-existent http one.
  • fast/loader/docLoaderFrame-expected.checksum: Removed.
  • fast/loader/docLoaderFrame-expected.png: Removed.
  • fast/loader/docLoaderFrame-expected.txt: Removed.
  • fast/loader/docLoaderFrame.html: Removed.
  • http/tests/xmlhttprequest/docLoaderFrame-expected.txt: Added.
  • http/tests/xmlhttprequest/docLoaderFrame.html: Added.
  • http/tests/xmlhttprequest/resources/docLoaderFrame-data.html: Added. Use a local resource. Also, turned the test into a text one, and used waitUntilDone to make it actually work.

Sep 2, 2006:

7:10 PM Changeset in webkit [16195] by weinig
  • 17 edits
    11 adds in trunk

WebCore:

Reviewed by Tim H.

Auto-generates DOMCSSCharsetRule, DOMCSSFontFaceRule, DOMCSSImportRule,
DOMCSSMediaRule, DOMCSSPageRule, DOMCSSPrimitiveValue, DOMCSSRule,
DOMCSSRuleList, DOMCSSStyleDeclaration, DOMCSSStyleRule, DOMCSSStyleSheet,
DOMCSSUnknownRule, DOMCSSValue, DOMCSSValueList, DOMCounter, and DOMRect.

Splits DOMRGBColor into its own files.

  • DerivedSources.make:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/objc/DOMCSS.h:
  • bindings/objc/DOMCSS.mm: (+[DOMCSSStyleSheet _CSSStyleSheetWith:]): (+[DOMCSSPrimitiveValue _valueWith:]):
  • bindings/objc/DOMCSSInternal.h:
  • bindings/objc/DOMExtensions.h:
  • bindings/objc/DOMInternal.h:
  • bindings/objc/DOMRGBColor.h: Added.
  • bindings/objc/DOMRGBColor.mm: Added. (getWrapperForRGB): (setWrapperForRGB): (removeWrapperForRGB): (-[DOMRGBColor dealloc]): (-[DOMRGBColor finalize]): (-[DOMRGBColor red]): (-[DOMRGBColor green]): (-[DOMRGBColor blue]): (-[DOMRGBColor alpha]): (-[DOMRGBColor color]): (-[DOMRGBColor _initWithRGB:WebCore::]): (+[DOMRGBColor _RGBColorWithRGB:WebCore::]):
  • bindings/scripts/CodeGeneratorObjC.pm:
  • css/CSSCharsetRule.idl: Added.
  • css/CSSFontFaceRule.idl: Added.
  • css/CSSImportRule.idl: Added.
  • css/CSSMediaRule.idl: Added.
  • css/CSSPageRule.idl: Added.
  • css/CSSRule.idl:
  • css/CSSStyleDeclaration.idl:
  • css/CSSStyleRule.idl: Added.
  • css/CSSStyleSheet.idl: Added.
  • css/CSSUnknownRule.idl: Added.
  • css/CSSValue.idl:
  • css/MediaList.idl:
  • css/RGBColor.idl: Added.
  • css/Rect.idl:
  • css/RectImpl.h:

WebKit:

Reviewed by Tim H.

  • MigrateHeaders.make:
12:19 PM Changeset in webkit [16194] by thatcher
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by Justin.

Testcase for:
http://bugzilla.opendarwin.org/show_bug.cgi?id=10579
AppleStyleCommand::applyBlockStyle crash

  • editing/style/table-selection-expected.checksum: Added.
  • editing/style/table-selection-expected.png: Added.
  • editing/style/table-selection-expected.txt: Added.
  • editing/style/table-selection.html: Added.

WebCore:

Reviewed by Justin.


http://bugzilla.opendarwin.org/show_bug.cgi?id=10579
AppleStyleCommand::applyBlockStyle crash

  • editing/ApplyStyleCommand.cpp: (WebCore::ApplyStyleCommand::applyBlockStyle): Prevent a crash by making sure that the 'beyondEnd' node is after the start node.
12:15 PM Changeset in webkit [16193] by thatcher
  • 10 edits in trunk

JavaScriptCore:

Reviewed by Tim H.

Bug 10454: Unix bakefile fixes
http://bugzilla.opendarwin.org/show_bug.cgi?id=10454

  • JavaScriptCoreSources.bkl:

WebCore:

Reviewed by Tim H.

Bug 10454: Unix bakefile fixes
http://bugzilla.opendarwin.org/show_bug.cgi?id=10454

  • WebCoreSources.bkl:
  • platform/gdk/RenderPopupMenuGdk.cpp: (WebCore::RenderPopupMenuGdk::RenderPopupMenuGdk):
  • platform/gdk/RenderPopupMenuGdk.h:
  • platform/gdk/RenderThemeGdk.cpp: (WebCore::RenderThemeGdk::createPopupMenu): (WebCore::RenderThemeGdk::systemFont):
  • platform/gdk/RenderThemeGdk.h:
  • platform/gdk/TemporaryLinkStubs.cpp:
  • webcore-base.bkl:
10:25 AM Changeset in webkit [16192] by adele
  • 6 edits in trunk/WebCore

Reviewed by Anders.

Test: fast/forms/password-doubleclick-selection.html

  • editing/visible_units.cpp: (WebCore::previousBoundary): When searching for boundaries in renderers that use the textSecurity property, convert characters in strings to alpha-numeric characters (in this case, all 'x's) so that every character isn't treated as a punctuation boundary. (WebCore::nextBoundary): ditto.
  • editing/TextIterator.cpp: (WebCore::SimplifiedBackwardsTextIterator::handleTextNode): Updated to use the renderer's string instead of the node value. This matches the base class implementation of handleTextNode.

Code cleanup.

  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::doApply): Avoid unnecessary check for password field case if the smart replace condition is false.
  • page/Frame.cpp: (WebCore::Frame::mayCopy): Removed duplicate code. Calls isSelectionInPasswordField now.
  • rendering/RenderText.cpp: (WebCore::RenderText::setText): Updated formatting.

Sep 1, 2006:

6:43 PM Changeset in webkit [16191] by hyatt
  • 6 edits in trunk/WebCore

Fix for 10682, refine the FOUC paint suppression logic so that it
is only triggered when FOUC would really have occurred.

Reviewed by aroben

  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::styleForElement):
  • dom/Document.cpp: (WebCore::Document::Document): (WebCore::Document::updateLayoutIgnorePendingStylesheets): (WebCore::Document::updateStyleSelector):
  • dom/Document.h: (WebCore::Document::haveStylesheetsLoaded): (WebCore::Document::): (WebCore::Document::didLayoutWithPendingStylesheets):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::paintChildren):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintLayer):
6:10 PM Changeset in webkit [16190] by aroben
  • 11 edits in trunk

2006-09-01 MorganL <morlmor@yahoo.com>

Reviewed by Darin. Updated/landed by Adam.

Fixes http://bugzilla.opendarwin.org/show_bug.cgi?id=10553
Windows build fixes

5:57 PM Changeset in webkit [16189] by beidson
  • 6 edits in trunk/WebCore

Reviewed by Tim Omernick

Added an SQLDatabase logging channel. Changed all SQLDatabase and SQLStatement errors to use this new channel.
Also, as a popular request from other contributers, added a log for all prepare(), step(), finalize(), and reset()s
Also, fixed a grammar error in my previous ChangeLog entry.

  • bridge/mac/WebCorePageBridge.mm: (initializeLoggingChannelsIfNecessary): Initialize the new channel
  • loader/icon/SQLDatabase.cpp: (WebCore::SQLDatabase::setBusyTimeout): Use SQLDatabase logging channel (WebCore::SQLDatabase::setBusyHandler): ditto (WebCore::SQLDatabase::clearAllTables): ditto (WebCore::SQLDatabase::runVacuumCommand): ditto
  • loader/icon/SQLStatement.cpp: (WebCore::SQLStatement::prepare): Added a log (WebCore::SQLStatement::step): ditto (WebCore::SQLStatement::finalize): ditto (WebCore::SQLStatement::reset): ditto (WebCore::SQLStatement::getColumnBlob): Use SQLDatabase logging channel (WebCore::SQLStatement::returnTextResults): ditto (WebCore::SQLStatement::returnTextResults16): ditto (WebCore::SQLStatement::returnIntResults): ditto (WebCore::SQLStatement::returnInt64Results): ditto (WebCore::SQLStatement::returnDoubleResults): ditto
  • platform/Logging.cpp: (WebCore::): Added new logging channel
  • platform/Logging.h: ditto
5:55 PM Changeset in webkit [16188] by adele
  • 20 edits in trunk

LayoutTests:

Reviewed by Tim Omernick.

Updated results for http://bugzilla.opendarwin.org/show_bug.cgi?id=6990
Switch to use new text field implementation for <input type="password">

  • dom/html/level2/html/HTMLInputElement22-expected.txt:
  • dom/xhtml/level2/html/HTMLInputElement22-expected.txt:
  • fast/events/event-sender-mouse-click.html:
  • fast/forms/input-appearance-height-expected.checksum:
  • fast/forms/input-appearance-height-expected.txt:
  • fast/forms/input-type-change-in-onfocus-mouse-expected.txt:
  • fast/forms/input-value-expected.checksum:
  • fast/forms/input-value-expected.png:
  • fast/forms/input-value-expected.txt:
  • tables/mozilla_expected_failures/bugs/bug92647-1-expected.checksum:
  • tables/mozilla_expected_failures/bugs/bug92647-1-expected.png:
  • tables/mozilla_expected_failures/bugs/bug92647-1-expected.txt:

Updated from switch to style-able selects

  • fast/forms/option-strip-whitespace-expected.checksum:
  • fast/forms/option-strip-whitespace-expected.png:
  • fast/forms/option-strip-whitespace-expected.txt:

WebCore:

Reviewed by Tim Omernick.

  • css/html4.css: Give input[password] the same style properties as normal text fields.
  • html/HTMLInputElement.cpp: Removed checks for TextFieldAppearance. (WebCore::HTMLInputElement::selectionStart): (WebCore::HTMLInputElement::selectionEnd): (WebCore::HTMLInputElement::setSelectionStart): (WebCore::HTMLInputElement::setSelectionEnd): (WebCore::HTMLInputElement::select): (WebCore::HTMLInputElement::setSelectionRange): (WebCore::HTMLInputElement::createRenderer):
  • html/HTMLInputElement.h: (WebCore::HTMLInputElement::isNonWidgetTextField):
5:03 PM Changeset in webkit [16187] by beidson
  • 8 edits in trunk/WebCore

Reviewed by John (though Sarge review a previous patch of mine in an attempt to review this one)

Added some constants for SQL Result Codes to SQLDatabase.h
This way, users of SQLDatabase can access all necessary SQLite functionality
without having access to sqlite3.h.
Also reordered some files in the xcodeproj and some #includes

  • WebCore.xcodeproj/project.pbxproj: Reordered some files
  • loader/icon/IconDataCache.cpp: (WebCore::IconDataCache::writeToDatabase): Use our new result codes
  • loader/icon/IconDatabase.cpp: (WebCore::IconDatabase::sharedIconDatabase): Use our new result codes (WebCore::IconDatabase::retainIconForPageURL): Use our new result codes (WebCore::IconDatabase::pruneUnretainedIconsOnStartup): Use our new result codes (WebCore::IconDatabase::addIconForIconURLQuery): Use our new result codes (WebCore::IconDatabase::hasIconForIconURLQuery): Use our new result codes
  • loader/icon/SQLDatabase.cpp: (WebCore::SQLDatabase::lastError): Moved from header to eliminate dependency on sqlite3.h in header (WebCore::SQLDatabase::lastErrorMsg): Ditto
  • loader/icon/SQLDatabase.h: Removed #include <sqlite3.h>
  • loader/icon/SQLStatement.cpp: Added #include <sqlite3.h>
  • loader/icon/SQLStatement.h: Removed #include <sqlite3.h>
4:53 PM Changeset in webkit [16186] by beidson
  • 1 edit in trunk/WebKit/ChangeLog

My ChangeLog grammar needs reviewing, apparently (fixed last ChangeLog)

4:20 PM Changeset in webkit [16185] by beidson
  • 2 edits in trunk/WebKit

Reviewed by Darin

A "never should be reach" method was reached - lets not release the shared database bridge, esp
since we never retain it!

  • Misc/WebIconDatabase.m: (-[WebIconDatabase _applicationWillTerminate:]): Don't release the bridge
3:57 PM Changeset in webkit [16184] by hyatt
  • 4 adds in trunk/LayoutTests/fast/flexbox

Add flexbox test.

3:56 PM Changeset in webkit [16183] by hyatt
  • 2 edits in trunk/WebCore

Fix for 10680, contrained flexbox is not shrinking flexible children
properly. (Fixing in case it's hit by the file upload control.)

Reviewed by aroben

Adding 026.html to fast/flexbox.

  • rendering/RenderFlexibleBox.cpp: (WebCore::RenderFlexibleBox::allowedChildFlex):
2:52 PM Changeset in webkit [16182] by eseidel
  • 4 edits in trunk/WebCore

2006-09-01 Nikolas Zimmermann <zimmermann@kde.org>

Reviewed by eseidel. Landed by eseidel.

Implement support for multiple parents in the SVG idls.

For class "FooBar" go through all parents recursively,
collect their attributes & functions & constants and
add them directly into the class "FooBar".

(Note: It is NOT enabled by default for now - still some things related to SVGAnimated* need to be solved first)

  • bindings/scripts/CodeGenerator.pm:
  • bindings/scripts/CodeGeneratorJS.pm:
  • bindings/scripts/generate-bindings.pl:
2:50 PM Changeset in webkit [16181] by eseidel
  • 6 edits in trunk

2006-09-02 Nikolas Zimmermann <zimmermann@kde.org>

Reviewed by hyatt. Landed by eseidel.

Fix build on Linux.

  • CMakeLists.txt: Add platform/CString.cpp.
  • platform/qt/FrameQt.cpp: Correct repsondToChangedContents() signature
  • platform/qt/FrameQt.h:: Correct respondToChangedContents() signature
2:41 PM Changeset in webkit [16180] by darin
  • 9 edits in trunk

WebCore:

Reviewed by Brady.

  • a few small tweaks to the icon database bridge
  • bridge/mac/WebCoreIconDatabaseBridge.h: Removed unnecessary declarations. Renamed sharedBridgeInstance to sharedInstance.
  • bridge/mac/WebCoreIconDatabaseBridge.mm: Put functions inside WebCore namespace. (WebCore::IconDatabase::loadIconFromURL): Updated for sharedInstance namespace.

WebKit:

Reviewed by Brady.

  • a few small tweaks to the icon database bridge
  • Misc/WebIconDatabase.m: (-[WebIconDatabase init]): Updated for name change.
  • WebCoreSupport/WebIconDatabaseBridge.h: Removed unneeded declarations.
  • WebCoreSupport/WebIconDatabaseBridge.m: (-[WebIconDatabaseBridge init]): Added. Always returns nil since you're not supposed to allocate one of these. (-[WebIconDatabaseBridge _init]): Renamed from init. Used internally to make the shared instance. Added the "self = [super init]" idiom even though it's not important in this case just to be consistent. (-[WebIconDatabaseBridge releaseCachedLoaderForIconURL:]): Moved this up in the file so it can be called without declaring it in the header. (+[WebIconDatabaseBridge sharedInstance]): Renamed. Calls the new _init. Also use CFRetain for compatibility. (-[WebIconDatabaseBridge dealloc]): Emptied this out and made it just assert (false). (-[WebIconDatabaseBridge finalize]): Added and made it assert (false) too.
2:21 PM Changeset in webkit [16179] by thatcher
  • 2 edits in trunk/WebKit

Reviewed by Adele.

Bug 10677: Omit "-webkit-text-security: none;" from the computed style list
http://bugzilla.opendarwin.org/show_bug.cgi?id=10677

2:00 PM Changeset in webkit [22139] by sfalken
  • 5 edits in branches/WindowsMerge
1:09 PM Changeset in webkit [16178] by thatcher
  • 5 edits in trunk

LayoutTests:

Reviewed by Darin.

  • updated results, null is not round-tripped through the bindings because of regressions the original change caused in other applications.
  • added an echo for "undefined".
  • plugins/bindings-test-expected.txt:
  • plugins/bindings-test.html:

WebCore:

Reviewed by Darin.

<rdar://problem/4651318> REGRESSION: can't drag library items to widget in Dashcode
<rdar://problem/4701626> REGRESSION: Unit Converter widget doesn't work convert data in its fields

jsNull should not be returned as NSNull because existing applications do not
expect that return value. Return as nil for backwards compatibility.

  • bindings/objc/WebScriptObject.mm: (+[WebScriptObject _convertValueToObjcValue:originExecutionContext:executionContext:]):
1:05 PM Changeset in webkit [16177] by ap
  • 4 edits in trunk/JavaScriptCore

2006-09-01 Nikolas Zimmermann <zimmermann@kde.org>

Reviewed and landed by ap.

Fix build on Linux (C89 without gcc extensions enabled).

  • pcre/pcre_internal.h: Use C style comments.
  • wtf/Assertions.h: Use C style comments.
  • wtf/Platform.h: Use C style comments.
12:48 PM Changeset in webkit [16176] by ap
  • 2 edits in trunk/WebCore

Build fix.

  • loader/Decoder.cpp: (Decoder::Decoder): Use String::latin1().
12:36 PM Changeset in webkit [16175] by ap
  • 12 edits
    18 adds in trunk

Reviewed by Eric.

WebCore:

  • loader/Decoder.cpp: (Decoder::Decoder): Decoder now knows what kind of content it is decoding. Also, the browser default encoding can now be passed directly to the constructor, to streamline the logic. (Decoder::decode): Add support for @charset, differentiate between HTML and XML. (Decoder::setEncodingName): Style cleanup. (Decoder::encodingName): Ditto. (Decoder::flush): Ditto.
  • loader/Decoder.h: (WebCore::Decoder::):
  • bridge/mac/WebCoreEncodings.mm: Pass a content type of text/html to Decoder.
  • loader/CachedCSSStyleSheet.cpp: (WebCore::CachedCSSStyleSheet::CachedCSSStyleSheet): (WebCore::CachedCSSStyleSheet::setCharset): (WebCore::CachedCSSStyleSheet::data):
  • loader/CachedCSSStyleSheet.h: Use Decoder instead of TextEncoding::toUnicode() to enable @charset support.
  • loader/CachedXBLDocument.cpp: (WebCore::CachedXBLDocument::CachedXBLDocument): Pass a content type.
  • loader/CachedXSLStyleSheet.cpp: (WebCore::CachedXSLStyleSheet::CachedXSLStyleSheet): Ditto. (WebCore::CachedXSLStyleSheet::data): Flush the decoder to be safe.
  • page/Frame.cpp: (WebCore::Frame::write): Pass a content type and a default encoding to the Decoder.
  • xml/XSLTProcessor.cpp: (WebCore::XSLTProcessor::createDocumentFromSource): Pass the output document MIME type.
  • xml/xmlhttprequest.cpp: (WebCore::XMLHttpRequest::receivedData): Ditto.

LayoutTests:

  • fast/encoding/css-charset-expected.txt: Added.
  • fast/encoding/css-charset.css: Added.
  • fast/encoding/css-charset.html: Added.
  • fast/encoding/css-charset-evil-expected.txt: Added.
  • fast/encoding/css-charset-evil.css: Added.
  • fast/encoding/css-charset-evil.html: Added.
  • fast/encoding/default-xhtml-encoding-expected.txt: Added.
  • fast/encoding/default-xhtml-encoding.xhtml: Added.
  • fast/encoding/meta-in-xhtml-expected.txt: Added.
  • fast/encoding/meta-in-xhtml.xhtml: Added.
  • fast/encoding/pseudo-xml-2-expected.txt: Added.
  • fast/encoding/pseudo-xml-2.html: Added.
  • fast/encoding/pseudo-xml-3-expected.txt: Added.
  • fast/encoding/pseudo-xml-3.html: Added.
  • fast/encoding/pseudo-xml-4-expected.txt: Added.
  • fast/encoding/pseudo-xml-4.html: Added.
  • fast/encoding/pseudo-xml-expected.txt: Added.
  • fast/encoding/pseudo-xml.html: Added.
12:01 PM Changeset in webkit [16174] by adele
  • 5 edits in trunk/WebCore

Reviewed by John.

  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::setSecureKeyboardEntry): Enable and disable Roman keyboards when switching in and out of this mode. (WebCore::FrameMac::isSecureKeyboardEntry): Changed secureKeyboardEntry to isSecureKeyboardEntry.
  • bridge/mac/FrameMac.h: ditto.
  • page/Frame.cpp: (WebCore::Frame::setIsActive): ditto.
  • page/Frame.h: (WebCore::Frame::isSecureKeyboardEntry): ditto.
11:07 AM Changeset in webkit [22138] by adachan
  • 3 edits in branches/WindowsMerge/WebCore

2006-09-01 Ada Chan <adachan@apple.com>

Reviewed by lamadio.

Fixed build break.

10:21 AM Changeset in webkit [16173] by adele
  • 2 edits in trunk/WebCore

Reviewed by John.

  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::doApply): Don't smart replace when the selection being replaces is in a password field.
Note: See TracTimeline for information about the timeline view.