Timeline



Dec 8, 2004:

5:20 PM Changeset in webkit [8159] by rjw
  • 3 edits in trunk/WebKit

Fixed <rdar://problem/3911719> REGRESSION: Images no longer scale vertically
Account for scaling correctly when taking into account progressively

loaded images.

Also added implementation of repetition count for animated GIF images.
Also replaced strings with new constants from CFImageProperties.h

Also fixed possible problem with -(NSSize)size implementation,
relevant to Panther only.

Reviewed by Chris.

  • WebCoreSupport.subproj/WebImageData.m: (-[WebImageData _floatProperty:type:at:]): (-[WebImageData _frameDurationAt:]): (-[WebImageData _repetitionCount]):
  • WebCoreSupport.subproj/WebImageRenderer.m: (-[WebImageRenderer size]):
2:54 PM Changeset in webkit [8158] by mjs
  • 3 edits in trunk/JavaScriptCore

Reviewed by Darin.

<rdar://problem/3908017> REGRESSION (172-173): assertion in ObjectImp::construct trying to create JS error (24hourfitness.com)

The fix was to implement copy constructor and assignment operator,
the ones that worked on the base class did not replace the
defaults apparently!

  • kjs/protect.h: (KJS::ProtectedValue::ProtectedValue): (KJS::ProtectedValue::operator=): (KJS::ProtectedObject::ProtectedObject): (KJS::ProtectedObject::operator=):

Also fixed a bug in the GC test mode that compares the results of
the old collector and the new collector.

  • kjs/value.cpp: (ValueImp::mark):
1:58 PM Changeset in webkit [8157] by cblu
  • 4 edits in trunk/WebKit

Removed NPN wrappers since these no longer need to be defined to make the QT plug-in work
since 3828925 has been fixed.

Reviewed by john.

  • Plugins.subproj/WebNetscapePluginPackage.m: (-[WebNetscapePluginPackage load]): use under-bar symbols since non-under-bar wrappers have been removed
  • Plugins.subproj/npapi.m: removed NPN wrappers
  • WebKit.exp: removed symbols
8:30 AM Changeset in webkit [8156] by kocienda
  • 2 edits in trunk/WebKit

Reviewed by Darin

  • WebView.subproj/WebHTMLView.m: (-[WebHTMLView drawRect:]): Work around for this bug: <rdar://problem/3908282> REGRESSION (Mail): No drag image dragging selected text in Blot and Mail The reason for the workaround is that this method is called explicitly from the code to generate a drag image, and at that time, getRectsBeingDrawn:count: will return a zero count. This code change uses the passed-in rect when the count is zero.

Dec 7, 2004:

4:27 PM Changeset in webkit [8155] by cblu
  • 3 edits in trunk/WebKit

WebKit:

Support for fix for:
<rdar://problem/3734309> Safari doesn't open folders in title bar menu with non-Roman names using Cmd+click

Reviewed by john.

  • Misc.subproj/WebNSURLExtras.h:
  • Misc.subproj/WebNSURLExtras.m: (+[NSURL _web_URLWithUserTypedString:relativeToURL:]): renamed to take relativeToURL parameter (+[NSURL _web_URLWithUserTypedString:]): call _web_URLWithUserTypedString:relativeToURL: with nil for URL

WebBrowser:

Fixed: <rdar://problem/3734309> Safari doesn't open folders in title bar menu with non-Roman names using Cmd+click

Reviewed by john.

  • TitleBarButton.m:
(-[TitleBarButton showPathRepresentingPopUpMenu]): use …
since the path returned by [NSURL path] is of the "user typed" variety
3:39 PM Changeset in webkit [8154] by rjw
  • 2 edits in trunk/WebKit

Fixed <rdar://problem/3905564> REGRESSION (Tiger); in History menu, pixel size appears but is wrong for standalone images in Safari.

Reviewed by Chris.

  • WebCoreSupport.subproj/WebImageRenderer.m: (-[WebImageRenderer size]):
3:06 PM Changeset in webkit [8153] by rjw
  • 17 edits
    4 adds in trunk

WebCore:

Support threaded image decoding on machines w/ > 2 CPUs.

Reviewed by Maciej and Chris.

  • khtml/misc/loader.cpp: (CachedImageCallback::notifyUpdate): (CachedImageCallback::notifyFinished): (CachedImageCallback::notifyDecodingError): (CachedImageCallback::handleError): (CachedImageCallback::clear): (CachedImage::CachedImage): (CachedImage::clear): (CachedImage::data): (CachedImage::checkNotify): (Loader::servePendingRequests): (Loader::slotFinished): (Loader::numRequests): (Loader::cancelRequests): (Loader::removeBackgroundDecodingRequest):
  • khtml/misc/loader.h: (khtml::CachedImageCallback::CachedImageCallback): (khtml::CachedImageCallback::ref): (khtml::CachedImageCallback::deref): (khtml::CachedImage::decoderCallback):
  • khtml/rendering/render_object.cpp: (RenderObject::setPixmap):
  • kwq/KWQPixmap.h:
  • kwq/KWQPixmap.mm: (-[WebImageCallback initWithCallback:khtml::]): (-[WebImageCallback _commonTermination]): (-[WebImageCallback dealloc]): (-[WebImageCallback finalize]): (-[WebImageCallback notify]): (-[WebImageCallback setImageSourceStatus:]): (-[WebImageCallback status]): (QPixmap::shouldUseThreadedDecoding): (QPixmap::receivedData):
  • kwq/WebCoreImageRenderer.h:
  • kwq/WebCoreImageRendererFactory.h:
  • kwq/WebCoreImageRendererFactory.m: (+[WebCoreImageRendererFactory shouldUseThreadedDecoding]): (+[WebCoreImageRendererFactory setShouldUseThreadedDecoding:]):

