Timeline

Aug 23, 2004:

9:13 PM Changeset in webkit [7333] by mjs

• 3 edits in trunk/WebCore

Reviewed by Richard.

• reduce cost of innerHTML from O(N^{2) to O(N*D) where N is the number of nodes and D is the maximum DOM tree depth.}

• khtml/xml/dom_nodeimpl.cpp: (NodeImpl::recursive_toString): New static helper method for recursive_toHTML - this is recursive for children but iterative for siblings. (NodeImpl::recursive_toHTML): Call the helper with this as the first argument.
• khtml/xml/dom_nodeimpl.h:

8:22 PM Changeset in webkit [7332] by hyatt

• 7 edits in trunk/WebCore

Fix for 3558334.

Init the encodedURL explicitly for CSSStyleSelectors. Construction time was too early to be passing in the
document m_url, since it doesn't get set during construction.

Fix for 3769643, crash on vancouverblast.org.

Reviewed by darin

• khtml/css/cssstyleselector.cpp: (khtml::CSSStyleSelector::CSSStyleSelector): (khtml::CSSStyleSelector::init): (khtml::CSSStyleSelector::setEncodedURL):
• khtml/css/cssstyleselector.h:
• khtml/xml/dom_docimpl.cpp: (DocumentImpl::DocumentImpl): (DocumentImpl::setURL): (DocumentImpl::recalcStyleSelector):
• khtml/xml/dom_docimpl.h: (DOM::DocumentImpl::URL):

7:50 PM Changeset in webkit [7331] by mjs

• 2 edits in trunk/WebKit

Reviewed by Kevin.

• remove annoying ERROR spew and replace with comment

• WebView.subproj/WebHTMLView.m: (-[WebHTMLView validAttributesForMarkedText]):

6:19 PM Changeset in webkit [7330] by mjs

• 2 edits in trunk/WebKit

Reviewed by Richard.

• a bit of code cleanup

• WebView.subproj/WebDataSource.m: (-[WebDataSource _stringWithData:]): Call textEncodingName instead of copying it's code. (-[WebDataSource textEncodingName]): Tweak formatting.

5:12 PM Changeset in webkit [7329] by cblu

• 2 edits in trunk/WebKit

Fixed build.

• WebKit.pbproj/project.pbxproj: make sure we're doing -f on a file, not a directory

4:36 PM Changeset in webkit [7328] by cblu

• 6 edits
  2 adds in trunk/WebKit

Fixed: <rdar://problem/3674867> use new Security framework SPI's to reenable cert acquisition

Reviewed by john.

• WebCoreSupport.subproj/WebKeyGeneration.cpp:
• WebCoreSupport.subproj/WebKeyGeneration.h:
• WebCoreSupport.subproj/WebKeyGenerator.h:
• WebCoreSupport.subproj/WebKeyGenerator.m: (-[WebKeyGenerator signedPublicKeyAndChallengeStringWithStrengthIndex:challenge:pageURL:]): re-enabled cert code, call Panther version on Panther, Tiger version on Tiger (-[WebKeyGenerator addCertificatesToKeychainFromData:]): ditto
• WebCoreSupport.subproj/WebNewKeyGeneration.c: Added. (gnrAddContextAttribute): (gnrGetSubjPubKey): (gnrNullAlgParams): (gnrSign): (gnrFreeCssmData): (nssArraySize): (signedPublicKeyAndChallengeString): (addCertificateToKeychainFromData): (addCertificatesToKeychainFromData):
• WebCoreSupport.subproj/WebNewKeyGeneration.h: Added.
• WebKit.pbproj/project.pbxproj:

4:02 PM Changeset in webkit [7327] by darin

• 1 edit in trunk/WebCore/ChangeLog-2005-08-23

Get rid of stray conflict markers.

2:17 PM Changeset in webkit [7326] by kdecker

• 2 edits in trunk/WebCore

Reviewed by Maciej.

• fixed rdar://problem/3681094> Crash in KJS::WindowFunc::tryCall with application/xhtml+xml Content-Type
• khtml/ecma/kjs_window.cpp: (WindowFunc::tryCall): since this is an xml document, we get the domain from the xmlDocImpl(), not docImpl().

