Timeline



Feb 24, 2008:

11:44 PM Changeset in webkit [30562] by Darin Adler
  • 2 edits in trunk/WebKitTools
  • Scripts/do-webcore-rename: Make some updates based on a trial run of the renaming script.
11:17 PM Changeset in webkit [30561] by Darin Adler
  • 2 edits in trunk/WebKitTools
  • Scripts/do-webcore-rename: More renaming plans.
11:11 PM Changeset in webkit [30560] by weinig@apple.com
  • 2 edits in trunk/JavaScriptCore

Reviewed by Mark Rowe.

​http://bugs.webkit.org/show_bug.cgi?id=17529
Add support for reading from stdin from testkjs

  • kjs/testkjs.cpp: (GlobalObject::GlobalObject): Add readline function to global object. (functionReadline): Added. Reads characters from stdin until a '\n' or EOF is encountered. The input is returned as a String to the caller.
11:09 PM Changeset in webkit [30559] by Darin Adler
  • 2 edits in trunk/WebCore
  • dom/Document.h: Removed stray duplicate declaration of RegisteredEventListenerList.
10:52 PM Changeset in webkit [30558] by Darin Adler
  • 2 edits in trunk/WebCore
  • dom/Range.cpp: (WebCore::Range::surroundContents): Removed incorrect comment. I added it a while ago, and I was just mixed up.
10:48 PM Changeset in webkit [30557] by Darin Adler
  • 2 edits in trunk/WebCore
  • quick follow-up to that last check-in
  • platform/graphics/cg/ImageSourceCG.cpp: (WebCore::ImageSource::createFrameAtIndex): Use CFEqual instead of WebCore::String's == for speed.
10:36 PM Changeset in webkit [30556] by Darin Adler
  • 3 edits
    6 adds in trunk

WebCore:

Reviewed by Darin.

Bug 8749: XBM rendered incorrectly as black on white

xbm images are now created and rendered using black on a
transparent background (same behavior as in Firefox)

Test: platform/mac/fast/canvas/canvas-draw-xbm-image.html

  • platform/graphics/cg/ImageSourceCG.cpp: (WebCore::ImageSource::createFrameAtIndex):

LayoutTests:

Reviewed by Darin.


  • platform/mac/fast/canvas/canvas-draw-xbm-image-expected.checksum: Added.
  • platform/mac/fast/canvas/canvas-draw-xbm-image-expected.png: Added.
  • platform/mac/fast/canvas/canvas-draw-xbm-image-expected.txt: Added.
  • platform/mac/fast/canvas/canvas-draw-xbm-image.html: Added.
  • platform/mac/fast/canvas/resources: Added.
  • platform/mac/fast/canvas/resources/smile.xbm: Added.
9:29 PM Changeset in webkit [30555] by weinig@apple.com
  • 4 edits in trunk/JavaScriptCore

Reviewed by Mark Rowe.

​http://bugs.webkit.org/show_bug.cgi?id=17528
Give testkjs a bath

  • JavaScriptCore.exp:
  • JavaScriptCore.xcodeproj/project.pbxproj: Make the testkjs.cpp use 4 space indentation.
  • kjs/testkjs.cpp: (StopWatch::getElapsedMS): (GlobalObject::className): (GlobalObject::GlobalObject): Rename GlobalImp to GlobalObject and setup the global functions in the GlobalObject's constructor. Also, use static functions for the implementation so we can use the standard PrototypeFunction class and remove TestFunctionImp. (functionPrint): Move print() functionality here. (functionDebug): Move debug() functionality here. (functionGC): Move gc() functionality here. (functionVersion): Move version() functionality here. (functionRun): Move run() functionality here. (functionLoad): Move load() functionality here. (functionQuit): Move quit() functionality here. (prettyPrintScript): Fix indentation. (runWithScripts): Since all the functionality of createGlobalObject is now in the GlobalObject constructor, just call new here. (parseArguments): Fix indentation. (kjsmain): Ditto (fillBufferWithContentsOfFile): Ditto.
7:08 PM Changeset in webkit [30554] by mitz@apple.com
  • 3 edits
    3 adds in trunk

WebCore:

Reviewed by Darin Adler.

Test: http/tests/misc/object-image-error.html

  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::attach): Added an early return in case updateFromElement() has changed the object to use fallback content.

LayoutTests:

Reviewed by Darin Adler.

  • http/tests/misc/object-image-error-expected.txt: Added.
  • http/tests/misc/object-image-error.html: Added.
  • http/tests/misc/resources/404image.php: Added.
5:34 PM Changeset in webkit [30553] by weinig@apple.com
  • 2 edits in trunk/JavaScriptCore

Reviewed by Oliver Hunt and Mark Rowe.

​http://bugs.webkit.org/show_bug.cgi?id=17505
Add support for getting command line arguments in testkjs

  • This slightly changes the behavior of parsing arguments by requiring a '-f' before all files.
  • kjs/testkjs.cpp: (createGlobalObject): Add a global property called 'arguments' which contains an array with the parsed arguments as strings. (runWithScripts): Pass in the arguments vector so that it can be passed to the global object. (parseArguments): Change parsing rules to require a '-f' before any script file. After all '-f' and '-p' arguments have been parsed, the remaining are added to the arguments vector and exposed to the script. If there is a chance of ambiguity (the user wants to pass the string '-f' to the script), the string '--' can be used separate the options from the pass through arguments. (kjsmain):
2:50 PM Changeset in webkit [30552] by mitz@apple.com
  • 1 edit
    651 moves
    651 deletes in trunk/LayoutTests

Rubber-stamped by Oliver Hunt.

  • replace expected results in platform/mac-tiger and platform/mac-leopard which were identical with a single copy in platform/mac
10:48 AM Changeset in webkit [30551] by mitz@apple.com
  • 3 edits
    4 adds in trunk

WebCore:

Reviewed by Darin Adler.

  • fix float positioning when a float that does not fit on the line is followed by a float that does

Test: fast/block/float/narrow-after-wide.html

  • rendering/bidi.cpp: (WebCore::RenderBlock::findNextLineBreak): Changed to not position any more floats on the line once a float that does not fit is encountered. That float should be pushed to the next line, and so should all floats that follow, regardless of whether they can fit on the current line.

LayoutTests:

Reviewed by Darin Adler.

  • test float positioning when a float that does not fit on the line is followed by a float that does
  • fast/block/float/narrow-after-wide.html: Added.
  • platform/mac/fast/block/float/narrow-after-wide-expected.checksum: Added.
  • platform/mac/fast/block/float/narrow-after-wide-expected.png: Added.
  • platform/mac/fast/block/float/narrow-after-wide-expected.txt: Added.
10:44 AM Changeset in webkit [30550] by mitz@apple.com
  • 2 edits in trunk/JavaScriptCore

Reviewed by Darin Adler.

  • wtf/Deque.h: (WTF::::expandCapacityIfNeeded): Fixed the case where m_start and m_end are both zero but the buffer capacity is non-zero. (WTF::::prepend): Added validity checks.
12:30 AM Changeset in webkit [30549] by Darin Adler
  • 21 edits
    3 adds in trunk

WebCore:

Reviewed by Sam.

Disentangle global history updating from the back/forward history.
There are many cases where we don't want to create a new back/forward
item, but we do still want to add to the global history (used for visited
link coloring) in those cases.

Test: fast/history/subframe-is-visited.html

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::updateGlobalHistory): Renamed from addHistoryForCurrentLocation and removed the back/forward handling. (WebCore::FrameLoader::updateHistoryForStandardLoad): Streamlined logic a bit. Replaced call to addHistoryForCurrentLocation with a call to addBackForwardItemClippedAtTarget. Added an unconditional call to updateGlobalHistory. (WebCore::FrameLoader::updateHistoryForClientRedirect): Added a FIXME; why doesn't this function update global history? (WebCore::FrameLoader::updateHistoryForBackForwardNavigation): Ditto. (WebCore::FrameLoader::updateHistoryForReload): Replaced the direct call the client with a call to the new updateGlobalHistory function. (WebCore::FrameLoader::updateHistoryForRedirectWithLockedHistory): Did the same changes as for updateHistoryForStandardLoad.
  • loader/FrameLoader.h: More of the same.
  • loader/FrameLoaderClient.h: Removed updateGlobalHistoryForReload and renamed updateGlobalHistoryForStandardLoad to updateGlobalHistory.
  • svg/graphics/SVGImageEmptyClients.h: (WebCore::SVGEmptyFrameLoaderClient::updateGlobalHistory): Updated to match the above.

WebKit/gtk:

Reviewed by Sam.

  • remove separate client calls for "standard" and "reload' history
  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::updateGlobalHistory):
  • WebCoreSupport/FrameLoaderClientGtk.h:

WebKit/mac:

Reviewed by Sam.

  • remove separate client calls for "standard" and "reload' history
  • WebCoreSupport/WebFrameLoaderClient.h:
  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::updateGlobalHistory):

WebKit/qt:

Reviewed by Sam.

  • remove separate client calls for "standard" and "reload' history
  • WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::updateGlobalHistory):
  • WebCoreSupport/FrameLoaderClientQt.h:

WebKit/win:

Reviewed by Sam.

  • remove separate client calls for "standard" and "reload' history
  • WebFrame.cpp: (WebFrame::updateGlobalHistory):
  • WebFrame.h:

WebKit/wx:

Reviewed by Sam.

  • remove separate client calls for "standard" and "reload' history
  • WebKitSupport/FrameLoaderClientWx.cpp: (WebCore::FrameLoaderClientWx::updateGlobalHistory):
  • WebKitSupport/FrameLoaderClientWx.h:

LayoutTests:

Reviewed by Sam.

  • fast/history/resources/subframe.html: Added.
  • fast/history/subframe-is-visited-expected.txt: Added.
  • fast/history/subframe-is-visited.html: Added.
12:09 AM Changeset in webkit [30548] by Darin Adler
  • 2 edits in trunk/WebKit/qt
  • another try at fixing the build
  • Api/qwebframe.cpp: (QWebFrame::load): Replace new FormData with FormData::create.

Feb 23, 2008:

11:55 PM Changeset in webkit [30547] by Darin Adler
  • 2 edits in trunk/WebCore

Reviewed by Sam.

  • html/HTMLFormElement.cpp: (WebCore::HTMLFormElement::submit): Restore the old logic, with its unpleasant trip through String do to the replace operation. It's inefficient, but there's no real reason to worry about the efficiency of mailto forms, since that feature is nearly obsolete.
11:36 PM Changeset in webkit [30546] by Darin Adler
  • 3 edits in trunk/WebCore

Reviewed by Sam.

  • replace SegmentedString's use of DeprecatedValueList with Deque

Testing indicates this is a slight speed-up for page loading.

  • platform/text/SegmentedString.cpp: (WebCore::SegmentedString::length): Update to use Deque. (WebCore::SegmentedString::setExcludeLineNumbers): Ditto. (WebCore::SegmentedString::append): Ditto. (WebCore::SegmentedString::prepend): Ditto. (WebCore::SegmentedString::advanceSubstring): Ditto. (WebCore::SegmentedString::toString): Ditto.
  • platform/text/SegmentedString.h: Tweak formatting. Don't bother making an entire class private with a single friend. Change m_sbstrings from a DeprecatedValueList to a Deque.
11:26 PM Changeset in webkit [30545] by Darin Adler
  • 2 edits in trunk/WebCore

Reviewed by Anders.

  • loader/TextResourceDecoder.cpp: (WebCore::findTextEncoding): Fix off-by-one in code that null-terminates the C string here.
11:10 PM Changeset in webkit [30544] by pewtermoose@webkit.org
  • 3 edits in trunk/WebCore

Not reviewed, build fix.

  • platform/network/cf/FormDataStreamCFNet.cpp: (WebCore::httpBodyFromRequest):
  • plugins/win/PluginViewWin.cpp: (WebCore::PluginView::handlePost):
11:07 PM Changeset in webkit [30543] by mitz@apple.com
  • 3 edits in trunk/WebCore
  • fix a typo
  • platform/graphics/win/GlyphPageTreeNodeCGWin.cpp: (WebCore::GlyphPage::fill):
  • platform/graphics/win/GlyphPageTreeNodeCairoWin.cpp: (WebCore::GlyphPage::fill):
11:07 PM Changeset in webkit [30542] by pewtermoose@webkit.org
  • 6 edits in trunk

2008-02-23 Jan Michael Alonzo <​jmalonzo@unpluggable.com>
.:

Rubber stamped by Darin.

Add separator '\' after libJavaScriptCore_la_LIBADD and cleanup
whitespaces introduced in the previous commit.

  • GNUmakefile.am:

JavaScriptCore:

Rubber stamped by Darin.

Add separator '\' after libJavaScriptCore_la_LIBADD and cleanup
whitespaces introduced in the previous commit.

  • GNUmakefile.am:

WebKitTools:

Rubber stamped by Darin.

Add separator '\' after libJavaScriptCore_la_LIBADD and cleanup
whitespaces introduced in the previous commit.

  • GNUmakefile.am:
10:59 PM Changeset in webkit [30541] by mitz@apple.com
  • 3 edits in trunk/WebCore

Reviewed by Anders Carlsson.

  • fix GlyphPage::fill() logic for partial page fills
  • platform/graphics/win/GlyphPageTreeNodeCGWin.cpp: (WebCore::GlyphPage::fill): Changed the test for non-BMP characters to work correctly when the length parameter is less than a full page size. Also updated the comment.
  • platform/graphics/win/GlyphPageTreeNodeCairoWin.cpp: (WebCore::GlyphPage::fill): Ditto.
9:42 PM Changeset in webkit [30540] by Darin Adler
  • 7 edits in trunk

.:

Rubber-stamped by Darin.

  • GNUmakefile.am: Add both GLOBALDEPS and WEBKITDEPS instead of DEPENDENCIES.
  • configure.ac: Break dependent modules into GLOBALDEPS and WEBKITDEPS.

JavaScriptCore:

  • GNUmakefile.am: Add GLOBALDEPS for testkjs and minidom.