WebKit:

Support threaded image decoding on machines w/ > 2 CPUs.

Reviewed by Maciej and Chris.

  • Misc.subproj/WebKitSystemBits.h:
  • Misc.subproj/WebKitSystemBits.m: (WebSystemMainMemory): (WebNumberOfCPUs):
  • WebCoreSupport.subproj/WebImageData.h:
  • WebCoreSupport.subproj/WebImageData.m: (+[WebImageData initialize]): (-[WebImageData init]): (-[WebImageData _commonTermination]): (-[WebImageData dealloc]): (-[WebImageData _invalidateImages]): (-[WebImageData _imageSourceOptions]): (-[WebImageData imageAtIndex:]): (-[WebImageData propertiesAtIndex:]): (-[WebImageData _createImages]): (-[WebImageData decodeData:isComplete:callback:]): (-[WebImageData incrementalLoadWithBytes:length:complete:callback:]): (drawPattern): (-[WebImageData tileInRect:fromPoint:context:]): (-[WebImageData isNull]): (-[WebImageData size]): (-[WebImageData _frameDurationAt:]): (-[WebImageData _frameDuration]): (+[WebImageData stopAnimationsInView:]): (-[WebImageData addAnimatingRenderer:inView:]): (-[WebImageData removeAnimatingRenderer:]):
  • WebCoreSupport.subproj/WebImageDecodeItem.h: Added.
  • WebCoreSupport.subproj/WebImageDecodeItem.m: Added. (+[WebImageDecodeItem decodeItemWithImage:data:isComplete:callback:]): (-[WebImageDecodeItem initWithImage:data:isComplete:callback:]): (-[WebImageDecodeItem finalize]): (-[WebImageDecodeItem dealloc]):
  • WebCoreSupport.subproj/WebImageDecoder.h: Added.
  • WebCoreSupport.subproj/WebImageDecoder.m: Added. (decoderNotifications): (+[WebImageDecoder initialize]): (+[WebImageDecoder notifyMainThread]): (+[WebImageDecoder sharedDecoder]): (+[WebImageDecoder performDecodeWithImage:data:isComplete:callback:]): (+[WebImageDecoder imageDecodesPending]): (+[WebImageDecoder decodeComplete:status:]): (-[WebImageDecoder init]): (-[WebImageDecoder dealloc]): (-[WebImageDecoder finalize]): (-[WebImageDecoder removeItem]): (-[WebImageDecoder addItem:]): (-[WebImageDecoder decodeItem:]): (decoderThread): (startDecoderThread):
  • WebCoreSupport.subproj/WebImageRenderer.m: (-[WebImageRenderer initWithData:MIMEType:]): (-[WebImageRenderer initWithContentsOfFile:]): (-[WebImageRenderer incrementalLoadWithBytes:length:complete:callback:]): (-[WebInternalImage incrementalLoadWithBytes:length:complete:callback:]):
  • WebKit.pbproj/project.pbxproj:
  • WebView.subproj/WebImageRepresentation.m: (-[WebImageRepresentation receivedData:withDataSource:]): (-[WebImageRepresentation receivedError:withDataSource:]): (-[WebImageRepresentation finishedLoadingWithDataSource:]):

WebBrowser:

Added a menu item to enable/disable threaded image decoding.

Reviewed by Maciej and Chris.

  • Debug/DebugUtilities.m: (-[DebugUtilities createDebugMenu]): (-[BrowserDocument toggleUseATSUForAllTextDrawing:]): (-[BrowserDocument toggleUseThreadedImageDecoding:]): (-[BrowserDocument validate_toggleUseThreadedImageDecoding:]):
12:46 PM Changeset in webkit [8152] by cblu
  • 2 edits in trunk/WebKit

Fix for performance regression.

Reviewed by kevin.

  • WebCoreSupport.subproj/WebBridge.m: (-[WebBridge objectLoadedFromCacheWithURL:response:data:]): construct the WebResource without copying the data
12:05 PM Changeset in webkit [8151] by cblu
  • 5 edits in trunk/WebKit

Fixed: <rdar://problem/3909243> REGRESSION: large standalone images stop loading part way through

Reviewed by kevin.

  • WebView.subproj/WebBaseResourceHandleDelegate.h:
  • WebView.subproj/WebMainResourceClient.m: (-[WebMainResourceClient addData:]): call super so it buffers the data
