Timeline



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

Aug 18, 2004:

5:30 PM Changeset in webkit [7290] by trey
  • 2 edits in trunk/WebCore

3765958 - downstreamPosition() can hit infinite loop when at end of doc

The problem was that I had a position that was after the maximum position in the text
node, because it was the old caret position before a backspace was processed. Later
I happened to call downstream() on that position, and hit the bug. Fix is to consider
a position past the end point of its node if it is *greater than* or equal to its max offset.

Reviewed by Ken.

  • khtml/xml/dom_positioniterator.cpp: (DOM::PositionIterator::atEnd):
5:15 PM Changeset in webkit [7289] by hyatt
  • 2 edits in trunk/WebCore

Make the header overlap the footer in the zero-line case. Changes to Emerson's template will ensure the
header draws over the footer.

Reviewed by darin

  • khtml/rendering/render_block.cpp: (khtml::getHeightForLineCount):
4:34 PM Changeset in webkit [7288] by rjw
  • 5 edits in trunk

WebKit:

Replace horrible pollForAppletInView: with new
webPlugInGetApplet. The details of how the applet instance
is provided now belong to the Java team. Yeh.

Reviewed by Chris.

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

WebCore:

Replace horrible pollForAppletInView: with new
webPlugInGetApplet. The details of how the applet instance
is provided now belong to the Java team. Yeh.

Reviewed by Chris.

  • kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::getAppletInstanceForView):
  • kwq/WebCoreBridge.h:
11:19 AM Changeset in webkit [7287] by cblu
  • 6 edits in trunk/WebCore

Fixed: <rdar://problem/3692199> 8A146: Safari crashes in toHTMLWithOptions, selection with no renderer (various sites)

Reviewed by trey.

  • khtml/xml/dom2_rangeimpl.cpp: (DOM::RangeImpl::toHTML): renamed, don't assume that nodes of the range had renderers, use the common ancestor of the range as the root
  • khtml/xml/dom2_rangeimpl.h:
  • khtml/xml/dom_nodeimpl.cpp: (NodeImpl::recursive_toHTML): renamed, removed code that determines whether to include the root in the HTML, leave this up to the caller
  • khtml/xml/dom_nodeimpl.h:
  • kwq/WebCoreBridge.mm: (-[WebCoreBridge markupStringFromNode:nodes:]): call renamed methods (-[WebCoreBridge markupStringFromRange:nodes:]): ditto
10:43 AM Changeset in webkit [7286] by kocienda
  • 2 edits in trunk/WebCore

Reviewed by me

  • khtml/css/css_valueimpl.cpp: (CSSStyleDeclarationImpl::copy): Roll back silly last minute change that broke this code. Note to self: read code before making changes to it.
10:34 AM Changeset in webkit [7285] by kocienda
  • 5 edits in trunk/WebCore

Coded by Darin and Ken

  • khtml/css/css_computedstyle.cpp: Added CopyProperties static array. This contains the properties we implement that we also want to copy in the new CSSComputedStyleDeclarationImpl::copy described below. (DOM::CSSComputedStyleDeclarationImpl::copy): New function. In this class, copies the computed values of all the properties listed in CopyProperties. In essence, this makes a freeze-dired version of a computed style. (DOM::CSSComputedStyleDeclarationImpl::diff): Removes every property from the passed-in CSSStyleDeclarationImpl that is also in the computed style.
  • khtml/css/css_computedstyle.h:
  • khtml/css/css_valueimpl.cpp: (CSSStyleDeclarationImpl::copy): New function. In this class, the copy operation is straightforward. Returns a copy that will be unchanged when the original changes.
  • khtml/css/css_valueimpl.h: (DOM::CSSStyleDeclarationImpl::values): Added a accessor suitable for use when the CSSStyleDeclarationImpl is const.

Aug 17, 2004:

8:13 PM Changeset in webkit [7284] by darin
  • 1 edit in trunk/WebKit/ChangeLog

Change some tabs to spaces.

6:16 PM Changeset in webkit [7283] by rjw
  • 5 edits in trunk/WebKit

Fixed <rdar://problem/3759093> Need PDF context menu: "Open in Preview" or other external app

Added support for "Open with ..." in PDF view.

Reviewed by Chris and Trey.

  • ChangeLog:
  • English.lproj/Localizable.strings:
  • English.lproj/StringsNotToBeLocalized.txt:
  • WebView.subproj/WebPDFView.h:
  • WebView.subproj/WebPDFView.m: (-[WebPDFView initWithFrame:]): (-[WebPDFView dealloc]): (applicationInfoForMIMEType): (-[WebPDFView path]): (-[WebPDFView menuForEvent:]): (-[WebPDFView setDataSource:]): (-[WebPDFView layout]): (-[WebPDFView viewDidMoveToHostWindow]): (-[WebPDFView openWithFinder:]):
5:09 PM Changeset in webkit [7282] by mjs
  • 2 edits in trunk/WebCore

Reviewed by Darin.

<rdar://problem/3703768> CrashTracer: ...50 crashes at com.apple.WebCore: KHTMLPart::xmlDocImpl const + 0

  • khtml/khtmlview.cpp: (KHTMLView::viewportMouseMoveEvent): Add a nil check and an assertion for m_part being null. It seems impossible for this to happen, so we want to debug it ourselves, but in the meantime, let's try to avoid causing crashes for our users.
5:07 PM Changeset in webkit [7281] by darin
  • 3 edits in trunk/JavaScriptCore

Reviewed by Adele.

  • fixed <rdar://problem/3746676> SAP WebDynpro app hangs inside JavaScript property map hash table code (deleted sentinel problem)
  • kjs/property_map.h: Added some private functions.
  • kjs/property_map.cpp: (KJS::PropertyMap::clear): Set sentinelCount to 0. (KJS::PropertyMap::put): Complete search for the element before choosing to use the deleted-element sentinel. Also keep sentinel count up to date when we destroy a sentinel by overwriting with a new added element. (KJS::PropertyMap::expand): Added. Calls rehash with a size 2x the old size, or 16. (KJS::PropertyMap::rehash): Added. Refactored the rehash code into a separate function. (KJS::PropertyMap::remove): Add one to sentinelCount, and rehash if 1/4 or more of the elements are deleted-element sentinels. (KJS::PropertyMap::checkConsistency): Check the sentinelCount.
4:19 PM Changeset in webkit [7280] by mjs
  • 2 edits in trunk/JavaScriptCore

Code change by Eric Albert, reviewd by me.

<rdar://problem/3571960> washingtonpost.com claims I don't have cookies enabled and won't let me read articles

  • kjs/date_object.cpp: (timetUsingCF): Clamp time to LONG_MAX (getting rid of time_t entirely would be even better, but is not required to fix this bug.
3:14 PM Changeset in webkit [7279] by rjw
  • 1 edit in trunk/WebKit/Plugins.subproj/WebJavaPlugIn.h

JNI needs both the jmethodID and return type. Changed API to
pass both.

Reviewed by Chris.

  • Plugins.subproj/WebJavaPlugIn.h:
3:03 PM Changeset in webkit [7278] by hyatt
  • 3 edits in trunk/WebCore

Fix the line truncation function for Emerson so that at the far left setting of the slider, only the header
is visible.

Reviewed by darin

  • khtml/rendering/render_block.cpp: (khtml::getHeightForLineCount):
  • khtml/rendering/render_flexbox.cpp: (khtml::RenderFlexibleBox::layoutVerticalBox):
2:53 PM Changeset in webkit [7277] by kocienda
  • 18 edits in trunk

Reviewed by Hyatt

Rewrite of the command that deletes a selection. I deleted great
big swaths of bug-ridden code to accomplish this and replaced it
with code that is much cleaner and smarter.

Also, renamed equivalentUpstreamPosition and equivalentDownstreamPosition to
upstream to downstream, respectively.

Added a couple of new helper methods.

  • khtml/editing/htmlediting.cpp: DeleteCollapsibleWhitespaceCommand and RemoveNodeAndPruneCommand now obsolete. A huge win.
  • khtml/editing/htmlediting.h: Ditto.
  • khtml/editing/htmlediting_impl.cpp: (khtml::debugPosition): Fix printf which had a placeholder, but no argument passed in the varargs. (khtml::CompositeEditCommandImpl::deleteUnrenderedText): New helper. Much simplified and cleaner version of (khtml::ApplyStyleCommandImpl::doApply): upstream/downstream name change (khtml::ApplyStyleCommandImpl::nodeFullySelected): upstream/downstream name change (khtml::DeleteSelectionCommandImpl::doApply): upstream/downstream name change (khtml::DeleteTextCommandImpl::DeleteTextCommandImpl): Add an assert to check that the passed offset is less than the length of the text node. (khtml::InputNewlineCommandImpl::insertNodeAfterPosition): upstream/downstream name change (khtml::InputNewlineCommandImpl::insertNodeBeforePosition): upstream/downstream name change (khtml::InputNewlineCommandImpl::doApply): upstream/downstream name change (khtml::InputTextCommandImpl::prepareForTextInsertion): upstream/downstream name change (khtml::InputTextCommandImpl::execute): upstream/downstream name change (khtml::InputTextCommandImpl::insertSpace): upstream/downstream name change (khtml::ReplaceSelectionCommandImpl::doApply): upstream/downstream name change (khtml::TypingCommandImpl::issueCommandForDeleteKey): upstream/downstream name change (khtml::TypingCommandImpl::deleteKeyPressed):
  • khtml/editing/htmlediting_impl.h:
  • khtml/xml/dom_position.cpp: (DOM::Position::previousWordBoundary): (DOM::Position::nextWordBoundary): (DOM::Position::upstream): (DOM::Position::downstream): (DOM::Position::inRenderedText): Add null check. (DOM::Position::isRenderedCharacter): New helper. (DOM::isWS): New helper in this file. (DOM::Position::leadingWhitespacePosition): New helper. Factored out from htmlediting_impl.cpp. (DOM::Position::trailingWhitespacePosition): Ditto. (DOM::Position::debugPosition): Add null check.
  • khtml/xml/dom_position.h:
  • khtml/xml/dom_selection.cpp: (DOM::Selection::toRange): upstream/downstream name change (DOM::Selection::validate): upstream/downstream name change (DOM::Selection::debugPosition): upstream/downstream name change
  • layout-tests/editing/deleting/delete-block-contents-003-expected.txt: Updated tests with new expected results.
  • layout-tests/editing/deleting/delete-contiguous-ws-001-expected.txt: Ditto.
  • layout-tests/editing/deleting/delete-selection-001-expected.txt: Ditto.
  • layout-tests/editing/deleting/delete-trailing-ws-001-expected.txt: Ditto.
  • layout-tests/editing/inserting/insert-br-case1-expected.txt: Ditto.
  • layout-tests/editing/inserting/insert-br-case2-expected.txt: Ditto.
  • layout-tests/editing/style/style-3681552-fix-002-expected.txt: Ditto.
2:41 PM Changeset in webkit [7276] by trey
  • 5 edits in trunk/WebCore

Various spelling fixes.

Reviewed by Ken.

  • khtml/khtml_part.cpp: (KHTMLPart::setSelection): No misspellings in the spelling code comments
  • khtml/xml/dom_docimpl.cpp: (DocumentImpl::removeMarker): Repaint if doc changes. Sometimes the markers were not being erased when you clicked in a word.
  • khtml/xml/dom_position.cpp: (DOM::Position::previousWordBoundary): Small optimization. Bail after first try if the second try will not come out any different. (DOM::Position::nextWordBoundary): Ditto
  • kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::markMisspellingsInSelection): Comment.