WebKitTools:

Rubber-stamped by Darin.

  • GNUmakefile.am: Add both GLOBALDEPS and WEBKITDEPS instead of DEPENDENCIES.
9:38 PM Changeset in webkit [30539] by kevino@webkit.org
  • 5 edits
    4 adds in trunk

Move text drawing into wxcode, as we need platform-dependent APIs for drawing non-kerned text, which wx doesn't yet have.

​http://bugs.webkit.org/show_bug.cgi?id=17396

9:19 PM Changeset in webkit [30538] by Darin Adler
  • 3 edits in trunk/JavaScriptCore

Reviewed by Anders.

  • wtf/Deque.h: Wrote an all-new version of this class that uses a circular buffer. Growth policy is identical to vector. Added iterators.
  • wtf/Vector.h: Made two small refinements while using this to implement Deque: Made VectorBufferBase derive from Noncopyable, which would have saved me some debugging time if it had been there. Renamed Impl and m_impl to Buffer and m_buffer.
9:13 PM Changeset in webkit [30537] by Darin Adler
  • 2 edits in trunk/WebCore

Pointed out by Mitz.

  • loader/TextResourceDecoder.cpp: (WebCore::find): Fix a "==" that was supposed to be ">". (WebCore::findIgnoringCase): Ditto.
9:12 PM Changeset in webkit [30536] by Darin Adler
  • 2 edits in trunk/WebCore

Reviewed by Sam.

  • html/HTMLFormElement.cpp: (WebCore::appendEncodedString): Fix buffer overrun in the code I just landed.
9:04 PM Changeset in webkit [30535] by Darin Adler
  • 7 edits in trunk/WebCore

Reviewed by Sam.

  • remove all but 1 remaining use of DeprecatedCString
  • change FormData to start refcount at 1
  • html/HTMLFormElement.cpp: (WebCore::appendString): Added. Helpers for Vector<char>. (WebCore::appendEncodedString): Renamed from encodeCString and changed to append to an existing Vector<char> to cut down a bit on memory allocation. (WebCore::HTMLFormElement::formData): Replace the DeprecatedCString called enc_string with a Vector<char> called encodedData. Change to use the new FormData::create function. (WebCore::HTMLFormElement::submit): Same thing, for the mailto form code.
  • loader/TextResourceDecoder.cpp: (WebCore::find): Added. Helper for searching in char* with length. (WebCore::findIgnoringCase): Ditto. (WebCore::findTextEncoding): Added. Helper for looking up a text encoding when we have a char* with length rather than a null-terminated char*. (WebCore::findXMLEncoding): Changed to use char* and length rather than DeprecatedCString. Also fixed some things that would be one-character buffer overruns in a string that's not null-terminated. (WebCore::TextResourceDecoder::checkForCSSCharset): Change to use findTextEncoding rather than constructing a CString to pass to the TextEncodingconstructor. (WebCore::TextResourceDecoder::checkForHeadCharset): Ditto.
  • platform/network/FormData.cpp: (WebCore::FormData::FormData): Changed to start refcount at 1. (WebCore::FormData::create): Added. (WebCore::FormData::copy): Call adoptRef since refcount starts at 1.
  • platform/network/FormData.h: Made constructor private. Added create functions.
  • platform/network/mac/ResourceRequestMac.mm: (WebCore::ResourceRequest::doUpdateResourceRequest): Call FormData::create.
  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::send): Ditto.
9:01 PM Changeset in webkit [30534] by Darin Adler
  • 57 edits in trunk

JavaScriptCore:

Reviewed by Anders.

  • API/JSCallbackObject.h: Removed attribute arguments.
  • API/JSCallbackObjectFunctions.h: (KJS::JSCallbackObject<Base>::put): Ditto.
  • API/JSObjectRef.cpp: (JSObjectSetProperty): Use initializeVariable or putDirect when necessary to set attribute values.
  • JavaScriptCore.exp: Updated.
  • bindings/objc/objc_runtime.h: Removed attribute arguments.
  • bindings/objc/objc_runtime.mm: (ObjcFallbackObjectImp::put): Ditto.
  • bindings/runtime_array.cpp: (RuntimeArray::put): Ditto.
  • bindings/runtime_array.h: Ditto.
  • bindings/runtime_object.cpp: (RuntimeObjectImp::put): Ditto.
  • bindings/runtime_object.h: Ditto. Also removed canPut which was only called from one place in WebCore that can use hasProperty instead.
  • kjs/Activation.h: Removed attribute argument from put and added the new initializeVariable function that's used to put variables in variable objects. Also made isActivationObject a const member.
  • kjs/JSGlobalObject.cpp: (KJS::JSGlobalObject::put): Removed attribute argument. (KJS::JSGlobalObject::initializeVariable): Added. Used to give variables their initial values, which can include the read-only property. (KJS::JSGlobalObject::reset): Removed obsolete comments about flags. Removed Internal flag, which is no longer needed.
  • kjs/JSGlobalObject.h: More of the same.
  • kjs/JSVariableObject.h: Added pure virtual initializeVariable function. (KJS::JSVariableObject::symbolTablePut): Removed checkReadOnly flag; we always check read-only. (KJS::JSVariableObject::symbolTableInitializeVariable): Added.
  • kjs/array_instance.cpp: (KJS::ArrayInstance::put): Removed attribute argument.
  • kjs/array_instance.h: Ditto.
  • kjs/function.cpp: (KJS::FunctionImp::put): Ditto. (KJS::Arguments::put): Ditto. (KJS::ActivationImp::put): Ditto. (KJS::ActivationImp::initializeVariable): Added.
  • kjs/function.h: Removed attribute arguments.
  • kjs/function_object.cpp: (KJS::FunctionObjectImp::construct): Removed Internal flag.
  • kjs/lookup.h: (KJS::lookupPut): Removed attributes argument. Also changed to use putDirect instead of calling JSObject::put. (KJS::cacheGlobalObject): Ditto.
  • kjs/nodes.cpp: (KJS::ConstDeclNode::handleSlowCase): Call initializeVariable to initialize the constant. (KJS::ConstDeclNode::evaluateSingle): Ditto. (KJS::TryNode::execute): Use putDirect to set up the new object. (KJS::FunctionBodyNode::processDeclarations): Removed Internal. (KJS::ProgramNode::processDeclarations): Ditto. (KJS::EvalNode::processDeclarations): Call initializeVariable to initialize the variables and functions. (KJS::FuncDeclNode::makeFunction): Removed Internal. (KJS::FuncExprNode::evaluate): Ditto.
  • kjs/object.cpp: Removed canPut, which was only being used in one code path, not the normal high speed one. (KJS::JSObject::put): Removed attribute argument. Moved the logic from canPut here, in the one code ath that was still using it.
  • kjs/object.h: Removed Internal attribute, ad canPut function. Removed the attributes argument to the put function. Made isActivationObject const.
  • kjs/regexp_object.cpp: (KJS::RegExpImp::put): Removed attributes argument. (KJS::RegExpImp::putValueProperty): Ditto. (KJS::RegExpObjectImp::put): Ditto. (KJS::RegExpObjectImp::putValueProperty): Ditto.
  • kjs/regexp_object.h: Ditto.
  • kjs/string_object.cpp: (KJS::StringInstance::put): Removed attributes argument.
  • kjs/string_object.h: Ditto.

WebCore:

Reviewed by Anders.

  • bindings/js/JSCSSStyleDeclarationCustom.cpp: (WebCore::JSCSSStyleDeclaration::customPut): Remove attributes argument.
  • bindings/js/JSCanvasPixelArrayCustom.cpp: (WebCore::JSCanvasPixelArray::indexGetter): Use early exit idiom. (WebCore::JSCanvasPixelArray::indexSetter): Moved length check into the CanvasPixelArray object, for consistency with the getter. Removed attributes argument.
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::customPut): Removed special case for variable initialization, which is not needed since that does use put any more. Removed attributes argument.
  • bindings/js/JSEventTargetBase.h: (WebCore::JSEventTargetBase::putValueProperty): Removed attributes argument. (WebCore::JSEventTargetBase::put): Ditto. (WebCore::JSEventTargetPrototype::self): Removed Internal flag.
  • bindings/js/JSEventTargetNode.cpp: (WebCore::JSEventTargetNode::put): Removed attributes argument. (WebCore::JSEventTargetNode::putValueProperty): Ditto.
  • bindings/js/JSEventTargetNode.h: Ditto.
  • bindings/js/JSHTMLAppletElementCustom.cpp: (WebCore::JSHTMLAppletElement::customPut): Ditto.
  • bindings/js/JSHTMLEmbedElementCustom.cpp: (WebCore::JSHTMLEmbedElement::customPut): Ditto.
  • bindings/js/JSHTMLInputElementBase.cpp: (WebCore::JSHTMLInputElementBase::put): Ditto. (WebCore::JSHTMLInputElementBase::putValueProperty): Ditto.
  • bindings/js/JSHTMLInputElementBase.h: Ditto.
  • bindings/js/JSHTMLObjectElementCustom.cpp: (WebCore::JSHTMLObjectElement::customPut): Ditto.
  • bindings/js/JSHTMLOptionsCollectionCustom.cpp: (WebCore::JSHTMLOptionsCollection::indexSetter): Ditto.
  • bindings/js/JSHTMLSelectElementCustom.cpp: (WebCore::JSHTMLSelectElement::indexSetter): Ditto.
  • bindings/js/JSHistoryCustom.cpp: (WebCore::JSHistory::customPut): Ditto.
  • bindings/js/JSLocation.cpp: (WebCore::JSLocation::put): Ditto.
  • bindings/js/JSLocation.h: Ditto.
  • bindings/js/JSXMLHttpRequest.cpp: (WebCore::JSXMLHttpRequest::put): Ditto. (WebCore::JSXMLHttpRequest::putValueProperty): Ditto.
  • bindings/js/JSXMLHttpRequest.h: Ditto.
  • bindings/js/kjs_dom.cpp: (WebCore::getRuntimeObject): Changed return type to JSObject*.
  • bindings/js/kjs_dom.h: Ditto.
  • bindings/js/kjs_events.cpp: (WebCore::JSClipboard::put): Removed attributes argument. (WebCore::JSClipboard::putValueProperty): Ditto.
  • bindings/js/kjs_events.h: Ditto.
  • bindings/js/kjs_html.cpp: (WebCore::runtimeObjectGetter): Updated for change to getRuntimeObject to return a JSObject. Used early exit idiom. (WebCore::runtimeObjectPropertyGetter): Ditto. (WebCore::runtimeObjectCustomGetOwnPropertySlot): Ditto. (WebCore::runtimeObjectCustomPut): Use hasProperty to check for properties that we should put with the property syntax instead of canPut. (WebCore::runtimeObjectImplementsCall): Ditto. (WebCore::runtimeObjectCallAsFunction): Ditto.
  • bindings/js/kjs_html.h: Removed attributes argument to runtimeObjectCustomPut.
  • bindings/js/kjs_window.cpp: (KJS::Window::put): Removed attributes argument.
  • bindings/js/kjs_window.h: Ditto.
  • bindings/scripts/CodeGeneratorJS.pm: Removed attributes argument from put, putValueProperty, customPut, and indexSetter.
  • html/CanvasPixelArray.h: (WebCore::CanvasPixelArray::set): Added index checking here, as in the get function. Before, the checking was done in the JavaScript bindings for set.
8:15 PM Changeset in webkit [30533] by pewtermoose@webkit.org
  • 7 edits in trunk/WebCore

2008-02-23 Brent Fulgham <​bfulgham@gmail.com>

Reviewed by Adam and Darin.

​http://bugs.webkit.org/show_bug.cgi?id=17442
Correct the Windows Cairo implementation of GraphicsContext so
that Windows 'WorldTransform' is kept in sync with the Cairo
transormations.

Also corrects an uninitialized variable in the Cairo Windows
image drag handler.

WARNING: NO TEST CASES ADDED OR CHANGED

  • platform/graphics/cairo/GraphicsContextCairo.cpp: Modify methods to call GraphicsContextPlatformPrivate calls, just as is done for the CoreGraphics version. These calls are nop's for everything but Windows. (WebCore::GraphicsContext::savePlatformState): (WebCore::GraphicsContext::restorePlatformState): (WebCore::GraphicsContext::clip): (WebCore::GraphicsContext::drawLineForMisspellingOrBadGrammar): (WebCore::GraphicsContext::translate): (WebCore::GraphicsContext::concatCTM): Re-enable for Windows (WebCore::GraphicsContext::beginTransparencyLayer): (WebCore::GraphicsContext::endTransparencyLayer): (WebCore::GraphicsContext::rotate): (WebCore::GraphicsContext::scale):
  • platform/graphics/cairo/GraphicsContextPlatformPrivateCairo.h:
  • platform/graphics/win/GraphicsContextCGWin.cpp: Move common code for handling WorldTransform calls to parent class.
  • platform/graphics/win/GraphicsContextCairoWin.cpp: Add common code for handling WorldTransform calls. (WebCore::GraphicsContextPlatformPrivate::concatCTM): Change implementation so that it only handles WorldTransform. The Cairo update is done in GraphicsContextCairo.cpp
  • platform/graphics/win/GraphicsContextWin.cpp: Call platform-private methods to keep WorldTransform in sync. (WebCore::GraphicsContextPlatformPrivate::save): (WebCore::GraphicsContextPlatformPrivate::restore): (WebCore::GraphicsContextPlatformPrivate::clip): (WebCore::GraphicsContextPlatformPrivate::scale): (WebCore::GraphicsContextPlatformPrivate::rotate): (WebCore::GraphicsContextPlatformPrivate::translate):
  • platform/win/DragImageCairoWin.cpp: (WebCore::createDragImageFromImage): Correct uninitialized value
8:10 PM Changeset in webkit [30532] by pewtermoose@webkit.org
  • 3 edits in trunk/WebKit/win

2008-02-23 Brent Fulgham <​bfulgham@gmail.com>