12:05 PM Changeset in webkit [7325] by vicki

• 3 edits in trunk

bump all version numbers to 160u, since I just submitted WebBrowser-159

11:06 AM Changeset in webkit [7324] by mjs

• 2 edits in trunk/WebCore

Reviewed by Darin.

<rdar://problem/3771426> assertion failed due to reentering dispatchImageLoadEventsNow

• khtml/xml/dom_docimpl.cpp: (DocumentImpl::dispatchImageLoadEventsNow): Avoid re-entering this function, since it uses a data member for the copy of the list of events to be dispatched.

10:52 AM Changeset in webkit [7323] by mjs

• 6 edits in trunk/WebCore

Reviewed by Darin.

<rdar://problem/3770306> XMLHttpRequest does not honor character set encoding

• khtml/ecma/xmlhttprequest.cpp: (KJS::XMLHttpRequest::slotData): Get encoding from the transfer job.
• kwq/KWQKJobClasses.h:
• kwq/KWQKJobClasses.mm: (KIO::TransferJobPrivate::TransferJobPrivate): Added retrievedCharset flag. (KIO::TransferJob::retrieveCharset): New method, gets the charset from the response. (KIO::TransferJob::queryMetaData): Handle charset. (KIO::TransferJob::emitReceivedResponse): Clear retreivedCharset flag.
• kwq/KWQLoader.h:
• kwq/KWQLoader.mm: (KWQResponseTextEncodingName): New function, gets the encoding from the response.

10:44 AM Changeset in webkit [7322] by hyatt

• 2 edits in trunk/WebCore

Apply leo's fix to marquees.

Reviewed by hyatt

• khtml/rendering/render_layer.cpp: (Marquee::start):

9:12 AM Changeset in webkit [7321] by darin

• 5 edits in trunk/WebCore

Reviewed by Maciej.

• added an ascii() member function to DOMString and DOMStringImpl to help debugging

• khtml/dom/dom_string.h: Add ascii member function for debugging.
• khtml/dom/dom_string.cpp: (DOM::DOMString::ascii): Added. Calls through to DOMStringImpl::ascii.
• khtml/xml/dom_stringimpl.h: Add ascii member function for debugging.
• khtml/xml/dom_stringimpl.cpp: (DOM::DOMStringImpl::ascii): Added. Makes a new buffer and puts a simple ASCII version in it. Maybe make it better about characters outside the 0x20-0x7E range some day, but for now this is way better than what we had before.

Aug 20, 2004:

5:58 PM Changeset in webkit [7320] by hyatt

• 4 edits in trunk/WebCore

Divorce the notion of a marquee being stopped from JS from the notion of being suspended by the back/forward cache.

• khtml/ecma/kjs_html.cpp: (KJS::HTMLElementFunction::tryCall):
• khtml/rendering/render_layer.cpp: (m_direction): (Marquee::start): (Marquee::stop): (Marquee::updateMarqueePosition):
• khtml/rendering/render_layer.h: (khtml::):

4:37 PM Changeset in webkit [7319] by hyatt

• 22 edits in trunk/LayoutTests

Fix layout tests.

4:12 PM Changeset in webkit [7318] by hyatt

• 2 edits in trunk/WebKit

Fix the directionality of the unicode hyphen so that on Panther it now matches Tiger.

Reviewed by darin

• Misc.subproj/WebUnicode.m: (_unicodeDirection):

4:09 PM Changeset in webkit [7317] by rjw

• 1 add in trunk/JavaScriptCore/bindings/jni/jni_objc.mm

Added.

2:57 PM Changeset in webkit [7316] by rjw

• 18 edits in trunk

WebKit:

Implemented new JNI abstraction. We no longer invoke Java methods
directly with JNI, rather we call into the plugin. This allows the
plugin to dispatch the call to the appropriate VM thread. This
change should (will?) fix a whole class of threading related problems with
the Java VM.

Reviewed by Hyatt.

• WebCoreSupport.subproj/WebBridge.m: (-[WebBridge getAppletInView:]):

WebCore:

