Timeline

Sep 18, 2004:

6:12 PM Changeset in webkit [7596] by darin

• 3 edits in trunk/WebCore

Reviewed by Maciej.

• fixed <rdar://problem/3805627> -[DOMRange _text] sometimes includes all text to the end of the document for certain ranges

• khtml/misc/khtml_text_operations.cpp: (khtml::TextIterator::advance): Correctly handle the case when we are already on m_endNode. The concept here is that m_endNode is a node you must never "leave" when iterating.

Sep 17, 2004:

12:22 PM Changeset in webkit [7595] by vicki

• 3 edits in trunk

change version number to 2.0, 165u for TOT

12:06 PM Changeset in webkit [7594]

• 3 copies in tags/Safari-164

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

12:06 PM Changeset in webkit [7593] by vicki

• 6 edits in trunk

Safari-164 stamp for everything except WebBrowser. In these projects, CFBundleShortVersionString matches CFBundleVersion (164 for both).

11:10 AM Changeset in webkit [7592] by cblu

• 2 edits in trunk/WebKit

Fixed:

<rdar://problem/3805757> don't unnecessarily put RTFD on the pasteboard
<rdar://problem/3805756> strip attachments before generating RTF

Reviewed by john.

• WebView.subproj/WebHTMLView.m: (-[WebHTMLView writeSelectionWithPasteboardTypes:toPasteboard:]): only put RTFD on the pasteboard if it has attachments, strip attachments when writing RTF

9:49 AM Changeset in webkit [7591] by kocienda

• 2 edits in trunk

Reviewed by me

• layout-tests/editing/selection/unrendered-004-expected.txt: Updated expected results.

9:44 AM Changeset in webkit [7590] by kocienda

• 2 edits in trunk/WebCore

Reviewed by Darin

Fix for this bug:

<rdar://problem/3780245> REGRESSION (Mail): some lines are skipped when doing arrow navigation

• khtml/xml/dom_position.cpp: (DOM::Position::previousLinePosition): One-line fix. Make sure that BRs at the end of blocks are not skipped.

9:42 AM Changeset in webkit [7589] by kocienda

• 4 edits in trunk/WebCore

Reviewed by Darin

Fix for this bug:

<rdar://problem/3805594> REGRESSION (Mail): Second return key stroke does not work when first was in blockquote

• khtml/xml/dom_selection.cpp: (DOM::Selection::toRange): Code to convert caret positions moved the position upstream before making a range-compliant position, but erroneously would allow the position to cross blocks. Now it will no longer do so.

Sep 16, 2004:

5:58 PM Changeset in webkit [7588] by rjw

• 9 edits in trunk/WebCore

Part 1 of the feature requested in
<rdar://problem/3752791> Dashboard: Need a better solution for control regions

This patch implements the CSS parsing necessary for dashboard regions.
Here's an example of the syntax we support:

<style>
#aDiv {

-apple-dashboard-region:

dashboard-region-circle(control 0 0 80 0)
dashboard-region-rectangle(control,20,0,20,0)
dashboard-region-circle(control 80 0 0 0);

}
</style>

Part 2 will determine the appropriate regions.

Reviewed by Chris.

• khtml/css/css_valueimpl.cpp: (CSSPrimitiveValueImpl::CSSPrimitiveValueImpl):
• khtml/css/css_valueimpl.h: (DOM::CSSPrimitiveValueImpl::): (DOM::DashboardRegionImpl::DashboardRegionImpl): (DOM::DashboardRegionImpl::~DashboardRegionImpl): (DOM::DashboardRegionImpl::setNext): (DOM::DashboardRegionImpl::setLabel):
• khtml/css/cssparser.cpp: (CSSParser::parseValue): (CSSParser::parseContent): (skipCommaInDashboardRegion): (CSSParser::parseDashboardRegions):
• khtml/css/cssparser.h:
• khtml/css/cssproperties.c: (hash_prop): (findProp):
• khtml/css/cssproperties.h:
• khtml/css/cssproperties.in:
• khtml/dom/css_value.h: (DOM::CSSPrimitiveValue::):

4:45 PM Changeset in webkit [7587] by darin

• 4 edits in trunk/WebCore

• fixed caret-drawing regression from my last patch

• khtml/xml/dom_selection.cpp: (DOM::Selection::xPosForVerticalArrowNavigation): Pass false, meaning "vertical line caret" rather than true meaning "big box for overtyping". (DOM::Selection::layoutCaret): Ditto.

4:09 PM Changeset in webkit [7586] by darin

• 2 edits in trunk/WebKit

