⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Timeline



Jun 8, 2006:

11:14 PM Changeset in webkit [14778] by justing
  • 1 edit in trunk/WebCore/ChangeLog

Tweaked ChangeLog

9:57 PM Changeset in webkit [14777] by justing
  • 39 edits
    26 adds in trunk

LayoutTests:

Reviewed by levi


<​http://bugzilla.opendarwin.org/show_bug.cgi?id=4468>
Implement execCommand(Insert{Un}OrderedList)

Added:

  • editing/execCommand/create-list-from-range-selection-expected.checksum: Added.
  • editing/execCommand/create-list-from-range-selection-expected.png: Added.
  • editing/execCommand/create-list-from-range-selection-expected.txt: Added.
  • editing/execCommand/create-list-from-range-selection.html: Added.
  • editing/execCommand/insert-list-and-stitch-expected.checksum: Added.
  • editing/execCommand/insert-list-and-stitch-expected.png: Added.
  • editing/execCommand/insert-list-and-stitch-expected.txt: Added.
  • editing/execCommand/insert-list-and-stitch.html: Added.
  • editing/execCommand/insert-list-with-id-expected.checksum: Added.
  • editing/execCommand/insert-list-with-id-expected.png: Added.
  • editing/execCommand/insert-list-with-id-expected.txt: Added.
  • editing/execCommand/insert-list-with-id.html: Added.
  • editing/execCommand/remove-list-from-range-selection-expected.checksum: Added.
  • editing/execCommand/remove-list-from-range-selection-expected.png: Added.
  • editing/execCommand/remove-list-from-range-selection-expected.txt: Added.
  • editing/execCommand/remove-list-from-range-selection.html: Added.
  • editing/execCommand/remove-list-items-expected.checksum: Added.
  • editing/execCommand/remove-list-items-expected.png: Added.
  • editing/execCommand/remove-list-items-expected.txt: Added.
  • editing/execCommand/remove-list-items.html: Added.
  • editing/execCommand/switch-list-type-expected.checksum: Added.
  • editing/execCommand/switch-list-type-expected.png: Added.
  • editing/execCommand/switch-list-type-expected.txt: Added.
  • editing/execCommand/switch-list-type.html: Added.


Fixes:

  • editing/pasteboard/drag-drop-modifies-page-expected.txt:
  • editing/pasteboard/paste-list-001-expected.txt:
  • editing/selection/drag-to-contenteditable-iframe-expected.checksum:
  • editing/selection/drag-to-contenteditable-iframe-expected.png:
  • editing/selection/drag-to-contenteditable-iframe-expected.txt:
  • editing/style/block-style-005-expected.txt:
  • fast/text/textIteratorNilRenderer-expected.checksum:
  • fast/text/textIteratorNilRenderer-expected.png:
  • fast/text/textIteratorNilRenderer-expected.txt:
  • editing/deleting/delete-line-016-expected.checksum:
  • editing/deleting/delete-line-016-expected.png:
  • editing/deleting/delete-line-016-expected.txt:

WebCore:

Reviewed by levi