Implemented new JNI abstraction. We no longer invoke Java methods
directly with JNI, rather we call into the plugin. This allows the
plugin to dispatch the call to the appropriate VM thread. This
change should (will?) fix a whole class of threading related problems with
the Java VM.

Reviewed by Hyatt.

• kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::getAppletInstanceForView):
• kwq/WebCoreBridge.mm: (rootForView): (-[WebCoreBridge executionContextForView:]):

JavaScriptCore:

Implemented new JNI abstraction. We no longer invoke Java methods
directly with JNI, rather we call into the plugin. This allows the
plugin to dispatch the call to the appropriate VM thread. This
change should (will?) fix a whole class of threading related problems with
the Java VM.

Reviewed by Hyatt.

• JavaScriptCore.pbproj/project.pbxproj:
• bindings/c/c_instance.h: (KJS::Bindings::CInstance::setExecutionContext): (KJS::Bindings::CInstance::executionContext):
• bindings/jni/jni_instance.cpp: (JavaInstance::JavaInstance): (JavaInstance::invokeMethod): (JavaInstance::setExecutionContext): (JavaInstance::executionContext):
• bindings/jni/jni_instance.h:
• bindings/jni/jni_jsobject.cpp: (JSObject::convertJObjectToValue):
• bindings/jni/jni_runtime.cpp: (JavaField::JavaField): (JavaArray::convertJObjectToArray): (JavaField::valueFromInstance): (JavaArray::JavaArray): (JavaArray::valueAt):
• bindings/jni/jni_runtime.h: (KJS::Bindings::JavaArray::operator=): (KJS::Bindings::JavaArray::executionContext):
• bindings/jni/jni_utility.h:
• bindings/objc/objc_instance.h: (KJS::Bindings::ObjcInstance::setExecutionContext): (KJS::Bindings::ObjcInstance::executionContext):
• bindings/runtime.cpp: (Instance::createBindingForLanguageInstance):
• bindings/runtime.h:
• bindings/runtime_root.h: (KJS::Bindings::RootObject::nativeHandle):

2:15 PM Changeset in webkit [7315] by mjs

• 2 edits in trunk/WebCore

Reviewed by Darin.

More text paint cleanup. Separated the background and foreground
passes instead of doing a weird for loop thing. Eliminated
redundant if conditions. Added comments.

• khtml/rendering/render_text.cpp: (RenderText::paint):

1:19 PM Changeset in webkit [7314] by kocienda

• 14 edits in trunk

Reviewed by Darin

• khtml/css/css_computedstyle.cpp: (DOM::): Changed CopyProperties constant to InheritableProperties. This reflects the name change of copy() to copyInheritableProperties() (DOM::CSSComputedStyleDeclarationImpl::copyInheritableProperties: Renamed from copy(). Now just copies those properties which can be inherited. (DOM::CSSComputedStyleDeclarationImpl::diff): Add a couple null checks.
• khtml/css/css_computedstyle.h: copyInheritableProperties name change. No longer needs to be virtual.
• khtml/css/css_valueimpl.cpp: Removed unneeded copy() function from CSSStyleDeclarationImpl.
• khtml/css/css_valueimpl.h: Ditto.
• khtml/editing/htmlediting.cpp: (khtml::EditCommand::typingStyle): Added. (khtml::EditCommand::setTypingStyle): Added.
• khtml/editing/htmlediting.h:
• khtml/editing/htmlediting_impl.cpp: (khtml::StyleChange::currentlyHasStyle): Fix leak of computed style used in this function. (khtml::EditCommandImpl::EditCommandImpl): Initialize m_typingStyle. (khtml::EditCommandImpl::~EditCommandImpl): Deref m_typingStyle. (khtml::EditCommandImpl::assignTypingStyle): New helper used in setting typing style. (khtml::EditCommandImpl::setTypingStyle): New setter. (khtml::DeleteSelectionCommandImpl::doApply): Use new method for managing typing style.
• khtml/editing/htmlediting_impl.h: (khtml::EditCommandImpl::typingStyle): New accessor.
• khtml/khtml_part.cpp: (KHTMLPart::appliedEditing): Restores typing style from command after setting selection. (KHTMLPart::applyStyle): Does a diff between the current style and the style of the caret.
• khtml/khtml_part.h:
• khtml/xml/dom_position.cpp: (DOM::Position::computedStyle): Now returns a CSSComputedStyleDeclarationImpl instead of a plain CSSStyleDeclarationImpl.
• khtml/xml/dom_position.h:
• layout-tests/editing/style/style-3681552-fix-002-expected.txt:

11:44 AM Changeset in webkit [7313] by trey

• 2 edits in trunk

Get rid of weird chars in bug titles from last checkin (maybe nb spaces?)

11:33 AM Changeset in webkit [7312] by vicki

• 1 edit in branches/Safari-1-0-branch/WebKit/ChangeLog

remove spurious ^{M characters from this ChangeLog, so that it is readable}

11:29 AM Changeset in webkit [7311] by trey

• 6 edits in trunk

WebCore:

3655407 - Editing:�complete:�ethod�nimplemented�WebKit�diting�PI)