Not reviewed, build fix.

  • WebIconDatabase.cpp:
  • WebIconDatabase.h:
8:05 PM Changeset in webkit [30531] by pewtermoose@webkit.org
  • 2 edits in trunk/JavaScriptCore

2008-02-23 Jan Michael Alonzo <​jmalonzo@unpluggable.com>

Not reviewed, Gtk build fix.

  • kjs/testkjs.pro:
8:00 PM Changeset in webkit [30530] by mitz@apple.com
  • 5 edits
    5 adds in trunk

WebCore:

Reviewed by Dave Hyatt.

  • make non-autowrapping text clear floats

Test: fast/text/whitespace/nowrap-clear-float.html

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::nextFloatBottomBelow): Renamed nearestFloat() to this and changed to avoid comparing bottom to 0 in each iteration. (WebCore::RenderBlock::getClearDelta): Updated comment for the rename.
  • rendering/RenderBlock.h:
  • rendering/bidi.cpp: (WebCore::RenderBlock::fitBelowFloats): Added. Factored out of findNextLineBreak() and simplified. (WebCore::RenderBlock::findNextLineBreak): Changed to call fitBelowFloats(). Fixed the bug by trying to fit below floats in the case of non-wrapping text. Removed some redundancy.

LayoutTests:

Reviewed by Dave Hyatt.

  • test that non-autowrapping text clear floats
  • fast/text/whitespace/nowrap-clear-float.html: Added.
  • platform/mac-leopard/fast/text/whitespace: Added.
  • platform/mac-leopard/fast/text/whitespace/nowrap-clear-float-expected.checksum: Added.
  • platform/mac-leopard/fast/text/whitespace/nowrap-clear-float-expected.png: Added.
  • platform/mac/fast/text/whitespace/nowrap-clear-float-expected.txt: Added.
5:35 PM Changeset in webkit [30529] by weinig@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Oliver Hunt.

Fix for ​http://bugs.webkit.org/show_bug.cgi?id=17504
Speed up DOM lists array subscription syntax by using the fast
getOwnPropertySlot and set paths

  • 6x speed improvement on Oliver's ImageData put test.
  • bindings/scripts/CodeGeneratorJS.pm: Add fast path for getOwnPropertySlot and put when an indexGetter or indexSetter is used. We can not use the fast path if an overridingNameGetter is used as there would be no way to override.
5:33 PM Changeset in webkit [30528] by kevino@webkit.org
  • 2 edits in trunk/WebCore

wx build fix after JSImageData.cpp was added.

1:52 PM Changeset in webkit [30527] by ap@webkit.org
  • 4 edits
    1 delete in trunk

Windows build fix - move ThreadCondition implementation from WebCore to WTF.

1:04 PM BuildingGtk edited by lethalman88@gmail.com
(diff)
12:49 PM Changeset in webkit [30526] by ap@webkit.org
  • 6 edits in trunk

Build fix.

JavaScriptCore:

Touch some files, hoping that Windows build bot will create JSC headers.

  • kjs/AllInOneFile.cpp:
  • kjs/array_instance.cpp:
  • wtf/HashTable.cpp:

WebKit/gtk:

  • webkit/webkitprivate.cpp: (webkit_init): Renamed initializeThreading to initializeThreadingAndMainThread.
12:29 PM Changeset in webkit [30525] by ap@webkit.org
  • 2 edits in trunk/JavaScriptCore

Qt/Wx build fix - this file was still in a wrong namespace, too.

  • wtf/ThreadingNone.cpp:
12:21 PM Changeset in webkit [30524] by ap@webkit.org
  • 2 edits in trunk/JavaScriptCore

More build fixing - fix mismatched braces.

12:10 PM Changeset in webkit [30523] by ap@webkit.org
  • 3 edits in trunk/JavaScriptCore

Wx and Gtk build fixes.

  • JavaScriptCore.pri: Don't try to compile ThreadingPthreads.
  • wtf/ThreadingGtk.cpp: Use a correct namespace.
11:50 AM Changeset in webkit [30522] by ap@webkit.org
  • 37 edits
    4 copies
    9 moves
    5 adds
    2 deletes in trunk

2008-02-23 Alexey Proskuryakov <​ap@webkit.org>

Reviewed by Darin.

Move basic threading support from WebCore to WTF.

10:54 AM Changeset in webkit [30521] by ddkilzer@apple.com
  • 1 edit in trunk/WebCore/bindings/js/JSCSSStyleDeclarationCustom.cpp

Fix typo in comment.

9:08 AM Changeset in webkit [30520] by ddkilzer@apple.com
  • 14 edits
    1 add in trunk

WebCore:

Please clarify licensing for some files
<​http://bugs.webkit.org/show_bug.cgi?id=14970>

Reviewed by Darin.

  • bindings/objc/WebScriptObject.h: Added Apple BSD-style license.
  • bindings/objc/WebScriptObjectPrivate.h: Ditto.
  • platform/text/mac/ShapeArabic.c: Added ICU license from WebCore/icu/LICENSE.

WebKit/mac:

Please clarify licensing for some files
<​http://bugs.webkit.org/show_bug.cgi?id=14970>

Reviewed by Darin.

  • Plugins/WebNetscapeDeprecatedFunctions.c: Updated copyright statement and added Apple BSD-style license.
  • Plugins/WebNetscapeDeprecatedFunctions.h: Ditto.

WebKitTools:

Please clarify licensing for some files
<​http://bugs.webkit.org/show_bug.cgi?id=14970>

Reviewed by Darin.

  • DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp: Added copyright statement. Replaced license with newer Apple BSD-style license.
  • DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.h: Ditto.
  • DumpRenderTree/TestNetscapePlugIn.subproj/TestObject.cpp: Ditto.
  • DumpRenderTree/TestNetscapePlugIn.subproj/TestObject.h: Ditto.
  • DumpRenderTree/TestNetscapePlugIn.subproj/main.cpp: Ditto.
  • DumpRenderTree/win/TestNetscapePlugin/main.c: Ditto.
  • mangleme/LICENSE: Added (LGPL).
12:09 AM Changeset in webkit [30519] by mitz@apple.com
  • 2 edits in trunk/LayoutTests
  • undo an accidental change to a test's expected results from r30431.
  • fast/js/date-DST-time-cusps-expected.txt:
12:03 AM Changeset in webkit [30518] by mitz@apple.com
  • 2 edits in trunk/LayoutTests

Rubber-stamped by Sam Weinig and Anders Carlsson.

  • updated results for <rdar://problem/5714333> Add document.getSelection()
  • fast/dom/Window/window-properties-expected.txt:

Feb 22, 2008:

11:52 PM Changeset in webkit [30517] by ap@webkit.org
  • 8 edits in trunk/LayoutTests

Reviewed by Darin.

A few tweaks to PCRE-based regex test suite.

  • fast/regex/pcre-test-runner.js: Added support for global regexes. Emulate PCRE test runner more precisely by trimming trailing whitespace from some lines.
  • fast/regex/test1-expected.txt:
  • fast/regex/test4-expected.txt:
  • fast/regex/testinput1:
  • fast/regex/testinput4:
  • fast/regex/testoutput1:
  • fast/regex/testoutput4: Updated to PCRE 7.6.
11:46 PM Changeset in webkit [30516] by mitz@apple.com
  • 30 edits in trunk/LayoutTests
  • update Tiger expected results following r30431
  • platform/mac-tiger/svg/W3C-SVG-1.1/text-tselect-02-f-expected.txt:
  • platform/mac-tiger/svg/custom/foreignObject-crash-on-hover-expected.txt:
  • platform/mac-tiger/svg/custom/text-dom-01-f-expected.txt:
  • platform/mac-tiger/svg/custom/text-hit-test-expected.txt:
  • platform/mac-tiger/svg/text/text-align-01-b-expected.txt:
  • platform/mac-tiger/svg/text/text-align-02-b-expected.txt:
  • platform/mac-tiger/svg/text/text-align-03-b-expected.txt:
  • platform/mac-tiger/svg/text/text-align-04-b-expected.txt:
  • platform/mac-tiger/svg/text/text-align-05-b-expected.txt:
  • platform/mac-tiger/svg/text/text-align-06-b-expected.txt:
  • platform/mac-tiger/svg/text/text-altglyph-01-b-expected.txt:
  • platform/mac-tiger/svg/text/text-deco-01-b-expected.txt:
  • platform/mac-tiger/svg/text/text-fonts-01-t-expected.txt:
  • platform/mac-tiger/svg/text/text-fonts-02-t-expected.txt:
  • platform/mac-tiger/svg/text/text-path-01-b-expected.txt:
  • platform/mac-tiger/svg/text/text-spacing-01-b-expected.txt:
  • platform/mac-tiger/svg/text/text-text-01-b-expected.txt:
  • platform/mac-tiger/svg/text/text-text-03-b-expected.txt:
  • platform/mac-tiger/svg/text/text-text-04-t-expected.txt:
  • platform/mac-tiger/svg/text/text-text-05-t-expected.txt:
  • platform/mac-tiger/svg/text/text-text-06-t-expected.txt:
  • platform/mac-tiger/svg/text/text-text-07-t-expected.txt:
  • platform/mac-tiger/svg/text/text-text-08-b-expected.txt:
  • platform/mac-tiger/svg/text/text-tref-01-b-expected.txt:
  • platform/mac-tiger/svg/text/text-tselect-01-b-expected.txt:
  • platform/mac-tiger/svg/text/text-tselect-02-f-expected.txt:
  • platform/mac-tiger/svg/text/text-tspan-01-b-expected.txt:
  • platform/mac-tiger/svg/text/text-ws-01-t-expected.txt:
  • platform/mac-tiger/svg/text/text-ws-02-t-expected.txt:
9:14 PM Changeset in webkit [30515] by justin.garcia@apple.com
  • 5 edits
    2 adds in trunk

WebCore:

Reviewed by Anders Carlsson.

<rdar://problem/5714333> Add document.getSelection()


Add a synonym for window.getSelection() that FF supports. There were requests in
this bugs dupes to also support the IE only document.selection, but IE's Selection
object is too different than ours currently for that to be safe (specifically, we
don't support enough of IE's TextRange methods on our DOM Range object yet). I
filed <rdar://problem/5761233> to cover that.

  • dom/Document.cpp: (WebCore::Document::getSelection): Added.
  • dom/Document.h:
  • dom/Document.idl: Ditto. JS only since this is about web compatibility.

LayoutTests:

Reviewed by Anders.


<rdar://problem/5714333> Add document.getSelection()

  • editing/selection/5714333-expected.txt: Added.
  • editing/selection/5714333.html: Added.
9:09 PM Changeset in webkit [30514] by oliver@apple.com
  • 2 edits in trunk/WebCore

WX build fixes

8:18 PM Changeset in webkit [30513] by mrowe@apple.com
  • 7 edits in trunk/WebCore

Roll out r30500 as it breaks non-Windows use of Cairo.

7:31 PM Changeset in webkit [30512] by oliver@apple.com
  • 4 edits in trunk/WebCore

Once more with the cairo/qt/wx build fixing

7:09 PM Changeset in webkit [30511] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Windows build fix. Add missing include of DeprecatedCString.h and sort existing includes.

7:03 PM Changeset in webkit [30510] by mrowe@apple.com
  • 3 edits in trunk/WebCore

Speculative Windows build fix. Add missing include of DeprecatedCString.h and sort existing includes.

6:15 PM Changeset in webkit [30509] by oliver@apple.com
  • 2 edits in trunk/WebCore

Build fix for windows, wx and qt

6:04 PM Changeset in webkit [30508] by oliver@apple.com
  • 2 edits in trunk/WebCore

Reviewed by NOBODY (build fix).

Unbreak windows vcproj

  • WebCore.vcproj/WebCore.vcproj:
5:27 PM Changeset in webkit [30507] by weinig@apple.com
  • 2 edits in trunk/WebKit/qt

Fix Qt build.

  • Api/qwebhistoryinterface.cpp:
5:16 PM Changeset in webkit [30506] by oliver@apple.com
  • 19 edits
    12 adds in trunk

Bug 17486: Support HTML5 Canvas.getImageData API
Support Canvas.getImageData and Canvas.createImageData

Reviewed by Sam W.

This patch adds support for all the pixel reading portions
of the HTML5 Canvas spec. There are two new types ImageData
and CanvasPixelArray which are used to provide the HTML5
ImageData object, and the required semantics for assignment
to the ImageData data array.

We only implement the CG version of ImageBuffer::getImageData,
but the logic is null safe, so this will not introduce any
crashes into other platforms, unfortunately it will result in
JS Object detection "lying" on non-CG platforms.

5:14 PM Changeset in webkit [30505] by mrowe@apple.com
  • 4 edits in tags/Safari-5525.11.1

Versioning.

5:14 PM Changeset in webkit [30504] by mrowe@apple.com
  • 2 edits in tags/Safari-5525.11.1/WebCore

Merge r30502.

5:07 PM Changeset in webkit [30503] by mrowe@apple.com
  • 1 copy in tags/Safari-5525.11.1

New tag.

5:06 PM Changeset in webkit [30502] by mrowe@apple.com
  • 2 edits in branches/Safari-3-1-branch/WebCore

2008-02-22 Mark Rowe <​mrowe@apple.com>

Build fix.

  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::send): Qualify references to symbols in the KJS namespace.
4:55 PM Changeset in webkit [30501] by weinig@apple.com
  • 32 edits
    4 deletes in trunk

WebCore:

Rubber-stamped by Adam Roben.