1:42 PM Changeset in webkit [7275] by darin
  • 1 edit in trunk/WebCore/ChangeLog-2005-08-23

Added a bug number to the textarea bug fix check-in,
so I can look it up here in ChangeLog.

1:41 PM Changeset in webkit [7274] by darin
  • 5 edits in trunk/WebCore

Reviewed by Maciej.

  • fixed <rdar://problem/3689700> crash loading page; stoxx.com (works in IE and Firefox)
  • khtml/khtml_part.h: Make completeURL public.
  • kwq/KWQKJavaAppletWidget.mm: (KJavaAppletWidget::KJavaAppletWidget): Complete the base URL before passing it across the bridge. This sidesteps the crashing bug in CFURL, filed as '<rdar://problem/3764632> CFURLCreateAbsoluteURLWithBytes crashes if passed the string "../.."' and also is obviously correct behavior that may fix other sites too.
  • fixed <rdar://problem/3547725> Crashes at csuohio.edu, list box vs. mouse event problem (Spoof No Fix)
  • kwq/KWQListBox.mm: (QListBox::~QListBox): Nil out the pointer from the KWQTableView back to the widget by calling a new detach method. (-[KWQTableView detach]): Set the pointer to the QListBox to 0. Also set the delegate and data source to nil, so we don't need nil checks in delegate and data source methods. (-[KWQTableView mouseDown:]): Add nil check. (-[KWQTableView keyDown:]): Add nil check. (-[KWQTableView keyUp:]): Add nil check. (-[KWQTableView becomeFirstResponder]): Add nil check. (-[KWQTableView resignFirstResponder]): Add nil check. (-[KWQTableView canBecomeKeyView]): Add nil check. (-[KWQTableView tableViewSelectionDidChange:]): Add nil checks, even though this is a delegate callback, to handle cases where calls to JavaScript result in the QListBox going away partway through. (-[KWQTableView drawRow:clipRect:]): Add nil check. (-[KWQTableView _accessibilityTableCell:tableColumn:]): Add nil check.
12:55 PM Changeset in webkit [7273] by trey
  • 2 edits in trunk/WebCore

Fix ASSERT in spelling marker management.

Reviewed by John.

  • khtml/xml/dom_docimpl.cpp: (DocumentImpl::shiftMarkers): Use assert instead of ASSERT. Tweak test to allow for a start position of 0.
12:48 PM Changeset in webkit [7272] by trey
  • 5 edits in trunk

WebCore:

3764147 - failure of subframe to load leaves links in parent doc broken

Reviewed by Maciej.

  • khtml/khtml_part.cpp: (KHTMLPart::childBegin): New method to mark part as not complete. (KHTMLPart::processObjectRequest): Mark child part imcomplete, so if we later get a failure on load it won't think it's already complete and do nothing.
  • khtml/khtml_part.h:

WebKit:

3764147 - failure of subframe to load leaves links in parent doc broken

Reviewed by Maciej.

  • WebView.subproj/WebDataSource.m: (-[WebDataSource _receivedMainResourceError:complete:]): Need to call [bridge end] in the case of an error, so WC can clean up.
12:05 PM Changeset in webkit [7271] by vicki
  • 3 edits in trunk

set version number to '158u'. the tree is open

11:59 AM Changeset in webkit [7270]
  • 3 copies in tags/Safari-157

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

11:59 AM Changeset in webkit [7269] by vicki
  • 6 edits in trunk

Safari-157 stamp

Note: See TracTimeline for information about the timeline view.