One new support routine here.

Reviewed by John

• kwq/WebCoreBridge.h:
• kwq/WebCoreBridge.mm: (-[WebCoreBridge caretRectAtNode:offset:]): New routine. (-[WebCoreBridge rangeByExpandingSelectionWithGranularity:]): Fixed former misleading method name.

WebKit:

3655407 - Editing:�complete:�ethod�nimplemented�WebKit�diting�PI)

This feature is mostly implemented. The only loose ends I know of are:

3769654 - text insertions done via complete: should preserve case of full replacement string
3769652 - positioning of complete: popup window wrong for right-to-left languages

Reviewed by John

• WebView.subproj/WebHTMLView.m: (-[WebHTMLViewPrivate dealloc]): Free new object. (-[WebHTMLView menuForEvent:]): Bail on completion session. (-[WebHTMLView windowDidResignKey:]): Ditto. (-[WebHTMLView windowWillClose:]): Ditto. (-[WebHTMLView mouseDown:]): Ditto. (-[WebHTMLView resignFirstResponder]): Ditto. (-[WebHTMLView keyDown:]): Bail on completion session if WebCore takes the event. Give the CompleteController a crack at the key event. (-[WebHTMLView _expandSelectionToGranularity:]): Adopt method rename (-[WebHTMLView complete:]): Make CompleteController, tell it to do its thing. (-[WebHTMLView checkSpelling:]): Add ERROR(). (-[WebHTMLView showGuessPanel:]): Add ERROR(). (-[WebHTMLView _changeSpellingToWord:]): Add ERROR(). (-[WebHTMLView ignoreSpelling:]): Add ERROR().

(-[WebTextCompleteController initWithHTMLView:]):
(-[WebTextCompleteController dealloc]):
(-[WebTextCompleteController _insertMatch:]): Stick the new string into the doc.
(-[WebTextCompleteController _buildUI]): Make popup window.
(-[WebTextCompleteController _placePopupWindow:]): Position popup window.
(-[WebTextCompleteController doCompletion]): Lookup matches, display window.
(-[WebTextCompleteController endRevertingChange:moveLeft:]): Bail on complete: session.
(-[WebTextCompleteController filterKeyDown:]): Process keys while popup is up.
(-[WebTextCompleteController _reflectSelection]): Handle choice in popup.
(-[WebTextCompleteController tableAction:]): Handle double click in popup
(-[WebTextCompleteController numberOfRowsInTableView:]): Fill table with matches.
(-[WebTextCompleteController tableView:objectValueForTableColumn:row:]): Ditto.
(-[WebTextCompleteController tableViewSelectionDidChange:]): Handle selection change.

• WebView.subproj/WebHTMLViewInternal.h:

9:57 AM Changeset in webkit [7310] by kocienda

• 2 adds in trunk/LayoutTests/editing/inserting

I neglected to add these files on my previous checkin

9:55 AM Changeset in webkit [7309] by kocienda

• 3 edits in trunk

Reviewed by Trey

Fix for this bug:

<rdar://problem/3768378> crash typing newline in Blot

