Timeline

Jan 21, 2005:

4:48 PM Changeset in webkit [8433] by rjw

• 2 edits in trunk/WebCore

Fixed <rdar://problem/3759399> Javascript / Liveconnect problems ((event handler):Undefined value)

Java applets specified with <object> or <embed> weren't scriptable. Now they are.

Reviewed by Chris.

• khtml/html/html_objectimpl.cpp: (HTMLEmbedElementImpl::getEmbedInstance): (HTMLObjectElementImpl::getObjectInstance):

4:16 PM Changeset in webkit [8432] by kocienda

• 2 edits
  2 adds in trunk

Reviewed by John

Fix for this bug:

<rdar://problem/3959464> REGRESSION (Mail): Insertion point goes back to beginning of document after deleting

• khtml/editing/htmlediting.cpp: (khtml::DeleteSelectionCommand::handleGeneralDelete): Add special case to handle retaining a fully-selected block. This fixes the bug.
• layout-tests/editing/deleting/delete-3959464-fix-expected.txt: Added.
• layout-tests/editing/deleting/delete-3959464-fix.html: Added.

3:01 PM Changeset in webkit [8431]

• 4 copies in tags/Safari-180

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

3:01 PM Changeset in webkit [8430] by rjw

• 2 edits in trunk/WebCore

Fixed <rdar://problem/3966998> REGRESSION(179-TOT) clicking on gmail message brings me to blank screen

getElementById() sometimes returns Undefined() instead of Null().

Reviewed by Hyatt.

• khtml/ecma/kjs_dom.cpp: (DOMDocumentProtoFunc::tryCall):

2:45 PM Changeset in webkit [8429] by hyatt

• 2 edits in trunk/WebCore

Fix for 3773809, make sure that overflow regions never end up at an invalid scroll offset because of a layout change.

Reviewed by kocienda

• khtml/rendering/render_layer.cpp: (RenderLayer::updateScrollInfoAfterLayout):

2:43 PM Changeset in webkit [8428] by darin

• 3 edits in trunk/WebKit

Reviewed by Kristin Forster.

• fixed <rdar://problem/3964972> update _initWithCGSEvent:eventRef: call in mouse moved workaround (breaks cursors in Carbon WebKit applications)

• Carbon.subproj/HIWebView.m: (MouseMoved): Instead of munging the event record's window number directly before calling _initWithCGSEvent, on Tiger we call _eventRelativeToWindow on the event after creating it. Also added a check so that with Macromedia Contribute's workaround in place we don't do anything at all to the event. Tested with both Contribute and CarbonWeb.

• English.lproj/StringsNotToBeLocalized.txt: Update for recent changes.

1:45 PM Changeset in webkit [8427] by hyatt

• 2 edits in trunk/WebCore

Fix for 3966349, hang loading page. Make sure that loops using popOneBlock properly check for a null
blockStack to avoid an infinite loop.

Reviewed by kocienda

• khtml/html/htmlparser.cpp: (KHTMLParser::parseToken): (KHTMLParser::insertNode): (KHTMLParser::popInlineBlocks):

10:55 AM Changeset in webkit [8426] by kocienda

• 2 edits in trunk/WebCore

Reviewed by John

Fix for this bug:

<rdar://problem/3957204> Mail crashed when sending a message in ~InsertParagraphSeparatorInQuotedContentCommand

• khtml/editing/htmlediting.cpp: (khtml::InsertParagraphSeparatorInQuotedContentCommand::InsertParagraphSeparatorInQuotedContentCommand): InsertParagraphSeparatorInQuotedContentCommand has this member variable: DOM::ElementImpl *m_breakNode; I failed to initialize this variable to null in the constructor, and there is an uncommonly-traveled code path which does not set this variable to something good. In the destructor, we check for null, and deref if non-null. Obvious "BOOM" potential. Fixed.

9:29 AM Changeset in webkit [8425] by kocienda

• 2 edits
  2 adds in trunk

Reviewed by John

Fix for this bug:

<rdar://problem/3966311> REGRESSION (Mail): Hitting return makes space character disappear

• khtml/editing/htmlediting.cpp: (khtml::InsertParagraphSeparatorCommand::doApply): When writing the code to insert a block in response to the return key, I did not write code to cover the case described in the bug. Now I have.
• layout-tests/editing/inserting/insert-div-025-expected.txt: Added.
• layout-tests/editing/inserting/insert-div-025.html: Added.

Jan 20, 2005:

6:25 PM Changeset in webkit [8424] by rjw

• 2 edits in trunk/WebKit

Fixed <rdar://problem/3815672> REGRESSION (Mail): Japanese text cannot be made bold

The AppKit's font substitution API doesn't match font traits!
It only find fonts that contain the appropriate glyphs. This
patch attempts to find the best variation within a family.

Also, made unrelated change to return printer/screen version of
substituted font.

Reviewed by Maciej.

• WebCoreSupport.subproj/WebTextRenderer.m: (-[WebTextRenderer _substituteFontForString:families:]):

5:50 PM Changeset in webkit [8423] by mjs

• 2 edits in trunk/WebCore

Reviewed by Ken.

<rdar://problem/3965196> security fix for javascript: exploit missed one case (already fixed in updates)

• khtml/ecma/kjs_window.cpp: (WindowFunc::tryCall): correct mistake in earlier fix for the following bug, caught by Adele:

<rdar://problem/3790449> REGRESSION (Mail): underline behavior is flaky because of how CSS handles text-decoration

5:36 PM Changeset in webkit [8422] by mjs

• 4 edits in trunk

Reviewed by Ken.

<rdar://problem/3965466> editing needs to insert text before applying typing style

• khtml/editing/htmlediting.cpp: (khtml::ApplyStyleCommand::addInlineStyleIfNeeded): Remove a FIXME comment. (khtml::InsertLineBreakCommand::doApply): Apply style to the BR node after inserting it. (khtml::InsertTextCommand::prepareForTextInsertion): Don't try to apply style to any new nodes created before inserting. (khtml::InsertTextCommand::input): Apply style to the inserted range after doing the text insert.
• khtml/editing/htmlediting.h:

This change leads to improved results on one of the layout tests:

• layout-tests/editing/inserting/insert-div-024-expected.txt:

5:22 PM Changeset in webkit [8421] by darin

• 3 edits in trunk/JavaScriptCore

Reviewed by me, changes by Han Ming Ong.

• <rdar://problem/3964302> SWB: A few files need to be updated to be compilable under GCC 4.0

• bindings/objc/WebScriptObjectPrivate.h: Make members public.
• kjs/lookup.h: Change "value.h" to "object.h" because we need KJS::Object to compile a template.

4:58 PM Changeset in webkit [8420] by vicki

• 1 edit in trunk/WebKit/ChangeLog

move the Safari-180 release-marker in the ChangeLog, since the fix for 3786659 will be included in Safari-180

4:50 PM Changeset in webkit [8419] by kocienda

• 2 edits in trunk/WebKit

Reviewed by John

Fix for this bug:

<rdar://problem/3786659> REGRESSION (Mail): editable WebViews don't work with "size up" and "size down" NSFontManager changes

• WebView.subproj/WebHTMLView.m: (-[WebHTMLView _addToStyle:fontA:fontB:]): This is the WebKit side of the fix. Replaced unimplemented code blocks with FIXME's in them for make bigger and make smaller with real working code.

4:41 PM Changeset in webkit [8418] by rjw

• 4 edits in trunk/JavaScriptCore

Fixed <rdar://problem/3964634> undefined property value from binding seems to evaluate to true in an if statement

The comprehensive fix for this problem requires new API, as described in 3965326. However,
given that we can't add new API at this point, the 'ObjcFallbackObjectImp' will behave
like and Undefined object if invokeUndefinedMethodFromWebScript:withArguments: isn't
implemented on the bound object.

Reviewed by Chris.

• bindings/objc/objc_runtime.h:
• bindings/objc/objc_runtime.mm: (ObjcFallbackObjectImp::type): (ObjcFallbackObjectImp::implementsCall): (ObjcFallbackObjectImp::toBoolean):
• bindings/testbindings.mm: (+[MyFirstInterface isSelectorExcludedFromWebScript:]): (+[MyFirstInterface isKeyExcludedFromWebScript:]):

4:01 PM Changeset in webkit [8417] by kocienda

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

Finished my thought in the changelog. I left a comment half-written earlier.

3:47 PM Changeset in webkit [8416] by vicki

• 3 edits in trunk

Versioning for TOT, Safari 2.0 (180+).

The tree is open!

3:33 PM Changeset in webkit [8415] by vicki

• 6 edits in trunk

Safari-180 stamp

2:02 PM Changeset in webkit [8414] by kocienda

• 2 edits in trunk/WebCore

Reviewed by John

Fix for this bug:

<rdar://problem/3964646> REGRESSION (179-180): Typing space at end of line makes following paragraph disappear

• khtml/editing/htmlediting.cpp: (khtml::CompositeEditCommand::removeBlockPlaceholderIfNeeded): Recent change to block placeholder removal code caused this regression. The code became too aggressive in removing block placeholders, and would remove them from blocks other than the block containing the selection.
• layout-tests/editing/inserting/insert-div-023-expected.txt: This file had a spur
• layout-tests/editing/inserting/insert-div-024-expected.txt:

12:36 PM Changeset in webkit [8413] by hyatt

• 2 edits in trunk/WebCore

Fix for oddness on albertsons.com. Make sure not to crash when setting/removing style properties on a node
with no document. The bug # is 3813900.

Reviewed by john

11:34 AM Changeset in webkit [8412] by harrison

• 2 edits in trunk/WebCore

Reviewed by Darin.

PARTIAL fix for following bug. Create attributed string with fonts and links. Still need to add attachments.
<rdar://problem/3942606> AX: Support kAXAttributedStringForTextMarkerRangeParameterizedAttribute

• kwq/KWQAccObject.mm: (-[KWQAccObject textUnderElement]): Touched up previous checkin to match code review comments.

(-[KWQAccObject value]):
Use plainText for this one instead of obsolete attributedString.

(-[KWQAccObject accessibilityParameterizedAttributeNames]):
Advertise AXAttributedStringForTextMarkerRange.

(AXAttributeStringAddFont):
(AXAttributeStringAddElement):
(-[KWQAccObject linkUIElementForNode:]):
(-[KWQAccObject _addAttachmentForElement:URL:needsParagraph:usePlaceholder:]):
(-[KWQAccObject accessibilityAttributedStringForRange:]):
New routines to support AXAttributedStringForTextMarkerRange.

(-[KWQAccObject doAXAttributedStringForTextMarkerRange:]):
Use new routines instead of obsolete attributedString.

11:01 AM Changeset in webkit [8411] by harrison

• 3 edits in trunk/WebCore

Reviewed by Darin.

<rdar://problem/3960196> AX Crash in DOM::Range::setStartBefore

• khtml/xml/dom_docimpl.cpp: (DocumentImpl::getAccObjectCache): Adjust when detecting cache in non-top level WebArea.
• kwq/KWQAccObject.mm: (-[KWQAccObject textUnderElement]): (-[KWQAccObject accessibilityAttributeValue:]): Make sure the elements document is the current one for the part.

10:05 AM Changeset in webkit [8410] by darin

• 6 edits in trunk/WebCore

Reviewed by John.

• fixed <rdar://problem/3922980> Mail not crashing, just quitting itself suddenly in -[WebHTMLView(MailExtras) findString:options:]

• khtml/dom/dom2_range.cpp: (DOM::operator==): Handle null and detached ranges without raising exceptions. The uncaught exception would make the entire program terminate.

• fixed <rdar://problem/3885729> attempting to modify a computed style does nothing, but should raise a DOM exception

• kwq/DOM-CSS.mm: (-[DOMCSSStyleDeclaration setProperty:::]): Uncommented this code which I had to disable back in November because Mail was stumbling over it.

• fixed <rdar://problem/3943049> focus() called during onload handler results in square text field on <input type=search>

• kwq/KWQLineEdit.mm: (QLineEdit::baselinePosition): Change computation so it will work even for fields that position their text in a way that depends on vertical size. In the case of a search field, the text is centered, so the old logic was broken.
• kwq/KWQTextField.mm: (-[KWQTextFieldController initWithTextField:QLineEdit:]): Set the frame size to something large enough to accomodate the field editor. If we start the frame at size 0,0 we run into AppKit trouble when it insets the frame to figure out the frame for the field editor. In the case of this bug this happens because we become first responder before being sized and positioned by the HTML layout code.

• improved debugging output when using "po" from gdb with Objective-C DOM

• kwq/DOM.mm: (-[DOMNode description]): Added. Includes node name ("<tr>") and node value (e.g., string for text node). (-[DOMRange description]): Tweaked format.