Rid the project of the Devil known as DeprecatedString!

  • GNUmakefile.am:
  • WebCore.base.exp:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • dom/Node.h:
  • html/HTMLFormElement.cpp:
  • platform/DeprecatedString.cpp: Removed.
  • platform/DeprecatedString.h: Removed.
  • platform/graphics/qt/IconQt.cpp:
  • platform/graphics/win/MediaPlayerPrivateQuickTimeWin.cpp:
  • platform/gtk/CookieJarGtk.cpp:
  • platform/gtk/CursorGtk.cpp:
  • platform/gtk/KeyEventGtk.cpp:
  • platform/mac/DeprecatedStringMac.mm: Removed.
  • platform/network/qt/ResourceHandleQt.cpp:
  • platform/network/win/CookieJarWin.cpp: (WebCore::cookies):
  • platform/qt/ClipboardQt.cpp:
  • platform/qt/CookieJarQt.cpp:
  • platform/qt/CursorQt.cpp:
  • platform/symbian/DeprecatedStringSymbian.cpp: Removed.
  • platform/text/AtomicString.cpp:
  • platform/text/AtomicString.h:
  • platform/text/PlatformString.h:
  • platform/text/String.cpp:
  • platform/text/StringImpl.cpp:
  • platform/text/qt/StringQt.cpp: (WebCore::String::String):
  • platform/text/wx/StringWx.cpp:
  • platform/win/BString.cpp:
  • platform/win/BString.h:
  • platform/win/PasteboardWin.cpp:

WebKit/qt:

Rubber-stamped by Adam Roben.

Rid the project of the Devil known as DeprecatedString!

  • Api/qwebhistory.cpp:
  • Api/qwebhistoryinterface.cpp:
4:23 PM Changeset in webkit [30500] by pewtermoose@webkit.org
  • 7 edits in trunk/WebCore

2008-02-22 Brent Fulgham <​bfulgham@gmail.com>

Reviewed by Adam Roben.

​http://bugs.webkit.org/show_bug.cgi?id=17442
Correct the Windows Cairo implementation of GraphicsContext so
that Windows 'WorldTransform' is kept in sync with the Cairo
transformations.

Also corrects an uninitialized variable in the Cairo Windows
image drag handler.

  • platform/graphics/cairo/GraphicsContextCairo.cpp: Modify methods to call GraphicsContextPlatformPrivate calls, just as is done for the CoreGraphics version. These calls are nop's for everything but Windows. (WebCore::GraphicsContext::savePlatformState): (WebCore::GraphicsContext::restorePlatformState): (WebCore::GraphicsContext::clip): (WebCore::GraphicsContext::drawLineForMisspellingOrBadGrammar): (WebCore::GraphicsContext::translate): (WebCore::GraphicsContext::concatCTM): Re-enable for Windows (WebCore::GraphicsContext::beginTransparencyLayer): (WebCore::GraphicsContext::endTransparencyLayer): (WebCore::GraphicsContext::rotate): (WebCore::GraphicsContext::scale):
  • platform/graphics/cairo/GraphicsContextPlatformPrivateCairo.h:
  • platform/graphics/win/GraphicsContextCGWin.cpp: Move common code for handling WorldTransform calls to parent class.
  • platform/graphics/win/GraphicsContextCairoWin.cpp: Add common code for handling WorldTransform calls. (WebCore::GraphicsContextPlatformPrivate::concatCTM): Change implementation so that it only handles WorldTransform. The Cairo update is done in GraphicsContextCairo.cpp
  • platform/graphics/win/GraphicsContextWin.cpp: Call platform-private methods to keep WorldTransform in sync. (WebCore::GraphicsContextPlatformPrivate::save): (WebCore::GraphicsContextPlatformPrivate::restore): (WebCore::GraphicsContextPlatformPrivate::clip): (WebCore::GraphicsContextPlatformPrivate::scale): (WebCore::GraphicsContextPlatformPrivate::rotate): (WebCore::GraphicsContextPlatformPrivate::translate):
  • platform/win/DragImageCairoWin.cpp: (WebCore::createDragImageFromImage): Correct uninitialized value
4:16 PM Changeset in webkit [30499] by weinig@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Geoff Garen.

  • Remove use of DeprecatedString in CSSStyleSelector.
  • css/CSSStyleSelector.cpp: (WebCore::findHash): Removed. Use find instead. (WebCore::findSlashDotDotSlash): Changed to take a UChar* and a length. (WebCore::findSlashSlash): Ditto. (WebCore::findSlashDotSlash): Ditto. (WebCore::containsColonSlashSlash): Ditto. (WebCore::cleanPath): Change to operate on a String. (WebCore::checkPseudoState): Changed to use a Vector as a buffer.
3:51 PM Changeset in webkit [30498] by mrowe@apple.com
  • 4 edits in branches/Safari-3-1-branch

Versioning.

3:50 PM Changeset in webkit [30497] by mrowe@apple.com
  • 1 copy in tags/Safari-5525.11

New tag.

3:49 PM Changeset in webkit [30496] by mrowe@apple.com
  • 4 edits in branches/Safari-3-1-branch

Versioning.

3:39 PM Changeset in webkit [30495] by mrowe@apple.com
  • 2 edits in branches/Safari-3-1-branch/WebCore

Merge r30493.

3:39 PM Changeset in webkit [30494] by mrowe@apple.com
  • 5 edits
    2 deletes in branches/Safari-3-1-branch

Merge r30492.

3:16 PM Changeset in webkit [30493] by ggaren@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Sam Weinig.


Fixed <rdar://problem/5057509> Repro leak of JSXMLHttpRequest and
associated objects @ www.viamichelin.it, which was probably an underlying
cause of <rdar://problem/5744037> Gmail out of memory (17455)


If SubresourceLoader::create returned NULL, we would ref() / gcProtect()
the XMLHttpRequest but think we hadn't, therefore never
calling deref() / gcUnprotect().


This could happen at gmail.com, since gmail.com attempts to send
XMLHttpRequests from unload handlers in order to gather usage statistics.
(According to comments in the code, SubresourceLoader::create returns
NULL when called from an unload handler.)

The solution is to ref() / gcProtect() only if SubresourceLoader::create
doesn't return NULL. This make sense, since we only need to protect the
request as long as it has an outstanding network transaction.


  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::XMLHttpRequest): (WebCore::XMLHttpRequest::send):
3:16 PM Changeset in webkit [30492] by ggaren@apple.com
  • 5 edits
    2 deletes in trunk

JavaScriptCore:

Reviewed by Sam Weinig.


Partial fix for <rdar://problem/5744037> Gmail out of memory (17455)


I'm removing KJS_MEM_LIMIT for the following reasons:


  • We have a few reports of KJS_MEM_LIMIT breaking important web applications, like GMail and Google Reader. (For example, if you simply open 12 GMail tabs, tab #12 will hit the limit.)
  • Firefox has no discernable JS object count limit, so any limit, even a large one, is a potential compatibility problem.


  • KJS_MEM_LIMIT does not protect against malicious memory allocation, since there are many ways to maliciously allocate memory without increasing the JS object count.


  • KJS_MEM_LIMIT is already mostly broken, since it only aborts the script that breaches the limit, not any subsequent scripts.


  • We've never gotten bug reports about websites that would have benefited from an unbroken KJS_MEM_LIMIT. The initial check-in of KJS_MEM_LIMIT (KJS revision 80061) doesn't mention a website that needed it.


  • Any website that brings you anywhere close to crashing due to the number of live JS objects will almost certainly put up the "slow script" dialog at least 20 times beforehand.
  • kjs/collector.cpp: (KJS::Collector::collect):
  • kjs/collector.h:
  • kjs/nodes.cpp: (KJS::TryNode::execute):

LayoutTests:

Reviewed by Sam Weinig.


Removing the test for KJS_MEM_LIMIT, since I removed KJS_MEM_LIMIT.

  • fast/js/out-of-memory-expected.txt: Removed.
  • fast/js/out-of-memory.html: Removed.
3:02 PM Changeset in webkit [30491] by mrowe@apple.com
  • 2 edits in branches/Safari-3-1-branch/WebCore

Merge r30489.

3:00 PM BuildingCairoOnWindows edited by bfulgham@macports.org
(diff)
2:30 PM Changeset in webkit [30490] by weinig@apple.com
  • 7 edits in trunk/WebCore

Reviewed, tweaked and landed by Sam.

  • Make RegularExpression operate on Strings instead of DeprecatedStrings.
  • dom/DOMImplementation.cpp: (WebCore::DOMImplementation::isXMLMIMEType): Use string instead of DeprecatedString to build up the RegularExpression.
  • page/Frame.cpp: (WebCore::createRegExpForLabels): Use String instead of DeprecatedString. (WebCore::Frame::searchForLabelsAboveCell): Ditto. (WebCore::Frame::searchForLabelsBeforeElement): Ditto. (WebCore::Frame::matchLabelsAgainstElement): Ditto.
  • page/Frame.h:
  • page/mac/FrameMac.mm: (WebCore::regExpForLabels): Ditto. (WebCore::Frame::searchForNSLabelsAboveCell): Ditto. (WebCore::Frame::searchForLabelsBeforeElement): Ditto. (WebCore::Frame::matchLabelsAgainstElement): Ditto.
  • platform/text/RegularExpression.cpp: (WebCore::RegularExpression::Private::Private): (WebCore::RegularExpression::Private::compile): (WebCore::RegularExpression::RegularExpression): (WebCore::RegularExpression::operator=): (WebCore::RegularExpression::pattern): (WebCore::RegularExpression::match): (WebCore::RegularExpression::search): (WebCore::RegularExpression::searchRev): (WebCore::replace): Added.
  • platform/text/RegularExpression.h: Change functions to take Strings as input instead of DeprecatedStrings and reduce the complexity of the class by removing unneeded globbing support.
2:15 PM Changeset in webkit [30489] by jhoneycutt@apple.com
  • 2 edits in trunk/WebCore

2008-02-22 Jon Honeycutt <​jhoneycutt@apple.com>

Reviewed by Anders.

<rdar://problem/5760360> REGRESSION(r30376): Crash loading plugin page
during stress test (after only 5 min) - null dereference

Full-frame plug-ins create PluginStream objects without loaders, as the
PluginView receives the loading callbacks. We were trying to call
setDefersLoading on these null pointers.

  • plugins/PluginStream.cpp: (WebCore::PluginStream::startStream): Add null check. (WebCore::PluginStream::destroyStream): Same. (WebCore::PluginStream::deliverData): Same.
2:07 PM BuildingCairoOnWindows edited by bfulgham@macports.org
(diff)
1:39 PM Changeset in webkit [30488] by weinig@apple.com
  • 3 edits in trunk/WebCore

Reviewed and tweaked by Jon Honeycutt. Reviewed and landed by Sam.

  • Remove uses of DeprecatedString in Windows plugin code.
  • plugins/win/PluginPackageWin.cpp: (WebCore::getVersionInfo): Cleanup formatting. (WebCore::PluginPackage::freeLibraryTimerFired): Remove un-needed variable name. (WebCore::PluginPackage::storeFileVersion): Move casts. (WebCore::PluginPackage::fetchInfo): Use OwnArrayPtr and switch to more efficient use of Vectors. (WebCore::PluginPackage::load): Fix whitespace. (WebCore::PluginPackage::hash): Make the hashCodes const
  • plugins/win/PluginViewWin.cpp: (WebCore::makeURL): Use String instead of DeprecatedString. (WebCore::parseRFC822HeaderFields): Ditto. (WebCore::PluginView::handlePost): Ditto. (WebCore::PluginView::status): Ditto.
1:29 PM Changeset in webkit [30487] by timothy@apple.com
  • 32 edits in branches/Safari-3-1-branch

Merge r30472.

1:16 PM Changeset in webkit [30486] by weinig@apple.com
  • 3 edits in trunk/WebCore

Reviewed, tweaked and landed by Sam.

  • Don't use DeprecatedString in HTMLTokenizer.
  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::finish):
  • platform/text/PlatformString.h: (WebCore::find):
12:58 PM Changeset in webkit [30485] by Adam Roben
  • 5 edits in trunk/WebKit/win

Move hasWebView to WebFrameLoaderClient

I added an m_webFrame member to WebFrameLoaderClient. This is slightly
strange because WebFrame inherits from WebFrameLoaderClient, but this
member will be needed once we remove the inheritance, so we might as
well prepare for that now.

Reviewed by Anders.

  • WebCoreSupport/WebFrameLoaderClient.cpp: (WebFrameLoaderClient::WebFrameLoaderClient): Changed to take a WebFrame* parameter. (WebFrameLoaderClient::hasWebView): Moved here from WebFrame.cpp.
  • WebCoreSupport/WebFrameLoaderClient.h: Added an m_webFrame parameter.
  • WebFrame.cpp: Removed hasWebView. (WebFrame::WebFrame): Updated for WebFrameLoaderClient constructor change.
  • WebFrame.h:
12:57 PM BuildingCairoOnWindows edited by bfulgham@macports.org
(diff)
12:21 PM Changeset in webkit [30484] by timothy@apple.com
  • 2 edits in branches/Safari-3-1-branch/WebKitTools

Merge r30480.

12:16 PM Changeset in webkit [30483] by timothy@apple.com
  • 5 edits
    4 deletes in branches/Safari-3-1-branch

Roll out r29649.

12:05 PM Changeset in webkit [30482] by andersca@apple.com
  • 2 edits in trunk/WebKitTools

Reviewed by Adam.

<rdar://problem/5748781>
​https://bugs.webkit.org/show_bug.cgi?id=17413
REGRESSION: Latest Nightly doesn't load Java plugin w/Safari 3.1b


Copy the Java plug-in over to the new location.


  • FindSafari/FindSafari.cpp: (_tmain):
11:53 AM Changeset in webkit [30481] by Adam Roben
  • 4 edits
    2 adds in trunk/WebKit/win

Start to move FrameLoaderClient methods off WebFrame onto a separate class

WebFrame now inherits from a new WebFrameLoaderClient class, which
will gradually assume all FrameLoaderClient responsibilities. Once
that process is complete, WebFrame will no longer inherit from
WebFrameLoaderClient.

In this first patch, I've only moved createDocumentLoader up to the
WebFrameLoaderClient class.

Reviewed by Anders.

  • WebCoreSupport/WebFrameLoaderClient.cpp: Added. (WebFrameLoaderClient::WebFrameLoaderClient): (WebFrameLoaderClient::~WebFrameLoaderClient): (WebFrameLoaderClient::createDocumentLoader): Moved here from WebFrame.cpp.
  • WebCoreSupport/WebFrameLoaderClient.h: Added.
  • WebFrame.cpp: Removed createDocumentLoader implementation.
  • WebFrame.h: Changed to inherit from WebFrameLoaderClient.
  • WebKit.vcproj/WebKit.vcproj: Added new files to the project.