• khtml/editing/htmlediting_impl.cpp: (khtml::InputNewlineCommandImpl::doApply): Adding an assert in a recent change showed up that inserting newlines that was not being handled correctly for the case described in the bug. I added a new case to handle inserting BR's when at the caret max offset for a node, and this new code runs instead of the fall-through case that should not have been running and triggered the assert.
• layout-tests/editing/inserting/insert-br-case2-expected.txt: Regenerated results.
• layout-tests/editing/inserting/insert-br-case6-expected.txt: Added.
• layout-tests/editing/inserting/insert-br-case6.html: Added.

9:53 AM Changeset in webkit [7308] by trey

• 2 edits in trunk/WebCore

Fixing: Spellchecker called once or twice for every char typed.

Reviewed by Ken

• khtml/khtml_part.cpp: (KHTMLPart::setSelection): Don't do any spell checking if we're typing (it's done elsewhere, in markMisspellingsAfterTyping)

9:06 AM Changeset in webkit [7307] by sullivan

• 2 edits in trunk/WebKit

• English.lproj/StringsNotToBeLocalized.txt: checked this in; the only difference is that the strings were out of order.

Aug 19, 2004:

8:26 PM Changeset in webkit [7306] by mjs

• 2 edits in trunk/WebCore

Reviewed by John.

• fixed <rdar://problem/3549369> Crash at www.e1.ru in HTMLTokenizer::notifyFinished

Probably also fixed the following likely duplicates:

<rdar://problem/3503938> Safari crashed opening many tabs (HTMLTokenizer::notifyFinished(khtml::CachedObject*))
<rdar://problem/3566332> CrashTracer: ..405 crashes at com.apple.WebCore: QString::QString[unified] + 0x5c
<rdar://problem/3703964> CrashTracer: ...86 crashes at com.apple.WebCore: QString::QString[unified] + 0x5c
<rdar://problem/3703969> CrashTracer: ..234 crashes at com.apple.WebCore: HTMLTokenizer::notifyFinished + 0x1c8

• khtml/xml/dom_docimpl.cpp: (DocumentImpl::open): call setParsing(true), because we need to know we are once again parsing when we re-open a document that has previously completed loading.

7:21 PM Changeset in webkit [7305] by mjs

• 2 edits in trunk/WebCore

Reviewed by Dave and Darin.

• khtml/rendering/render_text.cpp: (RenderText::paint): Split apple and non-apple code paths to allow further cleanup.

5:38 PM Changeset in webkit [7304] by darin

• 3 edits in trunk/WebCore

Reviewed by Dave.

• fixed <rdar://problem/3767274> crash in partForWidget inside setFocus (test page attached)

• kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::nextKeyViewInFrame): Remove code to do position the widget; no longer needed since we reworked how widgets get their positions.
• kwq/KWQWidget.mm: (QWidget::setFocus): Ditto. This was the one that caused the bug.

5:11 PM Changeset in webkit [7303] by rjw

• 2 edits in trunk/WebKit

Continue to call old pollForAppletInView: in Tiger until
we get an plugin that supports the new API.

Reviewed by Chris.

• WebCoreSupport.subproj/WebBridge.m: (-[WebBridge getAppletInView:]):
• WebKit.pbproj/project.pbxproj:

4:23 PM Changeset in webkit [7302] by hyatt

• 2 edits in trunk/WebCore

Fix crash when text is contained inside a table-colgroup.

Reviewed by john

• khtml/xml/dom_textimpl.cpp: (TextImpl::rendererIsNeeded):

4:17 PM Changeset in webkit [7301] by trey

• 2 edits in trunk/WebCore

Unexpected errors hit while finding word boundaries, leading to crash.

Reviewed by Ken.

• kwq/KWQTextUtilities.cpp: (KWQFindWordBoundary): Don't call UCFindTextBreak with edge cases it thinks are param errors, and pass correct mask for forward case. Also fix off-by-one crashers in fallback code.

3:50 PM Changeset in webkit [7300] by hyatt

• 10 edits in trunk/WebCore

Make XSLT imports/includes work. This code has to be turned off until the newer version of libxslt is
available.

Reviewed by kocienda

