Timeline



Feb 5, 2006:

11:49 PM Changeset in webkit [12593] by mjs
  • 5 edits in trunk/JavaScriptCore

Reviewed by Anders.


  • kjs/internal.cpp: (KJS::Parser::noteNodeCycle): (KJS::Parser::removeNodeCycle): (KJS::clearNewNodes):
  • kjs/internal.h:
  • kjs/nodes.cpp: (ElementNode::breakCycle): (PropertyListNode::breakCycle): (ArgumentListNode::breakCycle): (StatListNode::StatListNode): (StatListNode::breakCycle): (VarDeclListNode::breakCycle): (BlockNode::BlockNode): (ClauseListNode::breakCycle): (CaseBlockNode::CaseBlockNode): (ParameterNode::breakCycle): (SourceElementsNode::SourceElementsNode): (SourceElementsNode::breakCycle):
  • kjs/nodes.h: (KJS::Node::breakCycle): (KJS::ElementNode::ElementNode): (KJS::ArrayNode::ArrayNode): (KJS::PropertyListNode::PropertyListNode): (KJS::ObjectLiteralNode::ObjectLiteralNode): (KJS::ArgumentListNode::ArgumentListNode): (KJS::ArgumentsNode::ArgumentsNode): (KJS::VarDeclListNode::VarDeclListNode): (KJS::VarStatementNode::VarStatementNode): (KJS::ForNode::ForNode): (KJS::CaseClauseNode::CaseClauseNode): (KJS::ClauseListNode::ClauseListNode): (KJS::ParameterNode::ParameterNode): (KJS::FuncExprNode::FuncExprNode): (KJS::FuncDeclNode::FuncDeclNode):
11:25 PM Changeset in webkit [12592] by mjs
  • 11 edits in trunk/WebKit

Reviewed by Darin.


  • WebCoreSupport.subproj/WebFrameBridge.h:
  • WebCoreSupport.subproj/WebFrameBridge.m: (-[WebFrameBridge initWithPage:webView:frameName:view:]): (-[WebFrameBridge page]): (-[WebFrameBridge mainFrame]): (-[WebFrameBridge webView]): (-[WebFrameBridge createWindowWithURL:frameName:]): (-[WebFrameBridge showWindow]): (-[WebFrameBridge areToolbarsVisible]): (-[WebFrameBridge setToolbarsVisible:]): (-[WebFrameBridge isStatusbarVisible]): (-[WebFrameBridge setStatusbarVisible:]): (-[WebFrameBridge setWindowFrame:]): (-[WebFrameBridge windowFrame]): (-[WebFrameBridge setWindowContentRect:]): (-[WebFrameBridge windowContentRect]): (-[WebFrameBridge setWindowIsResizable:]): (-[WebFrameBridge windowIsResizable]): (-[WebFrameBridge firstResponder]): (-[WebFrameBridge makeFirstResponder:]): (-[WebFrameBridge closeWindowSoon]): (-[WebFrameBridge runJavaScriptAlertPanelWithMessage:]): (-[WebFrameBridge runJavaScriptConfirmPanelWithMessage:]): (-[WebFrameBridge canRunBeforeUnloadConfirmPanel]): (-[WebFrameBridge runBeforeUnloadConfirmPanelWithMessage:]): (-[WebFrameBridge runJavaScriptTextInputPanelWithPrompt:defaultText:returningText:]): (-[WebFrameBridge addMessageToConsole:]): (-[WebFrameBridge runOpenPanelForFileButtonWithResultListener:]): (-[WebFrameBridge setStatusText:]): (-[WebFrameBridge startLoadingResource:withURL:customHeaders:]): (-[WebFrameBridge startLoadingResource:withURL:customHeaders:postData:]): (-[WebFrameBridge syncLoadResourceWithURL:customHeaders:postData:finalURL:responseHeaders:statusCode:]): (-[WebFrameBridge focusWindow]): (-[WebFrameBridge createChildFrameNamed:withURL:referrer:renderPart:allowsScrolling:marginWidth:marginHeight:]): (-[WebFrameBridge userAgentForURL:]): (-[WebFrameBridge _nextKeyViewOutsideWebFrameViewsWithValidityCheck:]): (-[WebFrameBridge previousKeyViewOutsideWebFrameViews]): (-[WebFrameBridge defersLoading]): (-[WebFrameBridge setDefersLoading:]): (-[WebFrameBridge viewForPluginWithURL:attributeNames:attributeValues:MIMEType:]): (-[WebFrameBridge _preferences]): (-[WebFrameBridge selectWordBeforeMenuEvent]): (-[WebFrameBridge historyLength]): (-[WebFrameBridge canGoBackOrForward:]): (-[WebFrameBridge goBackOrForward:]): (-[WebFrameBridge print]): (-[WebFrameBridge pollForAppletInView:]): (-[WebFrameBridge respondToChangedContents]): (-[WebFrameBridge respondToChangedSelection]): (-[WebFrameBridge undoManager]): (-[WebFrameBridge issueCutCommand]): (-[WebFrameBridge issueCopyCommand]): (-[WebFrameBridge issuePasteCommand]): (-[WebFrameBridge issuePasteAndMatchStyleCommand]): (-[WebFrameBridge canPaste]): (-[WebFrameBridge overrideMediaType]): (-[WebFrameBridge isEditable]): (-[WebFrameBridge shouldChangeSelectedDOMRange:toDOMRange:affinity:stillSelecting:]): (-[WebFrameBridge shouldBeginEditing:]): (-[WebFrameBridge shouldEndEditing:]): (-[WebFrameBridge windowObjectCleared]): (-[WebFrameBridge spellCheckerDocumentTag]): (-[WebFrameBridge isContinuousSpellCheckingEnabled]): (-[WebFrameBridge didFirstLayout]): (-[WebFrameBridge dashboardRegionsChanged:]): (-[WebFrameBridge createModalDialogWithURL:]): (-[WebFrameBridge canRunModal]): (-[WebFrameBridge runModal]):
  • WebCoreSupport.subproj/WebPageBridge.h:
  • WebCoreSupport.subproj/WebPageBridge.m: (-[WebPageBridge initWithMainFrameName:webView:frameView:]): (-[WebPageBridge webView]):
  • WebView.subproj/WebDataSource.m: (-[WebDataSource _fileWrapperForURL:]): (-[WebDataSource _webView]): (-[WebDataSource _setLoading:]): (-[WebDataSource _startLoading:]): (-[WebDataSource _setTitle:]): (-[WebDataSource _setRequest:]): (-[WebDataSource _updateIconDatabaseWithURL:]): (-[WebDataSource _defersCallbacksChanged]): (-[WebDataSource _setWebFrame:]):
  • WebView.subproj/WebDataSourcePrivate.h:
  • WebView.subproj/WebFrame.m: (-[WebFrame _traverseNextFrameStayWithin::]): (-[WebFrame _detachFromParent]): (-[WebFrame _setDataSource:]): (-[WebFrame _loadDataSource:withLoadType:formState:]): (-[WebFrame _initWithWebFrameView:webView:bridge:]): (-[WebFrame dealloc]): (-[WebFrame finalize]): (-[WebFrame webView]):
  • WebView.subproj/WebFrameView.m: (-[WebFrameView _webView]): (-[WebFrameView _goBack]): (-[WebFrameView _goForward]):
  • WebView.subproj/WebFrameViewInternal.h:
  • WebView.subproj/WebView.m: (-[WebView _createFrameNamed:inParent:allowsScrolling:]): (-[WebView _commonInitializationWithFrameName:groupName:]):
11:06 PM Changeset in webkit [12591] by andersca
  • 5 edits
    6 adds in trunk/WebCore

2006-02-05 Anders Carlsson <​andersca@mac.com>

Reviewed by Maciej.

  • JSCore.cpp: Add new generated files.


  • bindings/scripts/CodeGeneratorJS.pm: Add "IsIndex" extended attribute for parameters. Any parameter that is "unsigned long" and has this attribute will be checked to make sure that it's not < 0. If it is, an index size error exception will be thrown.


  • khtml/ecma/kjs_dom.cpp: (KJS::toAttr): Use JSAttr::info.


(KJS::getDOMNode):
Use new constructors.


  • khtml/ecma/kjs_dom.h: Remove classes that are generated now.


  • khtml/xml/Attr.idl: Added.
  • khtml/xml/CharacterData.idl: Added.
  • khtml/xml/Entity.idl: Added.
  • khtml/xml/Notation.idl: Added.
  • khtml/xml/ProcessingInstruction.idl: Added.
  • khtml/xml/Text.idl: Added.
9:57 PM Changeset in webkit [12590] by eseidel
  • 2 edits in trunk/WebKitSite

2006-02-05 Eric Seidel <​eseidel@apple.com>

Reviewed by darin.

General update to SVG project page, also link to status page.

  • projects/svg/index.html:
9:36 PM Changeset in webkit [12589] by eseidel
  • 4 adds in trunk/WebKitSite

2006-02-05 Eric Seidel <​eseidel@apple.com>

Rubber stamped by darin.

  • projects/svg/status.xml: Added.
  • projects/svg/svg-status.css: Added.
  • projects/svg/svg-status.xsl: Added.

2006-02-05 Eric Seidel <​eseidel@apple.com>

Adding ChangeLog :)


  • ChangeLog: Added.
6:36 PM Changeset in webkit [12588] by darin
  • 4 edits in trunk/LayoutTests
  • converted three more files from UTF-16 to ASCII that had no reason to be UTF-16
  • fast/dom/resources/top.html: Just straight conversion to ASCII.
  • fast/events/updateLayoutForHitTest.html: Converted one character to an entity, and reformatted the file a bit.
  • fast/text/line-breaks.html: Just straight conversion to ASCII.
3:07 PM Changeset in webkit [12587] by mjs
  • 11 edits
    2 adds in trunk

LayoutTests:

Reviewed by Darin.


Re-landing test case for:

  • fast/events/iframe-object-onload-expected.txt: Added.
  • fast/events/iframe-object-onload.html: Added.


  • changed expected results and description for this test, unload should not fire in this case:


  • fast/events/onunload-body-property-expected.txt:
  • fast/events/onunload-body-property.html:


  • changed from UTF-16 to ASCII, there was no reason for this to be UTF-16.
  • fast/dom/attr_dead_doc.html:

WebCore:

Reviewed by Darin.


Re-landed the following with more fixes so it does not break tests:


Specific changes:


  • don't bother to nil-check the document, a node can never have a null document now
  • move temp event forgetting from dispatchEvent to dispatchGenericEvent
  • pass event down using RefPtr::release() to avoid ref thrashing
  • support default handlers even for non-bubbling events (only on target node) and skip calling default event handler explicitly in callers
  • dispatch a whole separate load event to a frame document's containing frame
  • don't let propagationStopped prevent this new event
  • remove bogus security check for iframe onload
  • dispatch window events on the document, not the body, and also for non-html
  • set onload, onunload, onbeforeunload from frameset tags on the window object
  • don't restrict load/unload events to HTML
  • send default handler to the dispatch object as previously
  • bridge/mac/MacFrame.mm: (WebCore::MacFrame::shouldClose):
  • khtml/html/html_baseimpl.cpp: (WebCore::HTMLFrameElementImpl::parseMappedAttribute): (WebCore::HTMLFrameSetElementImpl::parseMappedAttribute):
  • khtml/xml/DocumentImpl.cpp: (WebCore::DocumentImpl::implicitClose):
  • khtml/xml/NodeImpl.cpp: (WebCore::NodeImpl::dispatchEvent): (WebCore::NodeImpl::dispatchGenericEvent): (WebCore::NodeImpl::dispatchWindowEvent):
  • khtml/xml/NodeImpl.h:
  • page/Frame.cpp: (WebCore::Frame::stopLoading): (WebCore::Frame::setWindowHasFocus):
12:31 PM Changeset in webkit [12586] by darin
  • 8 edits
    1 add in trunk/WebCore