Reviewed by John.

• fixed <rdar://problem/3804648> 8A262: Safari crashed in -[WebView(WebPrivate) _editingDelegateForwarder] inside QuickTime Cocoa Plug-in during WebView deallocation

• WebView.subproj/WebView.m: (-[WebView _editingDelegateForwarder]): Check _private for nil before dereferencing it.

3:24 PM Changeset in webkit [7585] by darin

• 18 edits in trunk/WebCore

Reviewed by Ken.

• fixed <rdar://problem/3803280> crash in selectAll on page with no contents

• khtml/khtml_part.cpp: (KHTMLPart::selectAll): Handle case of 0 for documentElement().
• khtml/xml/dom_position.cpp: (DOM::Position::previousWordPosition): Ditto. (DOM::Position::nextWordPosition): Ditto.

• cleaned up caret code
• changed DOM::Selection to use CaretPosition more

• khtml/rendering/render_box.cpp: (RenderBox::caretRect): Change to use empty rectangles instead of an X value of -1 to mean no rectangle.
• khtml/rendering/render_image.cpp: (RenderImage::selectionRect): Take advantage of the fixed QRect constructor.
• khtml/rendering/render_object.cpp: (RenderObject::caretRect): Change to use empty rectangle instead of an X value of -1 to mean no rectangle.
• khtml/rendering/render_text.cpp: (RenderText::caretRect): Cleaned up, and changed to use an empty rectangle instead of an X value of -1 to mean no rectangle.

• khtml/xml/dom_caretposition.h: Made conversion from Position to CaretPosition something you can do implicitly, since it's an unambiguous conversion. Conversion in the other direction needs to be done explicitly. Moved EIncludeLineBreak here, and added startParagraphBoundary and endParagraphBoundary functions.
• khtml/xml/dom_caretposition.cpp: (DOM::startParagraphBoundary): Added. Replaces, and made from, member function of DOM::Position. (DOM::endParagraphBoundary): Ditto.

• khtml/xml/dom_position.h: Removed EIncludeLineBreak and startParagraphBoundary and endParagraphBoundary.
• khtml/xml/dom_position.cpp: Removed startParagraphBoundary and endParagraphBoundary.

• khtml/xml/dom_selection.h: Use CaretPosition instead of Position in a few places. Use a QRect for the caret rect. Change nodeIsBeforeNode to be a static member function.
• khtml/xml/dom_selection.cpp: (DOM::Selection::Selection): Change caret to store QRect instead of 3 separate fields. (DOM::Selection::init): Ditto. (DOM::Selection::operator=): Ditto. (DOM::Selection::modifyExtendingRightForward): Change to use CaretPosition instead of Position. Check for 0. (DOM::Selection::modifyMovingRightForward): Ditto. Check for 0. (DOM::Selection::modifyExtendingLeftBackward): Ditto. (DOM::Selection::modifyMovingLeftBackward): Ditto. (DOM::Selection::modify): Ditto. (DOM::Selection::layoutCaret): Change to use a QRect for the caret rect, and use an empty one to mean no rectangle. (DOM::Selection::getRepaintRect): Ditto. (DOM::Selection::paintCaret): Ditto. (DOM::Selection::validate): Change to use CaretPosition instead of Position. Also fixed PARAGRAPH_BOUNDARY, which was broken and doing the same thing as PARAGRAPH before, but it's not really used so that didn't matter. Check for 0. (DOM::Selection::nodeIsBeforeNode): Tightened up a bit and added some FIXMEs. This function has a number of problems and should probably be discarded in favor of the DOMStringImpl method that does the same thing.

2:16 PM Changeset in webkit [7584] by kocienda

• 2 edits in trunk/WebCore

Reviewed by Hyatt

Fix for this bug:

<rdar://problem/3787168> REGRESSION (Mail): Deleting text from the beginning of a quoted range removes the quoting from the rest

• khtml/editing/htmlediting_impl.cpp: (khtml::DeleteSelectionCommandImpl::moveNodesAfterNode): Only move the text nodes (and their siblings) when doing this move between blocks. This serves to mimic NSText behavior very well.

11:38 AM Changeset in webkit [7583] by cblu

• 3 edits in trunk/WebKit

Fixed: <rdar://problem/3779150> REGRESSION: images not copied when copying HTML in Safari and pasting into TextEdit

Reviewed by john.

• WebView.subproj/WebHTMLView.m:

(-[WebHTMLView writeSelectionWithPasteboardTypes:toPasteboard:]): use …

for RTFD

• WebView.subproj/WebHTMLViewPrivate.h: