Timeline



Dec 9, 2004:

8:38 PM Changeset in webkit [8179] by rjw
  • 2 edits in trunk/WebCore

Check to disable threaded decoding during
layout tests wasn't invoking function, just checking address of
function, which would always return true.

  • kwq/WebCoreImageRendererFactory.m: (+[WebCoreImageRendererFactory shouldUseThreadedDecoding]):
7:09 PM Changeset in webkit [8178] by rjw
  • 4 edits in trunk/WebKit

Fixed <rdar://problem/3905789> Burn Disc image vibrates rapidly

Restrict our support for animated images to GIF. We used to
use presence of more than one image in a resource to determine
if an image should be animated. This caused us to animate icns!
If we ever support any other animated image formats we'll have
to extend.

Reviewed by Hyatt.

  • WebCoreSupport.subproj/WebImageData.h:
  • WebCoreSupport.subproj/WebImageData.m: (-[WebImageData shouldAnimate]):
  • WebCoreSupport.subproj/WebImageRenderer.m: (-[WebImageRenderer _startOrContinueAnimationIfNecessary]):
7:05 PM Changeset in webkit [8177] by hyatt
  • 3 edits in trunk/WebCore

Fix for 3892686, left/top overflow was not being propagated properly up to containing blocks because of a math
error.

Also fix a bug I noticed on the same page where relative position offsets were not being added in properly for
all inlines when repainting.

Reviewed by rjw

  • khtml/rendering/render_block.cpp: (khtml::RenderBlock::layoutBlockChildren):
  • khtml/rendering/render_flow.cpp: (RenderFlow::getAbsoluteRepaintRect):
5:46 PM Changeset in webkit [8176] by rjw
  • 2 edits in trunk/WebKit

Make WebPluginDatabase.h private (Dashboard needs SPI).

  • WebKit.pbproj/project.pbxproj:
5:38 PM Changeset in webkit [8175] by hyatt
  • 1 add in trunk/LayoutTests/fast/block/float/tableshifting-expected.txt

New layout test to cover more details of tables shifting below floats.

5:38 PM Changeset in webkit [8174] by hyatt
  • 6 edits
    1 add in trunk

Fix for 3867545, finance.yahoo.com lays out incorrectly. Add a quirk that will prevent tables from moving
down below floats when there is insufficient space. Instead we will match Gecko and just spill out of the
containing block to the right. This appears to be the more common desired behavior, despite being wrong.
WinIE sometimes wraps and sometimes doesn't, but it's really hard for us to match its inconsistency.

Reviewed by john

  • khtml/rendering/render_block.cpp: (khtml::RenderBlock::getClearDelta):
  • layout-tests/apple-only/base/www.cnn.com/index-expected.txt:
  • layout-tests/apple-only/base/www.ebay.com/index-expected.txt:
  • layout-tests/apple-only/base/www.excite.com/index-expected.txt:
  • layout-tests/fast/block/margin-collapse/102-expected.txt:
  • layout-tests/fast/block/margin-collapse/102.html:
5:15 PM Changeset in webkit [8173] by rjw
  • 2 edits in trunk/WebCore

Fixed <rdar://problem/3914078> worldclock crashing gc related

Use ProtectedValue for Context2D instance members.

Reviewed by Maciej.

  • khtml/ecma/kjs_html.h:
4:37 PM Changeset in webkit [8172] by sullivan
  • 2 edits in trunk/WebCore

Reviewed by Dave.

  • fixed <rdar://problem/3731099> Move AXTitle string for image elements to AXDescription
  • kwq/KWQAccObject.mm: (-[KWQAccObject title]): moved image alt tag code out of here (-[KWQAccObject accessibilityDescription]): moved image alt tag code into this new method (-[KWQAccObject accessibilityAttributeNames]): include AXDescription in the set of attributes that ordinary elements return; this means that ordinary elements that aren't images will return a nil description, which isn't ideal, but is in keeping with the way the rest of these attributes work here. (-[KWQAccObject accessibilityAttributeValue:]): call accessibilityDescription when asked for AXDescription
4:10 PM Changeset in webkit [8171] by kocienda
  • 2 edits in trunk/WebCore

Reviewed by Harrison

Fix for this bug:

<rdar://problem/3910425> REGRESSION (Mail): Crash in ReplaceSelectionCommand; selection is empty, leading to null deref

  • khtml/editing/htmlediting.cpp: (khtml::MoveSelectionCommand::doApply): The node representing the destination for the move may have been deleted. If this is the case, set the destination to the node the delete command provides in its ending selection.
4:00 PM Changeset in webkit [8170] by vicki
  • 3 edits in trunk

versioning for TOT, Safari 2.0 (175+)

3:53 PM Changeset in webkit [8169]
  • 2 copies in branches/Safari-175-branch

This commit was manufactured by cvs2svn to create branch
'Safari-175-branch'.

3:53 PM Changeset in webkit [8168]
  • 2 copies in tags/Safari-175

This commit was manufactured by cvs2svn to create tag 'Safari-175'.

3:53 PM Changeset in webkit [8167] by vicki
  • 6 edits in trunk

Safari-175 stamp (skipped Safari-174 because it ends in a "4")

2:41 PM Changeset in webkit [8166] by kocienda
  • 1 edit in trunk/WebCore/ChangeLog-2005-08-23

Rolling out change. Tree was closed.

2:40 PM Changeset in webkit [8165] by kocienda
  • 2 edits in trunk/WebCore

Rolling out change. Tree was closed.

2:37 PM Changeset in webkit [8164] by kocienda
  • 2 edits in trunk/WebCore

Reviewed by Harrison

Fix for this bug:

<rdar://problem/3910425> REGRESSION (Mail): Crash in ReplaceSelectionCommand; selection is empty, leading to null deref

  • khtml/editing/htmlediting.cpp: (khtml::MoveSelectionCommand::doApply): The node representing the destination for the move may have been deleted. If this is the case, set the destination to the node the delete command provides in its ending selection.
11:49 AM Changeset in webkit [8163] by cblu
  • 2 edits in trunk/WebKit

Workaround for this exception being raised during download:
[WebDownload connection:willStopBufferingData:]: selector not recognized

Reviewed by john.

  • Misc.subproj/WebDownload.m: (-[WebDownload connection:willStopBufferingData:]): implement this method so no exception is raised. It is a bug in Foundation that this method is being called, but it's too late to fix Foundation since it has already been submitted this week. This workaround will prevent any problems from affecting users.
11:21 AM Changeset in webkit [8162] by kocienda
  • 4 edits in trunk/WebCore

Reviewed by John

Fix for this bug:

<rdar://problem/3912841> REGRESSION (173-TOT): Some images report 0x0 dimensions on layout tests, causes spurious test failures

The new threaded image decoding capability can throw off layout tests. The issue is that the decoding
callback may not be delivered before the program asks for the dimensions of an image in order to
wrote the layout dimensions. More generally, I think we need to ensure that there are no races in
layout tests, so I have added a flag to the render tree debug code that we can set when debugging.

  • kwq/KWQRenderTreeDebug.cpp: (debuggingRenderTree): New function. Returns flag which tells whether the program is debugging the render tree. (externalRepresentation): Sets debuggingRenderTree flag to true;
  • kwq/KWQRenderTreeDebug.h:
  • kwq/WebCoreImageRendererFactory.m: (+[WebCoreImageRendererFactory shouldUseThreadedDecoding]): Checks debuggingRenderTree flag and will not do threaded decoding in any case if the flag is set.
10:57 AM Changeset in webkit [8161] by cblu
  • 5 edits in trunk/WebCore

Fix for busting XMLHTTPRequest.

Reviewed by kocienda.

  • khtml/misc/loader.cpp: (Loader::servePendingRequests): pass true for deliverAllData
  • kwq/KWQKJob.h: (KIO::get): take deliverAllData param (KIO::http_post): ditto
  • kwq/KWQKJobClasses.h:
  • kwq/KWQKJobClasses.mm: (KIO::TransferJob::TransferJob): if deliverAllData, create signal with data param (KIO::TransferJob::emitResult): if deliverAllData, call signal with data param
7:32 AM Changeset in webkit [8160] by kocienda
  • 2 edits in trunk/WebCore

Reviewed by John

<rdar://problem/3911011> REGRESSION (Mail): Spaces at end of line causing word wrap lost when copied/pasted

  • khtml/xml/dom_nodeimpl.cpp: (NodeImpl::renderedText): Fixed the code so that spaces at the end of lines are not skipped.

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.
Note: See TracTimeline for information about the timeline view.