11:51 AM Changeset in webkit [30480] by andersca@apple.com
  • 2 edits in trunk/WebKitTools

Reviewed by Adam.

<rdar://problem/5747325> REGRESSION: HTTP layout tests hang


  • DumpRenderTree/win/DumpRenderTree.cpp: (runTest): Init the URL request with the correct timeout.
11:07 AM Changeset in webkit [30479] by Adam Roben
  • 3 edits
    2 adds in trunk/WebKit/win

Move FormValuesPropertyBag into its own files

Reviewed by Sam.

  • WebCoreSupport/FormValuesPropertyBag.cpp: Added. (FormValuesPropertyBag::QueryInterface): (FormValuesPropertyBag::AddRef): (FormValuesPropertyBag::Release): (FormValuesPropertyBag::Read): (FormValuesPropertyBag::Write): (FormValuesPropertyBag::CountProperties): (FormValuesPropertyBag::GetPropertyInfo): (FormValuesPropertyBag::LoadObject):
  • WebCoreSupport/FormValuesPropertyBag.h: Added. (FormValuesPropertyBag::FormValuesPropertyBag):
  • WebFrame.cpp: Deleted FormValuesPropertyBag code.
  • WebKit.vcproj/WebKit.vcproj: Added new files to the project.
11:07 AM Changeset in webkit [30478] by Adam Roben
  • 3 edits in trunk/WebKit/win

Remove some unused WebFrame methods

FrameWinClient was deleted back in r22965, but these methods were
never deleted.

Reviewed by Sam.

  • WebFrame.cpp:
  • WebFrame.h: Made one createFrame overload protected, since it's only called internally by WebFrame.
11:06 AM Changeset in webkit [30477] by Adam Roben
  • 2 edits
    10 moves
    1 add in trunk/WebKit/win

Move Client implementations into a WebCoreSupport directory

Reviewed by Anders.

  • WebKit.vcproj/WebKit.vcproj: Updated file paths, and added WebCoreSupport directory to the include path.
  • WebCoreSupport/WebChromeClient.cpp: Renamed from WebKit/win/WebChromeClient.cpp.
  • WebCoreSupport/WebChromeClient.h: Renamed from WebKit/win/WebChromeClient.h.
  • WebCoreSupport/WebContextMenuClient.cpp: Renamed from WebKit/win/WebContextMenuClient.cpp.
  • WebCoreSupport/WebContextMenuClient.h: Renamed from WebKit/win/WebContextMenuClient.h.
  • WebCoreSupport/WebDragClient.cpp: Renamed from WebKit/win/WebDragClient.cpp.
  • WebCoreSupport/WebDragClient.h: Renamed from WebKit/win/WebDragClient.h.
  • WebCoreSupport/WebEditorClient.cpp: Renamed from WebKit/win/WebEditorClient.cpp.
  • WebCoreSupport/WebEditorClient.h: Renamed from WebKit/win/WebEditorClient.h.
  • WebCoreSupport/WebInspectorClient.cpp: Renamed from WebKit/win/WebInspectorClient.cpp.
  • WebCoreSupport/WebInspectorClient.h: Renamed from WebKit/win/WebInspectorClient.h.
10:32 AM Changeset in webkit [30476] by sullivan@apple.com
  • 4 edits in trunk/WebKit/mac

Reviewed by Adam Roben


Reverted the changed from yesterday to add pasteAndMatchStyle:, as the existing
pasteAsPlainText: has the same behavior.

  • WebView/WebHTMLView.mm: (-[WebHTMLView _pasteWithPasteboard:allowPlainText:]): (-[WebHTMLView readSelectionFromPasteboard:]): (-[WebHTMLView validateUserInterfaceItemWithoutDelegate:]): (-[WebHTMLView pasteAsRichText:]): (-[WebHTMLView paste:]):
  • WebView/WebView.mm:
  • WebView/WebViewPrivate.h:
2:27 AM Changeset in webkit [30475] by oliver@apple.com
  • 5 edits in trunk

<rdar://problem/5759327> REGRESSION: while(NaN) acts like while(true)

Reviewed by Alexey P.

Fix yet another case where we incorrectly relied on implicit double
to bool coercion.

Feb 21, 2008:

11:04 PM Changeset in webkit [30474] by hyatt@apple.com
  • 1 edit in trunk/WebCore/css/MediaQueryEvaluator.cpp

Add null check to fix crash in media query code I checked in.

10:10 PM Changeset in webkit [30473] by alp@webkit.org
  • 2 edits in trunk

2008-02-21 Alp Toker <​alp@atoker.com>

Fix a configure script typo.

  • configure.ac:
7:35 PM Changeset in webkit [30472] by weinig@apple.com
  • 32 edits in trunk

WebCore:

Reviewed by Anders Carlsson.

Fix for <rdar://problem/5757946>

  • Parse URLs before checking whether they are javascript: urls (which require security checks).
  • bindings/js/JSAttrCustom.cpp: (WebCore::JSAttr::setValue):
  • bindings/js/JSElementCustom.cpp: (WebCore::allowSettingSrcToJavascriptURL):
  • bindings/js/JSHTMLFrameElementCustom.cpp: (WebCore::allowSettingJavascriptURL):
  • bindings/js/JSHTMLIFrameElementCustom.cpp: (WebCore::JSHTMLIFrameElement::setSrc):

LayoutTests:

Reviewed by Anders Carlsson.

Update tests and results for <rdar://problem/5757946>

  • http/tests/security/javascriptURL/javascriptURL-execution-context-frame-location-htmldom-expected.txt:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-frame-location-htmldom.html:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-frame-src-getAttribute-value-expected.txt:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-frame-src-getAttribute-value.html:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-frame-src-htmldom-expected.txt:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-frame-src-htmldom.html:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-frame-src-setAttribute-expected.txt:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-frame-src-setAttribute.html:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-frame-src-setAttributeNS-expected.txt:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-frame-src-setAttributeNS.html:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-frame-src-setAttributeNode-expected.txt:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-frame-src-setAttributeNode.html:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-frame-src-setAttributeNodeNS-expected.txt:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-frame-src-setAttributeNodeNS.html:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-iframe-src-getAttribute-value-expected.txt:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-iframe-src-getAttribute-value.html:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-iframe-src-htmldom-expected.txt:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-iframe-src-htmldom.html:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-iframe-src-setAttribute-expected.txt:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-iframe-src-setAttribute.html:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-iframe-src-setAttributeNS-expected.txt:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-iframe-src-setAttributeNS.html:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-iframe-src-setAttributeNode-expected.txt:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-iframe-src-setAttributeNode.html:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-iframe-src-setAttributeNodeNS-expected.txt:
  • http/tests/security/javascriptURL/javascriptURL-execution-context-iframe-src-setAttributeNodeNS.html:
6:21 PM Changeset in webkit [30471] by mrowe@apple.com
  • 3 edits in branches/Safari-3-1-branch/WebCore

Merge r30461.

6:21 PM Changeset in webkit [30470] by mrowe@apple.com
  • 3 edits
    2 adds in branches/Safari-3-1-branch

Merge r30458.

6:21 PM Changeset in webkit [30469] by mrowe@apple.com
  • 9 edits in branches/Safari-3-1-branch

Merge r30433.

5:07 PM Changeset in webkit [30468] by adachan@apple.com
  • 3 edits
    2 adds in trunk

<rdar://problem/5757873> Buffer overrun in DeprecatedCString::find() in WebCore
We could get a buffer overrun in DeprecatedCString::find() if the end of the
string matches a beginning portion of the substring, for example, if string is
"a" but the substring is "ab".
The code as is also will not match things correctly under certain situations
since the inner while loop increments the index. For example, we wouldn't find
a match if the string is "aab..." and the substring is "ab". Changed the
inner while loop to increment a temporary index into str.


Test: fast/loader/charset-parse.html

Reviewed by Dan Berstein.

  • platform/DeprecatedCString.cpp: (WebCore::DeprecatedCString::find):
4:15 PM Changeset in webkit [30467] by hyatt@apple.com
  • 5 adds in trunk/LayoutTests

Add test case for viewport media queries.

4:14 PM Changeset in webkit [30466] by hyatt@apple.com
  • 9 edits in trunk/WebCore

Fix for bug 17301. CSS media queries need to use the correct viewport
when contained in documents inside iframes (rather than always using the
top-level document's viewport). CSS media queries based on the viewport
also needed to be dynamic and update as you resize the window (this is
a HOT feature). :)

This patch gets Acid3 up to 86/100 with 3 colored boxes filled in.

Reviewed by olliej

Added fast/media/viewport-media-query.html

  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::CSSStyleSelector): (WebCore::CSSStyleSelector::addViewportDependentMediaQueryResult): (WebCore::CSSStyleSelector::affectedByViewportChange):
  • css/CSSStyleSelector.h: (WebCore::MediaQueryResult::MediaQueryResult):
  • css/MediaQueryEvaluator.cpp: (WebCore::MediaQueryEvaluator::MediaQueryEvaluator): (WebCore::MediaQueryEvaluator): (WebCore::MediaQueryEvaluator::eval): (WebCore::colorMediaFeatureEval): (WebCore::monochromeMediaFeatureEval): (WebCore::device_aspect_ratioMediaFeatureEval): (WebCore::device_pixel_ratioMediaFeatureEval): (WebCore::gridMediaFeatureEval): (WebCore::device_heightMediaFeatureEval): (WebCore::device_widthMediaFeatureEval): (WebCore::heightMediaFeatureEval): (WebCore::widthMediaFeatureEval): (WebCore::min_colorMediaFeatureEval): (WebCore::max_colorMediaFeatureEval): (WebCore::min_monochromeMediaFeatureEval): (WebCore::max_monochromeMediaFeatureEval): (WebCore::min_device_aspect_ratioMediaFeatureEval): (WebCore::max_device_aspect_ratioMediaFeatureEval): (WebCore::min_device_pixel_ratioMediaFeatureEval): (WebCore::max_device_pixel_ratioMediaFeatureEval): (WebCore::min_heightMediaFeatureEval): (WebCore::max_heightMediaFeatureEval): (WebCore::min_widthMediaFeatureEval): (WebCore::max_widthMediaFeatureEval): (WebCore::min_device_heightMediaFeatureEval): (WebCore::max_device_heightMediaFeatureEval): (WebCore::min_device_widthMediaFeatureEval): (WebCore::max_device_widthMediaFeatureEval):
  • css/MediaQueryEvaluator.h:
  • css/MediaQueryExp.cpp: (WebCore::MediaQueryExp::~MediaQueryExp):
  • css/MediaQueryExp.h: (WebCore::MediaQueryExp::value): (WebCore::MediaQueryExp::isViewportDependent):
  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::pickMedia):
  • page/FrameView.cpp: (WebCore::FrameView::layout):
1:23 PM Changeset in webkit [30465] by andersca@apple.com
  • 10 edits in trunk

WebCore:

Reviewed by Sam.

Make more classes start out with a refcount of 1.

  • dom/QualifiedName.cpp: (WebCore::QNameComponentsTranslator::translate): (WebCore::QualifiedName::QualifiedName):
  • dom/QualifiedName.h: (WebCore::QualifiedName::QualifiedNameImpl::create): (WebCore::QualifiedName::QualifiedNameImpl::QualifiedNameImpl):
  • history/BackForwardList.cpp: (WebCore::BackForwardList::BackForwardList):
  • history/BackForwardList.h: (WebCore::BackForwardList::create):
  • page/Page.cpp: (WebCore::Page::Page):
  • platform/text/CString.cpp: (WebCore::CString::init): (WebCore::CString::newUninitialized): (WebCore::CString::copyBufferIfNeeded):
  • platform/text/CString.h: (WebCore::CStringBuffer::create): (WebCore::CStringBuffer::CStringBuffer):

WebKit/mac:

Reviewed by Sam.

Use BackForwardList::create instead.


  • History/WebBackForwardList.mm: (-[WebBackForwardList init]):
1:21 PM Changeset in webkit [30464] by kevino@webkit.org
  • 2 edits in trunk/WebCore

wx build fix for the domString() -> string() rename in r30443.

12:35 PM Changeset in webkit [30463] by sullivan@apple.com
  • 4 edits in trunk/WebKit/mac

Reviewed by Jessica Kahn