<​http://bugzilla.opendarwin.org/show_bug.cgi?id=4468>
Implement execCommand(Insert{Un}OrderedList)

  • WebCore.xcodeproj/project.pbxproj: Added InsertListCommand.{h,cpp} to the project.
  • dom/Position.cpp: (WebCore::hasRenderedNonAnonymousDescendantsWithHeight): Added. A block with height is only a candidate if this is false. This should fix the problems getting carets into empty blocks. (WebCore::Position::inRenderedContent):
  • editing/AppendNodeCommand.cpp: (WebCore::AppendNodeCommand::doApply):
  • editing/CompositeEditCommand.cpp: (WebCore::hasARenderedDescendant): (WebCore::CompositeEditCommand::prune): A node can have DOM descendants that are rendered, yet have no rendered descendants. Fixed the pruning rule to avoid removing a node like this. (WebCore::CompositeEditCommand::mergeIdenticalElements): Put the two elements next to each other if they aren't already, as a convenience. (WebCore::CompositeEditCommand::moveParagraph): Added code to preserve the current selection. (WebCore::CompositeEditCommand::moveParagraphs): Ditto.
  • editing/CompositeEditCommand.h:
  • editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::initializeStartEnd): Expansion for special elements should continue to happen until it is no longer possible. (WebCore::DeleteSelectionCommand::handleGeneralDelete): The code to adjust the start node wouldn't always avoid removing the start block.
  • editing/InsertListCommand.cpp: Added. (WebCore::InsertListCommand::fixOrphanedListChild): Puts a list item that isn't inside a list into a list. (WebCore::InsertListCommand::InsertListCommand): (WebCore::InsertListCommand::modifyRange): (WebCore::InsertListCommand::doApply):
  • editing/InsertListCommand.h: Added. (WebCore::InsertListCommand::):
  • editing/InsertNodeBeforeCommand.cpp: (WebCore::InsertNodeBeforeCommand::doApply):
  • editing/InsertParagraphSeparatorCommand.cpp:
  • editing/JSEditor.cpp:
  • editing/TextIterator.cpp: (WebCore::TextIterator::advance): Don't handle a node if the end of the range used to create the iterator ends at the start of that node. used to create the iterator.
  • editing/VisiblePosition.cpp: (WebCore::VisiblePosition::next): (WebCore::VisiblePosition::previous):
  • editing/VisiblePosition.h: Added a parameter to next/previous to prevent them from going outside the current editable region.
  • editing/htmlediting.cpp: (WebCore::highestAncestor): (WebCore::enclosingList): (WebCore::enclosingListChild): (WebCore::outermostEnclosingList): (WebCore::createListItemElement):
  • editing/htmlediting.h:
  • page/Frame.cpp: (WebCore::Frame::selectionListState): Added.
  • page/Frame.h:

WebKit:

Reviewed by levi


<​http://bugzilla.opendarwin.org/show_bug.cgi?id=4468>
Implement execCommand(Insert{Un}OrderedList)

  • WebView/WebFrame.m: (-[WebFrame _findFrameWithSelection]): Removed an assertion that we only have one frame with a selection.
  • WebView/WebView.m: (-[WebView selectedFrame]): Ditto.
6:06 PM Changeset in webkit [14776] by thatcher
  • 2 edits
    2 adds in trunk/WebKitSite

Reviewed by Darin.

Include our own copies of the BSD and LGPL licenses.

  • coding/bsd-license.html: Added.
  • coding/lgpl-license.html: Added.
  • nav.inc: link to our copies of the licenses
3:10 PM Changeset in webkit [14775] by harrison
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by Geoff and John.

<rdar://problem/4558879> -[DOMCSSPrimitiveValue setStringValue:] throws an exception (12)

  • fast/dom/setPrimitiveValue-expected.checksum: Added.
  • fast/dom/setPrimitiveValue-expected.png: Added.
  • fast/dom/setPrimitiveValue-expected.txt: Added.
  • fast/dom/setPrimitiveValue.html: Added.

WebCore:

Reviewed by Geoff and John.

<rdar://problem/4558879> -[DOMCSSPrimitiveValue setStringValue:] throws an exception (12)


Problem was the primitive value was getting reset to a generic state before the parameter
check that relies on that state.

Test: fast/dom/setPrimitiveValue.html

  • css/CSSPrimitiveValue.cpp: (WebCore::CSSPrimitiveValue::setFloatValue): (WebCore::CSSPrimitiveValue::setStringValue): Move parameter checks ahead of call to cleanup().
2:38 PM Changeset in webkit [14774] by aliceli1
  • 4 adds in trunk/LayoutTests/fast/dom/Range

added tests for Range.isPointInRange and Range.comparePoint

2:35 PM Changeset in webkit [14773] by aliceli1
  • 4 edits in trunk/WebCore

Reviewed by Justin.

  • fixed <rdar://problem/4578115> support Range.isPointInRange
  • fixed <rdar://problem/4578123> support Range.comparePoint
  • dom/Range.cpp: (WebCore::Range::isPointInRange): added implementation (WebCore::Range::comparePoint): added implementation
  • dom/Range.h: added prototypes
  • dom/Range.idl: added bindings
2:20 PM Changeset in webkit [14772] by thatcher
  • 1 edit in trunk/WebKit/WebView/WebView.m

Build fix.

2:05 PM Changeset in webkit [14771] by thatcher
  • 6 edits in trunk/WebKit

Reviewed by Darin and John.

<rdar://problem/3600734> API: please add a way to turn vertical scrollbar
always on (for Mail, to avoid reflow when typing)