11:01 AM Changeset in webkit [8150] by kocienda
  • 1 edit in trunk/WebCore/ChangeLog-2005-08-23

Fix my NOBODY (OOPS!). John reviewed.

10:13 AM Changeset in webkit [8149] by kocienda
  • 2 edits in trunk/WebCore

Reviewed by NOBODY (OOPS!).

  • khtml/editing/htmlediting.cpp: (khtml::InsertParagraphSeparatorInQuotedContentCommand::doApply): Fix a problem with my change in thei code yesterday. Call to insertBlockPlaceholderIfNeeded() must be done after new block has been inserted, otherwise a crash can result. Shuffle down call to insertBlockPlaceholderIfNeeded() a couple of lines (where the node is inserted), and all is good.
10:01 AM Changeset in webkit [8148] by kocienda
  • 2 edits
    2 adds in trunk

Reviewed by John

Fix for this bug:

<rdar://problem/3907422> REGRESSION (Mail): Pasting quoted content can place content after body element

  • khtml/editing/htmlediting.cpp: (khtml::ReplaceSelectionCommand::doApply): Detect when the body element is the "reference block" used for determining the location for inserting content. Do not allow an insert before or after if the reference block is the body. Perform insertNodeAt(0) and appendNode, respectively, in the block-is-body case.
  • layout-tests/editing/inserting/insert-3907422-fix-expected.txt: Added.
  • layout-tests/editing/inserting/insert-3907422-fix.html: Added.
9:45 AM Changeset in webkit [8147] by darin
  • 2 edits in trunk/WebCore

Reviewed by Don.

  • fixed <rdar://problem/3908701> REGRESSION: Cursor does not change to "hand" over active links
  • kwq/KWQEvent.mm: (positionForEvent): Add NSMouseMoved to list of events that have mouse location.
9:02 AM Changeset in webkit [8146] by kocienda
  • 4 edits in trunk/WebCore

Reviewed by John

Changed name of constant from KHTMLInterchangeNewline to AppleInterchangeNewline.
I discussed this with Hyatt and he agreed that going with "Apple" names was OK.

  • khtml/editing/html_interchange.h: Name change, as described above.
  • khtml/editing/htmlediting.cpp: (khtml::ReplacementFragment::isInterchangeNewlineComment): Ditto.
  • khtml/xml/dom2_rangeimpl.cpp: (DOM::RangeImpl::toHTML): Ditto.
7:09 AM Changeset in webkit [8145] by kocienda
  • 1 edit
    4 adds in trunk

Reviewed by me

Added a couple more layout tests.

  • layout-tests/editing/deleting/delete-line-013-expected.txt: Added.
  • layout-tests/editing/deleting/delete-line-013.html: Added.
  • layout-tests/editing/deleting/delete-line-014-expected.txt: Added.
  • layout-tests/editing/deleting/delete-line-014.html: Added.

Dec 6, 2004:

8:11 PM Changeset in webkit [8144] by rjw
  • 2 edits in trunk/WebKit

Use the AppKit's font rendering mode. This fixes 3905347, but we still need to track down
and resolve why metrics have changed for Courier. This may be caused by changes in
AppKit for 3902394.

Reviewed by John.

  • WebCoreSupport.subproj/WebTextRenderer.m: (_AppkitGetCGRenderingMode): (getUncachedWidth): (_drawGlyphs):
6:26 PM Changeset in webkit [8143] by mjs
  • 2 edits in trunk/WebCore

Reviewed by Hyatt.

  • fixed <rdar://problem/3906974> assertion failure in QWidget::beforeMouseDown clicking on <select multiple>
  • kwq/KWQListBox.mm: (QListBox::QListBox): Initialize KWQListBoxScrollView with this. (-[KWQListBoxScrollView initWithListBox:]): Make this class a KWQWidgetHolder. (-[KWQListBoxScrollView widget]): See above. (-[KWQTableView mouseDown:]): Pass outerView rather than self to beforeMouseDown and afterMouseDown, to avoid triggering an assertion failure.
6:09 PM Changeset in webkit [8142] by hyatt
  • 2 adds in trunk/LayoutTests/fast/table

Add table test to cover the document.write line layout bug.

6:09 PM Changeset in webkit [8141] by hyatt
  • 5 edits in trunk/WebCore

Fix for 3615411, the linesAppended optimization was old and broken, and it's easier with the new code fixes
made by me, kocienda and harrison to just remove it.

Reviewed by mjs

  • khtml/rendering/bidi.cpp: (khtml::RenderBlock::layoutInlineChildren):
  • khtml/rendering/render_block.cpp: (khtml:::RenderFlow):
  • khtml/rendering/render_block.h:
  • khtml/rendering/render_flow.cpp: (RenderFlow::dirtyLinesFromChangedChild):
5:36 PM Changeset in webkit [8140] by hyatt
  • 5 edits in trunk/WebCore

Fix for 3787133, some web pages print with many blank pages. Make sure to use the real page print rect and
not a damage rect that can be changed when intersected with the clip regions of the web page.

Reviewed by john

  • khtml/rendering/render_canvas.h: (khtml::RenderCanvas::printRect): (khtml::RenderCanvas::setPrintRect):
  • khtml/rendering/render_flow.cpp: (RenderFlow::paintLines):
  • khtml/rendering/render_list.cpp: (RenderListMarker::paint):
  • kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::adjustPageHeight):
4:47 PM Changeset in webkit [8139] by cblu
  • 2 edits in trunk/WebKit

Forgot to commit copied header.

  • DOM.subproj/DOMPrivate.h:
4:45 PM Changeset in webkit [8138] by harrison
  • 2 edits in trunk/WebCore

Reviewed by Ken Kocienda and Dave Hyatt (OOPS!).

<rdar://problem/3849947> Typing after pasting line does not appear until after window resize.

  • khtml/rendering/render_flow.cpp: (RenderFlow::dirtyLinesFromChangedChild):

Dirty the line above because new child can inval the cached line break position of previous line.

4:34 PM Changeset in webkit [8137] by hyatt
  • 15 edits in trunk/WebCore

Fix for 3254464, radio buttons do not work for quiz on netscape.com. Left/top overflow needed to be implemented.
This also fixes 3106907, link hover color only partially set on rollover and the more general architecture bug
3126929, handle top/left overflow.

This patch also fixes 3902891, scroll bar of position:fixed content moves when a page is scrolled.

Finally, the Emerson regression 3869718 (error involving computing the rightmost/lowest position of overflow:auto
regions and web pages) has been fixed.

Reviewed by mjs

  • khtml/rendering/bidi.cpp: (khtml::RenderBlock::computeHorizontalPositionsForLine): (khtml::RenderBlock::checkLinesForOverflow):
  • khtml/rendering/render_block.cpp: (khtml:::RenderFlow): (khtml::RenderBlock::overflowHeight): (khtml::RenderBlock::overflowWidth): (khtml::RenderBlock::overflowLeft): (khtml::RenderBlock::overflowTop): (khtml::RenderBlock::overflowRect): (khtml::RenderBlock::layoutBlock): (khtml::RenderBlock::layoutBlockChildren): (khtml::RenderBlock::paint): (khtml::RenderBlock::floatRect): (khtml::RenderBlock::lowestPosition): (khtml::RenderBlock::rightmostPosition): (khtml::RenderBlock::leftmostPosition): (khtml::RenderBlock::nodeAtPoint):
  • khtml/rendering/render_block.h:
  • khtml/rendering/render_box.h: (khtml::RenderBox::borderBox): (khtml::RenderBox::borderTopExtra): (khtml::RenderBox::borderBottomExtra):
  • khtml/rendering/render_layer.cpp: (RenderLayer::paintScrollbars): (mustExamineRenderer): (RenderLayer::intersectsDamageRect): (RenderLayer::containsPoint):
  • khtml/rendering/render_line.cpp: (khtml::InlineFlowBox::placeBoxesHorizontally): (khtml::InlineFlowBox::verticallyAlignBoxes):
  • khtml/rendering/render_line.h: (khtml::InlineBox::leftOverflow): (khtml::InlineBox::rightOverflow): (khtml::InlineFlowBox::setVerticalOverflowPositions): (khtml::RootInlineBox::RootInlineBox): (khtml::RootInlineBox::leftOverflow): (khtml::RootInlineBox::rightOverflow): (khtml::RootInlineBox::setVerticalOverflowPositions): (khtml::RootInlineBox::setHorizontalOverflowPositions):
  • khtml/rendering/render_object.h: (khtml::RenderObject::borderBox): (khtml::RenderObject::overflowLeft): (khtml::RenderObject::overflowTop): (khtml::RenderObject::overflowRect): (khtml::RenderObject::floatRect):
  • khtml/rendering/render_table.cpp: (RenderTable::layout): (RenderTable::paint): (RenderTable::paintBoxDecorations): (RenderTable::calcMinMaxWidth):
  • khtml/rendering/render_table.h: (khtml::RenderTableCell::borderTopExtra): (khtml::RenderTableCell::borderBottomExtra):
  • kwq/KWQRect.h:
  • kwq/KWQRect.mm: (QRect::inflate):
4:24 PM Changeset in webkit [8136] by mjs
  • 2 edits in trunk/WebCore

Reviewed by John.

  • fixed <rdar://problem/3903797> scripts can cause other frames/windows to execute arbitrary script using javascript: URLs

I changed all unprotected places that can navigate a different
window or frame from script to check for a javascript: URL, and if
found, to check for safety using cross-site-script rules.

I considered a few other possible exploits and made no change:

  • document.location is already protected because the document object itself is protected
  • frame.src, frame.location, iframe.src and targetted links are all safe because setting the URL of a frame to a javascript: URL executes the script in the context of the parent
  • khtml/ecma/kjs_window.cpp: (WindowFunc::tryCall): (Location::put): (LocationFunc::tryCall):