support for pasteAndMatchStyle: command (see <rdar://problem/5723952>)

  • WebView/WebHTMLView.mm: (-[WebHTMLView _pasteWithPasteboard:allowPlainText:matchStyle:]): added matchStyle parameter, passed along to bridge (formerly always passed NO to bridge) (-[WebHTMLView readSelectionFromPasteboard:]): pass NO for new matchStyle parameter to match old behavior (-[WebHTMLView validateUserInterfaceItemWithoutDelegate:]): validate pasteAndMatchStyle the same way as pasteAsRichText (-[WebHTMLView pasteAndMatchStyle:]): just like pasteAsRichText but passes YES for matchStyle (-[WebHTMLView pasteAsRichText:]): pass NO for new matchStyle parameter to match old behavior (-[WebHTMLView paste:]): ditto


  • WebView/WebView.mm: added macro(pasteAndMatchStyle)


  • WebView/WebViewPrivate.h: added pasteAndMatchStyle: to WebViewEditingActionsPendingPublic category
12:16 PM Changeset in webkit [30462] by alp@webkit.org
  • 2 edits in trunk

2008-02-21 Alp Toker <​alp@atoker.com>

GTK+/autotools SVG experimental build fix

Don't enable SVG filters in --enable-svg-experimental.

This feature isn't supported at all yet. Developers can enable it
explicitly by passing --enable-svg-filters if needed.

  • configure.ac:
12:10 PM Changeset in webkit [30461] by Antti Koivisto
  • 3 edits in trunk/WebCore

Reviewed by Sam Weinig.

<rdar://problem/5753789>
REGRESSION: 1.5% -2% Sunspider regression from r30009 (ebay photo upload hang)


Ensure all versions of allowsAccessFrom are inlined to single functions.
This is a 2% win in browser hosted Sunspider.

  • bindings/js/kjs_window.cpp: (KJS::Window::allowsAccessFrom): (KJS::Window::allowsAccessFromPrivate):
  • bindings/js/kjs_window.h:
11:59 AM Changeset in webkit [30460] by mitz@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Sam Weinig.

  • minor cleanup
  • rendering/bidi.cpp: (WebCore::bidiNext): Removed redundant isBR() check -- isText() returns true for RenderLineBreak. (WebCore::bidiFirst): Ditto. (WebCore::shouldSkipWhitespaceAfterStartObject): (WebCore::RenderBlock::findNextLineBreak):
11:43 AM Changeset in webkit [30459] by alp@webkit.org
  • 4 edits in trunk

2008-02-21 Mike Auty <​mike.auty@gmail.com>

Reviewed by Alp Toker.

​http://bugs.webkit.org/show_bug.cgi?id=17445
[GTK] WebKit doesn't compile with LDFLAGS="-Wl,--as-needed"

The GNUmakefile.am files make use of the LDFLAGS variable to include library
additions such as -ljpeg etc. Unfortunately, if these inclusions aren't made
in LIBADD/LDADD variables, then they are mis-ordered during the linking.

The as-needed flag discards libraries whose functions have not been needed by
earlier libraries, which therefore makes the ordering important.

This moves all -l library inclusion statements from LDFLAGS variables to
LIBADD/LDADD variables.

  • GNUmakefile.am:
10:09 AM Changeset in webkit [30458] by ggaren@apple.com
  • 3 edits
    2 adds in trunk

WebCore:

Reviewed by David Harrison.


Fixed <rdar://problem/5756125> REGRESSION: A crash occurs at
WebCore::Frame::scriptProxy() when completing a search with Package Tracker widget

Test: fast/dom/script-element-without-frame-crash.html

  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::parseTag): Added back a NULL check that was accidentally removed in r30325.

LayoutTests:

Reviewed by David Harrison.


Test for <rdar://problem/5756125> REGRESSION: A crash occurs at
WebCore::Frame::scriptProxy() when completing a search with Package Tracker widget

  • fast/dom/script-element-without-frame-crash-expected.txt: Added.
  • fast/dom/script-element-without-frame-crash.html: Added.
9:30 AM Changeset in webkit [30457] by pewtermoose@webkit.org
  • 2 edits in trunk/WebCore

2008-02-21 Rodney Dawes <​dobey@wayofthemonkey.com>

GTK+ build fix. s/domString()/string()/

  • platform/graphics/gtk/FontPlatformDataGtk.cpp: (FontPlatformData::FontPlatformData):

Feb 20, 2008:

11:54 PM Changeset in webkit [30456] by weinig@apple.com
  • 2 edits in trunk/LayoutTests

Rubber-stamped by Dan Bernstein.

Gratuitous change to this test to use the instanceof operator
instead of the isPrototypeOf method.

  • fast/tokenizer/doctype-search-reset.html:
11:38 PM Changeset in webkit [30455] by mitz@apple.com
  • 3 edits
    2 adds in trunk

WebCore:

Reviewed by Dave Hyatt.

Test: fast/tokenizer/doctype-search-reset.html

  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::parseTag):

LayoutTests:

Reviewed by Dave Hyatt.

  • fast/tokenizer/doctype-search-reset-expected.txt: Added.
  • fast/tokenizer/doctype-search-reset.html: Added.
9:00 PM Changeset in webkit [30454] by mitz@apple.com
  • 3 edits
    4 adds in trunk

WebCore:

Reviewed by Dave Hyatt.

Test: fast/text/wbr-in-pre-crash.html

  • rendering/bidi.cpp: (WebCore::RenderBlock::findNextLineBreak):

LayoutTests:

Reviewed by Dave Hyatt.

  • fast/text/wbr-in-pre-crash.html: Added.
  • platform/mac-leopard/fast/text/wbr-in-pre-crash-expected.checksum: Added.
  • platform/mac-leopard/fast/text/wbr-in-pre-crash-expected.png: Added.
  • platform/mac/fast/text/wbr-in-pre-crash-expected.txt: Added.
8:50 PM Changeset in webkit [30453] by weinig@apple.com
  • 3 edits
    3 adds in trunk

JavaScriptCore:

Reviewed by Darin.

Fix for Bug 16753: date set methods with no args should result in NaN (Acid3 bug)
The set values result in NaN now when called with no args, NaN or +/- inf values.
The setYear, setFullYear and setUTCFullYear methods used on NaN dates work as
descripted in the standard.

  • kjs/date_object.cpp: (KJS::fillStructuresUsingTimeArgs): (KJS::fillStructuresUsingDateArgs): (KJS::setNewValueFromTimeArgs): (KJS::setNewValueFromDateArgs): (KJS::dateProtoFuncSetYear):

LayoutTests:

Reviewed by Darin.

  • test for Bug 16753: date set methods with no args should result in NaN (Acid3 bug)
  • fast/js/date-set-to-nan-expected.txt: Added.
  • fast/js/date-set-to-nan.html: Added.
  • fast/js/resources/date-set-to-nan.js: Added.
8:24 PM Changeset in webkit [30452] by oliver@apple.com
  • 3 edits
    2 adds in trunk

Bug 17303: Canvas crash in ImageBuffer

Reviewed by Hyatt.

We handle a null GraphicsContext everywhere, but we weren't checking for
a null ImageBuffer, which is what will result in a null GraphicsContext in
the first place.

8:06 PM Changeset in webkit [30451] by hyatt@apple.com
  • 1 edit in trunk/LayoutTests/platform/mac/fast/media/mq-js-stylesheet-media-03-expected.txt

Land new results for a layout test that now passes now that dynamic media lists work.

8:05 PM Changeset in webkit [30450] by hyatt@apple.com
  • 4 edits in trunk/WebCore

Fix the layout test failure that never should have passed in the first
place by making changes to media lists actually result in the recomputation
of the style selector. Now it passes for the right reasons and not because
of dumb luck.

Reviewed by Sam Weinig

  • css/MediaList.cpp: (WebCore::MediaList::deleteMedium): (WebCore::MediaList::setMediaText): (WebCore::MediaList::appendMedium): (WebCore::MediaList::notifyChanged):
  • css/MediaList.h:
  • dom/Document.cpp: (WebCore::Document::attach):
8:00 PM Changeset in webkit [30449] by weinig@apple.com
  • 2 edits in trunk/WebCore

Reviewed, tweaked and landed by Sam.

  • make markup functions not use DeprecatedString.
  • editing/markup.cpp: (WebCore::append): Added. (WebCore::escapeContentText): Build up string using a Vector. (WebCore::appendStartMarkup): Use String instead of DeprecatedString.
7:57 PM Changeset in webkit [30448] by weinig@apple.com
  • 1 edit in trunk/WebCore/ChangeLog

Fix changelog

7:54 PM Changeset in webkit [30447] by weinig@apple.com
  • 3 edits in trunk/WebCore

Reviewed, tweaked and landed by Sam.

  • make HTMLInterchange return a String instead of a DeprecatedString
  • editing/TextIterator.cpp: (WebCore::CharacterIterator::string): Build up the String using a Vector. (WebCore::WordAwareIterator::advance): Switch to using Vector functions. (WebCore::WordAwareIterator::length): Ditto. (WebCore::WordAwareIterator::characters): Ditto.
  • editing/TextIterator.h: Use a Vector<UChar> for the buffer instead of DeprecatedString.
7:42 PM Changeset in webkit [30446] by weinig@apple.com
  • 3 edits in trunk/WebCore

Reviewed, tweaked and landed by Sam.

  • make HTMLInterchange return a String instead of a DeprecatedString
  • editing/HTMLInterchange.cpp: (WebCore::): Return a String from convertedSpaceString. (WebCore::convertHTMLTextToInterchangeFormat): Use a Vector instead of a DeprecatedString to build up the return String.
  • editing/HTMLInterchange.h:
7:38 PM Changeset in webkit [30445] by eric@webkit.org
  • 3 edits in trunk/WebCore

Reviewed by Oliver.

Remove m_drawingContext and change m_data to m_imageBuffer

  • html/HTMLCanvasElement.cpp: (WebCore::HTMLCanvasElement::HTMLCanvasElement): (WebCore::HTMLCanvasElement::reset): (WebCore::HTMLCanvasElement::paint): (WebCore::HTMLCanvasElement::createImageBuffer): (WebCore::HTMLCanvasElement::buffer): (WebCore::HTMLCanvasElement::createPlatformImage):
  • html/HTMLCanvasElement.h:
6:26 PM Changeset in webkit [30444] by pewtermoose@webkit.org
  • 1 edit
    10 adds in trunk/WebKitTools

2008-02-20 Brent Fulgham <​bfulgham@gmail.com>

Reviewed by Alp.

This patch reenables the venerable Spinneret application,
changing its name to match the other lanch applications.

  • WinLauncher: Added.
  • WinLauncher/WinLauncher.cpp: Added. (WinLauncherWebHost::updateAddressBar): (WinLauncherWebHost::QueryInterface): (WinLauncherWebHost::AddRef): (WinLauncherWebHost::Release): (resizeSubViews): (_tWinMain): (MyRegisterClass): (InitInstance): (WndProc): (MyEditProc): (About): (loadURL):
  • WinLauncher/WinLauncher.h: Added. (WinLauncherWebHost::WinLauncherWebHost): (WinLauncherWebHost::didStartProvisionalLoadForFrame): (WinLauncherWebHost::didReceiveServerRedirectForProvisionalLoadForFrame): (WinLauncherWebHost::didFailProvisionalLoadWithError): (WinLauncherWebHost::didCommitLoadForFrame): (WinLauncherWebHost::didReceiveTitle): (WinLauncherWebHost::didReceiveIcon): (WinLauncherWebHost::didFinishLoadForFrame): (WinLauncherWebHost::didFailLoadWithError): (WinLauncherWebHost::didChangeLocationWithinPageForFrame): (WinLauncherWebHost::willPerformClientRedirectToURL): (WinLauncherWebHost::didCancelClientRedirectForFrame): (WinLauncherWebHost::willCloseFrame): (WinLauncherWebHost::windowScriptObjectAvailable):
  • WinLauncher/WinLauncher.ico: Added.
  • WinLauncher/WinLauncher.rc: Added.
  • WinLauncher/WinLauncher.vcproj: Added.
  • WinLauncher/resource.h: Added.
  • WinLauncher/small.ico: Added.
  • WinLauncher/stdafx.cpp: Added.
  • WinLauncher/stdafx.h: Added.
6:17 PM Changeset in webkit [30443] by andersca@apple.com
  • 28 edits in trunk/WebCore

Reviewed by Sam.

Rename AtomicString::domString() to AtomicString::string().


  • css/CSSComputedStyleDeclaration.cpp: (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
  • css/CSSFontSelector.cpp: (WebCore::CSSFontSelector::getFontData):
  • dom/Attr.cpp: (WebCore::Attr::createTextChild):
  • dom/Comment.cpp: (WebCore::Comment::nodeName):
  • dom/Document.cpp: (WebCore::Document::recalcStyleSelector): (WebCore::Document::setHTMLWindowEventListener): (WebCore::Document::formElementsState):
  • dom/StyledElement.cpp: (WebCore::StyledElement::parseMappedAttribute):
  • dom/Text.cpp: (WebCore::Text::nodeName):
  • editing/SelectionController.cpp: (WebCore::SelectionController::debugRenderer):
  • html/HTMLCollection.cpp: (WebCore::HTMLCollection::checkForNameMatch):
  • html/HTMLElement.cpp: (WebCore::HTMLElement::nodeName): (WebCore::HTMLElement::setHTMLEventListener):
  • html/HTMLFormCollection.cpp: (WebCore::HTMLFormCollection::getNamedFormItem):
  • html/HTMLImageElement.cpp: (WebCore::HTMLImageElement::parseMappedAttribute): (WebCore::HTMLImageElement::isURLAttribute):
  • html/HTMLLinkElement.cpp: (WebCore::HTMLLinkElement::parseMappedAttribute): (WebCore::HTMLLinkElement::tokenizeRelAttribute):
  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::isURLAttribute): (WebCore::HTMLObjectElement::containsJavaApplet):
  • html/HTMLParamElement.cpp: (WebCore::HTMLParamElement::isURLAttribute):
  • html/HTMLParser.cpp: (WebCore::HTMLParser::handleIsindex):
  • html/HTMLScriptElement.cpp: (WebCore::HTMLScriptElement::insertedIntoDocument): (WebCore::HTMLScriptElement::shouldExecuteAsJavaScript):
  • html/HTMLStyleElement.cpp: (WebCore::HTMLStyleElement::parseMappedAttribute):
  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::parseTag):
  • html/HTMLViewSourceDocument.cpp: (WebCore::HTMLViewSourceDocument::addViewSourceToken):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::saveDocumentState):
  • page/FrameTree.cpp: (WebCore::FrameTree::uniqueChildName):
  • platform/text/AtomicString.h: (WebCore::AtomicString::string): (WebCore::AtomicString::contains): (WebCore::AtomicString::find): (WebCore::AtomicString::startsWith): (WebCore::AtomicString::endsWith):
  • rendering/RenderPartObject.cpp: (WebCore::RenderPartObject::updateWidget):
  • svg/SVGElement.cpp: (WebCore::SVGElement::addSVGEventListener):
  • svg/SVGSVGElement.cpp: (WebCore::SVGSVGElement::addSVGWindowEventListener):
  • xml/XPathFunctions.cpp: (WebCore::XPath::FunLocalName::evaluate): (WebCore::XPath::FunNamespaceURI::evaluate): (WebCore::XPath::FunName::evaluate):