Reviewed by Geoff.

  • fix <rdar://problem/4197997> -[DOMHTMLSelectElement options] always returns an empty list
  • khtml/html/HTMLOptionsCollectionImpl.h: Changed to be a subclass of HTMLCollectionImpl.
  • khtml/html/HTMLOptionsCollectionImpl.cpp: Added.
  • WebCore.vcproj/WebCore/WebCore.vcproj: Added new source file.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • bindings/objc/DOMHTML.mm: (-[DOMHTMLOptionsCollection setLength:]): Handle possible exception. (-[DOMHTMLSelectElement options]): Add get() since this is now a PassRefPtr.
  • khtml/ecma/kjs_html.cpp: (KJS::HTMLElement::selectIndexGetter): Call options() instead of optionsHTMLCollection(). (KJS::HTMLElement::selectGetter): Ditto. (KJS::KJS::HTMLElement::put): Ditto. (KJS::HTMLElement::selectSetter): Ditto. (KJS::KJS::HTMLSelectCollection::put): Ditto.
  • khtml/html/HTMLSelectElementImpl.h: Tweaked formatting. Eliminated m_options. Changed m_minwidth and m_size to int instead of short. Removed optionsHTMLCollection(). Changed options() to return a PassRefPtr.
  • khtml/html/HTMLSelectElementImpl.cpp: (WebCore::HTMLSelectElementImpl::~HTMLSelectElementImpl): Get rid of code to drop m_options. (WebCore::HTMLSelectElementImpl::remove): Remove unnecesssary RefPtr use, since that's handled fine by removeChild. (WebCore::HTMLSelectElementImpl::options): Change to create a new HTMLOptionsCollectionImpl each time, like other collection calls. Maybe we should change this eventually to use a HashMap so we can pass out the same collection over and over again, or could have a pointer stored in the select element that's managed by the collection itself.
11:34 AM Changeset in webkit [12585] by darin
  • 1 edit
    2 adds in trunk/LayoutTests

Reviewed by Maciej.

  • fast/forms/select-replace-option.html: Added.
  • fast/forms/select-replace-option-expected.txt: Added.
11:33 AM Changeset in webkit [12584] by darin
  • 1 edit in trunk/WebCore/ChangeLog

Forgot to put reviewer name in.

11:33 AM Changeset in webkit [12583] by darin
  • 2 edits in trunk/WebCore

Reviewed by NOBODY (OOPS!).

Test: fast/forms/select-replace-option.html

  • khtml/ecma/kjs_html.cpp: (KJS::KJS::HTMLSelectCollection::put): Use optionsHTMLCollection, because options isn't really implemented yet.
11:18 AM Changeset in webkit [12582] by darin
  • 2 edits in trunk/WebCore

Reviewed by Maciej.

  • kwq/KWQTextField.mm: (-[KWQTextFieldController setHasFocus:]): Add check for nil in case the blur event causes the KWQLineEdit to be destroyed.
11:17 AM Changeset in webkit [12581] by darin
  • 1 edit
    4 adds in trunk/LayoutTests

Reviewed by Maciej.

  • fast/replaced/object-param-no-name.html: Added.
  • fast/replaced/object-param-no-name-expected.txt: Added.
  • fixed test that I landed incorrectly earlier
  • fast/overflow/onscroll-layer-self-destruct.html: Removed Subversion property junk.
  • fast/overflow/onscroll-layer-self-destruct-expected.txt: Ditto.
11:16 AM Changeset in webkit [12580] by darin
  • 2 edits in trunk/WebCore

Reviewed by Maciej.

Test: fast/replaced/object-param-no-name.html

  • rendering/render_frames.cpp: (WebCore::RenderPartObject::updateWidget): Check for empty name, and skip processing the name/value of that <param> element.
11:00 AM Changeset in webkit [12579] by darin
  • 1 edit
    2 deletes in trunk/LayoutTests
  • rolled out Maciej's event cleanup patch, which is causing problems

I believe he has a fix, and he'll re-land the patch with the fix later.

Original fix was <​http://bugzilla.opendarwin.org/show_bug.cgi?id=7079>.
Problems described in <​http://bugzilla.opendarwin.org/show_bug.cgi?id=7084>.

11:00 AM Changeset in webkit [12578] by darin
  • 4 edits in trunk/WebCore
  • rolled out Maciej's event cleanup patch, which is causing problems

I believe he has a fix, and he'll re-land the patch with the fix later.

Original fix was <​http://bugzilla.opendarwin.org/show_bug.cgi?id=7079>.
Problems described in <​http://bugzilla.opendarwin.org/show_bug.cgi?id=7084>.

10:19 AM Changeset in webkit [12577] by darin
  • 2 edits in trunk/WebCore
  • ksvg2/css/KSVGCSSParser.cpp: Remove config.h, since this is included by another .cpp file -- oops!
10:16 AM Changeset in webkit [12576] by darin
  • 7 edits
    2 deletes in trunk/WebCore

Discussed with Anders (didn't get real review).

  • fix build
  • WebCore+SVG/SVGNamesWrapper.cpp: Touch file, not getting rebuilt even though it includes config.h which changed.
  • WebCore+SVG/XLinkNamesWrapper.cpp: Ditto.
  • ksvg2/css/KSVGCSSParser.cpp: Added include of config.h.
  • platform/mac/ImageDecoder.cpp: Ditto.
  • WebCore.vcproj/WebCore/WebCore.vcproj: Removed IntPoint.cpp and IntSize.cpp.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • platform/IntPoint.cpp: Removed. Was empty.
  • platform/IntSize.cpp: Removed. Was empty.
9:21 AM Changeset in webkit [12575] by darin
  • 1 edit in trunk/LayoutTests/ChangeLog

Reviewed by Maciej.

  • fast/overflow/onscroll-layer-self-destruct.html: Added.
  • fast/overflow/onscroll-layer-self-destruct-expected.txt: Added.
3:48 AM Changeset in webkit [12574] by darin
  • 2 edits in trunk/WebCore

Reviewed by Maciej.

Test: onscroll-layer-self-destruct.html

  • rendering/render_layer.cpp: (WebCore::RenderLayer::scrollToOffset): Don't send the scroll event until after we've done everything else we need to do.
2:37 AM Changeset in webkit [12573] by darin
  • 99 edits
    1 copy
    6 moves in trunk/WebCore

Rubber stamped by Maciej.

  • renamed DOMString and DOMStringImpl to String and StringImpl, and moved into platform
  • config.h: Put in transitional defines so old code can still refer to the classes as DOMString and DOMStringImpl.
  • platform/AtomicString.cpp: Added.
  • platform/AtomicString.h: Added.
  • platform/AtomicStringImpl.h: Added.
  • platform/PlatformString.h: Added.
  • platform/String.cpp: Added.
  • platform/StringImpl.cpp: Added.
  • platform/StringImpl.h: Added. Moved files here and renamed them. PlatformString.h would be named String.h except that conflicts with <string.h> on non-case-sensitive file systems.
  • khtml/xml/dom_atomicstring.cpp: Removed.
  • khtml/xml/dom_atomicstring.h: Removed.
  • khtml/xml/dom_stringimpl.cpp: Removed.
  • khtml/xml/dom_stringimpl.h: Removed.
  • khtml/dom/dom_string.cpp: Removed.
  • khtml/dom/dom_string.h: Removed.
  • WebCore.xcodeproj/project.pbxproj: Updated for file adds and removes.
  • WebCore.vcproj/WebCore/WebCore.vcproj: Ditto.
  • WebCore+SVG/KDOMHeaders.h:
  • WebCore+SVG/Namespace.h:
  • WebCore+SVG/kdom.h:
  • bindings/objc/DOM.mm:
  • bindings/objc/DOMCSS.mm:
  • bindings/objc/DOMHTML.mm:
  • bindings/objc/DOMInternal.mm:
  • bridge/BrowserExtension.h:
  • css/css_base.h:
  • css/css_computedstyle.cpp:
  • css/css_grammar.y:
  • css/css_ruleimpl.h:
  • css/css_stylesheetimpl.cpp:
  • css/css_stylesheetimpl.h:
  • css/css_valueimpl.cpp:
  • css/csshelper.cpp:
  • css/csshelper.h:
  • css/cssparser.h:
  • css/cssstyleselector.h:
  • kcanvas/KCanvasTreeDebug.cpp:
  • khtml/dom/dom2_events.h:
  • khtml/ecma/JSXMLHttpRequest.cpp:
  • khtml/ecma/kjs_html.cpp:
  • khtml/ecma/kjs_window.cpp:
  • khtml/editing/SelectionController.cpp:
  • khtml/editing/apply_style_command.cpp:
  • khtml/editing/delete_from_text_node_command.h:
  • khtml/editing/insert_into_text_node_command.h:
  • khtml/editing/jsediting.h:
  • khtml/editing/rebalance_whitespace_command.h:
  • khtml/editing/remove_css_property_command.h:
  • khtml/editing/typing_command.h:
  • khtml/html/FormDataList.h:
  • khtml/html/HTMLNameCollectionImpl.h:
  • khtml/html/html_baseimpl.cpp:
  • khtml/html/html_objectimpl.cpp:
  • khtml/khtml_events.h:
  • khtml/xbl/xbl_binding.h:
  • khtml/xbl/xbl_protobinding.h:
  • khtml/xbl/xbl_protohandler.h:
  • khtml/xbl/xbl_protoimplementation.h:
  • khtml/xml/DOMImplementationImpl.cpp:
  • khtml/xml/EventNames.h:
  • khtml/xml/NameNodeListImpl.h:
  • khtml/xml/NamedNodeMapImpl.h:
  • khtml/xml/NodeImpl.cpp:
  • khtml/xml/NodeImpl.h:
  • khtml/xml/dom2_eventsimpl.h:
  • khtml/xml/dom_atomicstringlist.h:
  • khtml/xml/dom_elementimpl.h:
  • khtml/xml/dom_qname.h:
  • khtml/xml/dom_xmlimpl.cpp:
  • khtml/xml/xml_tokenizer.h:
  • khtml/xsl/xsl_stylesheetimpl.cpp:
  • khtml/xsl/xslt_processorimpl.h:
  • ksvg2/misc/KCanvasRenderingStyle.cpp:
  • ksvg2/svg/SVGAngleImpl.h:
  • ksvg2/svg/SVGAnimateColorElementImpl.cpp:
  • ksvg2/svg/SVGAnimatedStringImpl.h:
  • ksvg2/svg/SVGAnimationElementImpl.cpp:
  • ksvg2/svg/SVGColorImpl.h:
  • ksvg2/svg/SVGDOMImplementationImpl.cpp:
  • ksvg2/svg/SVGElementImpl.cpp:
  • ksvg2/svg/SVGFitToViewBoxImpl.cpp:
  • ksvg2/svg/SVGLangSpaceImpl.cpp:
  • ksvg2/svg/SVGLangSpaceImpl.h:
  • ksvg2/svg/SVGLengthImpl.h:
  • ksvg2/svg/SVGMarkerElementImpl.cpp:
  • ksvg2/svg/SVGPaintImpl.h:
  • ksvg2/svg/SVGPathSegImpl.h:
  • ksvg2/svg/SVGPreserveAspectRatioImpl.cpp:
  • ksvg2/svg/SVGSVGElementImpl.cpp:
  • ksvg2/svg/SVGScriptElementImpl.cpp:
  • ksvg2/svg/SVGStringListImpl.h:
  • ksvg2/svg/SVGStylableImpl.h:
  • ksvg2/svg/SVGStyleElementImpl.cpp:
  • ksvg2/svg/SVGStyledElementImpl.cpp:
  • ksvg2/svg/SVGSymbolElementImpl.cpp:
  • ksvg2/svg/SVGViewElementImpl.cpp:
  • kwq/KWQAccObject.mm:
  • kwq/KWQFontFamily.h:
  • kwq/KWQFontFamily.mm:
  • kwq/KWQKJobClasses.h:
  • kwq/KWQKJobClasses.mm:
  • loader/Cache.h:
  • loader/CachedObject.h:
  • loader/CachedXBLDocument.h:
  • loader/CachedXSLStyleSheet.h:
  • loader/DocLoader.h:
  • page/Frame.cpp:
  • page/FrameTreeNode.h:
  • platform/SegmentedString.h:
  • rendering/render_image.h:
  • rendering/render_style.cpp:
  • xml/xmlhttprequest.cpp: Updated all includes to use new filenames.
1:53 AM Changeset in webkit [12572] by mjs
  • 2 edits in trunk/JavaScriptCore

Reviewed by Hyatt.


  • fix default traits for classes to make sure default constructors get called
  • kxmlcore/VectorTraits.h: (KXMLCore::):
1:45 AM Changeset in webkit [12571] by mjs
  • 5 edits
    2 adds in trunk

LayoutTests:

Reviewed by Darin.


Test case for:

  • fast/events/iframe-object-onload-expected.txt: Added.
  • fast/events/iframe-object-onload.html: Added.

WebCore:

Reviewed by Darin.


Specific changes:


  • don't bother to nil-check the document, a node can never have a null document now
  • move temp event forgetting from dispatchEvent to dispatchGenericEvent
  • pass event down using RefPtr::release() to avoid ref thrashing
  • support default handlers even for non-bubbling events (only on target node) and skip calling default event handler explicitly in callers
  • dispatch a whole separate load event to a frame document's containing frame
  • don't let propagationStopped prevent this new event
  • remove bogus security check for iframe onload
  • bridge/mac/MacFrame.mm: (WebCore::MacFrame::shouldClose):
  • khtml/xml/NodeImpl.cpp: (WebCore::NodeImpl::dispatchEvent): (WebCore::NodeImpl::dispatchGenericEvent): (WebCore::NodeImpl::dispatchWindowEvent):
  • khtml/xml/NodeImpl.h:
12:49 AM Changeset in webkit [12570] by andersca
  • 3 edits in trunk/WebCore

2006-02-05 Maciej Stachowiak <​mjs@apple.com>

Reviewed and landed by Anders.


12:29 AM Changeset in webkit [12569] by ap
  • 2 edits in trunk/WebCore

Fix by David Kilzer, reviewed by Darin, landed by ap (with a tweak suggested by Darin).

  • kwq/KWQPainter.mm: Include "FloatRect.h".

Feb 4, 2006:

9:44 PM Changeset in webkit [12568] by mjs
  • 14 edits in trunk/WebCore

Reviewed by Darin.

  • bindings/objc/DOM.mm: (-[DOMNode insertBefore::]): (-[DOMNode replaceChild::]): (-[DOMNode removeChild:]): (-[DOMNode appendChild:]):
  • khtml/ecma/kjs_dom.cpp: (KJS::DOMNodeProtoFunc::callAsFunction):
  • khtml/html/HTMLOptGroupElementImpl.cpp: (WebCore::HTMLOptGroupElementImpl::insertBefore): (WebCore::HTMLOptGroupElementImpl::replaceChild): (WebCore::HTMLOptGroupElementImpl::removeChild): (WebCore::HTMLOptGroupElementImpl::appendChild):
  • khtml/html/HTMLOptGroupElementImpl.h:
  • khtml/html/HTMLSelectElementImpl.cpp: (WebCore::HTMLSelectElementImpl::insertBefore): (WebCore::HTMLSelectElementImpl::replaceChild): (WebCore::HTMLSelectElementImpl::removeChild): (WebCore::HTMLSelectElementImpl::appendChild):
  • khtml/html/HTMLSelectElementImpl.h:
  • khtml/html/HTMLTextAreaElementImpl.cpp: (WebCore::HTMLTextAreaElementImpl::setDefaultValue):
  • khtml/html/html_tableimpl.cpp: (WebCore::HTMLTableElementImpl::createTFoot):
  • khtml/xml/ContainerNodeImpl.cpp: (WebCore::ContainerNodeImpl::insertBefore): (WebCore::ContainerNodeImpl::replaceChild): (WebCore::ContainerNodeImpl::removeChild): (WebCore::ContainerNodeImpl::appendChild):
  • khtml/xml/ContainerNodeImpl.h:
  • khtml/xml/NodeImpl.cpp: (WebCore::NodeImpl::insertBefore): (WebCore::NodeImpl::replaceChild): (WebCore::NodeImpl::removeChild): (WebCore::NodeImpl::appendChild):
  • khtml/xml/NodeImpl.h:
  • khtml/xml/xml_tokenizer.cpp: (WebCore::XMLTokenizer::insertErrorMessageBlock):
9:24 PM Changeset in webkit [12567] by mjs
  • 17 edits
    110 deletes in trunk

LayoutTests:

  • reverted CSS media queries patch, as it causes random crashes on the layout tests with a debug build.


  • fast/media/001-expected.checksum: Removed.
  • fast/media/001-expected.png: Removed.
  • fast/media/001-expected.txt: Removed.
  • fast/media/001.html: Removed.
  • fast/media/002-expected.checksum: Removed.
  • fast/media/002-expected.png: Removed.
  • fast/media/002-expected.txt: Removed.
  • fast/media/002.html: Removed.
  • fast/media/003-expected.checksum: Removed.
  • fast/media/003-expected.png: Removed.
  • fast/media/003-expected.txt: Removed.
  • fast/media/003.html: Removed.
  • fast/media/004-expected.checksum: Removed.
  • fast/media/004-expected.png: Removed.
  • fast/media/004-expected.txt: Removed.
  • fast/media/004.html: Removed.
  • fast/media/005-expected.checksum: Removed.
  • fast/media/005-expected.png: Removed.
  • fast/media/005-expected.txt: Removed.
  • fast/media/005.html: Removed.
  • fast/media/006-expected.checksum: Removed.
  • fast/media/006-expected.png: Removed.
  • fast/media/006-expected.txt: Removed.
  • fast/media/006.html: Removed.
  • fast/media/007-expected.checksum: Removed.
  • fast/media/007-expected.png: Removed.
  • fast/media/007-expected.txt: Removed.
  • fast/media/007.html: Removed.
  • fast/media/008-expected.checksum: Removed.
  • fast/media/008-expected.png: Removed.
  • fast/media/008-expected.txt: Removed.
  • fast/media/008.html: Removed.
  • fast/media/009-expected.checksum: Removed.
  • fast/media/009-expected.png: Removed.
  • fast/media/009-expected.txt: Removed.
  • fast/media/009.html: Removed.
  • fast/media/010-expected.checksum: Removed.
  • fast/media/010-expected.png: Removed.
  • fast/media/010-expected.txt: Removed.
  • fast/media/010.html: Removed.
  • fast/media/011-expected.checksum: Removed.
  • fast/media/011-expected.png: Removed.
  • fast/media/011-expected.txt: Removed.
  • fast/media/011.html: Removed.
  • fast/media/012-expected.checksum: Removed.
  • fast/media/012-expected.png: Removed.
  • fast/media/012-expected.txt: Removed.
  • fast/media/012.html: Removed.
  • fast/media/013-expected.checksum: Removed.
  • fast/media/013-expected.png: Removed.
  • fast/media/013-expected.txt: Removed.
  • fast/media/013.html: Removed.
  • fast/media/014-expected.checksum: Removed.
  • fast/media/014-expected.png: Removed.
  • fast/media/014-expected.txt: Removed.
  • fast/media/014.html: Removed.
  • fast/media/015-expected.checksum: Removed.
  • fast/media/015-expected.png: Removed.
  • fast/media/015-expected.txt: Removed.
  • fast/media/015.html: Removed.
  • fast/media/016-expected.checksum: Removed.
  • fast/media/016-expected.png: Removed.
  • fast/media/016-expected.txt: Removed.
  • fast/media/016.html: Removed.
  • fast/media/017-expected.checksum: Removed.
  • fast/media/017-expected.png: Removed.
  • fast/media/017-expected.txt: Removed.
  • fast/media/017.html: Removed.
  • fast/media/018-expected.checksum: Removed.
  • fast/media/018-expected.png: Removed.
  • fast/media/018-expected.txt: Removed.
  • fast/media/018.html: Removed.
  • fast/media/019-expected.checksum: Removed.
  • fast/media/019-expected.png: Removed.
  • fast/media/019-expected.txt: Removed.
  • fast/media/019.html: Removed.
  • fast/media/020-expected.checksum: Removed.
  • fast/media/020-expected.png: Removed.
  • fast/media/020-expected.txt: Removed.
  • fast/media/020.html: Removed.
  • fast/media/021-expected.checksum: Removed.
  • fast/media/021-expected.png: Removed.
  • fast/media/021-expected.txt: Removed.
  • fast/media/021.html: Removed.
  • fast/media/022-expected.checksum: Removed.
  • fast/media/022-expected.png: Removed.
  • fast/media/022-expected.txt: Removed.
  • fast/media/022.html: Removed.
  • fast/media/023-expected.checksum: Removed.
  • fast/media/023-expected.png: Removed.
  • fast/media/023-expected.txt: Removed.
  • fast/media/023.html: Removed.
  • fast/media/024-expected.checksum: Removed.
  • fast/media/024-expected.png: Removed.
  • fast/media/024-expected.txt: Removed.
  • fast/media/024.html: Removed.
  • fast/media/025-expected.checksum: Removed.
  • fast/media/025-expected.png: Removed.
  • fast/media/025-expected.txt: Removed.
  • fast/media/025.html: Removed.
  • fast/media/026-expected.checksum: Removed.
  • fast/media/026-expected.png: Removed.
  • fast/media/026.html: Removed.
  • fast/media/import-p-green.css: Removed.

WebCore:

  • reverted CSS media queries patch, as it causes random crashes on the layout tests with a debug build.


  • WebCore.xcodeproj/project.pbxproj:
  • css/css_grammar.y:
  • css/css_mediaqueryeval.cpp: Removed.
  • css/css_mediaqueryeval.h: Removed.
  • css/css_mediaqueryimpl.cpp: Removed.
  • css/css_mediaqueryimpl.h: Removed.
  • css/css_stylesheetimpl.cpp: (MediaListImpl::MediaListImpl): (MediaListImpl::contains): (MediaListImpl::deleteMedium): (MediaListImpl::mediaText): (MediaListImpl::setMediaText):
  • css/css_stylesheetimpl.h: (DOM::MediaListImpl::length): (DOM::MediaListImpl::item): (DOM::MediaListImpl::appendMedium):
  • css/css_valueimpl.h:
  • css/csshelper.h:
  • css/cssmediafeatures.in: Removed.
  • css/cssparser.cpp:
  • css/cssparser.h:
  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::CSSStyleSelector): (WebCore::CSSStyleSelector::init): (WebCore::CSSStyleSelector::~CSSStyleSelector): (WebCore::CSSStyleSelector::loadDefaultStyle): (WebCore::CSSStyleSelector::styleForElement): (WebCore::CSSStyleSelector::styleRulesForElement): (WebCore::CSSRuleSet::addRulesFromSheet): (WebCore::CSSStyleSelector::getColorFromPrimitiveValue):
  • css/cssstyleselector.h:
  • css/makemediafeatures: Removed.
  • css/maketokenizer:
  • css/tokenizer.flex:
  • khtml/html/html_headimpl.cpp: (WebCore::HTMLLinkElementImpl::process): (WebCore::HTMLLinkElementImpl::setStyleSheet): (WebCore::HTMLStyleElementImpl::childrenChanged):
  • platform/Screen.h:
  • platform/mac/Screen.mm:
5:48 PM Changeset in webkit [12566] by darin
  • 3 edits in trunk/WebKit

Reviewed by Maciej.

  • Misc.subproj/WebCoreStatistics.h:
  • Misc.subproj/WebCoreStatistics.m: (+[WebCoreStatistics javaScriptRootObjectTypeCounts]): javaScriptRootObjecTypeCounts -> javaScriptRootObjectTypeCounts
5:43 PM Changeset in webkit [12565] by darin
  • 1 edit
    3 adds in trunk/LayoutTests
  • fast/js/for-in-var-scope.html: Added.
  • fast/js/resources/for-in-var-scope.js: Added.
  • fast/js/for-in-var-scope-expected.txt: Added.
5:43 PM Changeset in webkit [12564] by darin
  • 4 edits in trunk/JavaScriptCore

Reviewed by Maciej.

Test: fast/js/for-in-var-scope.html

  • kjs/nodes.cpp: (valueForReadModifyAssignment): Use ALWAYS_INLINE macro. (ForInNode::execute): Break out of the scope chain loop once we find and set the loop variable. We don't want to set multiple loop variables. (ForInNode::processVarDecls): Process the declaration of the loop variable.
  • other cleanup
  • kjs/object.cpp: (KJS::tryGetAndCallProperty): Use ALWAYS_INLINE macro.
  • kxmlcore/FastMalloc.cpp: Change to use ALWAYS_INLINE macro from AlwaysInline.h instead of defining it here a second time.
5:09 PM Changeset in webkit [12563] by darin
  • 1 edit in trunk/LayoutTests/ChangeLog

Oops, rolled out part that wasn't supposed to be here.

5:07 PM Changeset in webkit [12562] by darin
  • 2 edits in trunk/LayoutTests
  • updated test result due to revised fix for <ol> number
  • fast/lists/olstart-expected.txt: Updated.
5:03 PM Changeset in webkit [12561] by darin
  • 2 edits in trunk/WebKitTools
  • Scripts/webkitdirs.pm: Add a compatibilty hack for people with old Configuration files that say Development or Deployment in them.
4:49 PM Changeset in webkit [12560] by darin
  • 4 edits in trunk/WebCore

Reviewed and tweaked by Darin.

  • fixed regressions caused by fix for bug 5776 earlier today
  • rendering/render_list.h: Remove m_value from RenderListMarker, add it to RenderListItem. A few other tweaks, including getting rid of "friend" relationship.
  • rendering/render_list.cpp: (RenderListItem::RenderListItem): Initialize m_value. (RenderListItem::setStyle): Restore old behavior of making no marker for LNONE case. (RenderListItem::calcValue): Work on m_value, not m_marker->m_value. (RenderListItem::resetValue): Reset m_value, even if there's no marker. (RenderListMarker::RenderListMarker): Remove code to set up m_value. (RenderListMarker::calcMinMaxWidth): Get marker value from list item.
  • rendering/RenderContainer.cpp: (updateListMarkerNumbers): Call resetValue by its new name.
3:29 PM Changeset in webkit [12559] by mjs
  • 10 edits in trunk

JavaScriptCore:

Reviewed by Hyatt.



  • kjs/collector.cpp: (KJS::Collector::numProtectedObjects): renamed from numReferencedObjects (KJS::typeName): (KJS::Collector::rootObjectTypeCounts): renamed from rootObjectClasses, use HashSet
  • kjs/collector.h: (KJS::Collector::isOutOfMemory): Renamed from outOfMemory.
  • kjs/nodes.cpp:

WebCore:

Reviewed by Hyatt.

  • kwq/WebCoreJavaScript.h:
  • kwq/WebCoreJavaScript.mm: (+[WebCoreJavaScript protectedObjectCount]): Renamed from referencedObjectCounts (+[WebCoreJavaScript rootObjectTypeCounts]): Renamed from rootObjectClasses, changed from NSSet to NSCountedSet.

WebKit:

Reviewed by Hyatt.

  • Misc.subproj/WebCoreStatistics.h:
  • Misc.subproj/WebCoreStatistics.m: (+[WebCoreStatistics javaScriptProtectedObjectsCount]): new (+[WebCoreStatistics javaScriptRootObjecTypeCounts]): new (+[WebCoreStatistics javaScriptRootObjectClasses]): deprecated (+[WebCoreStatistics javaScriptReferencedObjectsCount]): deprecated (+[WebCoreStatistics javaScriptNoGCAllowedObjectsCount]): Just return 0. Deprecated.
2:59 PM Changeset in webkit [12558] by mjs
  • 11 edits in trunk/WebCore

Reviewed by Darin.

  • WebCore.xcodeproj/project.pbxproj: reordered slightly.
  • bridge/mac/MacFrame.h:
  • bridge/mac/MacFrame.mm: (WebCore::MacFrame::createFrame): Don't take isObject parameter, just check tag name
  • khtml/html/HTMLFormElementImpl.cpp: (WebCore::HTMLFormElementImpl::prepareSubmit): remove obsolete hasOnlyLocalReferences check
  • khtml/html/html_baseimpl.cpp: (WebCore::HTMLFrameElementImpl::isURLAllowed): remove obsolete hasOnlyLocalReferences check
  • loader/DocLoader.cpp: (khtml::DocLoader::requestImage): remove obsolete hasOnlyLocalReferences check (khtml::DocLoader::requestStyleSheet): remove obsolete hasOnlyLocalReferences check (khtml::DocLoader::requestScript): remove obsolete hasOnlyLocalReferences check (khtml::DocLoader::requestXSLStyleSheet): remove obsolete hasOnlyLocalReferences check
  • page/Frame.cpp: (WebCore::Frame::requestFrame): refactored and simplified (WebCore::Frame::requestObject): ditto (WebCore::Frame::shouldUsePlugin): ditto (WebCore::Frame::loadPlugin): ditto (WebCore::Frame::loadSubframe): ditto (WebCore::Frame::frameNames): remove obsolete m_bPreloaded check (WebCore::Frame::frames): ditto (WebCore::Frame::connectChild): check isFrame, not child type (WebCore::Frame::disconnectChild): ditto (WebCore::Frame::handleFallbackContent): greatly simplify using onwerElement (WebCore::Frame::updateBaseURLForEmptyDocument): use ownerElement, notChildFrame (WebCore::Frame::onlyLocalReferences): removed (WebCore::Frame::setOnlyLocalReferences): removed
  • page/Frame.h:
  • page/FramePrivate.h: (WebCore::ChildFrame::ChildFrame): remove most fields.
  • rendering/render_frames.cpp: (WebCore::RenderPartObject::updateWidget): Remove redundant setting of base URL for child document of iframe, Frame itself takes care of this already
10:51 AM Changeset in webkit [12557] by darin
  • 4 edits in trunk/WebKitTools
  • Scripts/make-js-test-wrappers: Don't create a wrapper if there's a disabled wrapper already in the directory.
  • Scripts/svn-apply: Handle additions and deletions properly -- I've been noticing these haven't been working at all.
  • Scripts/svn-unapply: Ditto.
10:30 AM Changeset in webkit [12556] by darin
  • 1 edit in trunk/LayoutTests/ChangeLog

Fixed reviewer name.

8:42 AM Changeset in webkit [12555] by darin
  • 1 edit in trunk/WebCore/khtml/html/html_listimpl.cpp

Reviewed and tweaked by Darin.



Uses the DOM tree to calculate previous nodes and nest correctly

Tests:

  • fast/lists/olstart.html
  • fast/lists/ol-display-types.html
  • khtml/html/html_listimpl.cpp: (DOM::HTMLLIElementImpl::attach): remove calculation of start node value
  • rendering/render_list.cpp: (RenderListItem::setStyle): set up an m_marker even if the style is to not display the marker, to get the numbering right (enclosingList): gets the list enclosing the given node (previousListItemElement): gets the previous element in the list (RenderListItem::calcListValue): calculate the value correctly

Feb 3, 2006:

8:57 PM Changeset in webkit [12554] by darin
  • 1 edit
    8 adds in trunk/LayoutTests

Reviewed by NOBODY (OOPS!).


Layout tests for bug 5776: Generated numbers of ordered lists (OL) are
not calculated right

  • fast/lists/ol-display-types-expected.checksum: Added.
  • fast/lists/ol-display-types-expected.png: Added.
  • fast/lists/ol-display-types-expected.txt: Added.
  • fast/lists/ol-display-types.html: Added.
  • fast/lists/olstart-expected.checksum: Added.
  • fast/lists/olstart-expected.png: Added.
  • fast/lists/olstart-expected.txt: Added.
  • fast/lists/olstart.html: Added.
8:56 PM Changeset in webkit [12553] by darin
  • 2 edits in trunk/WebCore

Reviewed and tweaked by Darin.



Uses the DOM tree to calculate previous nodes and nest correctly

Tests:

  • fast/lists/olstart.html
  • fast/lists/ol-display-types.html
  • khtml/html/html_listimpl.cpp: (DOM::HTMLLIElementImpl::attach): remove calculation of start node value
  • rendering/render_list.cpp: (RenderListItem::setStyle): set up an m_marker even if the style is to not display the marker, to get the numbering right (enclosingList): gets the list enclosing the given node (previousListItemElement): gets the previous element in the list (RenderListItem::calcListValue): calculate the value correctly
6:54 PM Changeset in webkit [12552] by thatcher
  • 1 edit in trunk/WebCore/ChangeLog

Correct the ChangeLog to credit Kimmo Kinnunen <​kimmo.t.kinnunen@nokia.com>

6:14 PM Changeset in webkit [12551] by ggaren
  • 1 edit in trunk/WebCore/ChangeLog

beth felt like the changelog was judging her

6:12 PM Changeset in webkit [12550] by ggaren
  • 3 edits in trunk/WebCore

Reviewed beth.

  • Fixed <rdar://problem/4394868> window.open fails to parse "width=200px"

Cause: QString::toInt() is too aggressive; it parses letters as numbers.
The fix is to switch to DOMString. It's all the rage these days,
and its toInt() behavior matches what we want.

  • manual-tests/window-open-features-parsing.html: Added "px" case to giant evil string.
  • khtml/ecma/kjs_window.cpp: (KJS::setWindowFeature): (KJS::parseWindowFeatures): (KJS::WindowFunc::callAsFunction):
6:01 PM Changeset in webkit [12549] by thatcher
  • 17 edits
    111 adds in trunk

LayoutTests:

Reviewed by Dave Hyatt, Tim H.

Layout tests for:
Bug 4127: WebCore doesn't support Media Queries (CSS3 module)
​http://bugzilla.opendarwin.org/show_bug.cgi?id=4127

  • ChangeLog:
  • fast/media: Added.
  • fast/media/001-expected.checksum: Added.
  • fast/media/001-expected.png: Added.
  • fast/media/001-expected.txt: Added.
  • fast/media/001.html: Added.
  • fast/media/002-expected.checksum: Added.
  • fast/media/002-expected.png: Added.
  • fast/media/002-expected.txt: Added.
  • fast/media/002.html: Added.
  • fast/media/003-expected.checksum: Added.
  • fast/media/003-expected.png: Added.
  • fast/media/003-expected.txt: Added.
  • fast/media/003.html: Added.
  • fast/media/004-expected.checksum: Added.
  • fast/media/004-expected.png: Added.
  • fast/media/004-expected.txt: Added.
  • fast/media/004.html: Added.
  • fast/media/005-expected.checksum: Added.
  • fast/media/005-expected.png: Added.
  • fast/media/005-expected.txt: Added.
  • fast/media/005.html: Added.
  • fast/media/006-expected.checksum: Added.
  • fast/media/006-expected.png: Added.
  • fast/media/006-expected.txt: Added.
  • fast/media/006.html: Added.
  • fast/media/007-expected.checksum: Added.
  • fast/media/007-expected.png: Added.
  • fast/media/007-expected.txt: Added.
  • fast/media/007.html: Added.
  • fast/media/008-expected.checksum: Added.
  • fast/media/008-expected.png: Added.
  • fast/media/008-expected.txt: Added.
  • fast/media/008.html: Added.
  • fast/media/009-expected.checksum: Added.
  • fast/media/009-expected.png: Added.
  • fast/media/009-expected.txt: Added.
  • fast/media/009.html: Added.
  • fast/media/010-expected.checksum: Added.
  • fast/media/010-expected.png: Added.
  • fast/media/010-expected.txt: Added.
  • fast/media/010.html: Added.
  • fast/media/011-expected.checksum: Added.
  • fast/media/011-expected.png: Added.
  • fast/media/011-expected.txt: Added.
  • fast/media/011.html: Added.
  • fast/media/012-expected.checksum: Added.
  • fast/media/012-expected.png: Added.
  • fast/media/012-expected.txt: Added.
  • fast/media/012.html: Added.
  • fast/media/013-expected.checksum: Added.
  • fast/media/013-expected.png: Added.
  • fast/media/013-expected.txt: Added.
  • fast/media/013.html: Added.
  • fast/media/014-expected.checksum: Added.
  • fast/media/014-expected.png: Added.
  • fast/media/014-expected.txt: Added.
  • fast/media/014.html: Added.
  • fast/media/015-expected.checksum: Added.
  • fast/media/015-expected.png: Added.
  • fast/media/015-expected.txt: Added.
  • fast/media/015.html: Added.
  • fast/media/016-expected.checksum: Added.
  • fast/media/016-expected.png: Added.
  • fast/media/016-expected.txt: Added.
  • fast/media/016.html: Added.
  • fast/media/017-expected.checksum: Added.
  • fast/media/017-expected.png: Added.
  • fast/media/017-expected.txt: Added.
  • fast/media/017.html: Added.
  • fast/media/018-expected.checksum: Added.
  • fast/media/018-expected.png: Added.
  • fast/media/018-expected.txt: Added.
  • fast/media/018.html: Added.
  • fast/media/019-expected.checksum: Added.
  • fast/media/019-expected.png: Added.
  • fast/media/019-expected.txt: Added.
  • fast/media/019.html: Added.
  • fast/media/020-expected.checksum: Added.
  • fast/media/020-expected.png: Added.
  • fast/media/020-expected.txt: Added.
  • fast/media/020.html: Added.
  • fast/media/021-expected.checksum: Added.
  • fast/media/021-expected.png: Added.
  • fast/media/021-expected.txt: Added.
  • fast/media/021.html: Added.
  • fast/media/022-expected.checksum: Added.
  • fast/media/022-expected.png: Added.
  • fast/media/022-expected.txt: Added.
  • fast/media/022.html: Added.
  • fast/media/023-expected.checksum: Added.
  • fast/media/023-expected.png: Added.
  • fast/media/023-expected.txt: Added.
  • fast/media/023.html: Added.
  • fast/media/024-expected.checksum: Added.
  • fast/media/024-expected.png: Added.
  • fast/media/024-expected.txt: Added.
  • fast/media/024.html: Added.
  • fast/media/025-expected.checksum: Added.
  • fast/media/025-expected.png: Added.
  • fast/media/025-expected.txt: Added.
  • fast/media/025.html: Added.
  • fast/media/026-expected.checksum: Added.
  • fast/media/026-expected.png: Added.
  • fast/media/026.html: Added.
  • fast/media/import-p-green.css: Added.

WebCore:

Reviewed by Dave Hyatt and Tim H.

Bug 4127: WebCore doesn't support Media Queries (CSS3 module)
​http://bugzilla.opendarwin.org/show_bug.cgi?id=4127

​http://www.w3.org/TR/css3-mediaqueries/

  • WebCore.xcodeproj/project.pbxproj:
  • css/css_grammar.y:
  • css/css_mediaqueryeval.cpp: Added. (MediaQueryEvaluator): (MediaQueryEvaluator::~MediaQueryEvaluator): (MediaQueryEvaluator::mediaTypeMatch): (applyRestrictor): (MediaQueryEvaluator::eval): (parseAspectRatio):
  • css/css_mediaqueryeval.h: Added.
  • css/css_mediaqueryimpl.cpp: Added. (MediaQueryExpImpl::MediaQueryExpImpl): (MediaQueryExpImpl::~MediaQueryExpImpl): (MediaQueryExpImpl::mediaFeature): (MediaQueryExpImpl::value): (MediaQueryImpl::MediaQueryImpl): (MediaQueryImpl::~MediaQueryImpl): (MediaQueryImpl::restrictor): (MediaQueryImpl::expressions): (MediaQueryImpl::mediaType): (MediaQueryImpl::operator==):
  • css/css_mediaqueryimpl.h: Added. (DOM::MediaQueryExpImpl::operator==): (DOM::MediaQueryExpListImpl::MediaQueryExpListImpl): (DOM::MediaQueryExpListImpl::append): (DOM::MediaQueryExpListImpl::list): (DOM::MediaQueryImpl::):
  • css/css_stylesheetimpl.cpp: (MediaListImpl::MediaListImpl): (MediaListImpl::deleteMedium): (mediaQueryAsDOMString): (MediaListImpl::mediaText): (MediaListImpl::setMediaText): (MediaListImpl::item): (MediaListImpl::appendMedium): (MediaListImpl::appendMediaQuery):
  • css/css_stylesheetimpl.h: (DOM::MediaListImpl::length): (DOM::MediaListImpl::mediaQueries):
  • css/css_valueimpl.h:
  • css/csshelper.h:
  • css/cssmediafeatures.in: Added.
  • css/cssparser.cpp: (CSSParser::parseMediaQuery):
  • css/cssparser.h:
  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::CSSStyleSelector): (WebCore::CSSStyleSelector::init): (WebCore::CSSStyleSelector::~CSSStyleSelector): (WebCore::CSSStyleSelector::loadDefaultStyle): (WebCore::CSSStyleSelector::styleForElement): (WebCore::CSSStyleSelector::styleRulesForElement): (WebCore::CSSRuleSet::addRulesFromSheet): (WebCore::CSSStyleSelector::defaultStyleForRoot):
  • css/cssstyleselector.h:
  • css/makemediafeatures: Added.
  • css/maketokenizer:
  • css/tokenizer.flex:
  • khtml/html/html_headimpl.cpp: (WebCore::HTMLLinkElementImpl::process): (WebCore::HTMLLinkElementImpl::setStyleSheet): (WebCore::HTMLStyleElementImpl::childrenChanged):
  • platform/Screen.h:
  • platform/mac/Screen.mm: (WebCore::screenDepthPerComponent): (WebCore::screenIsMonochrome):
5:48 PM Changeset in webkit [12548] by mjs
  • 2 edits in trunk/WebKit

Reviewed by Darin.


  • change spellchecker preflighting to happen via class methods instead of object methods.
  • WebView.subproj/WebView.m: (-[WebView setContinuousSpellCheckingEnabled:]): (+[WebView _preflightSpellCheckerNow:]): (+[WebView _preflightSpellChecker]):
3:34 PM Changeset in webkit [12547] by thatcher
  • 14 edits in trunk

JavaScriptCore:

Reviewed by Justin.

Renamed configuration names to Debug, Release and Production.

JavaScriptGlue:

Reviewed by Justin.

Renamed configuration names to Debug, Release and Production.

  • JavaScriptGlue.xcodeproj/project.pbxproj:

WebCore:

Reviewed by Justin.

Renamed configuration names to Debug, Release and Production.

  • WebCore.xcodeproj/project.pbxproj:

WebKit:

Reviewed by Justin.

Renamed configuration names to Debug, Release and Production.

  • WebKit.xcodeproj/project.pbxproj:

WebKitTools:

Reviewed by Justin.

Renamed configuration names to Debug, Release and Production.

  • DrawTest/DrawTest.xcodeproj/project.pbxproj:
  • DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj:
  • Scripts/set-webkit-configuration:
  • Scripts/webkitdirs.pm:
2:46 PM Changeset in webkit [12546] by bdakin
  • 2 edits in trunk/WebCore

Reviewed by Hyatt.

No test cases added because this is an unreproducible crash.

This is a speculative fix for <rdar://problem/4048026>
CrashTracer: [BOUNCE BACK] 1219 crashes in Safari at
com.apple.WebCore: KHTMLParser::popOneBlock + 36

It seems possible that current could be nil when this function is
called, so the speculative fix is to add a nil check.

3:22 AM Changeset in webkit [12545] by hyatt
  • 4 edits
    6 adds in trunk/WebCore

Refactoring of the image classes to make them even more
portable.

Reviewed by mjs

  • WebCore.xcodeproj/project.pbxproj:
  • platform/Image.cpp: (WebCore::m_animationObserver): (WebCore::Image::Image): (WebCore::Image::~Image): (WebCore::Image::resetAnimation): (WebCore::Image::setData): (WebCore::Image::isNull): (WebCore::Image::size): (WebCore::Image::rect): (WebCore::Image::width): (WebCore::Image::height): (WebCore::): (WebCore::Image::compositeOperatorFromString):
  • platform/ImageData.cpp: Added. (WebCore::FrameData::clear): (WebCore::ImageData::ImageData): (WebCore::ImageData::~ImageData): (WebCore::ImageData::invalidateData): (WebCore::ImageData::cacheFrame): (WebCore::ImageData::isNull): (WebCore::ImageData::size): (WebCore::ImageData::setData): (WebCore::ImageData::setNativeData): (WebCore::ImageData::frameCount): (WebCore::ImageData::isSizeAvailable): (WebCore::ImageData::frameAtIndex): (WebCore::ImageData::frameDurationAtIndex): (WebCore::ImageData::shouldAnimate): (WebCore::ImageData::startAnimation): (WebCore::ImageData::stopAnimation): (WebCore::ImageData::resetAnimation): (WebCore::ImageData::advanceAnimation):
  • platform/ImageData.h: Added. (KXMLCore::): (WebCore::FrameData::m_duration): (WebCore::FrameData::~FrameData): (WebCore::ImageData::setIsPDF): (WebCore::ImageData::currentFrame):
  • platform/ImageDecoder.h:
  • platform/mac/Image.mm: (WebCore::Image::loadResource): (WebCore::Image::supportsType): (WebCore::Image::getCGImageRef): (WebCore::Image::getNSImage): (WebCore::Image::getTIFFRepresentation): (WebCore::graphicsContext): (WebCore::setCompositingOperation): (WebCore::fillSolidColorInRect): (WebCore::Image::drawInRect): (WebCore::drawPattern): (WebCore::Image::tileInRect): (WebCore::Image::scaleAndTileInRect):
  • platform/mac/ImageData.mm: Added. (WebCore::setCompositingOperation): (WebCore::ImageData::invalidateAppleSpecificData): (WebCore::ImageData::checkForSolidColor): (WebCore::ImageData::getTIFFRepresentation): (WebCore::ImageData::getNSImage):
  • platform/mac/PDFDocumentImage.h: Added.
  • platform/mac/PDFDocumentImage.mm: Added. (WebCore::releasePDFDocumentData): (WebCore::PDFDocumentImage::PDFDocumentImage): (WebCore::PDFDocumentImage::~PDFDocumentImage): (WebCore::PDFDocumentImage::documentRef): (WebCore::PDFDocumentImage::mediaBox): (WebCore::PDFDocumentImage::bounds): (WebCore::PDFDocumentImage::adjustCTM): (WebCore::PDFDocumentImage::setCurrentPage): (WebCore::PDFDocumentImage::currentPage): (WebCore::PDFDocumentImage::pageCount): (WebCore::PDFDocumentImage::draw):
1:11 AM Changeset in webkit [12544] by hyatt
  • 3 edits
    2 adds in trunk/WebCore

Make a new class called ImageDecoder that wraps a CGImageSource.
This paves the way for ImageData to become portable as well.

Reviewed by mjs

  • WebCore.xcodeproj/project.pbxproj:
  • platform/ImageDecoder.h: Added.
  • platform/mac/Image.mm: (WebCore::ImageData::~ImageData): (WebCore::ImageData::invalidateData): (WebCore::ImageData::cacheFrame): (WebCore::ImageData::size): (WebCore::ImageData::setCFData): (WebCore::ImageData::frameCount): (WebCore::ImageData::isSizeAvailable): (WebCore::ImageData::drawInRect):
  • platform/mac/ImageDecoder.cpp: Added. (WebCore::ImageDecoder::ImageDecoder): (WebCore::ImageDecoder::~ImageDecoder): (WebCore::imageSourceOptions): (WebCore::ImageDecoder::initialized): (WebCore::ImageDecoder::setData): (WebCore::ImageDecoder::isSizeAvailable): (WebCore::ImageDecoder::size): (WebCore::ImageDecoder::repetitionCount): (WebCore::ImageDecoder::frameCount): (WebCore::ImageDecoder::createFrameAtIndex): (WebCore::ImageDecoder::frameDurationAtIndex):

Feb 2, 2006:

11:32 PM Changeset in webkit [12543] by hyatt
  • 5 edits
    1 add in trunk/WebCore

Move ImageAnimationObserver into its own file.

Reviewed by mjs

  • WebCore.xcodeproj/project.pbxproj:
  • loader/CachedImage.h:
  • platform/Image.h:
  • platform/ImageAnimationObserver.h: Added. (WebCore::ImageAnimationObserver::~ImageAnimationObserver):
  • platform/mac/Image.mm:
11:18 PM Changeset in webkit [12542] by mjs
  • 8 edits
    1 add in trunk/WebCore