Adds new methods to lock the scrolling mode on WebDynamicScrollBarsView.
Locking the scroll mode prevents WebCore from changing it as needed.
Also adds an SPI on WebView that will lock the "always on" mode
for each scroller.

  • WebKit.xcodeproj/project.pbxproj:
  • WebView/WebDynamicScrollBarsView.h:
  • WebView/WebDynamicScrollBarsView.m: (-[WebDynamicScrollBarsView setAllowsScrolling:]): (-[WebDynamicScrollBarsView allowsScrolling]): (-[WebDynamicScrollBarsView setAllowsHorizontalScrolling:]): (-[WebDynamicScrollBarsView setAllowsVerticalScrolling:]): (-[WebDynamicScrollBarsView setHorizontalScrollingMode:]): (-[WebDynamicScrollBarsView setVerticalScrollingMode:]): (-[WebDynamicScrollBarsView setScrollingMode:]): (-[WebDynamicScrollBarsView setHorizontalScrollingModeLocked:]): (-[WebDynamicScrollBarsView setVerticalScrollingModeLocked:]): (-[WebDynamicScrollBarsView setScrollingModesLocked:]): (-[WebDynamicScrollBarsView horizontalScrollingModeLocked]): (-[WebDynamicScrollBarsView verticalScrollingModeLocked]):
  • WebView/WebView.m: (-[WebView setAlwaysShowVerticalScroller:]): (-[WebView alwaysShowVerticalScroller]): (-[WebView setAlwaysShowHorizontalScroller:]): (-[WebView alwaysShowHorizontalScroller]):
  • WebView/WebViewPrivate.h:
9:43 AM Changeset in webkit [14770] by darin
  • 6 edits in trunk

WebCore:

Reviewed by Justin.

  • bridge/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge getInnerNonSharedNode:innerNode:URLElement:atPoint:allowShadowContent:]): Changed to call nodeInfoAtPoint directly. (-[WebCoreFrameBridge _visiblePositionForPoint:]): Changed to call nodeInfoAtPoint directly. Also added code to convert coordinates so it works for points that are in nodes in subframes.
  • page/Frame.cpp: (WebCore::Frame::setMark): Added assertions to catch if we attempt to set selection endpoints in another document. (WebCore::Frame::setSelection): Ditto.

WebKit:

Reviewed by Justin.

  • WebView/WebHTMLView.m: (-[WebHTMLView _topHTMLView]): Added. (-[WebHTMLView _isTopHTMLView]): Added. (-[WebHTMLView _insideAnotherHTMLView]): Changed to use _topHTMLView. (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]): Forward to the top HTML view, so that only the top view ever starts a dragging operation. Change dragging code to not assume that the dragged node is in the current view's document. Added checks that the node is an element in a couple places and coordinate conversions. (-[WebHTMLView _mayStartDragAtEventLocation:]): Forward to the top HTML view. (-[NSArray addMouseMovedObserver]): Change to do nothing when the dataSource field is 0, since we now use the dataSource field to get to the WebView. (-[NSArray removeMouseMovedObserver]): Added a comment. (-[NSArray dragImage:at:offset:event:pasteboard:source:slideBack:]): Forward to the top HTML view. (-[NSArray draggingSourceOperationMaskForLocal:]): Assert that it's the top HTML view. (-[NSArray draggedImage:movedTo:]): Ditto. (-[NSArray draggedImage:endedAt:operation:]): Ditto. (-[NSArray namesOfPromisedFilesDroppedAtDestination:]): Ditto. (-[NSArray _canProcessDragWithDraggingInfo:]): Ditto. (-[NSArray _isMoveDrag]): Ditto. (-[NSArray draggingUpdatedWithDraggingInfo:actionMask:]): Ditto. (-[NSArray draggingCancelledWithDraggingInfo:]): Ditto. (-[NSArray concludeDragForDraggingInfo:actionMask:]): Ditto. Also added code to work with the appropriate bridge when receiving a drag. This fixes the problem where the top level frame got a selection intended for the inner frame; the source of the bug. (-[NSArray elementAtPoint:allowShadowContent:]): Added code to convert the coordinates so this works properly when returning an element from an inner frame. (-[NSArray setDataSource:]): Added a call to addMouseMovedObserver, needed now that addMouseMovedObserver won't do anything if called when dataSource is nil. (-[WebHTMLView _delegateDragSourceActionMask]): Forward to the top HTML view.
  • WebView/WebView.m: (-[WebViewPrivate dealloc]): Removed code to release dragCaretBridge since that field is now gone. (-[WebView moveDragCaretToPoint:]): Always call the main frame's bridge, since the drag caret is now a page-level item. Later we'll move it to the page bridge. (-[WebView removeDragCaret]): Ditto.
