Timeline

Nov 20, 2004:

2:10 PM Changeset in webkit [8059] by mjs

• 3 edits in trunk/WebKit

WebKit:

Reviewed by John.

<rdar://problem/3710101> _web_userVisibleString makes URL autocomplete roughly 2x slower

• Misc.subproj/WebNSURLExtras.h:

• Misc.subproj/WebNSURLExtras.m: (-[NSString _web_isUserVisibleURL]): New SPI to check if a URL string is already in user-visible form (i.e. converting it to an NSURL and then back via _web_userVisibleString would not change anything).

WebBrowser:

Reviewed by John,

<rdar://problem/3710101> _web_userVisibleString makes URL autocomplete roughly 2x slower

• URLCompletionController.m: (_rawStringFromItem): use URLString from history item when possible, only convert to user visible string when we really have to.

1:44 PM Changeset in webkit [8058] by mjs

• 2 edits in trunk/WebKit

WebKit:

Reviewed by NOBODY (OOPS!).

<rdar://problem/3190977> closing window with many tabs in it can be quite slow

• WebView.subproj/WebFrame.m: (-[WebFrame _detachFromParent]): autorelease bridge instead of releasing it, to make window and tab closing more responsive - this way the deallocation happens after the windoow or tab appears to close.

WebBrowser:

Reviewed by NOBODY (OOPS!).

<rdar://problem/3190977> closing window with many tabs in it can be quite slow

• BrowserDocument.m: (-[BrowserDocument close]): Don't make an autorelease pool here. This should not be needed for PLT world leak detection any more, and it is necessary to make window closing responsive.

Nov 19, 2004:

6:24 PM Changeset in webkit [8057] by mjs

• 10 edits in trunk/WebCore

Reviewed by Darin.

<rdar://problem/3482935> JavaScript so slow it seems like a hang (hrweb.apple.com) (HTMLCollection?)
<rdar://problem/3759149> PeopleSoft page in Safari twice as slow as Mozilla engine (HTMLFormCollection)
<rdar://problem/3888368> selecting an item on the Apache bugzilla query page can be improved 95% (HTMLFormCollection)

Many optimizations to HTMLFormCollection. Iterating it should not
be N^{2 any more, though finding items by name could still be.}

• khtml/html/html_formimpl.cpp: (DOM::HTMLFormElementImpl::~HTMLFormElementImpl): (DOM::HTMLFormElementImpl::length): (DOM::HTMLFormElementImpl::submitClick): (DOM::HTMLFormElementImpl::formData): (DOM::HTMLFormElementImpl::submit): (DOM::HTMLFormElementImpl::reset): (DOM::HTMLFormElementImpl::radioClicked): (DOM::appendToVector): (DOM::removeFromVector): (DOM::HTMLFormElementImpl::registerFormElement): (DOM::HTMLFormElementImpl::removeFormElement): (DOM::HTMLFormElementImpl::makeFormElementDormant): (DOM::HTMLFormElementImpl::registerImgElement): (DOM::HTMLFormElementImpl::removeImgElement):
• khtml/html/html_formimpl.h:
• khtml/html/html_miscimpl.cpp: (HTMLFormCollectionImpl::FormCollectionInfo::FormCollectionInfo): (void::HTMLFormCollectionImpl::FormCollectionInfo::reset): (HTMLFormCollectionImpl::resetCollectionInfo): (HTMLFormCollectionImpl::calcLength): (HTMLFormCollectionImpl::item): (HTMLFormCollectionImpl::getNamedItem): (HTMLFormCollectionImpl::getNamedFormItem): (HTMLFormCollectionImpl::firstItem): (HTMLFormCollectionImpl::nextItem):
• khtml/html/html_miscimpl.h: (DOM::HTMLFormCollectionImpl::~HTMLFormCollectionImpl):
• khtml/xml/dom_elementimpl.cpp: (ElementImpl::setAttribute): (ElementImpl::setAttributeMap):
• kwq/KWQPtrVector.h: (QPtrVector::findRef):
• kwq/KWQVectorImpl.h:
• kwq/KWQVectorImpl.mm: (KWQVectorImpl::findRef):
• kwq/WebCoreBridge.mm: (-[WebCoreBridge elementWithName:inForm:]): (-[WebCoreBridge controlsInForm:]):

5:36 PM Changeset in webkit [8056] by harrison

• 2 edits in trunk/WebCore

Reviewed by Ken and Darin.

<rdar://problem/3856215> Cannot remove bold from the beginning of a message

Problem is that KHTMLPart::computeAndSetTypingStyle always looked upstream
for the existing style, but in this case (hitting cmd-B with caret at top of
file) there is nothing upstream. Changed this to use the VisiblePosition
deepEquivalent instead.

• khtml/khtml_part.cpp: (KHTMLPart::computeAndSetTypingStyle):

5:13 PM Changeset in webkit [8055] by adele

• 1 edit in branches/Safari-1-2-branch/WebCore/ChangeLog-2005-08-23

* empty log message *

5:09 PM Changeset in webkit [8054] by adele

• 1 edit in branches/Safari-1-2-branch/WebCore/khtml/xml/dom_docimpl.cpp