3:57 PM Changeset in webkit [8135] by kocienda
  • 4 edits in trunk

Reviewed by Maciej

Fix for this bug:

<rdar://problem/3890955> 8A314: Forward delete sometimes fails to delete the selected quoted text

  • khtml/editing/htmlediting.cpp: (khtml::DeleteSelectionCommand::handleSpecialCaseBRDelete): Fixed bonehead coding mistake in the check for one of the special cases being checked for in this function. The specific case intends to check for a selection that is only a <br> after a block ends (as in </div><br>). If it sees such markup, it deletes only the <br> and bails. However, this code would run in *any* case where a selection ended in a <br> after a block and would not delete any part of the selection preceding the <br>. Bad. I have tightened the check to see that only a <br> is selected.

Fixing the bug above was accomplished with an additional call to DOM::Position::downstream. This
new use of the function exposed this bug:

<rdar://problem/3907666> Incorrectly coded loop in Position::downstream can lead to infinite loop

  • khtml/xml/dom_position.cpp: (DOM::Position::downstream): I am ashamed of my first cut at this. Rewrote the loop so it does not have this fatal flaw. It is a much better design as well.
  • layout-tests/editing/deleting/delete-3800834-fix-expected.txt: Changes made this test have what I consider to be a better result. Going with it.
3:39 PM Changeset in webkit [8134] by cblu
  • 3 edits in trunk/WebCore

Fixed: <rdar://problem/3871718> REGRESSION (125-168): text marked bold with font that does not have bold variant copies as non-bold

Reviewed by hyatt.

  • kwq/DOM.mm: (-[DOMElement _font]): new SPI for AppKit
  • kwq/DOMPrivate.h:
2:59 PM Changeset in webkit [8133]
  • 3 copies
    3 deletes in tags/WebKit-173~1

This commit was manufactured by cvs2svn to create tag 'WebKit-173~1'.

2:59 PM Changeset in webkit [8132] by vicki
  • 1 edit in trunk/WebKit/WebKit.pbproj/project.pbxproj

Bump version number to 173.1. We're doing a dot submission today for <rdar://problem/3903749>.

2:20 PM Changeset in webkit [8131] by cblu
  • 11 edits in trunk/WebKit

Fixed: <rdar://problem/3907381> NSURLConnection and WebKit buffer 2 copies of incoming data

Reviewed by darin.

  • WebView.subproj/WebBaseResourceHandleDelegate.h:
  • WebView.subproj/WebBaseResourceHandleDelegate.m: (+[WebBaseResourceHandleDelegate initialize]): cache check to see if Foundation supports access to its buffered data (-[WebBaseResourceHandleDelegate addData:]): don't buffer data if Foundation is buffering it for us (-[WebBaseResourceHandleDelegate saveResource]): when creating a WebResource, pass NO for copyData since we know it won't be mutated (-[WebBaseResourceHandleDelegate resourceData]): return the buffered data from the connection if it supports it (-[WebBaseResourceHandleDelegate willStopBufferingData:]): make a mutable copy of the data from NSURLConnection so we can continue buffering (-[WebBaseResourceHandleDelegate willCacheResponse:]): removed optimization that used the cached response data to save the resource since that is obsolete by this change (-[WebBaseResourceHandleDelegate connection:willStopBufferingData:]): new callback from NSURLConnection, informs us that NSURLConnection has given up buffering
  • WebView.subproj/WebDataSource.m: (-[WebDataSource _receivedData:]): removed buffering code since that's done by NSURLConnection and the main client (-[WebDataSource _setData:]): removed unnecessary cast since the resourceData ivar is now an NSData instead of NSMutableData (-[WebDataSource data]): return resourceData ivar, else return the resourceData from the main client
  • WebView.subproj/WebDataSourcePrivate.h:
  • WebView.subproj/WebMainResourceClient.m: (-[WebMainResourceClient releaseResources]): store resourceData on the data source so it can continue to have data after the main client has gone away (-[WebMainResourceClient connection:didReceiveData:lengthReceived:]):don't call [dataSource data] just to get the length of data received since [dataSource data] can now cause data to be copied (-[WebMainResourceClient connectionDidFinishLoading:]): ditto
  • WebView.subproj/WebResource.m: (-[WebResource initWithData:URL:MIMEType:textEncodingName:frameName:]): call following method with YES for copyData (-[WebResource _initWithData:URL:MIMEType:textEncodingName:frameName:copyData:]): new initializer, allows caller to choose whether or not the data is copied
  • WebView.subproj/WebResourcePrivate.h:
1:49 PM Changeset in webkit [8130] by darin
  • 2 edits in trunk/WebCore

Reviewed by Maciej.

  • fixed <rdar://problem/3906327> Select All of a large document is slow (>15 secs on my machine for attached specimen)
  • kwq/KWQScrollView.mm: (QScrollView::updateContents): Intersect with visibleRect before calling through to NSView to dirty; NSView could also be more efficient in this case (I filed 3906343).
1:09 PM Changeset in webkit [8129] by rjw
  • 2 edits in trunk/WebKit