9:36 AM Changeset in webkit [14769] by darin
  • 1 edit in trunk/LayoutTests/ChangeLog

Added change log entry that I apparently forgot.

Jun 7, 2006:

9:04 PM Changeset in webkit [14768] by justing
  • 17 edits in trunk

LayoutTests:

Reviewed by levi

  • editing/deleting/delete-4038408-fix-expected.txt:
  • editing/input/attributed-substring-from-range-lines-expected.txt:
  • editing/inserting/insert-3654864-fix-expected.txt:
  • editing/inserting/insert-3659587-fix-expected.txt:
  • editing/inserting/insert-3775316-fix-expected.txt:
  • editing/inserting/insert-at-end-01-expected.txt:
  • editing/inserting/insert-at-end-02-expected.txt:
  • editing/inserting/insert-br-001-expected.txt:
  • editing/inserting/insert-br-005-expected.txt:

WebCore:

Reviewed by levi


Fixed some comments and removed an unused variable.

  • editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::removeBlockPlaceholder): Don't remove a br if it isn't at the start of a block, since it isn't really a "block placeholder".
  • editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::initializePositionData): (WebCore::DeleteSelectionCommand::mergeParagraphs): (WebCore::DeleteSelectionCommand::doApply):
  • editing/InsertLineBreakCommand.cpp: (WebCore::InsertLineBreakCommand::doApply):
  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::doApply): (WebCore::ReplaceSelectionCommand::completeHTMLReplacement):
  • editing/htmlediting.cpp:
6:43 PM Changeset in webkit [14767] by hyatt
  • 5 edits in trunk/WebKit

Add support for custom highlighting to WebKit.

Reviewed by justin

  • WebCoreSupport/WebFrameBridge.m: (-[WebFrameBridge paintCustomHighlight:forBox:onLine:behindText:]):
  • WebView/WebHTMLView.m: (-[WebHTMLView _highlighterForType:]):
  • WebView/WebHTMLViewInternal.h:
  • WebView/WebHTMLViewPrivate.h:
6:41 PM Changeset in webkit [14766] by hyatt
  • 8 edits in trunk/WebCore

Add support for custom highlighting. This is all ifdefed to be Mac-only.

Reviewed by justin

  • WebCore.xcodeproj/project.pbxproj:
  • bridge/mac/FrameMac.h:
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::paintCustomHighlight):
  • bridge/mac/WebCoreFrameBridge.h:
  • platform/mac/ClipboardMac.h:
  • rendering/InlineTextBox.cpp: (WebCore::InlineTextBox::paint): (WebCore::InlineTextBox::paintCustomHighlight):
  • rendering/InlineTextBox.h:
6:25 PM Changeset in webkit [14765] by adele
  • 17 edits
    2 adds in trunk

WebCore:

Reviewed by Hyatt

Added support for cross-platform resize property.

  • platform/PlatformMouseEvent.h: Removed isMouseButtonDown, since we already cache this info when we handle mouse down and mouse up.
  • platform/mac/MouseEventMac.mm: ditto.
  • platform/win/TemporaryLinkStubs.cpp: ditto.
  • page/Frame.cpp: (WebCore::Frame::autoscrollTimerFired): Use d->m_bMousePressed instead of isMouseButtonDown.
  • page/FrameView.cpp: (WebCore::FrameView::handleMouseMoveEvent): Passes event to layer resize method. (WebCore::FrameView::mousePressed): Added.
  • page/FrameView.h: Added mousePressed.
  • rendering/RenderTheme.h: Removed paintResizeControl. Since its not dependent on the theme, the layer can just paint this.
  • rendering/RenderTheme.cpp: ditto.
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::RenderLayer): Initializes m_resizeCornerImage. (WebCore::RenderLayer::~RenderLayer): Deletes m_resizeCornerImage. (WebCore::RenderLayer::resize): Uses passed in event instead of creating a new one. (WebCore::RenderLayer::paintResizeControl): Now loads and paints the resize image.
  • rendering/RenderLayer.h: Added m_resizeCornerImage. Important to have one per-layer for when we allow more customization.
  • platform/cairo/ImageCairo.cpp: (WebCore::Image::loadResource): Loads image resources.