6:12 PM Changeset in webkit [30442] by pewtermoose@webkit.org
  • 1 edit in trunk/WebCore/ChangeLog

Remove stray conflict marker

6:10 PM Changeset in webkit [30441] by pewtermoose@webkit.org
  • 10 edits
    2 copies
    1 move
    3 adds
    1 delete in trunk/WebCore

2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>2008-02-20 Brent Fulgham <​bfulgham@gmail.com>

Reviewed by Dan.

​http://bugs.webkit.org/show_bug.cgi?id=17336
Provide implementations for Windows (Cairo) build of WebKit that
handles font formatting.

  • Split font implementation files to allow maximal code sharing between CG and Cairo back-ends.
  • WebCore.vcproj/WebCore.vcproj:
  • platform/graphics/SimpleFontData.h: Add signatures for private win initialization functions.
  • platform/graphics/win/FontCacheWin.cpp: (WebCore::FontCache::platformInit): (WebCore::FontCache::createFontPlatformData):
  • platform/graphics/win/FontCairoWin.cpp: Removed. Universal version is now part of platform/graphics/cairo.
  • platform/graphics/win/FontCustomPlatformDataCairo.cpp: Added. (WebCore::FontCustomPlatformDataCairo::~FontCustomPlatformDataCairo): (WebCore::FontCustomPlatformDataCairo::fontPlatformData): (WebCore::releaseData): (WebCore::createFontCustomPlatformData):
  • platform/graphics/win/FontCustomPlatformDataCairo.h: Added. (WebCore::FontCustomPlatformDataCairo::FontCustomPlatformDataCairo):
  • platform/graphics/win/FontPlatformData.h: (WebCore::FontPlatformData::FontPlatformData): (WebCore::FontPlatformData::fontFace): (WebCore::FontPlatformData::scaledFont): (WebCore::FontPlatformData::operator==):
  • platform/graphics/win/FontPlatformDataCGWin.cpp: Copied from WebCore/platform/graphics/win/FontPlatformDataWin.cpp. (WebCore::FontPlatformData::platformDataInit):
  • platform/graphics/win/FontPlatformDataCairoWin.cpp: Added. (WebCore::FontPlatformData::platformDataInit): (WebCore::FontPlatformData::FontPlatformData): (WebCore::FontPlatformData::setFont):
  • platform/graphics/win/FontPlatformDataWin.cpp: Moved CG-specific code to FontPlatformDataCG.cpp. (WebCore::FontPlatformData::FontPlatformData):
  • platform/graphics/win/GlyphPageTreeNodeCGWin.cpp: Copied from WebCore/platform/graphics/win/GlyphPageTreeNodeWin.cpp.
  • platform/graphics/win/GlyphPageTreeNodeCairoWin.cpp: Copied from WebCore/platform/graphics/win/GlyphPageTreeNodeWin.cpp. (WebCore::GlyphPage::fill):
  • platform/graphics/win/GlyphPageTreeNodeWin.cpp: Removed. Replaced with CG- and Cairo-specific versions.
  • platform/graphics/win/SimpleFontDataCGWin.cpp: (WebCore::SimpleFontData::platformInit): (WebCore::SimpleFontData::platformDestroy): (WebCore::SimpleFontData::platformWidthForGlyph):
  • platform/graphics/win/SimpleFontDataCairoWin.cpp: (WebCore::SimpleFontData::platformInit): (WebCore::SimpleFontData::platformDestroy): (WebCore::SimpleFontData::platformWidthForGlyph): (WebCore::SimpleFontData::setFont):
  • platform/graphics/win/SimpleFontDataWin.cpp: (WebCore::SimpleFontData::initGDIFont): (WebCore::SimpleFontData::platformCommonDestroy): (WebCore::SimpleFontData::widthForGDIGlyph):
6:03 PM Changeset in webkit [30440] by Darin Adler
  • 2 edits in trunk/WebCore

Reviewed by Sam.

  • dom/Node.h: Took out unneeded forward declaration of TextStream.
6:01 PM Changeset in webkit [30439] by Darin Adler
  • 2 edits in trunk/WebCore

Reviewed by Sam.

  • rendering/RenderObject.h: Took out unneeded forward declaration of TextStream.
5:59 PM Changeset in webkit [30438] by hyatt@apple.com
  • 4 edits in trunk/WebCore

Fix for bug 16760, incorrect <object> MIME type handling and fallback
handling.

Reviewed by darin

  • html/HTMLImageLoader.cpp: (WebCore::HTMLImageLoader::notifyFinished): If the image had an error, make sure to do <object> fallback.
  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::renderFallbackContent): Before doing fallback check if there is a MIME type mismatch between an image type and a non-image type. If so, detach and re-attach after storing the correct MIME type.
  • loader/loader.cpp: (WebCore::Loader::didReceiveData): Consider it an error when a 404 is encountered on a CachedResource load.
5:41 PM Changeset in webkit [30437] by andersca@apple.com
  • 4 edits in trunk/WebCore

Reviewed by Sam.

StringImpl constructors used by AtomicString should start with a refcount of 1.


  • platform/text/AtomicString.cpp: (WebCore::AtomicString::add):
  • platform/text/AtomicString.h:
  • platform/text/StringImpl.cpp: (WebCore::StringImpl::StringImpl):
5:06 PM Changeset in webkit [30436] by Darin Adler
  • 2 edits in trunk/WebCore
  • bindings/js/kjs_navigator.cpp: (WebCore::needsYouTubeQuirk): Tweak comments.
4:55 PM Changeset in webkit [30435] by timothy@apple.com
  • 1 copy in tags/Safari-5525.10

New tag.

3:54 PM Changeset in webkit [30434] by andersca@apple.com
  • 16 edits in trunk/WebCore

Reviewed by Darin.

Change all refcounted classes in page/ to start with a refcount of 1.

  • page/BarInfo.cpp: (WebCore::BarInfo::BarInfo):
  • page/BarInfo.h: (WebCore::BarInfo::create):
  • page/Console.cpp: (WebCore::Console::Console):
  • page/Console.h: (WebCore::Console::create):
  • page/DOMSelection.cpp: (WebCore::DOMSelection::DOMSelection):
  • page/DOMSelection.h: (WebCore::DOMSelection::create):
  • page/DOMWindow.cpp: (WebCore::DOMWindow::DOMWindow): (WebCore::DOMWindow::screen): (WebCore::DOMWindow::history): (WebCore::DOMWindow::locationbar): (WebCore::DOMWindow::menubar): (WebCore::DOMWindow::personalbar): (WebCore::DOMWindow::scrollbars): (WebCore::DOMWindow::statusbar): (WebCore::DOMWindow::toolbar): (WebCore::DOMWindow::console): (WebCore::DOMWindow::getSelection):
  • page/DOMWindow.h: (WebCore::DOMWindow::create):
  • page/Frame.cpp: (WebCore::Frame::domWindow):
  • page/History.cpp: (WebCore::History::History):
  • page/History.h: (WebCore::History::create):
  • page/InspectorController.cpp: (WebCore::InspectorResource::create): (WebCore::InspectorResource::InspectorResource): (WebCore::InspectorDatabaseResource::create): (WebCore::InspectorDatabaseResource::InspectorDatabaseResource): (WebCore::InspectorController::didLoadResourceFromMemoryCache): (WebCore::InspectorController::identifierForInitialRequest): (WebCore::InspectorController::didOpenDatabase):
  • page/Plugin.h: (WebCore::Plugin::create): (WebCore::Plugin::Plugin):
  • page/Screen.cpp: (WebCore::Screen::Screen):
  • page/Screen.h: (WebCore::Screen::create):
3:04 PM Changeset in webkit [30433] by weinig@apple.com
  • 9 edits in trunk

WebCore:

Reviewed by Darin and Geoff.

  • <rdar://problem/5754378> work around missing video on YouTube front page with a site-specific hack
  • WebCore.base.exp: Updated.
  • bindings/js/kjs_navigator.cpp: (WebCore::needsYouTubeQuirk): Added. Return true on Windows only when the quirk is needed. (WebCore::Navigator::getValueProperty): For the appVersion property, if needsYouTubeQuirk return true, then return the empty string.
  • page/Settings.cpp: (WebCore::Settings::Settings): Set m_needsSiteSpecificQuirks to false. (WebCore::Settings::setNeedsSiteSpecificQuirks): Added.
  • page/Settings.h: Added m_needsSiteSpecificQuirks. (WebCore::Settings::needsSiteSpecificQuirks): Added.

WebKit/mac:

Reviewed by Darin and Geoff.

  • WebKit part of <rdar://problem/5754378> work around missing video on YouTube front page with a site-specific hack
  • WebView/WebView.mm: (-[WebView _preferencesChangedNotification:]): Added a call to Settings::setNeedsSiteSpecificQuirks. There are currently no site-specific quirks on Mac, but we will propagate the state to WebCore to avoid possible mistakes later.

WebKit/win:

Reviewed by Darin and Geoff.

  • WebKit part of <rdar://problem/5754378> work around missing video on YouTube front page with a site-specific hack
  • WebView.cpp: (WebView::notifyPreferencesChanged): Added a call to Settings::setNeedsSiteSpecificQuirks. (WebView::setAllowSiteSpecificHacks): Added a comment about the problem Darin noticed, where after you disable the site-specific hacks they persist until you open a new window or tweak some other preference.
2:57 PM Changeset in webkit [30432] by hyatt@apple.com
  • 1 edit in trunk/WebCore/dom/DocumentType.cpp

Land additional doctype piece. toString fix of doctype.

2:47 PM Changeset in webkit [30431] by hyatt@apple.com
  • 113 edits
    15 adds in trunk

Fix for bug 12751, doctype nodes aren't part of the Document (Acid3).

Reviewed by Sam Weinig

Many tests added in fast/doctypes.

  • dom/DOMImplementation.cpp: (WebCore::DOMImplementation::createDocument): (WebCore::DOMImplementation::createHTMLDocument):
  • dom/Document.cpp: (WebCore::Document::Document): (WebCore::Document::setDocType): (WebCore::Document::attach): (WebCore::Document::getImageMap):
  • dom/Document.h: (WebCore::Document::doctype): (WebCore::Document::): (WebCore::Document::determineParseMode): (WebCore::Document::setParseMode): (WebCore::Document::parseMode): (WebCore::Document::inCompatMode): (WebCore::Document::inAlmostStrictMode): (WebCore::Document::inStrictMode):
  • dom/DocumentType.cpp: (WebCore::DocumentType::cloneNode): (WebCore::DocumentType::insertedIntoDocument): (WebCore::DocumentType::removedFromDocument):
  • dom/DocumentType.h:
  • dom/Node.cpp: (WebCore::Node::childAllowed):
  • dom/StyledElement.cpp: (WebCore::StyledElement::attributeChanged):
  • dom/XMLTokenizer.cpp: (WebCore::XMLTokenizer::internalSubset): (WebCore::):
  • editing/markup.cpp: (WebCore::appendStartMarkup):
  • html/HTMLAppletElement.cpp: (WebCore::HTMLAppletElement::createRenderer):
  • html/HTMLDocument.cpp: (WebCore::HTMLDocument::HTMLDocument): (WebCore::HTMLDocument::childAllowed): (WebCore::HTMLDocument::determineParseMode):
  • html/HTMLDocument.h:
  • html/HTMLMapElement.cpp: (WebCore::HTMLMapElement::parseMappedAttribute):
  • html/HTMLParamElement.cpp: (WebCore::HTMLParamElement::parseMappedAttribute):
  • html/HTMLParser.cpp: (WebCore::HTMLParser::parseDoctypeToken):
  • html/HTMLParser.h:
  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::reset): (WebCore::HTMLTokenizer::parseDoctype): (WebCore::HTMLTokenizer::parseTag): (WebCore::HTMLTokenizer::write): (WebCore::HTMLTokenizer::finish): (WebCore::HTMLTokenizer::processDoctypeToken):
  • html/HTMLTokenizer.h: (WebCore::): (WebCore::DoctypeToken::DoctypeToken): (WebCore::DoctypeToken::reset): (WebCore::DoctypeToken::state): (WebCore::DoctypeToken::setState): (WebCore::HTMLTokenizer::State::inDoctype): (WebCore::HTMLTokenizer::State::setInDoctype): (WebCore::HTMLTokenizer::State::needsSpecialWriteHandling): (WebCore::HTMLTokenizer::State::):
  • html/HTMLViewSourceDocument.cpp: (WebCore::HTMLViewSourceDocument::addViewSourceToken): (WebCore::HTMLViewSourceDocument::addViewSourceDoctypeToken):
  • html/HTMLViewSourceDocument.h:
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::write): (WebCore::FrameLoader::switchOutLowBandwidthDisplayIfReady):
  • page/Frame.cpp: (WebCore::Frame::documentTypeString):
  • page/inspector/utilities.js:
  • xml/XSLTProcessor.cpp: (WebCore::XSLTProcessor::createDocumentFromSource):
2:28 PM Changeset in webkit [30430] by andersca@apple.com
  • 60 edits in trunk/WebCore

Reviewed by Darin.