Fixed <rdar://problem/3903749> REGRESSION (8A321): WebKit gets incorrect glyph metrics due to change in how AppKit uses CGFont

Use CGFontRef direction when both getting font metrics and drawing
glyphs, instead on depending on [NSFont set].

Reviewed by John.

  • WebCoreSupport.subproj/WebTextRenderer.m: (getUncachedWidth): (_drawGlyphs):
11:48 AM Changeset in webkit [8128] by sullivan
  • 2 edits in trunk/WebCore

Darin found what appears to be the real leak that we were falsely blaming
on the 'leaks' tool (3880245). I made the change, and ran layout tests and PLT to make
sure nothing barfed.

  • khtml/css/cssparser.cpp: (CSSParser::parseValue): call clearProperties() instead of just setting numParsedProperties to 0 (CSSParser::parseDeclaration): ditto (CSSParser::createStyleDeclaration): ditto
10:38 AM Changeset in webkit [8127] by kocienda
  • 1 edit
    18 adds in trunk

Reviewed by me

New layout tests.

  • layout-tests/editing/inserting/insert-div-001-expected.txt: Added.
  • layout-tests/editing/inserting/insert-div-001.html: Added.
  • layout-tests/editing/inserting/insert-div-002-expected.txt: Added.
  • layout-tests/editing/inserting/insert-div-002.html: Added.
  • layout-tests/editing/inserting/insert-div-003-expected.txt: Added.
  • layout-tests/editing/inserting/insert-div-003.html: Added.
  • layout-tests/editing/inserting/insert-div-004-expected.txt: Added.
  • layout-tests/editing/inserting/insert-div-004.html: Added.
  • layout-tests/editing/inserting/insert-div-005-expected.txt: Added.
  • layout-tests/editing/inserting/insert-div-005.html: Added.
  • layout-tests/editing/inserting/insert-div-006-expected.txt: Added.
  • layout-tests/editing/inserting/insert-div-006.html: Added.
  • layout-tests/editing/inserting/insert-div-007-expected.txt: Added.
  • layout-tests/editing/inserting/insert-div-007.html: Added.
  • layout-tests/editing/inserting/insert-div-008-expected.txt: Added.
  • layout-tests/editing/inserting/insert-div-008.html: Added.
  • layout-tests/editing/inserting/insert-div-009-expected.txt: Added.
  • layout-tests/editing/inserting/insert-div-009.html: Added.
10:21 AM Changeset in webkit [8126] by kocienda
  • 2 edits in trunk/WebCore

Reviewed by Harrison

Fix for this bug:

<rdar://problem/3906948> REGRESSION (Mail): Insert paragraph code can make the insertion point "stick" in place.

  • khtml/editing/htmlediting.cpp: (khtml::InsertParagraphSeparatorCommand::doApply): Call insertBlockPlaceholderIfNeeded(), passing block being added to this function. This ensures that the added block has a height. (khtml::InsertParagraphSeparatorInQuotedContentCommand::doApply): Ditto.
10:21 AM Changeset in webkit [8125] by kocienda
  • 2 edits in trunk/WebKit

Reviewed by Harrison

Fix for this bug:

<rdar://problem/3906930> Hitting return key in editable content inserts br elements instead of blocks

  • WebView.subproj/WebHTMLView.m: (-[WebHTMLView insertNewline:]): One-line change to call insert-block rather than insert-br method on bridge.
9:39 AM Changeset in webkit [8124] by kocienda
  • 6 edits in trunk

Reviewed by John

  • khtml/dom/dom_string.cpp: (DOM::DOMString::substring): Expose method already on DOMStrimgImpl.
  • khtml/dom/dom_string.h: Ditto.
  • khtml/editing/htmlediting.cpp: (khtml::CompositeEditCommand::rebalanceWhitespace): New helper to create and execute a RebalanceWhitespaceCommand instance. (khtml::DeleteSelectionCommand::doApply): Call rebalanceWhitespace() after running command. (khtml::InsertLineBreakCommand::doApply): Ditto. (khtml::InsertParagraphSeparatorCommand::doApply): Ditto. (khtml::InsertParagraphSeparatorInQuotedContentCommand::doApply): Ditto. (khtml::InsertTextCommand::input): Ditto. (khtml::RebalanceWhitespaceCommand::RebalanceWhitespaceCommand): New command. (khtml::RebalanceWhitespaceCommand::~RebalanceWhitespaceCommand): Ditto. (khtml::RebalanceWhitespaceCommand::doApply): Ditto. (khtml::RebalanceWhitespaceCommand::doUnapply): Ditto. (khtml::RebalanceWhitespaceCommand::preservesTypingStyle): Ditto. (khtml::ReplaceSelectionCommand::completeHTMLReplacement): Ditto.
  • khtml/editing/htmlediting.h: Ditto. (khtml::RebalanceWhitespaceCommand::): Ditto.

Dec 5, 2004:

12:40 PM Changeset in webkit [8123] by darin
  • 2 edits in trunk/WebCore
  • fixed small problem in my check-in from yesterday
  • kwq/KWQEvent.mm: (positionForEvent): Get location from event without raising exception if it's the wrong type. (clickCountForEvent): Same, for clickCount. (QMouseEvent::QMouseEvent): Use the new helper functions so this can be constructed even with the wrong type of NSEvent. Required for cases where a keyboard event causes a "click" and we need to synthesize a QMouseEvent for KHTML internal use, using the key down NSEvent.

Dec 4, 2004:

2:31 PM Changeset in webkit [8122] by darin
  • 15 edits in trunk

WebCore:

Reviewed by John.

  • fixed <rdar://problem/3878329> REGRESSION (169-170): colors are wrong for my.yahoo.com due to CSS background parsing changes
  • khtml/css/cssparser.cpp: (CSSParser::parseBackgroundShorthand): Changed function so it doesn't rely on the position attribute being at the end of the array and then moved position attribute before color attribute so it takes precedence. Since "0" can be both the X coordinate of a position and a legal color (meaning black), we need to do position first.
  • fixed <rdar://problem/3760869> click events for input type=button or type=checkbox don't have flags like shiftKey set
  • khtml/rendering/render_form.h: Remove unused RenderFormElement fields.
  • khtml/rendering/render_form.cpp: (RenderFormElement::RenderFormElement): Take out code to set a bunch of unused fields. (RenderFormElement::slotClicked): Change to create the QMouseEvent from the actual mouse event rather than creating it with all the flags set to 0, using the new QMouseEvent constructor that does so.
  • kwq/KWQEvent.h: Added constructor that takes no parameters which uses the current event from AppKit. Made the click count getter const and added an isDoubleClick that matches the logic used elsewhere. Added a fixState helper method so the constructors can save code.
  • kwq/KWQEvent.mm: (QMouseEvent::QMouseEvent): Factored out the state-fixing code that was in the two existing constructors and added a third constructor that uses the "current event" from AppKit (used above). (QMouseEvent::fixState): Compute state and click count based on event type.
  • fixed first symptom of <rdar://problem/3830936> REGRESSION (125-165): crash due to null font family, hang at changeforamerica.com
  • kwq/KWQFontFamily.mm: (KWQFontFamily::getNSFamily): Handle empty strings specially so we don't run into trouble when the family name is a null string. This prevents the crash, but there are still other problems that may have the same underlying cause in CSS.
  • fixed <rdar://problem/3829808> Safari crashes when adding a DOM node that was removed from an XMLHTTP request result
  • khtml/html/html_baseimpl.cpp: (HTMLBodyElementImpl::insertedIntoDocument): Added nil check before calling scheduleRelayout. This is new code so the nil-dereference is a recent regression.

WebKit:

Reviewed by John.

  • fixed <rdar://problem/3846079> assertion failure in WebHTMLView(WebPrivate) removeTrackingRect at boots.com
  • fixed <rdar://problem/3857737> REGRESSION (165-166): clicking in a text field that's scrolled to the right causes it to scroll all the way left
  • fixed <rdar://problem/3861952> REGRESSION (165-166): selection is cleared when you start to scroll a frame
  • WebView.subproj/WebHTMLViewInternal.h: Added handlingMouseDown flag.
  • WebView.subproj/WebHTMLView.m: (-[WebHTMLView _addTrackingRect:owner:userData:assumeInside:useTrackingNum:]): Allow passing in a tracking number of 0, which means no existing tracking number. (-[WebHTMLView _addTrackingRects:owner:userDataList:assumeInsideList:trackingNums:count:]): Ditto. (-[WebHTMLView removeTrackingRect:]): Allow removing a tracking number of 0, which is a no-op. (-[WebHTMLView _removeTrackingRects:count:]): Ditto. (-[WebHTMLView acceptsFirstResponder]): Changed check to use handlingMouseDown flag instead of mouseDownEvent field since that field is set up too early in the mouse down event handling process. (-[WebHTMLView mouseDown:]): Added code to set handlingMouseDown flag.
  • fixed part of <rdar://problem/3829808> Safari crashes when adding a DOM node that was removed from an XMLHTTP request result
  • WebCoreSupport.subproj/WebBridge.m: (-[WebBridge syncLoadResourceWithURL:customHeaders:postData:finalURL:responseHeaders:statusCode:]): Changed code around so that it won't try to create a WebResource when the load fails.
  • moved next/previous links into private structure with the rest of WebFrame fields (We have a rule against putting new fields into obejcts that are part of our public API.)
  • WebView.subproj/WebFrame.h: Remove _nextSibling and _previousSibling.
  • WebView.subproj/WebFramePrivate.h: Added nextSibling and previousSibling fields to private class.
  • WebView.subproj/WebFrame.m: Got rid of some tabs in this file. (-[WebFrame _addChild:]): Changed code to use fields inside _private. (-[WebFrame _removeChild:]): Ditto. (-[WebFrame _nextFrameWithWrap:]): Ditto. (-[WebFrame _previousFrameWithWrap:]): Ditto.