WebKit:

Reviewed by Hyatt.

Added resources for missingImage and textAreaResizeCorner.

  • COM/WebKitDLL.cpp: (loadResourceIntoArray): Added. Returns a DeprecatedByteArray with the resource's data.
  • WebKit.vcproj/WebKit.rc: Added missing image and resize pngs as resources.
  • WebKit.vcproj/WebKit.vcproj: Added pngs.
  • WebKit.vcproj/missingImage.png: Added.
  • WebKit.vcproj/resource.h: Added entries for pngs.
  • WebKit.vcproj/textAreaResizeCorner.png: Added.
5:18 PM Changeset in webkit [14764] by hyatt
  • 4 edits in trunk/WebKit

Add SPI for setting and removing custom highlighters.

Reviewed by Tim H

  • WebView/WebHTMLView.m: (-[WebTextCompleteController dealloc]): (-[WebHTMLView _setHighlighter:ofType:]): (-[WebHTMLView _removeHighlighterOfType:]):
  • WebView/WebHTMLViewInternal.h:
  • WebView/WebHTMLViewPrivate.h:
4:08 PM Changeset in webkit [14763] by hyatt
  • 7 edits in trunk/WebCore

Add support for a new property for custom highlighting. This patch just
adds support for the parsing of the property. It doesn't do anything yet.

Reviewed by beth

  • css/CSSComputedStyleDeclaration.cpp: (WebCore::): (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
  • css/CSSPropertyNames.in:
  • css/cssparser.cpp: (WebCore::CSSParser::parseValue):
  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::applyProperty):
  • rendering/render_style.cpp: (WebCore::StyleCSS3InheritedData::StyleCSS3InheritedData): (WebCore::StyleCSS3InheritedData::operator==): (WebCore::RenderStyle::diff):
  • rendering/render_style.h: (WebCore::RenderStyle::highlight): (WebCore::RenderStyle::setHighlight): (WebCore::RenderStyle::initialHighlight):
3:59 PM Changeset in webkit [14762] by ggaren
  • 1 edit
    1 delete in trunk/JavaScriptCore

Removed API directory I prematurely/accidentally added.

  • API: Removed.
9:33 AM Changeset in webkit [14761] by darin
  • 2 edits
    4 adds in trunk

Fix suggested by Rob Buis. Reviewed by Darin.

Test: fast/css/font-shorthand-weight-only.html

  • css/cssparser.cpp: (WebCore::CSSParser::parseFont): Revert part of the change made to fix bug 5564.
9:29 AM Changeset in webkit [14760] by darin
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by Hyatt, tweaked by Mitz.

  • fast/table/text-field-baseline-expected.checksum: Added.
  • fast/table/text-field-baseline-expected.png: Added.
  • fast/table/text-field-baseline-expected.txt: Added.
  • fast/table/text-field-baseline.html: Added.

WebCore:

Reviewed by Hyatt.

Test: fast/table/text-field-baseline.html

  • rendering/RenderTableCell.cpp: (WebCore::RenderTableCell::baselinePosition): Do not descend into replaced objects, just use their baseline.
9:26 AM Changeset in webkit [14759] by darin
  • 5 edits
    4 adds in trunk

LayoutTests:

Reviewed by Hyatt.

  • fast/block/float/nopaint-after-layer-destruction-expected.checksum: Added.
  • fast/block/float/nopaint-after-layer-destruction-expected.png: Added.
  • fast/block/float/nopaint-after-layer-destruction-expected.txt: Added.
  • fast/block/float/nopaint-after-layer-destruction.html: Added.

WebCore:

Reviewed by Hyatt (concept) and Darin (some coding details).

Pixel test: fast/block/float/nopaint-after-layer-destruction.html

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::setPaintsFloatingObject): Added. Changes the noPaint flag in the block's FloatingObject for the given float and calls setChildNeedsLayout.
  • rendering/RenderBlock.h:
  • rendering/RenderBox.cpp: (WebCore::blockThatPaintsFloat): Added this helper function. (WebCore::RenderBox::setStyle): Added. Calls to setPaintsFloatingObject when a float gains or loses its layer.
Note: See TracTimeline for information about the timeline view.