• khtml/css/css_ruleimpl.cpp: (CSSImportRuleImpl::init):
• khtml/xml/dom_docimpl.cpp: (DocumentImpl::DocumentImpl): (DocumentImpl::~DocumentImpl): (DocumentImpl::applyXSLTransform):
• khtml/xml/dom_docimpl.h: (DOM::DocumentImpl::setTransformSource): (DOM::DocumentImpl::transformSource):
• khtml/xml/dom_xmlimpl.cpp: (DOM::ProcessingInstructionImpl::checkStyleSheet):
• khtml/xml/xml_tokenizer.cpp: (khtml::matchFunc): (khtml::openFunc): (khtml::readFunc): (khtml::writeFunc): (khtml::createQStringParser): (khtml::XMLTokenizer::setTransformSource):
• khtml/xsl/xsl_stylesheetimpl.cpp: (DOM::XSLStyleSheetImpl::XSLStyleSheetImpl): (DOM::XSLStyleSheetImpl::~XSLStyleSheetImpl): (DOM::XSLStyleSheetImpl::isLoading): (DOM::XSLStyleSheetImpl::clearDocuments): (DOM::XSLStyleSheetImpl::parseString): (DOM::XSLStyleSheetImpl::loadChildSheets): (DOM::XSLStyleSheetImpl::loadChildSheet): (DOM::XSLImportRuleImpl::parentStyleSheet): (DOM::XSLStyleSheetImpl::compileStyleSheet): (DOM::XSLStyleSheetImpl::locateStylesheetSubResource): (DOM::XSLImportRuleImpl::XSLImportRuleImpl): (DOM::XSLImportRuleImpl::~XSLImportRuleImpl): (DOM::XSLImportRuleImpl::setStyleSheet): (DOM::XSLImportRuleImpl::isLoading): (DOM::XSLImportRuleImpl::loadSheet):
• khtml/xsl/xsl_stylesheetimpl.h: (DOM::XSLStyleSheetImpl::setOwnerDocument): (DOM::XSLStyleSheetImpl::setDocument): (DOM::XSLStyleSheetImpl::markAsProcessed): (DOM::XSLStyleSheetImpl::processed): (DOM::XSLImportRuleImpl::href): (DOM::XSLImportRuleImpl::styleSheet): (DOM::XSLImportRuleImpl::isImportRule):
• khtml/xsl/xslt_processorimpl.cpp: (DOM::m_sourceDocument): (DOM::stylesheetLoadFunc): (DOM::XSLTProcessorImpl::transformDocument):
• khtml/xsl/xslt_processorimpl.h:

2:00 PM Changeset in webkit [7299] by vicki

• 3 edits in trunk

set version to Safari 2.0, (v159u) for tip of tree.

1:54 PM Changeset in webkit [7298]

• 3 copies in tags/Safari-159

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

1:54 PM Changeset in webkit [7297]

• 3 copies in tags/Safari-158

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

1:54 PM Changeset in webkit [7296]

• 3 copies in branches/ParisExpo-2004-branch

This commit was manufactured by cvs2svn to create branch
'ParisExpo-2004-branch'.

1:54 PM Changeset in webkit [7295]

• 3 copies in tags/ParisExpo-2004-anchor

This commit was manufactured by cvs2svn to create tag
'ParisExpo-2004-anchor'.

1:54 PM Changeset in webkit [7294] by vicki

• 6 edits in trunk

Safari-158 stamp

10:34 AM Changeset in webkit [7293] by cblu

• 2 edits in trunk/WebKit

Fixed typo in comment.

• Misc.subproj/WebIconDatabase.m: (-[WebIconDatabase _updateFileDatabase]):

10:33 AM Changeset in webkit [7292] by cblu

• 2 edits in trunk/WebKit

Fixed: <rdar://problem/3566336> CrashTracer: .2403 crashes at com.apple.WebKit: -[WebFileDatabase performSetObject:forKey:] + 0x94

Reviewed by trey.

• Misc.subproj/WebIconDatabase.m: (-[WebIconDatabase _updateFileDatabase]): pass WebFileDatabase copies of the mutable dictionaries or else they may be accessed on a separate thread as the main thread is modifying them

9:21 AM Changeset in webkit [7291] by vicki

• 2 edits in trunk/JavaScriptCore

• initialize deletedElementIndex to make the compiler happy