10:37 AM Changeset in webkit [8121] by cblu
  • 22 edits in trunk

WebCore:

New fixes for:
<rdar://problem/3685766> WebDataSource is missing subresources when they use cached WebCore data
<rdar://problem/3722434> REGRESSION?: Assertion failure trying to drag image in iframe (itapema.sc.gov.br)
<rdar://problem/3903173> REGRESSION (172-TOT): assertion failure and crash in slotAllData logging into hotmail account
<rdar://problem/3902749> REGRESSION (Tiger): missing image symbol does not appear

Reviewed by darin, rjw, kocienda.

  • khtml/misc/loader.cpp: (CachedObject::~CachedObject): (CachedCSSStyleSheet::checkNotify): (Loader::servePendingRequests): (Loader::slotFinished): (Loader::slotReceivedResponse): (Cache::requestImage): (Cache::requestScript):
  • khtml/misc/loader.h: (khtml::CachedObject::CachedObject): (khtml::CachedObject::response): (khtml::CachedObject::allData):
  • kwq/KWQKJobClasses.h:
  • kwq/KWQKJobClasses.mm: (KIO::TransferJobPrivate::TransferJobPrivate): (KIO::TransferJobPrivate::~TransferJobPrivate): (KIO::TransferJob::TransferJob): (KIO::TransferJob::assembleResponseHeaders): (KIO::TransferJob::retrieveCharset): (KIO::TransferJob::emitResult): (KIO::TransferJob::emitReceivedResponse):
  • kwq/KWQLoader.h:
  • kwq/KWQLoader.mm: (KWQHeaderStringFromDictionary): (KWQCheckCacheObjectStatus): (KWQIsResponseURLEqualToURL): (KWQResponseURL): (KWQResponseMIMEType): (KWQCacheObjectExpiresTime): (khtml::CachedObject::setResponse): (khtml::CachedObject::setAllData):
  • kwq/KWQPixmap.h:
  • kwq/KWQPixmap.mm: (QPixmap::QPixmap):
  • kwq/KWQResourceLoader.mm: (-[KWQResourceLoader finishJobAndHandle:]): (-[KWQResourceLoader cancel]): (-[KWQResourceLoader reportError]): (-[KWQResourceLoader finishWithData:]):
  • kwq/KWQSignal.h:
  • kwq/KWQSignal.mm: (KWQSignal::call):
  • kwq/KWQSlot.h:
  • kwq/KWQSlot.mm: (KWQSlot::KWQSlot): (KWQSlot::call):
  • kwq/WebCoreBridge.h:
  • kwq/WebCoreResourceLoader.h:

WebKit:

New fixes for:
<rdar://problem/3685766> WebDataSource is missing subresources when they use cached WebCore data
<rdar://problem/3722434> REGRESSION?: Assertion failure trying to drag image in iframe (itapema.sc.gov.br)
<rdar://problem/3903173> REGRESSION (172-TOT): assertion failure and crash in slotAllData logging into hotmail account
<rdar://problem/3902749> REGRESSION (Tiger): missing image symbol does not appear

Reviewed by darin, rjw, kocienda.

  • WebCoreSupport.subproj/WebBridge.m: (-[WebBridge objectLoadedFromCacheWithURL:response:data:]): (-[WebBridge syncLoadResourceWithURL:customHeaders:postData:finalURL:responseHeaders:statusCode:]):
  • WebCoreSupport.subproj/WebSubresourceClient.m: (-[WebSubresourceClient didFinishLoading]):
  • WebView.subproj/WebFrame.m: (-[WebFrame _opened]): (-[WebFrame _internalLoadDelegate]): (-[WebFrame _sendResourceLoadDelegateMessagesForURL:response:length:]):
  • WebView.subproj/WebFrameInternal.h:
10:30 AM Changeset in webkit [8120] by darin
  • 2 edits in trunk/WebCore

Reviewed by John.

  • fixed <rdar://problem/3876093> REGRESSION (166-167): Setting slider control's value from JavaScript has no effect (breaks RSS)
  • khtml/rendering/render_form.cpp: (RenderSlider::updateFromElement): Call setValue to update the value of the DOM element rather than modifying the m_value data member directly. We don't use m_value at all for sliders now, and in fact the code relies on the fact that m_value is null. Setting m_value to a non-null value was causing the bug. (RenderSlider::slotSliderValueChanged): Ditto.
7:31 AM Changeset in webkit [8119] by darin
  • 3 edits in trunk/WebKit

Reviewed by Maciej.

  • fixed remaining bit of <rdar://problem/3814237> REGRESSION (Mail): Copy/paste style does not set color in Mail compose window
  • WebView.subproj/WebHTMLView.m: (-[WebHTMLView _styleFromFontAttributes:]): When translating from an attribute dictionary to a CSS declaration, treat missing values according to the defaults defined in <AppKit/NSAttributedString.h>. Before the code was treating them as "no change", which is incorrect.
  • English.lproj/StringsNotToBeLocalized.txt: Add a string from the above change.
Note: See TracTimeline for information about the timeline view.