Change most SVG related classes to start out with a ref count of 1.


  • bindings/js/JSSVGPointListCustom.cpp: (WebCore::JSSVGPointList::initialize): (WebCore::JSSVGPointList::insertItemBefore): (WebCore::JSSVGPointList::replaceItem): (WebCore::JSSVGPointList::appendItem):
  • bindings/js/JSSVGTransformListCustom.cpp: (WebCore::JSSVGTransformList::initialize): (WebCore::JSSVGTransformList::insertItemBefore): (WebCore::JSSVGTransformList::replaceItem): (WebCore::JSSVGTransformList::appendItem):
  • rendering/SVGCharacterLayoutInfo.h: (WebCore::SVGCharOnPath::create): (WebCore::SVGCharOnPath::SVGCharOnPath):
  • rendering/SVGRootInlineBox.cpp: (WebCore::SVGRootInlineBox::buildLayoutInformationForTextBox):
  • svg/SVGAnimationElement.cpp: (WebCore::SVGAnimationElement::parseBeginOrEndValue):
  • svg/SVGClipPathElement.cpp: (WebCore::SVGClipPathElement::canvasResource):
  • svg/SVGElementInstance.cpp: (WebCore::SVGElementInstance::childNodes):
  • svg/SVGElementInstanceList.cpp: (WebCore::SVGElementInstanceList::SVGElementInstanceList):
  • svg/SVGElementInstanceList.h: (WebCore::SVGElementInstanceList::create):
  • svg/SVGFitToViewBox.cpp: (WebCore::SVGFitToViewBox::SVGFitToViewBox):
  • svg/SVGGradientElement.cpp: (WebCore::SVGGradientElement::SVGGradientElement): (WebCore::SVGGradientElement::canvasResource):
  • svg/SVGImageElement.cpp: (WebCore::SVGImageElement::SVGImageElement):
  • svg/SVGLengthList.h: (WebCore::SVGLengthList::create):
  • svg/SVGList.h: (WebCore::SVGList::SVGList): (WebCore::SVGPODListItem::create): (WebCore::SVGPODListItem::copy): (WebCore::SVGPODListItem::SVGPODListItem): (WebCore::SVGPODList::initialize): (WebCore::SVGPODList::insertItemBefore): (WebCore::SVGPODList::replaceItem): (WebCore::SVGPODList::appendItem): (WebCore::SVGPODList::SVGPODList):
  • svg/SVGMarkerElement.cpp: (WebCore::SVGMarkerElement::canvasResource):
  • svg/SVGMaskElement.cpp: (WebCore::SVGMaskElement::canvasResource):
  • svg/SVGNumberList.h: (WebCore::SVGNumberList::create):
  • svg/SVGPathElement.cpp: (WebCore::SVGPathElement::createSVGPathSegClosePath): (WebCore::SVGPathElement::createSVGPathSegMovetoAbs): (WebCore::SVGPathElement::createSVGPathSegMovetoRel): (WebCore::SVGPathElement::createSVGPathSegLinetoAbs): (WebCore::SVGPathElement::createSVGPathSegLinetoRel): (WebCore::SVGPathElement::createSVGPathSegCurvetoCubicAbs): (WebCore::SVGPathElement::createSVGPathSegCurvetoCubicRel): (WebCore::SVGPathElement::createSVGPathSegCurvetoQuadraticAbs): (WebCore::SVGPathElement::createSVGPathSegCurvetoQuadraticRel): (WebCore::SVGPathElement::createSVGPathSegArcAbs): (WebCore::SVGPathElement::createSVGPathSegArcRel): (WebCore::SVGPathElement::createSVGPathSegLinetoHorizontalAbs): (WebCore::SVGPathElement::createSVGPathSegLinetoHorizontalRel): (WebCore::SVGPathElement::createSVGPathSegLinetoVerticalAbs): (WebCore::SVGPathElement::createSVGPathSegLinetoVerticalRel): (WebCore::SVGPathElement::createSVGPathSegCurvetoCubicSmoothAbs): (WebCore::SVGPathElement::createSVGPathSegCurvetoCubicSmoothRel): (WebCore::SVGPathElement::createSVGPathSegCurvetoQuadraticSmoothAbs): (WebCore::SVGPathElement::createSVGPathSegCurvetoQuadraticSmoothRel): (WebCore::SVGPathElement::pathSegList):
  • svg/SVGPathElement.h:
  • svg/SVGPathSeg.h: (WebCore::SVGPathSeg::SVGPathSeg):
  • svg/SVGPathSegArc.h: (WebCore::SVGPathSegArcAbs::create): (WebCore::SVGPathSegArcRel::create):
  • svg/SVGPathSegClosePath.h: (WebCore::SVGPathSegClosePath::create):
  • svg/SVGPathSegCurvetoCubic.h: (WebCore::SVGPathSegCurvetoCubicAbs::create): (WebCore::SVGPathSegCurvetoCubicRel::create):
  • svg/SVGPathSegCurvetoCubicSmooth.h: (WebCore::SVGPathSegCurvetoCubicSmoothAbs::create): (WebCore::SVGPathSegCurvetoCubicSmoothRel::create):
  • svg/SVGPathSegCurvetoQuadratic.h: (WebCore::SVGPathSegCurvetoQuadraticAbs::create): (WebCore::SVGPathSegCurvetoQuadraticRel::create):
  • svg/SVGPathSegCurvetoQuadraticSmooth.h: (WebCore::SVGPathSegCurvetoQuadraticSmoothAbs::create): (WebCore::SVGPathSegCurvetoQuadraticSmoothRel::create):
  • svg/SVGPathSegLineto.h: (WebCore::SVGPathSegLinetoAbs::create): (WebCore::SVGPathSegLinetoRel::create):
  • svg/SVGPathSegLinetoHorizontal.h: (WebCore::SVGPathSegLinetoHorizontalAbs::create): (WebCore::SVGPathSegLinetoHorizontalRel::create):
  • svg/SVGPathSegLinetoVertical.h: (WebCore::SVGPathSegLinetoVerticalAbs::create): (WebCore::SVGPathSegLinetoVerticalRel::create):
  • svg/SVGPathSegList.h: (WebCore::SVGPathSegList::create):
  • svg/SVGPathSegMoveto.h: (WebCore::SVGPathSegMovetoAbs::create): (WebCore::SVGPathSegMovetoRel::create):
  • svg/SVGPatternElement.cpp: (WebCore::SVGPatternElement::SVGPatternElement): (WebCore::SVGPatternElement::canvasResource):
  • svg/SVGPointList.h: (WebCore::SVGPointList::create):
  • svg/SVGPolyElement.cpp: (WebCore::SVGPolyElement::points):
  • svg/SVGPreserveAspectRatio.cpp: (WebCore::SVGPreserveAspectRatio::SVGPreserveAspectRatio):
  • svg/SVGPreserveAspectRatio.h: (WebCore::SVGPreserveAspectRatio::create):
  • svg/SVGRenderingIntent.h: (WebCore::SVGRenderingIntent::SVGRenderingIntent):
  • svg/SVGStringList.h: (WebCore::SVGStringList::create):
  • svg/SVGStyledTransformableElement.cpp: (WebCore::SVGStyledTransformableElement::SVGStyledTransformableElement):
  • svg/SVGTests.cpp: (WebCore::SVGTests::requiredFeatures): (WebCore::SVGTests::requiredExtensions): (WebCore::SVGTests::systemLanguage):
  • svg/SVGTextElement.cpp: (WebCore::SVGTextElement::SVGTextElement):
  • svg/SVGTextPositioningElement.cpp: (WebCore::SVGTextPositioningElement::SVGTextPositioningElement):
  • svg/SVGTransformList.h: (WebCore::SVGTransformList::create):
  • svg/SVGUnitTypes.h: (WebCore::SVGUnitTypes::SVGUnitTypes):
  • svg/SVGViewElement.cpp: (WebCore::SVGViewElement::viewTarget):
  • svg/SVGViewSpec.cpp: (WebCore::SVGViewSpec::SVGViewSpec):
  • svg/graphics/SVGPaintServer.cpp: (WebCore::SVGPaintServer::sharedSolidPaintServer):
  • svg/graphics/SVGPaintServer.h:
  • svg/graphics/SVGPaintServerGradient.h: (WebCore::SVGPaintServerGradient::SharedStopCache::create): (WebCore::SVGPaintServerGradient::SharedStopCache::SharedStopCache):
  • svg/graphics/SVGPaintServerLinearGradient.h: (WebCore::SVGPaintServerLinearGradient::create):
  • svg/graphics/SVGPaintServerPattern.h: (WebCore::SVGPaintServerPattern::create):
  • svg/graphics/SVGPaintServerRadialGradient.h: (WebCore::SVGPaintServerRadialGradient::create):
  • svg/graphics/SVGPaintServerSolid.h: (WebCore::SVGPaintServerSolid::create):
  • svg/graphics/SVGResource.cpp: (WebCore::SVGResource::SVGResource):
  • svg/graphics/SVGResource.h:
  • svg/graphics/SVGResourceClipper.h: (WebCore::SVGResourceClipper::create):
  • svg/graphics/SVGResourceMarker.h: (WebCore::SVGResourceMarker::create):
  • svg/graphics/SVGResourceMasker.h: (WebCore::SVGResourceMasker::create):
  • svg/graphics/cg/SVGPaintServerGradientCg.cpp: (WebCore::SVGPaintServerGradient::updateQuartzGradientStopsCache):
1:36 PM Changeset in webkit [30429] by Darin Adler
  • 7 edits in trunk/WebCore

Reviewed by Sam.

  • make conversion from CSS ParseString to String and AtomicString automatic and remove all the explicit calls to do the conversion
  • fix CSS parsing to do fewer allocations, mostly by using the equalIgnoringCase function in CSSParser
  • css/CSSGrammar.y: Take out all the explicit atomicString and domString calls now that ParseString knows how to convert itself.
  • css/CSSParser.cpp: (WebCore::equal): Added. (WebCore::equalIgnoringCase): Allow non-lettters. (WebCore::ParseString::lower): Used charactersAreAllASCII. (WebCore::unitFromString): Use equal. (WebCore::CSSParser::parseValue): Removed unneeded call to domString. (WebCore::CSSParser::parseContent): Use equalIgnoringCase. (WebCore::CSSParser::parseBackgroundImage): Removed unneeded call to domString. (WebCore::CSSParser::parseTransitionTimingFunction): Use equalIgnoringCase. (WebCore::CSSParser::parseDashboardRegions): More of the same. (WebCore::CSSParser::parseCounterContent): Ditto. (WebCore::CSSParser::parseShape): Use equalIgnoringCase. (WebCore::CSSParser::parseFontFamily): Removed unneeded calls to domString. (WebCore::CSSParser::parseFontFaceSrc): More. (WebCore::CSSParser::parseFontFaceUnicodeRange): More. (WebCore::CSSParser::parseColor): Don't lowercase here -- setNamedColor now handles the case folding. (WebCore::CSSParser::parseColorFromValue): More. (WebCore::CSSParser::parseBorderImage): More. (WebCore::CSSParser::parseCounter): More. (WebCore::TransformOperationInfo::TransformOperationInfo): More. (WebCore::CSSParser::parseTransform): More. (WebCore::CSSParser::createCharsetRule): More. (WebCore::CSSParser::createImportRule): More.
  • css/CSSParser.h: Removed domString and atomicString functions. (WebCore::ParseString::operator String): Added. Allows conversion to String without an explicit function call. (WebCore::ParseString::operator AtomicString): Ditto.
  • css/MediaQueryExp.cpp: (WebCore::MediaQueryExp::MediaQueryExp): Removed a call to domString.
  • css/SVGCSSParser.cpp: (WebCore::CSSParser::parseSVGValue): Removed calls to domString.
  • platform/graphics/Color.cpp: (WebCore::findNamedColor): Call toASCIILower on each character as we copy it into the 8-bit character buffer to make the operation fold case.
1:05 PM Changeset in webkit [30428] by timothy@apple.com
  • 2 edits in branches/Safari-3-1-branch/WebCore

Merged r30422 and r30423.

1:03 PM Changeset in webkit [30427] by hyatt@apple.com
  • 1 edit in trunk/LayoutTests/svg/dynamic-updates/resources/SVGTestCase.js

Fix a bug in this SVG test harness. Make sure it uses document.documentElement instead of document.firstChild.

12:59 PM Changeset in webkit [30426] by timothy@apple.com
  • 2 edits in branches/Safari-3-1-branch/WebKit/win

Merge r30400.

9:48 AM Changeset in webkit [30425] by justin.garcia@apple.com
  • 3 edits in trunk/WebCore

Reviewed by Darin Adler.

<rdar://problem/5694920> Typing (esp. deleting) is slower due to TOT WebCore changes

  • editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::initializeStartEnd): The common case here is where there are no special elements. Avoid creating VisiblePositions in that case. Additionally, this change postpones the more expensive creation of an upstream VisiblePosition until the last possible moment. (WebCore::DeleteSelectionCommand::saveTypingStyleState): (WebCore::DeleteSelectionCommand::calculateTypingStyleAfterDelete): Add an early return for a common case: deleting characters that are all inside the same text node. In that case the style at the start of the selection will not change during the delete, so there is no need to save/recompute it. (WebCore::DeleteSelectionCommand::saveFullySelectedAnchor): Early return before VisiblePosition creation if the ends of the selection aren't enclosed by an anchor.
  • editing/TypingCommand.cpp: (WebCore::TypingCommand::markMisspellingsAfterTyping): Early return if spell checking isn't enabled.
4:35 AM Changeset in webkit [30424] by ap@webkit.org
  • 3 edits
    2 adds in trunk

Incorporates some improvements made by Dan Bernstein.

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=17106
<rdar://problem/5750722> Debug build ASSERTs on page load

Test: fast/encoding/GBK/close-gbk-converter.html

  • platform/text/TextCodecICU.cpp: (WebCore::getGbkEscape): Changed to use a switch instead of a HashMap, as there are only four values. (WebCore::gbkCallbackEscape): Check the reason why the function is called, and do not attempt getting an escape character if it's not UCNV_UNASSIGNED. (WebCore::gbkCallbackSubstitute): Ditto.
3:39 AM Changeset in webkit [30423] by ap@webkit.org
  • 2 edits in trunk/WebCore

Build fix.

  • xml/XMLHttpRequest.cpp: (WebCore::isSafeRequestHeader): (WebCore::XMLHttpRequest::setRequestHeader):
3:10 AM Changeset in webkit [30422] by ap@webkit.org
  • 2 edits in trunk/WebCore

Reviewed by Darin.

<rdar://problem/5749455> Unable to set the Referer header in Dashboard using XMLHttpRequest

Cannot be tested in DRT.

  • xml/XMLHttpRequest.cpp: (WebCore::canSetRequestHeader): Assume that a request that can load local files can also set any headers.
12:08 AM BuildingCairoOnWindows edited by bfulgham@macports.org
(diff)
Note: See TracTimeline for information about the timeline view.