Reviewed by Adele.

  • remove all use of the ChildFrame from outside Frame.cpp, in preparation for removing it
  • bridge/mac/MacFrame.h:
  • bridge/mac/MacFrame.mm: (WebCore::MacFrame::frameDetached): Factor out the ChildFrame manipulations into a base class method. (WebCore::MacFrame::objectContentType): factored out from createPart, calls the bridge to determine contents for an Object tag. (WebCore::MacFrame::createPlugin): factor out code from createPart that makes a Plugin. (WebCore::MacFrame::createFrame): ditto for code that makes a Frame. (WebCore::MacFrame::createEmptyDocument): factored out part into the superclass
  • khtml/xml/DocumentImpl.cpp: (WebCore::DocumentImpl::ownerElement): Call new ownerElement method on Frame instead of mucking at Frame internals.
  • page/Frame.cpp: (WebCore::Frame::createPart): Moved here from MacFrame, but some of the guts were factored out. (WebCore::Frame::ownerElement): New method, logic moved from DocumentImpl. (WebCore::Frame::frameDetached): New virtual method, the part of this that pokes at ChildFrame structures. (WebCore::Frame::updateBaseURLForEmptyDocument): Helper that looks at the ChildFrame array for the benefit of createEmptyDocument.
  • page/Frame.h:
  • page/Page.h: Fix up modeline.
  • page/Plugin.h: Added. (WebCore::Plugin::Plugin): Moved here from MacFrame.mm, since it is now used by Frame. (WebCore::Plugin::view): ditto
9:47 PM Changeset in webkit [12541] by hyatt
  • 2 edits in trunk/WebCore

Avoid extra copies of all the decoded image bytes.

Reviewed by mjs

  • platform/mac/Image.mm: (WebCore::ImageData::setData):
5:16 PM Changeset in webkit [12540] by justing
  • 141 edits
    4 adds in trunk/LayoutTests

Reviewed by harrison


Turned on continuous spell checking to test the fix for:
<​http://bugzilla.opendarwin.org/show_bug.cgi?id=6586>
REGRESSION: visible_text.cpp assertion failure in Mail.app

Added:

  • editing/selection/end-of-document.html

Pixel results for editing tests now include misspelling markers.
A few of them are incorrect, filed 7038 to cover this.

  • editing/deleting/delete-3928305-fix-expected.checksum:
  • editing/deleting/delete-3928305-fix-expected.png:
  • editing/deleting/delete-3959464-fix-expected.checksum:
  • editing/deleting/delete-3959464-fix-expected.png:
  • editing/deleting/delete-after-span-ws-001-expected.checksum:
  • editing/deleting/delete-after-span-ws-001-expected.png:
  • editing/deleting/delete-after-span-ws-002-expected.checksum:
  • editing/deleting/delete-after-span-ws-002-expected.png:
  • editing/deleting/delete-after-span-ws-003-expected.checksum:
  • editing/deleting/delete-after-span-ws-003-expected.png:
  • editing/deleting/delete-and-undo-expected.checksum:
  • editing/deleting/delete-and-undo-expected.png:
  • editing/deleting/delete-block-contents-003-expected.checksum:
  • editing/deleting/delete-block-contents-003-expected.png:
  • editing/deleting/delete-block-merge-contents-010-expected.checksum:
  • editing/deleting/delete-block-merge-contents-010-expected.png:
  • editing/deleting/delete-contiguous-ws-001-expected.checksum:
  • editing/deleting/delete-contiguous-ws-001-expected.png:
  • editing/deleting/delete-tab-002-expected.checksum:
  • editing/deleting/delete-tab-002-expected.png:
  • editing/deleting/delete-tab-003-expected.checksum:
  • editing/deleting/delete-tab-003-expected.png:
  • editing/deleting/delete-trailing-ws-002-expected.checksum:
  • editing/deleting/delete-trailing-ws-002-expected.png:
  • editing/deleting/delete-ws-fixup-001-expected.checksum:
  • editing/deleting/delete-ws-fixup-001-expected.png:
  • editing/deleting/delete-ws-fixup-002-expected.checksum:
  • editing/deleting/delete-ws-fixup-002-expected.png:
  • editing/inserting/editing-empty-divs-expected.checksum:
  • editing/inserting/editing-empty-divs-expected.png:
  • editing/inserting/insert-3659587-fix-expected.checksum:
  • editing/inserting/insert-3659587-fix-expected.png:
  • editing/inserting/insert-br-004-expected.checksum:
  • editing/inserting/insert-br-004-expected.png:
  • editing/inserting/insert-br-005-expected.checksum:
  • editing/inserting/insert-br-005-expected.png:
  • editing/inserting/insert-br-007-expected.checksum:
  • editing/inserting/insert-br-007-expected.png:
  • editing/inserting/insert-div-007-expected.checksum:
  • editing/inserting/insert-div-007-expected.png:
  • editing/inserting/insert-div-012-expected.checksum:
  • editing/inserting/insert-div-012-expected.png:
  • editing/inserting/insert-div-013-expected.checksum:
  • editing/inserting/insert-div-013-expected.png:
  • editing/inserting/insert-div-014-expected.checksum:
  • editing/inserting/insert-div-014-expected.png:
  • editing/inserting/insert-div-015-expected.checksum:
  • editing/inserting/insert-div-015-expected.png:
  • editing/inserting/insert-div-016-expected.checksum:
  • editing/inserting/insert-div-016-expected.png:
  • editing/inserting/insert-div-017-expected.checksum:
  • editing/inserting/insert-div-017-expected.png:
  • editing/inserting/insert-div-018-expected.checksum:
  • editing/inserting/insert-div-018-expected.png:
  • editing/inserting/insert-div-019-expected.checksum:
  • editing/inserting/insert-div-019-expected.png:
  • editing/inserting/insert-div-020-expected.checksum:
  • editing/inserting/insert-div-020-expected.png:
  • editing/inserting/insert-div-021-expected.checksum:
  • editing/inserting/insert-div-021-expected.png:
  • editing/inserting/insert-div-022-expected.checksum:
  • editing/inserting/insert-div-022-expected.png:
  • editing/inserting/insert-div-025-expected.checksum:
  • editing/inserting/insert-div-025-expected.png:
  • editing/inserting/insert-div-027-expected.checksum:
  • editing/inserting/insert-div-027-expected.png:
  • editing/inserting/insert-tab-002-expected.checksum:
  • editing/inserting/insert-tab-002-expected.png:
  • editing/inserting/insert-tab-003-expected.checksum:
  • editing/inserting/insert-tab-003-expected.png:
  • editing/inserting/insert-text-with-newlines-expected.checksum:
  • editing/inserting/insert-text-with-newlines-expected.png:
  • editing/inserting/return-key-with-selection-001-expected.checksum:
  • editing/inserting/return-key-with-selection-001-expected.png:
  • editing/inserting/return-key-with-selection-002-expected.checksum:
  • editing/inserting/return-key-with-selection-002-expected.png:
  • editing/inserting/return-key-with-selection-003-expected.checksum:
  • editing/inserting/return-key-with-selection-003-expected.png:
  • editing/inserting/typing-002-expected.checksum:
  • editing/inserting/typing-002-expected.png:
  • editing/inserting/typing-around-br-001-expected.checksum:
  • editing/inserting/typing-around-br-001-expected.png:
  • editing/pasteboard/cut-text-001-expected.checksum:
  • editing/pasteboard/cut-text-001-expected.png:
  • editing/pasteboard/paste-list-001-expected.checksum:
  • editing/pasteboard/paste-text-001-expected.checksum:
  • editing/pasteboard/paste-text-001-expected.png:
  • editing/pasteboard/paste-text-012-expected.checksum:
  • editing/pasteboard/paste-text-012-expected.png:
  • editing/pasteboard/paste-text-013-expected.checksum:
  • editing/pasteboard/paste-text-013-expected.png:
  • editing/pasteboard/paste-text-014-expected.checksum:
  • editing/pasteboard/paste-text-014-expected.png:
  • editing/pasteboard/paste-text-019-expected.checksum:
  • editing/pasteboard/paste-text-019-expected.png:
  • editing/selection/3690703-2-expected.checksum:
  • editing/selection/3690703-expected.checksum:
  • editing/selection/3690719-expected.checksum:
  • editing/selection/extend-by-word-001-expected.checksum:
  • editing/selection/extend-by-word-001-expected.png:
  • editing/selection/iframe-expected.checksum:
  • editing/selection/iframe-expected.png:
  • editing/selection/move-backwords-by-word-001-expected.checksum:
  • editing/selection/move-backwords-by-word-001-expected.png:
  • editing/selection/move-by-character-001-expected.checksum:
  • editing/selection/move-by-character-001-expected.png:
  • editing/selection/move-by-line-001-expected.checksum:
  • editing/selection/move-by-line-001-expected.png:
  • editing/selection/move-by-word-001-expected.checksum:
  • editing/selection/move-by-word-001-expected.png:
  • editing/selection/replaced-boundaries-1-expected.checksum:
  • editing/selection/replaced-boundaries-1-expected.png:
  • editing/selection/unrendered-002-expected.checksum:
  • editing/selection/unrendered-002-expected.png:
  • editing/style/block-styles-007-expected.checksum:
  • editing/style/block-styles-007-expected.png:
  • editing/style/create-block-for-style-002-expected.checksum:
  • editing/style/create-block-for-style-002-expected.png:
  • editing/style/create-block-for-style-003-expected.checksum:
  • editing/style/create-block-for-style-003-expected.png:
  • editing/style/create-block-for-style-004-expected.checksum:
  • editing/style/create-block-for-style-004-expected.png:
  • editing/style/create-block-for-style-006-expected.checksum:
  • editing/style/create-block-for-style-006-expected.png:
  • editing/style/create-block-for-style-008-expected.checksum:
  • editing/style/create-block-for-style-008-expected.png:
  • editing/style/create-block-for-style-009-expected.checksum:
  • editing/style/create-block-for-style-009-expected.png:
  • editing/style/create-block-for-style-011-expected.checksum:
  • editing/style/create-block-for-style-011-expected.png:
  • editing/style/create-block-for-style-012-expected.checksum:
  • editing/style/create-block-for-style-012-expected.png:
  • editing/style/create-block-for-style-013-expected.checksum:
  • editing/style/create-block-for-style-013-expected.png:
  • editing/unsupported-content/list-type-after-expected.checksum:
  • editing/unsupported-content/list-type-before-expected.checksum:
  • editing/unsupported-content/table-type-after-expected.checksum:
  • editing/unsupported-content/table-type-before-expected.checksum:
  • fast/dom/blur-contenteditable-expected.checksum:
  • fast/dom/blur-contenteditable-expected.png:
5:14 PM Changeset in webkit [12539] by justing
  • 2 edits in trunk/WebKitTools

Reviewed by thatcher


Enabled spell checking for layout tests.


  • DumpRenderTree/DumpRenderTree.m: (main):
5:14 PM Changeset in webkit [12538] by justing
  • 3 edits in trunk/WebCore

Reviewed by darin, harrison


<​http://bugzilla.opendarwin.org/show_bug.cgi?id=6586>
REGRESSION: visible_text.cpp assertion failure in Mail.app

Calling range() on a WordIterator destroyed its RangeImpl, use
RefPtr instead of PassRefPtr.
Turned on continuous spell checking to test the fix, found
a crasher caused by invalid positions created in
previousBoundary.

Added layout test:

  • editing/selection/end-of-document.html
  • khtml/editing/visible_text.h:
  • khtml/editing/visible_units.cpp: (khtml::previousBoundary):
4:02 PM Changeset in webkit [12537] by hyatt
  • 2 edits in trunk/WebCore

Vectorize Image. Optimize so that we no longer throw away
all CGImageRefs. Now we just throw away the last frame.

Reviewed by mjs

  • platform/mac/Image.mm: (WebCore::FrameData::m_duration): (WebCore::FrameData::~FrameData): (WebCore::FrameData::clear): (WebCore::ImageData::invalidateData): (WebCore::ImageData::cacheFrame): (WebCore::ImageData::frameAtIndex): (WebCore::ImageData::frameDurationAtIndex):
3:18 PM Changeset in webkit [12536] by ggaren
  • 6 edits in trunk/WebCore

Reviewed by mjs.

Cleaned up some stuff I came across when working with JS enabled/
disabled code:

(1) Renamed initScript to initScriptIfNeeded, because that's what it

does

(2) Removed dead or otherwise irrelevent code
(3) Fixed up comments
(4) Removed NULL checks for interpreter() since it never returns NULL,

replaced with single assert().

No test case because there's no change in behavior.

  • khtml/ecma/kjs_proxy.cpp: (WebCore::KJSProxyImpl::evaluate): 1 (WebCore::KJSProxyImpl::createHTMLEventHandler): 1 (WebCore::KJSProxyImpl::createSVGEventHandler): 1 (WebCore::KJSProxyImpl::interpreter): 4 (WebCore::KJSProxyImpl::initScriptIfNeeded): 1
  • khtml/ecma/kjs_proxy.h: 1
  • page/Frame.cpp: (Frame::didOpenURL): 2, 3 (Frame::stopLoading): 2 (Frame::jScriptEnabled): 2 (Frame::jScript): changed method call to data member access to match

style of rest of class

(Frame::javaEnabled): 2
(Frame::pluginsEnabled): 2
(Frame::begin): 3
(Frame::userGestureHint): 4
(Frame::pauseTimeouts): collapsed a line
(Frame::resumeTimeouts): ditto
(Frame::saveInterpreterBuiltins): 4
(Frame::restoreInterpreterBuiltins): 4

  • page/Frame.h: 2
  • page/FramePrivate.h: 2 (FramePrivate::FramePrivate): 2
1:19 PM Changeset in webkit [12535] by hyatt
  • 2 edits in trunk/WebCore

Fix for PLT performance regression. Make sure that the document's load event timer
isn't started again if it's already active.

Reviewed by mjs

  • khtml/xml/DocumentImpl.cpp: (WebCore::DocumentImpl::dispatchImageLoadEventSoon):
1:00 PM Changeset in webkit [12534] by staikos
  • 2 edits in trunk/JavaScriptCore

Reviewed by Maciej.

  • kjs/lookup.h: Fix compile, merged from KDE.
9:32 AM Changeset in webkit [12533] by darin
  • 1 edit
    4 adds in trunk/LayoutTests
  • fast/events/onunload-body-property.html: Added.
  • fast/events/onunload-body-property-expected.txt: Added.
  • fast/events/onunload-window-property.html: Added.
  • fast/events/onunload-window-property-expected.txt: Added.
9:31 AM Changeset in webkit [12532] by darin
  • 3 edits in trunk/WebCore

Reviewed by Maciej.

Tests:

  • fast/events/onunload-body-property.html
  • fast/events/onunload-window-property.html
  • khtml/ecma/kjs_dom.cpp: (KJS::DOMNode::getValueProperty): Change onunload case to use unloadEvent, not scrollEvent. (KJS::DOMNode::putValueProperty): Ditto.
  • khtml/ecma/kjs_window.cpp: (KJS::Window::getValueProperty): Ditto. (KJS::Window::putValueProperty): Ditto.

Reviewed by Hyatt.

  • khtml/ecma/kjs_window.cpp: (KJS::WindowQObject::installTimeout): When converting the timeout number from an integer ms to a double seconds, use a 10 ms minimum.
9:15 AM Changeset in webkit [12531] by darin
  • 2 edits in trunk/WebCore
  • added a FIXME about an incorrect change made to FloatRect to fix the Win32 compile
  • platform/FloatRect.cpp: (WebCore::enclosingIntRect): Added FIXME because this should do a ceil operation, not a round operation.
1:29 AM Changeset in webkit [12530] by hyatt
  • 3 edits in trunk/WebCore

Fix win32 bustage. #ifdef enclosingIntRect using. Remove floorf and ceilf from enclosingIntRect impl since it's not needed. Fix the bad #import in qmatrix.h

1:00 AM Changeset in webkit [12529] by darin
  • 2 deletes in trunk/WebCore/platform

Deleted these files, Dave checked them in by mistake.

12:52 AM Changeset in webkit [12528] by hyatt
  • 1 edit in trunk/LayoutTests/fast/text/whitespace/006-expected.txt

Check in revised results for 006.html test.

12:40 AM Changeset in webkit [12527] by hyatt
  • 33 edits
    2 copies in trunk/WebCore

Fix for bug 6957, rewrite image rendering in C++ and move it to
WebCore. Animation now stops lazily and just uses the CachedObject
notification system to push updates so that rects no longer need to
be cached (or sets of animating renderers in specific views).


Reviewed by darin

  • WebCore.exp:
  • bindings/objc/DOM.mm: (-[DOMElement _image]): (-[DOMElement _imageTIFFRepresentation]):
  • bindings/objc/DOMPrivate.h:
  • bridge/mac/MacFrame.mm: (WebCore::MacFrame::khtmlMouseMoveEvent): (WebCore::MacFrame::fileWrapperForElement):
  • bridge/mac/WebCoreFrameBridge.h:
  • bridge/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge elementAtPoint:]):
  • kcanvas/RenderSVGImage.cpp: (RenderSVGImage::imageChanged):
  • kcanvas/RenderSVGImage.h:
  • kcanvas/device/quartz/KCanvasFilterQuartz.mm: (KCanvasFEImageQuartz::getCIFilter):
  • khtml/ecma/kjs_html.cpp: (KJS::drawPattern):
  • kwq/KWQClipboard.h: (WebCore::KWQClipboard::setDragHasStarted):
  • kwq/KWQClipboard.mm: (WebCore::KWQClipboard::dragNSImage):
  • kwq/KWQCursor.mm: (WebCore::createCustomCursor):
  • kwq/KWQPainter.h: (WebCore::QPainter::):
  • kwq/KWQPainter.mm: (WebCore::QPainter::drawFloatImage): (WebCore::QPainter::drawTiledImage): (WebCore::QPainter::drawScaledAndTiledImage):
  • kwq/KWQScrollView.h:
  • kwq/KWQScrollView.mm: (QScrollView::inWindow):
  • kwq/WebCoreImageRendererFactory.h:
  • kwq/WebCoreImageRendererFactory.m:
  • loader/CachedImage.cpp: (WebCore::CachedImage::ref): (WebCore::CachedImage::imageRect): (WebCore::CachedImage::notifyObservers): (WebCore::CachedImage::data): (WebCore::CachedImage::error): (WebCore::CachedImage::shouldStopAnimation): (WebCore::CachedImage::animationAdvanced):
  • loader/CachedImage.h: (WebCore::CachedImage::canRender):
  • loader/CachedObjectClient.h: (WebCore::CachedObjectClient::imageChanged): (WebCore::CachedObjectClient::willRenderImage):
  • platform/Image.h: (WebCore::ImageAnimationObserver::~ImageAnimationObserver): (WebCore::Image::animationObserver): (WebCore::Image::):
  • platform/mac/Image.mm: (WebCore::releasePDFDocumentData): (WebCore::PDFDocumentImage::PDFDocumentImage): (WebCore::PDFDocumentImage::~PDFDocumentImage): (WebCore::PDFDocumentImage::documentRef): (WebCore::PDFDocumentImage::mediaBox): (WebCore::PDFDocumentImage::bounds): (WebCore::PDFDocumentImage::adjustCTM): (WebCore::PDFDocumentImage::setCurrentPage): (WebCore::PDFDocumentImage::currentPage): (WebCore::PDFDocumentImage::pageCount): (WebCore::PDFDocumentImage::draw): (WebCore::ImageData::setIsPDF): (WebCore::ImageData::currentFrame): (WebCore::m_PDFDoc): (WebCore::ImageData::~ImageData): (WebCore::ImageData::invalidateData): (WebCore::ImageData::cacheFrame): (WebCore::ImageData::checkForSolidColor): (WebCore::ImageData::isNull): (WebCore::ImageData::size): (WebCore::ImageData::setData): (WebCore::ImageData::setCFData): (WebCore::ImageData::imageSourceOptions): (WebCore::ImageData::frameCount): (WebCore::ImageData::isSizeAvailable): (WebCore::ImageData::frameAtIndex): (WebCore::ImageData::getTIFFRepresentation): (WebCore::ImageData::getNSImage): (WebCore::ImageData::frameDurationAtIndex): (WebCore::ImageData::shouldAnimate): (WebCore::ImageData::startAnimation): (WebCore::ImageData::stopAnimation): (WebCore::ImageData::resetAnimation): (WebCore::ImageData::advanceAnimation): (WebCore::ImageData::setCompositingOperation): (WebCore::ImageData::fillSolidColorInRect): (WebCore::ImageData::drawInRect): (WebCore::drawPattern): (WebCore::): (WebCore::ImageData::tileInRect): (WebCore::ImageData::scaleAndTileInRect): (WebCore::Image::loadResource): (WebCore::Image::supportsType): (WebCore::m_animationObserver): (WebCore::Image::Image): (WebCore::Image::~Image): (WebCore::Image::getCGImageRef): (WebCore::Image::getNSImage): (WebCore::Image::getTIFFRepresentation): (WebCore::Image::resetAnimation): (WebCore::Image::setData): (WebCore::Image::isNull): (WebCore::Image::size): (WebCore::Image::rect): (WebCore::graphicsContext): (WebCore::Image::drawInRect): (WebCore::Image::tileInRect): (WebCore::Image::scaleAndTileInRect):
  • rendering/render_box.cpp: (WebCore::RenderBox::paintBackgroundExtended):
  • rendering/render_image.cpp: (WebCore::RenderImage::setCachedImage): (WebCore::RenderImage::imageChanged):
  • rendering/render_image.h:
  • rendering/render_line.cpp: (WebCore::InlineFlowBox::paintBackground): (WebCore::InlineFlowBox::paintBackgroundAndBorder):
  • rendering/render_list.cpp: (RenderListMarker::imageChanged):
  • rendering/render_list.h:
  • rendering/render_object.cpp: (WebCore::RenderObject::mustRepaintBackgroundOrBorder): (WebCore::RenderObject::paintBorderImage): (WebCore::RenderObject::paintBorder): (WebCore::RenderObject::imageChanged): (WebCore::RenderObject::willRenderImage):
  • rendering/render_object.h:
12:37 AM Changeset in webkit [12526] by hyatt
  • 10 edits in trunk/WebKit

Fix for bug 6957, rewrite image rendering in C++ and move it to
WebCore. Animation now stops lazily and just uses the CachedObject
notification system to push updates so that rects no longer need to
be cached (or sets of animating renderers in specific views).

Reviewed by darin

  • Misc.subproj/WebNSPasteboardExtras.h:
  • Misc.subproj/WebNSPasteboardExtras.m: (-[NSPasteboard _web_writeImage:element:URL:title:archive:types:]): (-[NSPasteboard _web_declareAndWriteDragImage:element:URL:title:archive:source:]):
  • Misc.subproj/WebNSViewExtras.h:
  • Misc.subproj/WebNSViewExtras.m: (-[NSView _web_dragImage:element:rect:event:pasteboard:source:offset:]):
  • WebCoreSupport.subproj/WebImageData.m: (-[WebImageData init]):
  • WebCoreSupport.subproj/WebImageRendererFactory.m: (+[WebImageRendererFactory shouldUseThreadedDecoding]): (+[WebImageRendererFactory setShouldUseThreadedDecoding:]): (-[WebImageRendererFactory setPatternPhaseForContext:inUserSpace:]): (-[WebImageRendererFactory imageDataForName:]):
  • WebView.subproj/WebHTMLView.m: (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]): (-[NSArray elementAtPoint:]):
  • WebView.subproj/WebImageView.m: (-[WebImageView writeImageToPasteboard:types:]): (-[WebImageView elementAtPoint:]): (-[WebImageView mouseDragged:]):
  • WebView.subproj/WebView.m: (-[WebView _writeImageElement:withPasteboardTypes:toPasteboard:]):
12:36 AM Changeset in webkit [12525] by darin
  • 1 edit
    4 adds in trunk/LayoutTests

Reviewed by Darin.

  • fast/block/positioning/height-change-expected.checksum: Added.
  • fast/block/positioning/height-change-expected.png: Added.
  • fast/block/positioning/height-change-expected.txt: Added.
  • fast/block/positioning/height-change.html: Added.
12:35 AM Changeset in webkit [12524] by darin
  • 3 edits in trunk/WebCore

Test: LayoutTests/fast/block/positioning/height-change.html

Reviewed by Hyatt.

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::layoutBlock): Relayout positioned children if the block's height changed during layout.
  • rendering/render_flexbox.cpp: (WebCore::RenderFlexibleBox::layoutBlock): Ditto.
12:22 AM Changeset in webkit [12523] by darin
  • 10 edits
    3 adds in trunk/JavaScriptCore

Reviewed by Maciej.

  • kxmlcore/Noncopyable.h: Added.
  • kxmlcore/OwnArrayPtr.h: Added.
  • kxmlcore/OwnPtr.h: Added.
  • kjs/function.h:
  • kjs/function.cpp: Use OwnPtr for Parameter pointers.
  • kjs/internal.h: Use Noncopyable for LabelStack.
  • kjs/list.cpp: Use OwnArrayPtr for overflow.
  • kjs/property_map.h:
  • kjs/property_map.cpp: Use OwnArrayPtr for SavedProperties. Use Vector for some stack buffers.
  • kjs/regexp_object.h:
  • kjs/regexp_object.cpp: Use OwnArrayPtr for lastOvector.
12:07 AM Changeset in webkit [12522] by darin
  • 1 edit
    4 adds in trunk/JavaScriptGlue
  • added some forwarding headers to make new JavaScriptCore changes work
  • kxmlcore/Assertions.h: Added.
  • kxmlcore/Noncopyable.h: Added.
  • kxmlcore/OwnPtr.h: Added.
  • kxmlcore/OwnArrayPtr.h: Added.

Feb 1, 2006:

11:42 PM Changeset in webkit [12521] by mjs
  • 16 edits in trunk/WebCore

Reviewed by Darin.

  • bridge/mac/BrowserExtensionMac.mm: (WebCore::BrowserExtensionMac::createNewWindow):
  • bridge/mac/MacFrame.h:
  • bridge/mac/MacFrame.mm: (WebCore::MacFrame::generateFrameName): (WebCore::MacFrame::submitForm): (WebCore::Frame::frameDetached): (WebCore::MacFrame::nextKeyViewInFrameHierarchy): (WebCore::MacFrame::createEmptyDocument):
  • bridge/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge setParent:]): (-[WebCoreFrameBridge parent]): (-[WebCoreFrameBridge setName:]): (-[WebCoreFrameBridge name]):
  • khtml/ecma/kjs_window.cpp: (KJS::Window::getValueProperty): (KJS::Window::put): (KJS::Window::isSafeScript): (KJS::WindowFunc::callAsFunction):
  • khtml/html/html_baseimpl.cpp: (WebCore::HTMLFrameElementImpl::isURLAllowed):
  • khtml/html/html_headimpl.cpp: (WebCore::HTMLLinkElementImpl::process):
  • khtml/xml/DocumentImpl.cpp: (WebCore::DocumentImpl::ownerElement): (WebCore::DocumentImpl::parentDocument):
  • page/Frame.cpp: (WebCore::Frame::checkCompleted): (WebCore::Frame::redirectionTimerFired): (WebCore::Frame::requestFrameName): (WebCore::Frame::slotChildCompleted): (WebCore::Frame::incrementFrameCount): (WebCore::Frame::decrementFrameCount): (WebCore::Frame::topLevelFrameCount): (WebCore::Frame::selectFrameElementInParentIfFullySelected): (WebCore::Frame::handleFallbackContent): (WebCore::Frame::userGestureHint): (WebCore::Frame::canCachePage): (WebCore::Frame::updatePolicyBaseURL):
  • page/Frame.h:
  • page/FramePrivate.h: (WebCore::FramePrivate::FramePrivate):
  • page/FrameTreeNode.cpp: (WebCore::FrameTreeNode::setName): (WebCore::FrameTreeNode::appendChild): (WebCore::FrameTreeNode::removeChild):
  • page/FrameTreeNode.h: (WebCore::FrameTreeNode::FrameTreeNode): (WebCore::FrameTreeNode::name): (WebCore::FrameTreeNode::parent): (WebCore::FrameTreeNode::setParent):
  • rendering/render_frames.cpp: (WebCore::isURLAllowed):
11:04 PM Changeset in webkit [12520] by hyatt
  • 1 edit in trunk/LayoutTests/fast/block/float/031.html

Back out the changes to 031.html's HTML because of the 4959 partial backout.

11:02 PM Changeset in webkit [12519] by hyatt
  • 4 deletes in trunk/LayoutTests/fast/parser

Remove residual style span form test case because of the backout.

10:45 PM Changeset in webkit [12518] by hyatt
  • 1 edit in trunk/WebCore/khtml/html/htmlparser.cpp

Back out half of the residual style fix for bug 4959, since treating <span> like residual style breaks fast/block/float/031.html (despite buildbot and people on the outside not seeing it... how weird is that).

9:16 PM Changeset in webkit [12517] by darin
  • 1 edit
    2 adds in trunk/LayoutTests
  • fast/dom/Window/clear-timeout.html: Added.
  • fast/dom/Window/clear-timeout-expected.txt: Added.
9:15 PM Changeset in webkit [12516] by darin
  • 2 edits in trunk/WebCore

Checking in without review because I *just* broke this with my last check-in.

Test: fast/dom/Window/clear-timeout.html

  • khtml/ecma/kjs_window.cpp: (WindowQObject::clearTimeout): Fixed a != that should have been a ==.
6:51 PM Changeset in webkit [12515] by darin
  • 30 edits
    3 adds
    3 deletes in trunk/WebCore

Reviewed by Hyatt.

  • use Timer for everything, kill timer features of QObject, QTimer, QTimerEvent
  • ForwardingHeaders/kxmlcore/Noncopyable.h: Added.
  • ForwardingHeaders/kxmlcore/OwnArrayPtr.h: Added.
  • ForwardingHeaders/kxmlcore/OwnPtr.h: Added.
  • ForwardingHeaders/qtimer.h: Removed.
  • kwq/KWQTimer.cpp: Removed.
  • kwq/KWQTimer.h: Removed.
  • kwq/KWQObject.h:
  • kwq/KWQObject.cpp: Remove timer-related features.
  • kwq/KWQEvent.h: Remove QTimerEvent.
  • WebCore.xcodeproj/project.pbxproj: Updated for file changes.
  • WebCore.vcproj/WebCore/WebCore.vcproj: Ditto.
  • platform/Timer.h: Change to use nextFireInterval rather than nextFireTime. More convenient and also no need to define what the epoch is. Change TimerBase to use a virtual function rather than a function pointer. Added setDeferringTimers function to ensure timers don't fire while handling an event.
  • platform/mac/Timer.cpp: (WebCore::timerFired): Added deferral support. (WebCore::fireDeferred): Added. (WebCore::TimerBase::TimerBase): Simplify to remove function pointer. (WebCore::TimerBase::~TimerBase): Put in here, no longer inline. (WebCore::TimerBase::start): Change to use interval. (WebCore::TimerBase::startRepeating): Change since we use interval now. (WebCore::TimerBase::startOneShot): Ditto. (WebCore::TimerBase::stop): Added deferral support. (WebCore::TimerBase::nextFireInterval): Added. (WebCore::TimerBase::fire): Added. (WebCore::isDeferringTimers): Added. (WebCore::setDeferringTimers): Added.
  • bridge/mac/MacFrame.mm: (WebCore::MacFrame::startRedirectionTimer): Update for change to Timer class. (WebCore::MacFrame::passMouseDownEventToWidget): Update to use setDeferringTimers instead of QObject::setDefersTimers.
  • khtml/ecma/kjs_window.h:
  • khtml/ecma/kjs_window.cpp: (KJS::DOMWindowTimer): Added. Class for JavaScript "timeout". (KJS::WindowQObject::parentDestroyed): Update to use DOMWindowTimer. (KJS::WindowQObject::installTimeout): Ditto. (KJS::WindowQObject::pauseTimeouts): Ditto. (KJS::WindowQObject::resumeTimeouts): Ditto. (KJS::WindowQObject::clearTimeout): Ditto. (KJS::WindowQObject::timerFired): Ditto. (KJS::DOMWindowTimer::fired): Added.
  • khtml/html/htmltokenizer.h:
  • khtml/html/htmltokenizer.cpp: (WebCore::HTMLTokenizer::HTMLTokenizer): Use Timer instead of timer ID. (WebCore::HTMLTokenizer::reset): Ditto. (WebCore::HTMLTokenizer::continueProcessing): Ditto. (WebCore::HTMLTokenizer::write): Ditto. (WebCore::HTMLTokenizer::stopParsing): Ditto. (WebCore::HTMLTokenizer::processingData): Ditto. (WebCore::HTMLTokenizer::timerFired): Ditto. (WebCore::HTMLTokenizer::end): Ditto. (WebCore::HTMLTokenizer::finish): Ditto.
  • khtml/xml/DocumentImpl.h:
  • khtml/xml/DocumentImpl.cpp: (WebCore::DocumentImpl::DocumentImpl): Updated for changes to use RefPtr more cases, and not use it for m_svgExtensions. (WebCore::DocumentImpl::~DocumentImpl): Delete m_svgExtensions instead of just deref'ing it since it's no longer shared. (WebCore::DocumentImpl::implementation): Update for RefPtr. (WebCore::DocumentImpl::elementSheet): Ditto. (WebCore::DocumentImpl::styleSheets): Ditto. (WebCore::DocumentImpl::recalcStyleSelector): Ditto. (WebCore::DocumentImpl::dispatchImageLoadEventSoon): Update to use new style timer. (WebCore::DocumentImpl::removeImage): Ditto. (WebCore::DocumentImpl::dispatchImageLoadEventsNow): Ditto. (WebCore::DocumentImpl::imageLoadEventTimerFired): Ditto. (WebCore::DocumentImpl::svgExtensions): Update for RefPtr. (WebCore::DocumentImpl::accessSVGExtensions): Ditto.
  • ksvg2/misc/KSVGTimeScheduler.h: (WebCore::TimeScheduler::document):
  • ksvg2/misc/KSVGTimeScheduler.cpp: (WebCore::SVGTimer::downcast): (WebCore::SVGTimer::SVGTimer): (WebCore::SVGTimer::start): (WebCore::SVGTimer::calculateTimePercentage): (WebCore::SVGTimer::notifyAll): (WebCore::SVGTimer::addNotify): (WebCore::SVGTimer::removeNotify): (WebCore::TimeScheduler::TimeScheduler): (WebCore::TimeScheduler::~TimeScheduler): (WebCore::TimeScheduler::addTimer): (WebCore::TimeScheduler::connectIntervalTimer): (WebCore::TimeScheduler::disconnectIntervalTimer): (WebCore::TimeScheduler::startAnimations): (WebCore::TimeScheduler::toggleAnimations): (WebCore::TimeScheduler::timerFired): Rework to use new timer.
  • ksvg2/misc/SVGDocumentExtensions.h:
  • ksvg2/misc/SVGDocumentExtensions.cpp: Make SVGDocumentExtensions no longer be shared, be solely owned by the document.
  • ksvg2/svg/SVGDocumentImpl.cpp: (WebCore::SVGDocumentImpl::createStyleSelector): Update for RefPtr use.
  • kwq/KWQSlot.cpp: Remove endLifeSupport and timerNotify slots.
  • page/Frame.h:
  • page/Frame.cpp: (WebCore::Frame::selectionLayoutChanged): Update to use new style timer. (WebCore::Frame::caretBlinkTimerFired): Ditto.
  • page/FramePrivate.h: (WebCore::FramePrivate::FramePrivate): Ditto.
  • page/FrameView.h:
  • page/FrameView.cpp: (WebCore::FrameViewPrivate::FrameViewPrivate): Update to use new style timer. (WebCore::FrameView::clear): Ditto. (WebCore::FrameView::layout): Ditto. (WebCore::FrameView::layoutTimerFired): Ditto. (WebCore::FrameView::scheduleRelayout): Ditto. (WebCore::FrameView::layoutPending): Ditto. (WebCore::FrameView::haveDelayedLayoutScheduled): Ditto. (WebCore::FrameView::unscheduleRelayout): Ditto.
  • page/ObjectContents.h: Remove unneeded includes.
  • rendering/render_frames.cpp: Removed include of <qtimer.h>.
  • rendering/render_layer.h:
  • rendering/render_layer.cpp: (WebCore::Marquee::Marquee): Use new style timers. (WebCore::Marquee::start): Ditto. (WebCore::Marquee::suspend): Ditto. (WebCore::Marquee::stop): Ditto. (WebCore::Marquee::updateMarqueeStyle): Ditto. (WebCore::Marquee::timerFired): Ditto.
2:36 PM Changeset in webkit [12514] by bdakin
  • 3 edits in trunk/WebCore

Patch by Darin. Reviewed by Beth.

Fix for <rdar://problem/4424126> REGRESSION(412-420+): yellow
highlight fails to follow cursor when mousing over star rating
(6232)

Event coordinates cannot be calculated until there is a target.
This patch restructures the code so that the coordinates are
initialized when the mouse event is created but are not calculated
until the target is set.

Note: See TracTimeline for information about the timeline view.