Fixed by Maciej, merged by Darin for SUPanPlum <rdar://problem/3872586>.

2004-11-19 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Darin.

<rdar://problem/3864151> REGRESSION (125-167): Chrysler.com never stops loading

• khtml/xml/dom_docimpl.cpp: (DocumentImpl::close): Don't fire the onload handler if there is a redirect pending. This is a very long-standing bug that was masked by our previously incorrect redirect logic. It used to be that an older redirect would always win. Recently we changed things so that a newer redirect would win, but a script that causes a redirect would stop parsing once complete (so if there are two redirects in the same script, the latter wins). However, we should have also prevented onload in this case. Testing with other browsers shows that onload handlers do not run at all when there is a pending redirect.

4:57 PM Changeset in webkit [8053] by vicki

• 3 edits in trunk

versioning for TOT, Safari 2.0 (v172+)

The tree is OPEN!

4:48 PM Changeset in webkit [8052]

• 2 copies in tags/Safari-172

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

4:48 PM Changeset in webkit [8051] by vicki

• 6 edits in trunk

Safari-172 stamp

3:21 PM Changeset in webkit [8050] by cblu

• 2 edits in trunk/WebKit

Fixed: <rdar://problem/3880387> REGRESSION: www.shockplay.com site gives "Unexpected server response"

Reviewed by mjs.

• Plugins.subproj/WebBaseNetscapePluginView.m: (-[NSData _web_locationAfterFirstBlankLine]): support both formats ("\r\n\n" and "\r\n\r\n") for separating header data from body data because Shockwave still sends the prior format

1:25 PM Changeset in webkit [8049] by mjs

• 2 edits in trunk/WebCore

Reviewed by Darin.

<rdar://problem/3864151> REGRESSION (125-167): Chrysler.com never stops loading

• khtml/xml/dom_docimpl.cpp: (DocumentImpl::close): Don't fire the onload handler if there is a redirect pending. This is a very long-standing bug that was masked by our previously incorrect redirect logic. It used to be that an older redirect would always win. Recently we changed things so that a newer redirect would win, but a script that causes a redirect would stop parsing once complete (so if there are two redirects in the same script, the latter wins). However, we should have also prevented onload in this case. Testing with other browsers shows that onload handlers do not run at all when there is a pending redirect.

11:58 AM Changeset in webkit [8048] by kocienda

• 2 edits in trunk/WebCore

Reviewed by Harrison

Fix some object lifetime issues in these two commands. This fixes some crashes
I am seeing in some new code I am working on, but have not yet reproduced otherwise.

• khtml/editing/htmlediting.cpp: (khtml::InsertParagraphSeparatorCommand::~InsertParagraphSeparatorCommand): No longer deref nodes in the ancestor list. They are not ref'ed when put on list. D'uh. (khtml::InsertParagraphSeparatorCommand::doApply): Ref all cloned nodes that are created by the command before putting them on the cloned nodes list. This are still deref'ed in the destructor. (khtml::InsertParagraphSeparatorInQuotedContentCommand::~InsertParagraphSeparatorInQuotedContentCommand): Ditto destructor comment. (khtml::InsertParagraphSeparatorInQuotedContentCommand::doApply): Ditto doApply comment.

10:43 AM Changeset in webkit [8047] by kocienda

• 9 edits in trunk

WebCore:

Reviewed by Harrison

Fix for this bug:

<rdar://problem/3655241> setTypingStyle: does not set the real typing style, and typingStyle does not return it

• khtml/khtml_part.cpp: (KHTMLPart::computeAndSetTypingStyle): New helper that does the work of reducing a passed-in style declaration given the current selection, and then sets the minimum necessary style as the typing style on the part. (KHTMLPart::applyStyle): Call new computeAndSetTypingStyle. The guts of computeAndSetTypingStyle used to be here in the selection-as-caret case. But now [WebCoreBridge setTypingStyle:] needs this code as well.
• khtml/khtml_part.h: Declare new computeAndSetTypingStyle() function.
• kwq/WebCoreBridge.h: Declare new typingStyle and setTypingStyle: methods.
• kwq/WebCoreBridge.mm: (-[WebCoreBridge typingStyle]): Calls through to the part to retrieve the typing style. (-[WebCoreBridge setTypingStyle:]): Calls through to the part to set the typing style.

WebKit:

Reviewed by Harrison

Fix for this bug:

<rdar://problem/3655241> setTypingStyle: does not set the real typing style, and typingStyle does not return it

• WebCoreSupport.subproj/WebBridge.m: (-[WebBridge respondToChangedContents]): No longer call through to WebKit to set the typing style. The call was part of the misguided use of the setTypingStyle: and typingStyle as a cache of what was stored on the WebCore side. (-[WebBridge respondToChangedSelection]): Ditto.
• WebView.subproj/WebView.m: (-[WebViewPrivate dealloc]): Object no longer has typingStyle ivar. (-[WebView setTypingStyle:]): Call over the bridge to set typing style. (-[WebView typingStyle]): Call over the bridge to retrieve typing style.
• WebView.subproj/WebViewInternal.h: Object no longer has typingStyle ivar.