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

Timeline



Oct 31, 2006:

11:53 PM Changeset in webkit [17515] by bdash
  • 2 edits in trunk/WebCore

2006-10-31 Mitz Pettel <​mitz@webkit.org>

Reviewed by Maciej.

  • fix image dragging

This is covered by editing/selection/drag-to-contenteditable-iframe.html

  • rendering/HitTestResult.cpp: (WebCore::HitTestResult::altDisplayString): Changed imageTag to imgTag, for HTML IMG elements. (WebCore::HitTestResult::absoluteImageURL): Ditto.
11:15 PM Changeset in webkit [17514] by bdash
  • 2 edits in trunk/WebKit

2006-10-31 Mark Rowe <​bdash@webkit.org>

Reviewed by Maciej.

Fix null pointer dereference while running editing/pasteboard/drag-drop-modifies-page.html

  • Misc/WebElementDictionary.m: (-[WebElementDictionary _image]): Add null check.
10:13 PM Changeset in webkit [17513] by bdash
  • 2 edits in trunk/WebKitSite

2006-10-31 Mark Rowe <​bdash@webkit.org>

Reviewed by Mitz.

Update WebKit blog link to prevent pointless redirects.

  • nav.inc:
9:45 PM Changeset in webkit [17512] by bdash
  • 3 edits in trunk/WebKitTools

2006-11-01 Mark Rowe <​bdash@webkit.org>

Reviewed by Mitz.

Update references to webkit.opendarwin.org to webkit.org in Spinneret and WebKit.app.

  • Spinneret/Spinneret/Spinneret.cpp: (_tWinMain):
  • WebKitLauncher/WebKitNightlyEnabler.m: (cleanUpAfterOurselves):
7:22 PM Changeset in webkit [17511] by bdakin
  • 6 edits in trunk

WebCore:

Reviewed by Maciej.

This adds the back-end of the remaining WebElementDictionary
functions into HitTestResult.

  • WebCore.exp:
  • rendering/HitTestResult.cpp: (WebCore::HitTestResult::title): (WebCore::displayString): This is nearly identical to the displayString() defined in DOMInternal.mm except that it returns a String instead of an NSString. The old code path used the DOMInternal method, so I made a new one here for the new code path. (WebCore::HitTestResult::altDisplayString): (WebCore::HitTestResult::image): (WebCore::HitTestResult::absoluteImageURL): (WebCore::HitTestResult::absoluteLinkURL): (WebCore::HitTestResult::titleDisplayString): (WebCore::HitTestResult::textContent):
  • rendering/HitTestResult.h:

WebKit:

Reviewed by Maciej.

This creates local functions for the remaining WebElementDictionary
members that calls into HitTestResult instead of doing magical
things with the Objective-C DOM classes.

  • ChangeLog:
  • Misc/WebElementDictionary.m: (addLookupKey): The values of the dictionary are now just selectors. They used to be WebElementMethods which were WebElementTargetObjects associated with selectors, but none of that is needed any more. (+[WebElementDictionary initializeLookupTable]): All selectors are now local functions, no more WebElementTargetObjects. (-[WebElementDictionary objectForKey:]): No more target objects! (-[WebElementDictionary _domNode]): Call into HitTestResult member variable. (-[WebElementDictionary _altDisplayString]): Same. (-[WebElementDictionary _image]): Same. (-[WebElementDictionary _absoluteImageURL]): Same. (-[WebElementDictionary _title]): Same. (-[WebElementDictionary _absoluteLinkURL]): Same. (-[WebElementDictionary _targetWebFrame]): Same. (-[WebElementDictionary _titleDisplayString]): Same. (-[WebElementDictionary _textContent]): Same.
7:11 PM Changeset in webkit [17510] by bdash
  • 3 edits in trunk/WebKitTools

2006-10-31 Mark Rowe <​bdash@webkit.org>

Reviewed by Stephanie.

Limit build slaves to a single build to prevent concurrent builds on a single slave from significantly increasing build time.

  • BuildSlaveSupport/build.webkit.org-config/webkit/builders.py: Adjust slave distribution, and make use of a SlaveLock.
  • BuildSlaveSupport/build.webkit.org-config/webkit/factories.py: Remove unused Qt build factory.
6:59 PM Changeset in webkit [17509] by sullivan
  • 2 edits in trunk/WebCore
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::markMisspellings): Tiger build fix: added an #ifndef BUILDING_ON_TIGER where one was needed.
6:49 PM Changeset in webkit [17508] by sullivan
  • 6 edits in trunk/WebCore

Reviewed by Maciej


  • fixed <rdar://problem/4804627> ToolTips do not appear for grammar suggestions


The foundation of this was in my last checkin. This checkin is all about displaying
the correct string in the toolTip.

  • dom/DocumentMarker.h: New description field in this struct.
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::advanceToNextMisspelling): When adding a grammar marker, supply the appropriate description. Also, added a comment about the remaining work to make grammar checking return sensible answers. (WebCore::FrameMac::markMisspellings): ditto (yes, still needs some refactoring to minimize duplicated code)


  • dom/Document.h:
  • dom/Document.cpp: (WebCore::Document::addMarker): Now takes an optional description string (WebCore::Document::markerContainingPoint): New function, returns a pointer to the (first) marker of the specified type whose rect contains the specified point, or 0 if none.


  • rendering/HitTestResult.cpp: (WebCore::HitTestResult::spellingToolTip): Replaced hardwired string placeholder implementation with code that uses markerContainingPoint and gets the description from the marker.
6:33 PM Changeset in webkit [22453] by mjs
  • 2 edits in branches/WindowsMerge/WebKitWin

Reviewed by Maciej.


  • merged changes for:



Most of htis was already in this tree.

  • WebFrame.cpp: (WebFrame::stopLoading): Implement. Added FIXME.
6:14 PM Changeset in webkit [17507] by ggaren
  • 3 edits
    2 adds in trunk

JavaScriptCore:

Reviewed by Beth.


Fixed ​http://bugs.webkit.org/show_bug.cgi?id=11477
REGRESSION: GMail crashes in KJS::FunctionImp::callerGetter

  • kjs/function.cpp: (KJS::FunctionImp::argumentsGetter): Removed unnecessary braces. (KJS::FunctionImp::callerGetter): More logical NULL checking.

LayoutTests:

Added test for accessing the 'caller' property from inside an event
listener.

  • fast/events/caller-access-from-event-listener-expected.txt: Added.
  • fast/events/caller-access-from-event-listener.html: Added.
5:35 PM Changeset in webkit [17506] by ggaren
  • 3 edits in trunk/WebCore

Reviewed by Maciej.


Fixed crash resulting from Darin's last patch to remove BrowserExtension.

  • loader/mac/FrameLoaderMac.mm: (WebCore::FrameLoader::createWindow):
  • manual-tests/window-open-features-parsing.html: Updated for clarity.
4:56 PM Changeset in webkit [17505] by thatcher
  • 1 edit in trunk/WebKit/WebView/WebView.mm

Build fix for Leopard.

4:33 PM Changeset in webkit [22452] by sfalken
  • 2 edits in branches/WindowsMerge/WebCore

2006-10-31 Steve Falkenburg <​sfalken@apple.com>

Yet more B&I build fixes
Another missing .h file from the vcproj

  • WebCore.vcproj/WebCore.vcproj:
3:56 PM Changeset in webkit [17504] by ggaren
  • 20 edits
    2 adds in trunk

LayoutTests:

Added layout test for copying password field.

  • editing/pasteboard/copy-in-password-field-expected.txt: Added.
  • editing/pasteboard/copy-in-password-field.html: Added.

WebCore:

Reviewed by Alice.

Moved some Editing code from WebKit, the bridge, and WebCore::Frame down
to WebCore::Editor.


Layout tests pass.


Renamed "may*" to "can*" because "can" is more accurate (these functions
aren't just about permission) and it matches WebKit.

(WebCore::FrameMac::handleMouseMoveEvent): Directly test for dragging in a
password field. Now that WebCore fully implements canCopy(), it doesn't just
mean "the selection is not in a password field" anymore.
(-[WebCoreFrameBridge _shouldAllowAccessFrom:]): Removed this #ifed-out code.
The fact that it's not called anymore may represent a security issue, but I
don't see how commented-out code will help reveal the issue, and the
bridge is going away, anyway.

  • editing/SelectionController.h: Changed selection() calls to references to m_sel, to match the rest of the file.

WebKit:

Reviewed by Alice.


Moved some Editing code from WebKit, the bridge, and WebCore::Frame down
to WebCore::Editor.

  • WebCoreSupport/WebFrameBridge.mm:
  • WebView/WebHTMLView.m: (-[WebHTMLView _shouldDeleteRange:]): (-[WebHTMLView _canCopy]): (-[WebHTMLView _canCut]): (-[WebHTMLView _canDelete]): (-[WebHTMLView _canPaste]): (-[WebHTMLView _canEdit]): (-[WebHTMLView _canEditRichly]): (-[WebHTMLView _isEditable]): (-[WebHTMLView _isSelectionInPasswordField]): (-[NSArray validateUserInterfaceItem:]): (-[NSArray _expandSelectionToGranularity:]):
3:37 PM Changeset in webkit [17503] by justing
  • 13 edits in trunk

LayoutTests:

Reviewed by harrison

Updated these tests to mention the bug that they demonstrate:

  • editing/pasteboard/nested-blocks-with-text-area-expected.checksum:
  • editing/pasteboard/nested-blocks-with-text-area-expected.png:
  • editing/pasteboard/nested-blocks-with-text-area-expected.txt:
  • editing/pasteboard/nested-blocks-with-text-area.html:
  • editing/pasteboard/nested-blocks-with-text-field-expected.checksum:
  • editing/pasteboard/nested-blocks-with-text-field-expected.png:
  • editing/pasteboard/nested-blocks-with-text-field-expected.txt:
  • editing/pasteboard/nested-blocks-with-text-field.html:

WebCore:

Reviewed by harrison


<rdar://problem/4711063>
Pasting 10K lines into Mail/Blot takes ~7sec, in TextEdit it takes ~1.5sec

  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplacementFragment::ReplacementFragment): Don't do the test insertion and plain text string creation unless we need the string for a BeforeTextInserted event handler or for a plain text only region. (WebCore::ReplacementFragment::removeInterchangeNodes): Added, moved code here from ReplacementFragment's constructor. (WebCore::ReplaceSelectionCommand::completeHTMLReplacement): Added a FIXME.
  • editing/ReplaceSelectionCommand.h:
3:10 PM Changeset in webkit [17502] by adachan
  • 6 edits in trunk/WebCore

2006-10-31 Ada Chan <​adachan@apple.com>

Reviewed by Adam

Correct forward declarations of the HitTestRequest struct.

3:03 PM Changeset in webkit [17501] by sullivan
  • 11 edits in trunk

WebCore:

Reviewed by Beth and Adam

Support for displaying tooltips for bad grammar. Currently this always displays the same tooltip;
next I'll make it use a string that's relevant for a specific grammar error.

  • WebCore.exp: export symbol for spellingToolTip function


  • rendering/HitTestResult.h:
  • rendering/HitTestResult.cpp: (WebCore::HitTestResult::spellingToolTip): new function, returns the string to be used in a tool tip that describes the questionable grammar


  • rendering/InlineTextBox.h:
  • rendering/InlineTextBox.cpp: (WebCore::InlineTextBox::paintSpellingOrGrammarMarker): now takes a style and font, needed to compute the rect representing the range containing questionable grammar. Computes the rect and associates it with the marker. (WebCore::InlineTextBox::paintDocumentMarkers): Pass in the style and font now needed by paintSpellingOrGrammarMarker

WebKit:

Reviewed by Beth and Adam


Display a tooltip when hovering over marked bad grammar.

  • Misc/WebElementDictionary.m: (+[WebElementDictionary initializeLookupTable]): support spelling tool tip (-[WebElementDictionary _spellingToolTip]): new method, calls through to HitTestResult


  • WebView/WebHTMLView.m: (-[WebHTMLView _updateMouseoverWithEvent:]): Check for a spelling tool tip; if found, prefer it over the other possible tool tips. Check for empty strings instead of just nil strings being, since values from WebElementDictionary are empty strings.


  • WebView/WebViewPrivate.h: declare new string constant WebElementSpellingToolTipKey
  • WebView/WebView.mm: define new string constant WebElementSpellingToolTipKey
3:01 PM Changeset in webkit [17500] by adele
  • 2 edits in trunk/WebCore

Removed commented out variable from last checkin.

  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::listBoxDefaultEventHandler):
2:51 PM Changeset in webkit [17499] by beidson
  • 2 edits in trunk/WebCore

Build fix

2:15 PM Changeset in webkit [22451] by sfalken
  • 2 edits in branches/WindowsMerge/WebCore
2:10 PM Changeset in webkit [17498] by adele
  • 10 edits
    4 adds in trunk

LayoutTests:

Reviewed by Adam.

Tests for ​http://bugs.webkit.org/show_bug.cgi?id=11127 NativeListBox: arrow and drag selection should pivot around one list item
and ​http://bugs.webkit.org/show_bug.cgi?id=11173 REGRESSION (NativeListBox): Shift-clicking items in list box doesn't expand the current selection
and ​http://bugs.webkit.org/show_bug.cgi?id=11417 REGRESSION: onchange does not fire for list-style select elements

  • fast/forms/listbox-onchange-expected.txt: Added.
  • fast/forms/listbox-onchange.html: Added.
  • fast/forms/listbox-selection-expected.txt: Added.
  • fast/forms/listbox-selection.html: Added.

WebCore:

Reviewed by Adam.

Tests:

  • LayoutTests/fast/forms/listbox-selection.html
  • LayoutTests/fast/forms/listbox-onchange.html
  • html/HTMLSelectElement.h: Added m_selectedListIndexBase and m_selectedListIndexExtent to track indices for the active selection in progress. Added 2 vectors to cache selection state. One is kept so that the previous selection state can be restored as the active selection grows and shrinks. And one for onChange, that is updated after onChange is fired. Added m_activeSelectionState to keep track of whether the current drag selection is selecting or deselecting.


  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::HTMLSelectElement): Initialized new variables. (WebCore::HTMLSelectElement::setSelectedIndex): If needed, initialize m_selectedListIndexBase and m_selectedListIndexExtent. (WebCore::HTMLSelectElement::dispatchBlurEvent): Only fire the onChange event here for menu lists. (WebCore::HTMLSelectElement::listBoxDefaultEventHandler): Updates base and extent variables for mouse and key events. (WebCore::HTMLSelectElement::setBase): Added. Also caches the selection state. (WebCore::HTMLSelectElement::setExtent): Added. (WebCore::HTMLSelectElement::updateListBoxSelection): Added. (WebCore::HTMLSelectElement::listBoxOnChange): Added.
  • rendering/RenderListBox.cpp: (WebCore::RenderListBox::updateFromElement): Only scroll to reveal the first index if both the first and last indices aren't visible. (WebCore::RenderListBox::listIndexAtOffset): Added. Replaces optionAtPoint, which is no longer used. (WebCore::RenderListBox::autoscroll): Now sets the selection using the select's base and extent. (WebCore::RenderListBox::stopAutoscroll): Added. Tells the select element to fire onChange. This is needed because the autoscroll can end from a mouseUp outside of the list box, and the select element won't get a mouseUp event directly. But the frame will stop the autoscroll at that point, and now we can notify the select element from here. (WebCore::RenderListBox::scrollToRevealElementAtListIndex): Checks new listIndexIsVisible method. (WebCore::RenderListBox::listIndexIsVisible): Added. (WebCore::RenderListBox::valueChanged): Removed unnecessary printf.
  • page/Frame.cpp: (WebCore::Frame::stopAutoscrollTimer): Added rendererIsBeingDestroyed argument, so when the renderer calls this during destruction, we don't try to use the pointer to that renderer to call stopAutoscroll. This is done so a renderer that's still alive has a chance to do some cleanup after autoscroll.
  • rendering/RenderListBox.h: (WebCore::RenderListBox::shouldAutoscroll): Always returns true now, since we're also updating selection from the autoscroll timer.
  • rendering/RenderObject.h: (WebCore::RenderObject::stopAutoscroll): Added.
  • rendering/RenderObject.cpp: (WebCore::RenderObject::destroy): Calls stopAutoscrollTimer with rendererIsBeingDestroyed argument.
2:04 PM Changeset in webkit [17497] by zbujtas
  • 2 edits in S60/trunk/WebCore

2006-10-31 sareen <​shyam.sareen@nokia.com>

Reviewed by <​zbujtas@gmail.com>.
DESC:
​http://bugs.webkit.org/show_bug.cgi?id=11473

WARNING: NO TEST CASES ADDED OR CHANGED

  • kwq/KWQObjectWidget.cpp: (QObjectWidget::hasResized): (QObjectWidget::focusable):
1:55 PM Changeset in webkit [17496] by bdakin
  • 1 edit in trunk/WebCore/ChangeLog

Fixing typo in ChangeLog. This is the last one, I swear.

1:53 PM Changeset in webkit [17495] by bdakin
  • 1 edit
    1 add in trunk/WebCore

Forgot to check this in a minute ago. Oops!! Thanks Mitz!

  • rendering/HitTestRequest.h: Added. (WebCore::HitTestRequest::HitTestRequest):
1:48 PM Changeset in webkit [17494] by bdakin
  • 55 edits in trunk

WebCore:

Reviewed by Maciej.

Fix for ​http://bugs.webkit.org/show_bug.cgi?id=11461 HitTestResult
should be split into HitTestRequest and HitTestResult

This patch creates a new struct called HitTestRequest that holds
the three boolean values (readonly, active, and mouseMove) that
were formerly a part of HitTestRequest. all hitTest() and
nodeAtPoint() functions now take a HitTestRequest in addition to
the HitTestResult.

WebKit:

Reviewed by Maciej.

Small tweaks to WebKit because of ​http://bugs.webkit.org/
show_bug.cgi?id=11461 HitTestResult should be split into
HitTestRequest and HitTestResult

  • WebKit.xcodeproj/project.pbxproj:
  • WebView/WebHTMLView.m: (-[WebHTMLView elementAtPoint:allowShadowContent:]): The HitTestResult initializer now just takes a point.
1:32 PM Changeset in webkit [22450] by sfalken
  • 2 edits in branches/WindowsMerge/WebCore

2006-10-31 Steve Falkenburg <​sfalken@apple.com>

Remove extra files I accidently added

  • WebCore.vcproj/WebCore.vcproj:
1:03 PM Changeset in webkit [17493] by thatcher
  • 6 edits in trunk/WebCore

Reviewed by Maciej.

Bug 11442: [CSS 3] support for cursor: all-scroll
​http://bugs.webkit.org/show_bug.cgi?id=11442

  • css/CSSComputedStyleDeclaration.cpp: (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
  • css/CSSValueKeywords.in:
  • css/cssparser.cpp: (WebCore::CSSParser::parseValue):
  • page/FrameView.cpp: (WebCore::selectCursor):
  • rendering/RenderStyle.h:
12:56 PM Changeset in webkit [17492] by oliver
  • 8 edits
    5 moves in trunk/WebCore

2006-10-31 Oliver Hunt <​oliver@apple.com>

Reviewed by Geoff.

Converting Obj-C++ to C++ in kcanvas

  • WebCore.xcodeproj/project.pbxproj:
  • kcanvas/device/quartz/KCanvasFilterQuartz.h:
  • kcanvas/device/quartz/KCanvasFilterQuartz.mm:
  • kcanvas/device/quartz/KCanvasItemQuartz.cpp: Added.
  • kcanvas/device/quartz/KCanvasItemQuartz.mm: Removed.
  • kcanvas/device/quartz/KRenderingDeviceQuartz.cpp: Added.
  • kcanvas/device/quartz/KRenderingDeviceQuartz.h:
  • kcanvas/device/quartz/KRenderingDeviceQuartz.mm: Removed.
  • kcanvas/device/quartz/KRenderingPaintServerGradientQuartz.mm:
  • kcanvas/device/quartz/KRenderingPaintServerQuartz.h:
  • kcanvas/device/quartz/QuartzSupport.cpp: Added.
  • kcanvas/device/quartz/QuartzSupport.h:
  • kcanvas/device/quartz/QuartzSupport.mm: Removed.
  • platform/graphics/svg/cg/SVGResourceClipperCg.cpp: Added. (WebCore::SVGResourceClipper::applyClip):
  • platform/graphics/svg/cg/SVGResourceClipperCg.mm: Removed.
  • platform/graphics/svg/cg/SVGResourceImageCg.cpp: Added.
  • platform/graphics/svg/cg/SVGResourceImageCg.mm: Removed.
12:52 PM Changeset in webkit [17491] by oliver
  • 2 edits in trunk/JavaScriptCore

2006-10-31 Oliver Hunt <​oliver@apple.com>

Reviewed by Geoff.

Adding definition for PLATFORM(CI)

  • wtf/Platform.h:
12:34 PM Changeset in webkit [17490] by justing
  • 11 edits
    1 add
    1 delete in trunk

LayoutTests:

Reviewed by harrison

  • editing/execCommand/create-list-1.html: Removed.
  • editing/execCommand/create-list-with-hr-expected.checksum:
  • editing/execCommand/create-list-with-hr-expected.png:
  • editing/execCommand/create-list-with-hr-expected.txt:
  • editing/execCommand/remove-list-1-expected.checksum:
  • editing/execCommand/remove-list-1-expected.png:
  • editing/execCommand/remove-list-1-expected.txt:
  • fast/text/attributed-substring-from-range-001-expected.txt:

WebCore:

Reviewed by harrison


<rdar://problem/4808375>
REGRESSION: TextIterator slowed down, affecting Find on Page & Copy (etc.) [11460}


Completely back out r17276 because of performance issues.

  • editing/TextIterator.cpp: (WebCore::TextIterator::TextIterator): (WebCore::TextIterator::advance): (WebCore::TextIterator::handleTextNode): (WebCore::TextIterator::handleTextBox): (WebCore::TextIterator::handleReplacedElement): (WebCore::TextIterator::handleNonTextNode): (WebCore::TextIterator::exitNode): (WebCore::TextIterator::emitCharacter): (WebCore::TextIterator::range): (WebCore::SimplifiedBackwardsTextIterator::SimplifiedBackwardsTextIterator): (WebCore::SimplifiedBackwardsTextIterator::advance): (WebCore::SimplifiedBackwardsTextIterator::handleTextNode): (WebCore::SimplifiedBackwardsTextIterator::handleReplacedElement): (WebCore::SimplifiedBackwardsTextIterator::emitCharacter): (WebCore::SimplifiedBackwardsTextIterator::emitNewline): (WebCore::SimplifiedBackwardsTextIterator::range): (WebCore::CharacterIterator::range): (WebCore::TextIterator::rangeFromLocationAndLength):
  • editing/TextIterator.h: (WebCore::TextIterator::atEnd): (WebCore::SimplifiedBackwardsTextIterator::atEnd):
12:01 PM Changeset in webkit [17489] by thatcher
  • 1 edit in trunk/WebCore/WebCore.xcodeproj/project.pbxproj

Build fix. Generated headers need to be added to 'Copy Generated Headers' and not 'Copy Headers'.

11:34 AM Changeset in webkit [22449] by sfalken
  • 4 edits in branches/WindowsMerge

Fixes for Boomer submission, bumped version number.

11:31 AM Changeset in webkit [17488] by darin
  • 14 edits
    3 adds in trunk

WebCore:

Reviewed by Brady.

  • got "action dictionary" code out of FrameLoader, replacing with a class called NavigationAction
  • loader/DocumentLoader.h: Changed m_triggeringAction to a NavigationAction.
  • loader/mac/DocumentLoaderMac.mm: (WebCore::DocumentLoader::triggeringAction): Ditto. (WebCore::DocumentLoader::setTriggeringAction): Ditto.
  • loader/FrameLoader.h: Changed action parameters to NavigationAction.
  • loader/FrameLoaderClient.h: Changed action parameters to NavigationAction. Removed elementForEvent.
  • loader/mac/FrameLoaderMac.mm: (WebCore::FrameLoader::load): Ditto. (WebCore::FrameLoader::reload): Ditto. (WebCore::FrameLoader::checkNewWindowPolicy): Ditto. (WebCore::FrameLoader::checkNavigationPolicy): Ditto. (WebCore::FrameLoader::continueLoadAfterNewWindowPolicy): Ditto. (WebCore::FrameLoader::post): Ditto.
  • loader/NavigationAction.h: Added.
  • loader/NavigationAction.cpp: Added.
  • loader/mac/NavigationActionMac.mm: Added.
  • loader/mac/MainResourceLoaderMac.mm: Fixed copyright.
  • WebCore.xcodeproj/project.pbxproj: Updated for new files.
  • WebCore.exp: Updated.

WebKit:

Reviewed by Brady.

  • got "action dictionary" code out of FrameLoader, replacing with a class called NavigationAction
  • WebCoreSupport/WebFrameLoaderClient.h: Changed parameter types to NavigationAction. Made elementForEvent non-virtual. Added actionDictionary function.
  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::dispatchDecidePolicyForNewWindowAction): Changed parameter type, and used actionDictionary to make the action dictionary. (WebFrameLoaderClient::dispatchDecidePolicyForNavigationAction): Ditto. (WebFrameLoaderClient::actionDictionary): Added. Code that was originally in WebCore that creates the action dictionary (from a NavigationAction).
  • WebView/WebFrame.mm: (-[WebFrame _loadItem:withLoadType:]): Use NavigationAction instead of a dictionary for the action parameters.
11:19 AM Changeset in webkit [17487] by sfalken
  • 2 edits in trunk/WebCore

2006-10-31 Steve Falkenburg <​sfalken@apple.com>

Reviewed by Adam.

Build fix

  • platform/win/CookieJarWin.cpp: (WebCore::cookies):
11:03 AM Changeset in webkit [17486] by aroben
  • 9 edits
    2 moves
    2 adds
    1 delete in trunk/WebCore

Reviewed by Maciej. Landed by Adam.

Fixes: ​http://bugs.webkit.org/show_bug.cgi?id=11463

Move KCanvasTreeDebug into rendering, named as SVGRenderTreeAsText.*,
as dicussed with Dave. Also kill the outdated DESIGN document.

  • CMakeLists.txt:
  • WebCore.xcodeproj/project.pbxproj:
  • kcanvas/DESIGN: Removed.
  • kcanvas/KCanvasFilters.cpp:
  • kcanvas/KCanvasTreeDebug.cpp: Moved to rendering/SVGRenderTreeAsText
  • kcanvas/KCanvasTreeDebug.h: Ditto.
  • kcanvas/device/KRenderingPaintServerGradient.cpp:
  • kcanvas/device/KRenderingPaintServerPattern.cpp:
  • kcanvas/device/KRenderingPaintServerSolid.cpp:
  • platform/graphics/svg/SVGResourceClipper.cpp:
  • rendering/RenderTreeAsText.cpp:
  • rendering/SVGRenderTreeAsText.cpp: Added.
  • rendering/SVGRenderTreeAsText.h: Added. (WebCore::operator<<):
10:43 AM Changeset in webkit [17485] by ap
  • 2 edits in trunk/WebCore

2006-10-31 MorganL <​morganl.webkit@yahoo.com>

Reviewed by Maciej.

Fixes ​http://bugs.webkit.org/show_bug.cgi?id=11286
Includes some CRLF -> LF fixups.

  • platform/win/CookieJarWin.cpp: (WebCore::cookies):
10:39 AM Changeset in webkit [22448] by kmccullo
  • 2 edits in branches/WindowsMerge/WebKitWin

WebKitWin:

Reviewed by Steve.

  • Added basic functionality for running javascript from the address bar. Currently no return results are evaluated so use alert messages to see the results.
  • WebView.cpp: (WebView::stringByEvaluatingJavaScriptFromString):
10:23 AM Changeset in webkit [22447] by aroben
  • 11 edits in branches/WindowsMerge

WebCoreWin:

Build fixes.

  • WebCore.vcproj/WebCore.vcproj: Add new platform/graphics directory to our include path, and copy headers from there for WebKit's benefit.
  • bridge/win/FrameWin.cpp: Update accessors of FrameLoadRequest. (WebCore::FrameWin::urlSelected): (WebCore::FrameWin::submitForm):
  • plugins/win/PluginViewWin.cpp: Ditto. (WebCore::PluginViewWin::start): (WebCore::PluginViewWin::performRequest): (WebCore::PluginViewWin::load): (WebCore::PluginViewWin::getURLNotify): (WebCore::PluginViewWin::getURL): (WebCore::PluginViewWin::handlePost):
  • plugins/win/PluginViewWin.h: Correct forward declaration of FrameLoadRequest.

WebKitWin:

Build fixes.

  • WebEditorClient.cpp: Add stubs for new methods. (WebEditorClient::isContinuousSpellCheckingEnabled): (WebEditorClient::isGrammarCheckingEnabled): (WebEditorClient::spellCheckerDocumentTag):
  • WebEditorClient.h: Add declarations for new methods.
  • WebFrame.cpp: Fix #includes to use new platform/graphics directory.
  • WebView.cpp: Ditto.
  • WebKit.vcproj/WebKit.vcproj: Add platform/graphics to include path.
10:19 AM Changeset in webkit [17484] by aroben
  • 3 edits in trunk/WebCore

Reviewed by Steve.

Fix some incorrect forward declarations.

  • loader/FrameLoader.h:
  • page/Frame.h:
10:16 AM Changeset in webkit [17483] by ap
  • 6 edits
    3 adds in trunk

2006-10-31 Vladimir Olexa <​vladimir.olexa@gmail.com>

Reviewed by Geoff.

​http://bugs.webkit.org/show_bug.cgi?id=4166
Function object does not support caller property

Test: fast/js/caller-property.html

  • kjs/function.cpp: (KJS::FunctionImp::callerGetter): added (KJS::FunctionImp::getOwnPropertySlot): added if statement to handle callerGetter()
  • kjs/function.h: added callerGetter() declaration
  • kjs/identifier.h: added caller property macro
  • tests/mozilla/expected.html:
9:45 AM Changeset in webkit [17482] by mjs
  • 3 edits in trunk/WebKit

Reviewed by Maciej.


  • COM/WebFrame.cpp: (WebFrame::stopLoading): Implement.
  • COM/WebView.cpp: (WebView::stopLoading): ditto (WebView::reload): ditto
9:20 AM Changeset in webkit [17481] by ap
  • 2 edits in trunk/WebKitQt

2006-10-31 Simon Hausmann <​hausmann@kde.org>

Reviewed by Maciej.

  • QtLauncher/main.cpp: (main): Declare our frame as main frame to the Page object. Fixes crashes on various web sites
8:49 AM Changeset in webkit [17480] by darin
  • 2 edits in trunk/WebKitTools
  • DumpRenderTree/DumpRenderTree.m: Instead of allocating a single local pasteboard, allocate any number of local pasteboards. (main): Allocate the dictionary of pasteboards. (+[DumpRenderTreePasteboard _pasteboardWithName:]): Allocate a pasteboard, given a name. (+[LocalPasteboard alloc]): Added, so we don't have to call NSAllocateObject explicitly elsewhere. (-[LocalPasteboard addTypes:owner:]): Added a check that the owner responds to the selector rather than calling unconditionally.
8:40 AM Changeset in webkit [17479] by darin
  • 2 edits in trunk/WebCore
  • WebCore.xcodeproj/project.pbxproj: Change GraphicsTypes.h to a private header to try to fix the build on the buildbot.
8:16 AM Changeset in webkit [17478] by darin
  • 2 edits in trunk/WebCore
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::advanceToNextMisspelling): Build fix for release Tiger builds. (WebCore::FrameMac::markMisspellings): Ditto.
6:26 AM Changeset in webkit [17477] by sullivan
  • 2 edits in trunk/WebCore
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::advanceToNextMisspelling): build fix: needed #ifndef BUILDING_ON_TIGER in one more place
6:18 AM Changeset in webkit [17476] by sullivan
  • 6 edits in trunk/WebCore

Reviewed by Adam Roben


  • fixes <rdar://problem/4804614> Bad grammar ranges are not visibly marked


This patch introduces much of the guts of grammar checking, though still not enough to actually
check grammar sensibly, due to:


<rdar://problem/4811175> Many false reports of bad grammar appear, caused by insufficient
context passed to grammar checker

  • platform/Logging.h:
  • platform/Logging.cpp: new log channel SpellingAndGrammar
  • bridge/mac/WebCorePageBridge.mm: (initializeLoggingChannelsIfNecessary): initialize new log channel
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::advanceToNextMisspelling): Compute bad grammar range when computing misspelling range. Find first detailed grammar range from the set NSSpellChecker determines. Compare it with misspelling range to see which is earliest (or shortest in the event of a tie), and do further processing with that one (select range; create marker that causes range to be visibly marked with a funky underline; update spelling panel appropriately). (WebCore::FrameMac::markMisspellings): More or less the same types of changes as in advanceToNextMisspelling The loops are structured just differently enough to make sharing code between these two functions a little tricky, so I decided to save that for a later patch.


(WebCore::FrameMac::respondToChangedSelection):
remove grammar markers when we remove spelling markers

4:36 AM Changeset in webkit [17475] by bdash
  • 2 edits in trunk/WebCore

2006-10-31 Nikolas Zimmermann <​zimmermann@kde.org>

Reviewed by Mitz.

Fix Qt/Linux build with older gcc3.3.4.

  • bindings/js/kjs_window.cpp: (KJS::WindowFunc::callAsFunction):
4:24 AM Changeset in webkit [17474] by bdash
  • 1 delete in trunk/WebCore/kcanvas/device/qt/KCanvasClipperQt.h

Remove obsolete KCanvasClipperQt.h which should have been removed in r17467.

3:47 AM Changeset in webkit [17473] by bdash
  • 11 edits in trunk

2006-10-31 Zack Rusin <​zack@kde.org>

Reviewed by Mitz.

Fix the Qt build after last nights changes.

  • platform/qt/FrameQt.cpp:
  • platform/qt/EditorClientQt.h:
  • platform/qt/EditorClientQt.cpp:
  • platform/graphics/svg/qt/SVGResourceImageQt.cpp:
  • platform/graphics/svg/qt/SVGResourceClipperQt.cpp:
  • CMakeLists.txt:
  • kcanvas/device/qt/KRenderingDeviceQt.cpp:

2006-10-31 Zack Rusin <​zack@kde.org>

Reviewed by Mitz.

Fix the Qt build after last nights changes.

  • QtLauncher/CMakeLists.txt:
  • WebKitPart/CMakeLists.txt:

Oct 30, 2006:

9:50 PM Changeset in webkit [17472] by darin
  • 2 edits in trunk/WebKitTools
  • fixed build
  • DumpRenderTree/DumpRenderTree.m: (-[LocalPasteboard setString:forType:]): Don't use CFPasteboard.
9:49 PM Changeset in webkit [22446] by darin
  • 5 edits in branches/WindowsMerge/WebCore

Reviewed by Anders.

  • remove BrowserExtension
  • WebCore.vcproj/WebCore.vcproj:
  • bridge/win/FrameWin.cpp: (WebCore::FrameWin::FrameWin):
  • platform/win/TemporaryLinkStubs.cpp:
  • plugins/win/PluginViewWin.cpp: (WebCore::PluginViewWin::performRequest):
9:46 PM Changeset in webkit [17471] by darin
  • 17 edits
    6 deletes in trunk/WebCore

Reviewed by Anders.

  • remove BrowserExtension
  • WebCore.vcproj/WebCore/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/kjs_window.cpp: (KJS::createNewWindow): (KJS::WindowFunc::callAsFunction):
  • bridge/BrowserExtension.h: Removed.
  • bridge/mac/BrowserExtensionMac.h: Removed.
  • bridge/mac/BrowserExtensionMac.mm: Removed.
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::FrameMac): (WebCore::FrameMac::submitForm): (WebCore::FrameMac::urlSelected):
  • bridge/win/BrowserExtensionWin.cpp: Removed.
  • bridge/win/BrowserExtensionWin.h: Removed.
  • bridge/win/FrameWin.cpp: (WebCore::FrameWin::FrameWin):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::createWindow):
  • loader/FrameLoader.h:
  • loader/icon/IconLoader.cpp:
  • loader/mac/FrameLoaderMac.mm: (WebCore::FrameLoader::safeLoad): (WebCore::FrameLoader::load): (WebCore::FrameLoader::createWindow):
  • page/Frame.cpp: (WebCore::Frame::urlSelected): (WebCore::Frame::requestFrame): (WebCore::Frame::submitForm): (WebCore::Frame::scheduleHistoryNavigation): (WebCore::Frame::redirectionTimerFired):
  • page/Frame.h:
  • page/FrameLoadRequest.h: (WebCore::FrameLoadRequest::FrameLoadRequest): (WebCore::FrameLoadRequest::isEmpty): (WebCore::FrameLoadRequest::resourceRequest): (WebCore::FrameLoadRequest::frameName): (WebCore::FrameLoadRequest::setFrameName):
  • page/FramePrivate.h: (WebCore::FramePrivate::FramePrivate): (WebCore::FramePrivate::~FramePrivate):
  • platform/gdk/BrowserExtensionGdk.h: Removed.
  • platform/gdk/FrameGdk.cpp: (WebCore::FrameGdk::FrameGdk):
  • platform/gdk/TemporaryLinkStubs.cpp:
  • platform/network/ResourceRequest.h: (WebCore::ResourceRequest::isEmpty):
9:45 PM Changeset in webkit [17470] by darin
  • 2 edits in trunk/WebKit
  • Plugins/WebBaseNetscapePluginView.mm: (-[WebBaseNetscapePluginView evaluateJavaScriptPluginRequest:]): Fix comment.
9:29 PM Changeset in webkit [17469] by thatcher
  • 2 edits in trunk/WebKitTools

Reviewed by Tim H.

Fixes: ​http://bugs.webkit.org/show_bug.cgi?id=11353

  • Drosera/debugger.js: ParsedURL() object now recognizes local files
8:48 PM Changeset in webkit [17468] by darin
  • 5 edits in trunk/WebKitTools

Reviewed by Tim Hatcher.

  • DumpRenderTree/DumpRenderTree.m: Changed to allocate a local pasteboard. This should make our buildbot tests no longer need a pasteboard server.
8:17 PM Changeset in webkit [17467] by oliver
  • 57 edits
    13 moves
    7 adds
    10 deletes in trunk/WebCore

2006-10-31 Nikolas Zimmermann <​zimmermann@kde.org>

Reviewed by Oliver.

Modified from original due to earlier reversion

Fixes: ​http://bugs.webkit.org/show_bug.cgi?id=11436

Better SVG integration in WebKit, Part I.

The patch is mostly about creating a new platform/graphics directory, and
moving the kcanvas resources (clipper/masker/marker) there (in svg budir),
with a new name (KCanvasClipper -> SVGResourceClipper). Also fix several ownership
issues, by using ref counting (the SVG classes now store RefPtrs to the resources).

KCanvasFilters is still left in kcanvas/ subdirectory, to be converted in a next patch.
All details of the patch, and upcoming patches can be found in the bug report.

  • CMakeLists.txt:
  • WebCore.xcodeproj/project.pbxproj:
  • graphics/svg/SVGResource.cpp: Removed.
  • graphics/svg/SVGResourceClipper.cpp: Removed.
  • graphics/svg/SVGResourceClipper.h: Removed.
  • graphics/svg/SVGResourceImage.h: Removed.
  • graphics/svg/SVGResourceListener.h: Removed.
  • graphics/svg/SVGResourceMarker.cpp: Removed.
  • graphics/svg/SVGResourceMarker.h: Removed.
  • graphics/svg/SVGResourceMasker.cpp: Removed.
  • graphics/svg/SVGResourceMasker.h: Removed.
  • kcanvas/KCanvasClipper.cpp: Removed.
  • kcanvas/KCanvasClipper.h: Removed.
  • kcanvas/KCanvasCreator.cpp: Removed.
  • kcanvas/KCanvasCreator.h: Removed.
  • kcanvas/KCanvasFilters.cpp: (WebCore::getFilterById):
  • kcanvas/KCanvasFilters.h:
  • kcanvas/KCanvasImage.h: Removed.
  • kcanvas/KCanvasMarker.cpp: Removed.
  • kcanvas/KCanvasMarker.h: Removed.
  • kcanvas/KCanvasMasker.cpp: Removed.
  • kcanvas/KCanvasMasker.h: Removed.
  • kcanvas/KCanvasResource.cpp: Removed.
  • kcanvas/KCanvasResource.h: Removed.
  • kcanvas/KCanvasResourceListener.h: Removed.
  • kcanvas/KCanvasTreeDebug.cpp: (WebCore::writeRenderResources):
  • kcanvas/device/KRenderingDevice.h:
  • kcanvas/device/KRenderingPaintServer.h: (WebCore::KRenderingPaintServer::KRenderingPaintServer):
  • kcanvas/device/KRenderingPaintServerGradient.cpp: (WebCore::KRenderingPaintServerGradient::listener): (WebCore::KRenderingPaintServerGradient::setListener):
  • kcanvas/device/KRenderingPaintServerGradient.h:
  • kcanvas/device/KRenderingPaintServerPattern.cpp: (WebCore::KRenderingPaintServerPattern::KRenderingPaintServerPattern): (WebCore::KRenderingPaintServerPattern::~KRenderingPaintServerPattern): (WebCore::KRenderingPaintServerPattern::tile): (WebCore::KRenderingPaintServerPattern::setTile): (WebCore::KRenderingPaintServerPattern::listener): (WebCore::KRenderingPaintServerPattern::setListener):
  • kcanvas/device/KRenderingPaintServerPattern.h:
  • kcanvas/device/KRenderingPaintServerSolid.h:
  • kcanvas/device/qt/KCanvasClipperQt.cpp: Removed.
  • kcanvas/device/qt/KCanvasClipperQt.h:
  • kcanvas/device/qt/KRenderingDeviceQt.cpp: (WebCore::KRenderingDeviceQt::contextForImage): (WebCore::KRenderingDeviceQt::createResource): (WebCore::KRenderingDeviceQt::createPaintServer):
  • kcanvas/device/qt/KRenderingDeviceQt.h:
  • kcanvas/device/qt/KRenderingPaintServerGradientQt.cpp:
  • kcanvas/device/qt/KRenderingPaintServerPatternQt.cpp:
  • kcanvas/device/qt/KRenderingPaintServerQt.cpp:
  • kcanvas/device/qt/KRenderingPaintServerSolidQt.cpp:
  • kcanvas/device/qt/RenderPathQt.h: Removed.
  • kcanvas/device/quartz/KCanvasFilterQuartz.mm: (WebCore::KCanvasFilterQuartz::getCIFilterStack):
  • kcanvas/device/quartz/KCanvasItemQuartz.mm:
  • kcanvas/device/quartz/KCanvasMaskerQuartz.h: Removed.
  • kcanvas/device/quartz/KCanvasMaskerQuartz.mm: Removed.
  • kcanvas/device/quartz/KCanvasResourcesQuartz.h: Removed.
  • kcanvas/device/quartz/KCanvasResourcesQuartz.mm: Removed.
  • kcanvas/device/quartz/KRenderingDeviceQuartz.h:
  • kcanvas/device/quartz/KRenderingDeviceQuartz.mm: (WebCore::KRenderingDeviceQuartz::contextForImage): (WebCore::KRenderingDeviceQuartz::createPaintServer): (WebCore::KRenderingDeviceQuartz::createResource):
  • kcanvas/device/quartz/KRenderingPaintServerGradientQuartz.mm: (WebCore::KRenderingPaintServerGradientQuartz::KRenderingPaintServerGradientQuartz): (WebCore::KRenderingPaintServerGradientQuartz::~KRenderingPaintServerGradientQuartz): (WebCore::KRenderingPaintServerGradientQuartz::setup): (WebCore::KRenderingPaintServerGradientQuartz::teardown):
  • kcanvas/device/quartz/KRenderingPaintServerQuartz.h:
  • kcanvas/device/quartz/KRenderingPaintServerQuartz.mm: (WebCore::patternCallback): (WebCore::KRenderingPaintServerPatternQuartz::setup):
  • kcanvas/device/quartz/QuartzSupport.mm:
  • ksvg2/misc/KCanvasRenderingStyle.cpp: (WebCore::sharedSolidPaintServer):
  • ksvg2/svg/SVGClipPathElement.cpp: (WebCore::SVGClipPathElement::SVGClipPathElement): (WebCore::SVGClipPathElement::~SVGClipPathElement): (WebCore::SVGClipPathElement::canvasResource):
  • ksvg2/svg/SVGClipPathElement.h:
  • ksvg2/svg/SVGFEImageElement.cpp:
  • ksvg2/svg/SVGFilterElement.cpp: (WebCore::SVGFilterElement::SVGFilterElement): (WebCore::SVGFilterElement::~SVGFilterElement): (WebCore::SVGFilterElement::canvasResource):
  • ksvg2/svg/SVGFilterElement.h:
  • ksvg2/svg/SVGGradientElement.cpp: (WebCore::SVGGradientElement::SVGGradientElement): (WebCore::SVGGradientElement::~SVGGradientElement): (WebCore::SVGGradientElement::canvasResource): (WebCore::SVGGradientElement::resourceNotification):
  • ksvg2/svg/SVGGradientElement.h:
  • ksvg2/svg/SVGImageElement.cpp:
  • ksvg2/svg/SVGLinearGradientElement.cpp: (WebCore::SVGLinearGradientElement::buildGradient):
  • ksvg2/svg/SVGLinearGradientElement.h:
  • ksvg2/svg/SVGMarkerElement.cpp: (WebCore::SVGMarkerElement::SVGMarkerElement): (WebCore::SVGMarkerElement::~SVGMarkerElement): (WebCore::SVGMarkerElement::canvasResource):
  • ksvg2/svg/SVGMarkerElement.h:
  • ksvg2/svg/SVGMaskElement.cpp: (WebCore::SVGMaskElement::SVGMaskElement): (WebCore::SVGMaskElement::~SVGMaskElement): (WebCore::SVGMaskElement::drawMaskerContent): (WebCore::SVGMaskElement::canvasResource):
  • ksvg2/svg/SVGMaskElement.h:
  • ksvg2/svg/SVGPatternElement.cpp: (WebCore::SVGPatternElement::SVGPatternElement): (WebCore::SVGPatternElement::~SVGPatternElement): (WebCore::SVGPatternElement::fillAttributesFromReferencePattern): (WebCore::SVGPatternElement::drawPatternContentIntoTile): (WebCore::SVGPatternElement::canvasResource):
  • ksvg2/svg/SVGPatternElement.h:
  • ksvg2/svg/SVGRadialGradientElement.cpp: (WebCore::SVGRadialGradientElement::buildGradient):
  • ksvg2/svg/SVGRadialGradientElement.h:
  • ksvg2/svg/SVGStyledElement.h: (WebCore::SVGStyledElement::canvasResource):
  • platform/GraphicsContext.cpp: Removed.
  • platform/GraphicsContext.h: Removed.
  • platform/GraphicsTypes.cpp: Removed.
  • platform/GraphicsTypes.h: Removed.
  • platform/graphics/GraphicsContext.cpp: (WebCore::GraphicsContextState::GraphicsContextState): (WebCore::GraphicsContextPrivate::GraphicsContextPrivate): (WebCore::GraphicsContext::createGraphicsContextPrivate): (WebCore::GraphicsContext::destroyGraphicsContextPrivate): (WebCore::GraphicsContext::save): (WebCore::GraphicsContext::restore): (WebCore::GraphicsContext::font): (WebCore::GraphicsContext::setFont): (WebCore::GraphicsContext::pen): (WebCore::GraphicsContext::setPen): (WebCore::GraphicsContext::setFillColor): (WebCore::GraphicsContext::fillColor): (WebCore::GraphicsContext::updatingControlTints): (WebCore::GraphicsContext::setUpdatingControlTints): (WebCore::GraphicsContext::setPaintingDisabled): (WebCore::GraphicsContext::paintingDisabled): (WebCore::GraphicsContext::drawImage): (WebCore::GraphicsContext::drawText): (WebCore::GraphicsContext::drawHighlightForText): (WebCore::GraphicsContext::initFocusRing): (WebCore::GraphicsContext::clearFocusRing): (WebCore::GraphicsContext::focusRingBoundingRect): (WebCore::GraphicsContext::addFocusRingRect): (WebCore::GraphicsContext::focusRingWidth): (WebCore::GraphicsContext::focusRingOffset): (WebCore::GraphicsContext::focusRingRects): (WebCore::GraphicsContext::drawTiledImage):
  • platform/graphics/GraphicsContext.h:
  • platform/graphics/GraphicsTypes.cpp: (WebCore::): (WebCore::parseCompositeOperator): (WebCore::compositeOperatorName): (WebCore::parseLineCap): (WebCore::lineCapName): (WebCore::parseLineJoin): (WebCore::lineJoinName):
  • platform/graphics/GraphicsTypes.h: (WebCore::):
  • platform/graphics/svg/SVGResource.cpp: Added. (WebCore::SVGResource::SVGResource): (WebCore::SVGResource::~SVGResource): (WebCore::SVGResource::invalidate): (WebCore::SVGResource::addClient): (WebCore::SVGResource::clients): (WebCore::SVGResource::idInRegistry): (WebCore::SVGResource::setIdInRegistry): (WebCore::SVGResource::externalRepresentation): (WebCore::getResourceById): (WebCore::getPaintServerById): (WebCore::operator<<):
  • platform/graphics/svg/SVGResource.h: Added. (WebCore::): (WebCore::SVGResource::isPaintServer): (WebCore::SVGResource::isFilter): (WebCore::SVGResource::isClipper): (WebCore::SVGResource::isMarker): (WebCore::SVGResource::isMasker): (WebCore::SVGResourceListener::~SVGResourceListener):
  • platform/graphics/svg/SVGResourceClipper.cpp: Added. (WebCore::SVGResourceClipper::SVGResourceClipper): (WebCore::SVGResourceClipper::~SVGResourceClipper): (WebCore::SVGResourceClipper::resetClipData): (WebCore::SVGResourceClipper::addClipData): (WebCore::SVGResourceClipper::clipData): (WebCore::SVGResourceClipper::externalRepresentation): (WebCore::operator<<): (WebCore::getClipperById):
  • platform/graphics/svg/SVGResourceClipper.h: Added. (WebCore::ClipDataList::addPath): (WebCore::SVGResourceClipper::isClipper):
  • platform/graphics/svg/SVGResourceImage.h: Added.
  • platform/graphics/svg/SVGResourceMarker.cpp: Added. (WebCore::SVGResourceMarker::SVGResourceMarker): (WebCore::SVGResourceMarker::~SVGResourceMarker): (WebCore::SVGResourceMarker::setMarker): (WebCore::SVGResourceMarker::setRef): (WebCore::SVGResourceMarker::draw): (WebCore::SVGResourceMarker::externalRepresentation): (WebCore::getMarkerById):
  • platform/graphics/svg/SVGResourceMarker.h: Added. (WebCore::SVGResourceMarker::refX): (WebCore::SVGResourceMarker::refY): (WebCore::SVGResourceMarker::setAngle): (WebCore::SVGResourceMarker::setAutoAngle): (WebCore::SVGResourceMarker::angle): (WebCore::SVGResourceMarker::setUseStrokeWidth): (WebCore::SVGResourceMarker::useStrokeWidth): (WebCore::SVGResourceMarker::isMarker):
  • platform/graphics/svg/SVGResourceMasker.cpp: Added. (WebCore::SVGResourceMasker::SVGResourceMasker): (WebCore::SVGResourceMasker::~SVGResourceMasker): (WebCore::SVGResourceMasker::setMask): (WebCore::SVGResourceMasker::mask): (WebCore::SVGResourceMasker::externalRepresentation): (WebCore::getMaskerById):
  • platform/graphics/svg/SVGResourceMasker.h: Added. (WebCore::SVGResourceMasker::isMasker):
  • platform/graphics/svg/cg/SVGResourceClipperCg.mm: Added. (WebCore::SVGResourceClipper::applyClip):
  • platform/graphics/svg/cg/SVGResourceImageCg.mm: Added. (WebCore::SVGResourceImage::SVGResourceImage): (WebCore::SVGResourceImage::~SVGResourceImage): (WebCore::SVGResourceImage::init): (WebCore::SVGResourceImage::size): (WebCore::SVGResourceImage::cgLayer): (WebCore::SVGResourceImage::setCGLayer):
  • platform/graphics/svg/cg/SVGResourceMaskerCg.mm: Added. (WebCore::applyLuminanceToAlphaFilter): (WebCore::applyExpandAlphatoGrayscaleFilter): (WebCore::transformImageIntoGrayscaleMask): (WebCore::SVGResourceMasker::applyMask):
  • platform/graphics/svg/qt/SVGResourceClipperQt.cpp: Added. (WebCore::SVGResourceClipper::applyClip):
  • platform/graphics/svg/qt/SVGResourceImageQt.cpp: Added. (WebCore::SVGResourceImage::init): (WebCore::SVGResourceImage::size):
  • platform/graphics/svg/qt/SVGResourceMaskerQt.cpp: Added. (WebCore::SVGResourceMasker::applyMask):
  • platform/qt/GraphicsContextQt.cpp:
  • rendering/RenderPath.cpp: (WebCore::RenderPath::paint): (WebCore::DrawMarkersData::DrawMarkersData): (WebCore::RenderPath::drawMarkersIfNeeded):
  • rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::paint):
  • rendering/RenderSVGImage.cpp: (WebCore::RenderSVGImage::paint):
  • rendering/SVGInlineFlowBox.cpp: (WebCore::paintSVGInlineFlow):
7:53 PM Changeset in webkit [17466] by sullivan
  • 2 edits in trunk/WebCore

Reviewed by Brady


  • minor cleanup to make future patches clearer
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::advanceToNextMisspelling): renamed misspelling to misspellingNSRange for clarity; changed > 0 test to == 0 test with "continue" to better match structure of similar code in markMisspellings; a few other style tweaks.


(WebCore::FrameMac::markMisspellings):
renamed misspelling to misspellingNSRange for clarity; removed unnecessary braces around
a block just after a break and outdented accordingly.

7:23 PM Changeset in webkit [17465] by ggaren
  • 3 edits
    2 adds in trunk/WebKitTools

Reviewed by Beth.


Reorganized project file into Delegates and Controllers groups, and split
UIDelegate stuff into a UIDelegate class.


A little birdy told me that I might end up adding some UIDelegate methods
to DRT soon.

  • DumpRenderTree/DumpRenderTree.m: (main): (runTest):
  • DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj:
6:54 PM Changeset in webkit [17464] by sullivan
  • 5 edits in trunk

WebCore:

Reviewed by Geoff Garen


WebCore part of change to push the code that updates the spelling panel
into WebCore, in preparation for some grammar-checking stuff.

  • bridge/mac/FrameMac.h:
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::advanceToNextMisspelling): no more return value for this method, and update the spelling panel with the misspelled word here rather than in the WebKit callers.

WebKit:

Reviewed by Geoff Garen

  • WebView/WebHTMLView.m: (-[NSArray checkSpelling:]): removed code to update spelling panel; WebCore handles that now (-[NSArray showGuessPanel:]): ditto
6:36 PM Changeset in webkit [17463] by sullivan
  • 3 edits in trunk/WebKit
  • English.lproj/WebViewEditingContextMenu.nib/info.nib:
  • English.lproj/WebViewEditingContextMenu.nib/objects.nib: Another wording change to match framework, post-Tiger: "Check Spelling" -> "Check Document Now"
6:33 PM Changeset in webkit [17462] by oliver
  • 72 edits
    2 copies
    13 adds in trunk/WebCore

liver Hunt <​oliver@apple.com>

Reviewed by Anders.

Roll out last patch

  • CMakeLists.txt:
  • ChangeLog:
  • WebCore.xcodeproj/project.pbxproj:
  • graphics/svg/SVGResource.cpp:
  • graphics/svg/SVGResourceClipper.cpp:
  • graphics/svg/SVGResourceClipper.h:
  • graphics/svg/SVGResourceImage.h:
  • graphics/svg/SVGResourceListener.h:
  • graphics/svg/SVGResourceMarker.cpp:
  • graphics/svg/SVGResourceMarker.h:
  • graphics/svg/SVGResourceMasker.cpp:
  • graphics/svg/SVGResourceMasker.h:
  • kcanvas/KCanvasClipper.cpp: Added. (WebCore::operator<<): (WebCore::KCanvasClipper::KCanvasClipper): (WebCore::KCanvasClipper::~KCanvasClipper): (WebCore::KCanvasClipper::resetClipData): (WebCore::KCanvasClipper::addClipData): (WebCore::KCanvasClipper::clipData): (WebCore::KCanvasClipper::externalRepresentation): (WebCore::getClipperById):
  • kcanvas/KCanvasClipper.h: Added. (WebCore::KCClipData::windRule): (WebCore::KCClipDataList::KCClipDataList): (WebCore::KCClipDataList::addPath): (WebCore::KCanvasClipper::isClipper):
  • kcanvas/KCanvasFilters.cpp: (WebCore::getFilterById):
  • kcanvas/KCanvasFilters.h:
  • kcanvas/KCanvasImage.h: Added. (WebCore::KCanvasImage::KCanvasImage): (WebCore::KCanvasImage::~KCanvasImage):
  • kcanvas/KCanvasMarker.cpp: Added. (WebCore::KCanvasMarker::KCanvasMarker): (WebCore::KCanvasMarker::~KCanvasMarker): (WebCore::KCanvasMarker::setMarker): (WebCore::KCanvasMarker::setRef): (WebCore::KCanvasMarker::refX): (WebCore::KCanvasMarker::refY): (WebCore::KCanvasMarker::setAngle): (WebCore::KCanvasMarker::angle): (WebCore::KCanvasMarker::setAutoAngle): (WebCore::KCanvasMarker::setUseStrokeWidth): (WebCore::KCanvasMarker::useStrokeWidth): (WebCore::KCanvasMarker::draw): (WebCore::KCanvasMarker::externalRepresentation): (WebCore::getMarkerById):
  • kcanvas/KCanvasMarker.h: Added. (WebCore::KCanvasMarker::isMarker):
  • kcanvas/KCanvasMasker.cpp: Added. (WebCore::KCanvasMasker::KCanvasMasker): (WebCore::KCanvasMasker::~KCanvasMasker): (WebCore::KCanvasMasker::setMask): (WebCore::KCanvasMasker::externalRepresentation): (WebCore::getMaskerById):
  • kcanvas/KCanvasMasker.h: Added. (WebCore::KCanvasMasker::isMasker): (WebCore::KCanvasMasker::mask):
  • kcanvas/KCanvasResource.cpp: Added. (WebCore::operator<<): (WebCore::KCanvasResource::KCanvasResource): (WebCore::KCanvasResource::~KCanvasResource): (WebCore::KCanvasResource::addClient): (WebCore::KCanvasResource::clients): (WebCore::KCanvasResource::invalidate): (WebCore::KCanvasResource::idInRegistry): (WebCore::KCanvasResource::setIdInRegistry): (WebCore::KCanvasResource::externalRepresentation): (WebCore::getResourceById): (WebCore::getPaintServerById):
  • kcanvas/KCanvasResource.h: (WebCore::): (WebCore::KCanvasResource::isPaintServer): (WebCore::KCanvasResource::isFilter): (WebCore::KCanvasResource::isClipper): (WebCore::KCanvasResource::isMarker): (WebCore::KCanvasResource::isMasker):
  • kcanvas/KCanvasResourceListener.h: Added. (KCanvasResourceListener::KCanvasResourceListener): (KCanvasResourceListener::~KCanvasResourceListener):
  • kcanvas/KCanvasTreeDebug.cpp: (WebCore::writeRenderResources):
  • kcanvas/device/KRenderingDevice.h:
  • kcanvas/device/KRenderingPaintServer.h: (WebCore::KRenderingPaintServer::KRenderingPaintServer): (WebCore::KRenderingPaintServer::idInRegistry): (WebCore::KRenderingPaintServer::setIdInRegistry):
  • kcanvas/device/KRenderingPaintServerGradient.cpp: (WebCore::KRenderingPaintServerGradient::listener): (WebCore::KRenderingPaintServerGradient::setListener):
  • kcanvas/device/KRenderingPaintServerGradient.h:
  • kcanvas/device/KRenderingPaintServerPattern.cpp: (WebCore::KRenderingPaintServerPattern::KRenderingPaintServerPattern): (WebCore::KRenderingPaintServerPattern::~KRenderingPaintServerPattern): (WebCore::KRenderingPaintServerPattern::tile): (WebCore::KRenderingPaintServerPattern::setTile): (WebCore::KRenderingPaintServerPattern::listener): (WebCore::KRenderingPaintServerPattern::setListener):
  • kcanvas/device/KRenderingPaintServerPattern.h:
  • kcanvas/device/KRenderingPaintServerSolid.h:
  • kcanvas/device/qt/KCanvasClipperQt.cpp: (WebCore::KCanvasClipperQt::applyClip):
  • kcanvas/device/qt/KCanvasClipperQt.h: (WebCore::KCanvasClipperQt::KCanvasClipperQt):
  • kcanvas/device/qt/KRenderingDeviceQt.cpp: (WebCore::KRenderingDeviceQt::contextForImage): (WebCore::KRenderingDeviceQt::createResource): (WebCore::KRenderingDeviceQt::createPaintServer):
  • kcanvas/device/qt/KRenderingDeviceQt.h:
  • kcanvas/device/qt/KRenderingPaintServerGradientQt.cpp:
  • kcanvas/device/qt/KRenderingPaintServerPatternQt.cpp:
  • kcanvas/device/qt/KRenderingPaintServerQt.cpp:
  • kcanvas/device/qt/KRenderingPaintServerSolidQt.cpp:
  • kcanvas/device/quartz/KCanvasFilterQuartz.mm: (WebCore::KCanvasFilterQuartz::getCIFilterStack):
  • kcanvas/device/quartz/KCanvasItemQuartz.mm:
  • kcanvas/device/quartz/KCanvasMaskerQuartz.h: (WebCore::KCanvasMaskerQuartz::KCanvasMaskerQuartz):
  • kcanvas/device/quartz/KCanvasMaskerQuartz.mm: (WebCore::applyLuminanceToAlphaFilter): (WebCore::applyExpandAlphatoGrayscaleFilter): (WebCore::transformImageIntoGrayscaleMask): (WebCore::KCanvasMaskerQuartz::applyMask):
  • kcanvas/device/quartz/KCanvasResourcesQuartz.h: (WebCore::KCanvasClipperQuartz::KCanvasClipperQuartz): (WebCore::KCanvasImageQuartz::KCanvasImageQuartz): (WebCore::KCanvasImageQuartz::init): (WebCore::KCanvasImageQuartz::size):
  • kcanvas/device/quartz/KCanvasResourcesQuartz.mm: (WebCore::KCanvasClipperQuartz::applyClip): (WebCore::KCanvasImageQuartz::~KCanvasImageQuartz): (WebCore::KCanvasImageQuartz::cgLayer): (WebCore::KCanvasImageQuartz::setCGLayer):
  • kcanvas/device/quartz/KRenderingDeviceQuartz.h:
  • kcanvas/device/quartz/KRenderingDeviceQuartz.mm: (WebCore::KRenderingDeviceQuartz::contextForImage): (WebCore::KRenderingDeviceQuartz::createPaintServer): (WebCore::KRenderingDeviceQuartz::createResource):
  • kcanvas/device/quartz/KRenderingPaintServerGradientQuartz.mm: (WebCore::KRenderingPaintServerGradientQuartz::KRenderingPaintServerGradientQuartz): (WebCore::KRenderingPaintServerGradientQuartz::~KRenderingPaintServerGradientQuartz): (WebCore::KRenderingPaintServerGradientQuartz::setup): (WebCore::KRenderingPaintServerGradientQuartz::teardown):
  • kcanvas/device/quartz/KRenderingPaintServerQuartz.h:
  • kcanvas/device/quartz/KRenderingPaintServerQuartz.mm: (WebCore::patternCallback): (WebCore::KRenderingPaintServerPatternQuartz::setup):
  • kcanvas/device/quartz/QuartzSupport.mm:
  • ksvg2/misc/KCanvasRenderingStyle.cpp: (WebCore::sharedSolidPaintServer):
  • ksvg2/svg/SVGClipPathElement.cpp: (WebCore::SVGClipPathElement::SVGClipPathElement): (WebCore::SVGClipPathElement::~SVGClipPathElement): (WebCore::SVGClipPathElement::canvasResource):
  • ksvg2/svg/SVGClipPathElement.h:
  • ksvg2/svg/SVGFEImageElement.cpp:
  • ksvg2/svg/SVGFilterElement.cpp: (WebCore::SVGFilterElement::SVGFilterElement): (WebCore::SVGFilterElement::~SVGFilterElement): (WebCore::SVGFilterElement::canvasResource):
  • ksvg2/svg/SVGFilterElement.h:
  • ksvg2/svg/SVGGradientElement.cpp: (WebCore::SVGGradientElement::SVGGradientElement): (WebCore::SVGGradientElement::~SVGGradientElement): (WebCore::SVGGradientElement::canvasResource): (WebCore::SVGGradientElement::resourceNotification):
  • ksvg2/svg/SVGGradientElement.h:
  • ksvg2/svg/SVGImageElement.cpp:
  • ksvg2/svg/SVGLinearGradientElement.cpp: (WebCore::SVGLinearGradientElement::buildGradient):
  • ksvg2/svg/SVGLinearGradientElement.h:
  • ksvg2/svg/SVGMarkerElement.cpp: (WebCore::SVGMarkerElement::SVGMarkerElement): (WebCore::SVGMarkerElement::~SVGMarkerElement): (WebCore::SVGMarkerElement::canvasResource):
  • ksvg2/svg/SVGMarkerElement.h:
  • ksvg2/svg/SVGMaskElement.cpp: (WebCore::SVGMaskElement::SVGMaskElement): (WebCore::SVGMaskElement::~SVGMaskElement): (WebCore::SVGMaskElement::drawMaskerContent): (WebCore::SVGMaskElement::canvasResource):
  • ksvg2/svg/SVGMaskElement.h:
  • ksvg2/svg/SVGPatternElement.cpp: (WebCore::SVGPatternElement::SVGPatternElement): (WebCore::SVGPatternElement::~SVGPatternElement): (WebCore::SVGPatternElement::fillAttributesFromReferencePattern): (WebCore::SVGPatternElement::drawPatternContentIntoTile): (WebCore::SVGPatternElement::canvasResource):
  • ksvg2/svg/SVGPatternElement.h:
  • ksvg2/svg/SVGRadialGradientElement.cpp: (WebCore::SVGRadialGradientElement::buildGradient):
  • ksvg2/svg/SVGRadialGradientElement.h:
  • ksvg2/svg/SVGStyledElement.h: (WebCore::SVGStyledElement::canvasResource):
  • platform/GraphicsContext.cpp: Added. (WebCore::GraphicsContextState::GraphicsContextState): (WebCore::GraphicsContextPrivate::GraphicsContextPrivate): (WebCore::GraphicsContext::createGraphicsContextPrivate): (WebCore::GraphicsContext::destroyGraphicsContextPrivate): (WebCore::GraphicsContext::save): (WebCore::GraphicsContext::restore): (WebCore::GraphicsContext::font): (WebCore::GraphicsContext::setFont): (WebCore::GraphicsContext::pen): (WebCore::GraphicsContext::setPen): (WebCore::GraphicsContext::setFillColor): (WebCore::GraphicsContext::fillColor): (WebCore::GraphicsContext::updatingControlTints): (WebCore::GraphicsContext::setUpdatingControlTints): (WebCore::GraphicsContext::setPaintingDisabled): (WebCore::GraphicsContext::paintingDisabled): (WebCore::GraphicsContext::drawImage): (WebCore::GraphicsContext::drawText): (WebCore::GraphicsContext::drawHighlightForText): (WebCore::GraphicsContext::initFocusRing): (WebCore::GraphicsContext::clearFocusRing): (WebCore::GraphicsContext::focusRingBoundingRect): (WebCore::GraphicsContext::addFocusRingRect): (WebCore::GraphicsContext::focusRingWidth): (WebCore::GraphicsContext::focusRingOffset): (WebCore::GraphicsContext::focusRingRects): (WebCore::GraphicsContext::drawTiledImage):
  • platform/GraphicsContext.h: Added.
  • platform/GraphicsTypes.cpp: Added. (WebCore::): (WebCore::parseCompositeOperator): (WebCore::compositeOperatorName): (WebCore::parseLineCap): (WebCore::lineCapName): (WebCore::parseLineJoin): (WebCore::lineJoinName):
  • platform/GraphicsTypes.h: Added. (WebCore::):
  • platform/graphics/GraphicsContext.cpp:
  • platform/graphics/GraphicsContext.h:
  • platform/graphics/GraphicsTypes.cpp:
  • platform/graphics/GraphicsTypes.h:
  • platform/qt/GraphicsContextQt.cpp:
  • rendering/RenderPath.cpp: (WebCore::RenderPath::paint): (WebCore::DrawMarkersData::DrawMarkersData): (WebCore::RenderPath::drawMarkersIfNeeded):
  • rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::paint):
  • rendering/RenderSVGImage.cpp: (WebCore::RenderSVGImage::paint):
  • rendering/SVGInlineFlowBox.cpp: (WebCore::paintSVGInlineFlow):
5:46 PM Changeset in webkit [17461] by oliver
  • 59 edits
    13 moves
    3 adds
    2 deletes in trunk/WebCore

2006-10-31 Nikolas Zimmermann <​zimmermann@kde.org>

Reviewed by Oliver.

Fixes: ​http://bugs.webkit.org/show_bug.cgi?id=11436

Better SVG integration in WebKit, Part I.

The patch is mostly about creating a new platform/graphics directory, and
moving the kcanvas resources (clipper/masker/marker) there (in svg budir),
with a new name (KCanvasClipper -> SVGResourceClipper). Also fix several ownership
issues, by using ref counting (the SVG classes now store RefPtrs to the resources).

KCanvasFilters is still left in kcanvas/ subdirectory, to be converted in a next patch.
All details of the patch, and upcoming patches can be found in the bug report.

  • CMakeLists.txt:
  • WebCore.xcodeproj/project.pbxproj:
  • kcanvas/KCanvasClipper.cpp: Removed.
  • kcanvas/KCanvasClipper.h: Removed.
  • kcanvas/KCanvasCreator.cpp: Removed.
  • kcanvas/KCanvasCreator.h: Removed.
  • kcanvas/KCanvasFilters.cpp: (WebCore::getFilterById):
  • kcanvas/KCanvasFilters.h:
  • kcanvas/KCanvasImage.h: Removed.
  • kcanvas/KCanvasMarker.cpp: Removed.
  • kcanvas/KCanvasMarker.h: Removed.
  • kcanvas/KCanvasMasker.cpp: Removed.
  • kcanvas/KCanvasMasker.h: Removed.
  • kcanvas/KCanvasResource.cpp: Removed.
  • kcanvas/KCanvasResource.h: Removed.
  • kcanvas/KCanvasResourceListener.h: Removed.
  • kcanvas/KCanvasTreeDebug.cpp: (WebCore::writeRenderResources):
  • kcanvas/device/KRenderingDevice.h:
  • kcanvas/device/KRenderingPaintServer.h: (WebCore::KRenderingPaintServer::KRenderingPaintServer):
  • kcanvas/device/KRenderingPaintServerGradient.cpp: (WebCore::KRenderingPaintServerGradient::listener): (WebCore::KRenderingPaintServerGradient::setListener):
  • kcanvas/device/KRenderingPaintServerGradient.h:
  • kcanvas/device/KRenderingPaintServerPattern.cpp: (WebCore::KRenderingPaintServerPattern::KRenderingPaintServerPattern): (WebCore::KRenderingPaintServerPattern::~KRenderingPaintServerPattern): (WebCore::KRenderingPaintServerPattern::tile): (WebCore::KRenderingPaintServerPattern::setTile): (WebCore::KRenderingPaintServerPattern::listener): (WebCore::KRenderingPaintServerPattern::setListener):
  • kcanvas/device/KRenderingPaintServerPattern.h:
  • kcanvas/device/KRenderingPaintServerSolid.h:
  • kcanvas/device/qt/KCanvasClipperQt.cpp: Removed.
  • kcanvas/device/qt/KCanvasClipperQt.h: Removed.
  • kcanvas/device/qt/KRenderingDeviceQt.cpp: (WebCore::KRenderingDeviceQt::contextForImage): (WebCore::KRenderingDeviceQt::createResource): (WebCore::KRenderingDeviceQt::createPaintServer):
  • kcanvas/device/qt/KRenderingDeviceQt.h:
  • kcanvas/device/qt/KRenderingPaintServerGradientQt.cpp:
  • kcanvas/device/qt/KRenderingPaintServerPatternQt.cpp:
  • kcanvas/device/qt/KRenderingPaintServerQt.cpp:
  • kcanvas/device/qt/KRenderingPaintServerSolidQt.cpp:
  • kcanvas/device/qt/RenderPathQt.h: Removed.
  • kcanvas/device/quartz/KCanvasFilterQuartz.mm: (WebCore::KCanvasFilterQuartz::getCIFilterStack):
  • kcanvas/device/quartz/KCanvasItemQuartz.mm:
  • kcanvas/device/quartz/KCanvasMaskerQuartz.h: Removed.
  • kcanvas/device/quartz/KCanvasMaskerQuartz.mm: Removed.
  • kcanvas/device/quartz/KCanvasResourcesQuartz.h: Removed.
  • kcanvas/device/quartz/KCanvasResourcesQuartz.mm: Removed.
  • kcanvas/device/quartz/KRenderingDeviceQuartz.h:
  • kcanvas/device/quartz/KRenderingDeviceQuartz.mm: (WebCore::KRenderingDeviceQuartz::contextForImage): (WebCore::KRenderingDeviceQuartz::createPaintServer): (WebCore::KRenderingDeviceQuartz::createResource):
  • kcanvas/device/quartz/KRenderingPaintServerGradientQuartz.mm: (WebCore::KRenderingPaintServerGradientQuartz::KRenderingPaintServerGradientQuartz): (WebCore::KRenderingPaintServerGradientQuartz::~KRenderingPaintServerGradientQuartz): (WebCore::KRenderingPaintServerGradientQuartz::setup): (WebCore::KRenderingPaintServerGradientQuartz::teardown):
  • kcanvas/device/quartz/KRenderingPaintServerQuartz.h:
  • kcanvas/device/quartz/KRenderingPaintServerQuartz.mm: (WebCore::patternCallback): (WebCore::KRenderingPaintServerPatternQuartz::setup):
  • kcanvas/device/quartz/QuartzSupport.mm:
  • ksvg2/misc/KCanvasRenderingStyle.cpp: (WebCore::sharedSolidPaintServer):
  • ksvg2/svg/SVGClipPathElement.cpp: (WebCore::SVGClipPathElement::SVGClipPathElement): (WebCore::SVGClipPathElement::~SVGClipPathElement): (WebCore::SVGClipPathElement::canvasResource):
  • ksvg2/svg/SVGClipPathElement.h:
  • ksvg2/svg/SVGFEImageElement.cpp:
  • ksvg2/svg/SVGFilterElement.cpp: (WebCore::SVGFilterElement::SVGFilterElement): (WebCore::SVGFilterElement::~SVGFilterElement): (WebCore::SVGFilterElement::canvasResource):
  • ksvg2/svg/SVGFilterElement.h:
  • ksvg2/svg/SVGGradientElement.cpp: (WebCore::SVGGradientElement::SVGGradientElement): (WebCore::SVGGradientElement::~SVGGradientElement): (WebCore::SVGGradientElement::canvasResource): (WebCore::SVGGradientElement::resourceNotification):
  • ksvg2/svg/SVGGradientElement.h:
  • ksvg2/svg/SVGImageElement.cpp:
  • ksvg2/svg/SVGLinearGradientElement.cpp: (WebCore::SVGLinearGradientElement::buildGradient):
  • ksvg2/svg/SVGLinearGradientElement.h:
  • ksvg2/svg/SVGMarkerElement.cpp: (WebCore::SVGMarkerElement::SVGMarkerElement): (WebCore::SVGMarkerElement::~SVGMarkerElement): (WebCore::SVGMarkerElement::canvasResource):
  • ksvg2/svg/SVGMarkerElement.h:
  • ksvg2/svg/SVGMaskElement.cpp: (WebCore::SVGMaskElement::SVGMaskElement): (WebCore::SVGMaskElement::~SVGMaskElement): (WebCore::SVGMaskElement::drawMaskerContent): (WebCore::SVGMaskElement::canvasResource):
  • ksvg2/svg/SVGMaskElement.h:
  • ksvg2/svg/SVGPatternElement.cpp: (WebCore::SVGPatternElement::SVGPatternElement): (WebCore::SVGPatternElement::~SVGPatternElement): (WebCore::SVGPatternElement::fillAttributesFromReferencePattern): (WebCore::SVGPatternElement::drawPatternContentIntoTile): (WebCore::SVGPatternElement::canvasResource):
  • ksvg2/svg/SVGPatternElement.h:
  • ksvg2/svg/SVGRadialGradientElement.cpp: (WebCore::SVGRadialGradientElement::buildGradient):
  • ksvg2/svg/SVGRadialGradientElement.h:
  • ksvg2/svg/SVGStyledElement.h: (WebCore::SVGStyledElement::canvasResource):
  • platform/GraphicsContext.cpp: Removed.
  • platform/GraphicsContext.h: Removed.
  • platform/GraphicsTypes.cpp: Removed.
  • platform/GraphicsTypes.h: Removed.
  • platform/graphics/GraphicsContext.cpp: Added. (WebCore::GraphicsContextState::GraphicsContextState): (WebCore::GraphicsContextPrivate::GraphicsContextPrivate): (WebCore::GraphicsContext::createGraphicsContextPrivate): (WebCore::GraphicsContext::destroyGraphicsContextPrivate): (WebCore::GraphicsContext::save): (WebCore::GraphicsContext::restore): (WebCore::GraphicsContext::font): (WebCore::GraphicsContext::setFont): (WebCore::GraphicsContext::pen): (WebCore::GraphicsContext::setPen): (WebCore::GraphicsContext::setFillColor): (WebCore::GraphicsContext::fillColor): (WebCore::GraphicsContext::updatingControlTints): (WebCore::GraphicsContext::setUpdatingControlTints): (WebCore::GraphicsContext::setPaintingDisabled): (WebCore::GraphicsContext::paintingDisabled): (WebCore::GraphicsContext::drawImage): (WebCore::GraphicsContext::drawText): (WebCore::GraphicsContext::drawHighlightForText): (WebCore::GraphicsContext::initFocusRing): (WebCore::GraphicsContext::clearFocusRing): (WebCore::GraphicsContext::focusRingBoundingRect): (WebCore::GraphicsContext::addFocusRingRect): (WebCore::GraphicsContext::focusRingWidth): (WebCore::GraphicsContext::focusRingOffset): (WebCore::GraphicsContext::focusRingRects): (WebCore::GraphicsContext::drawTiledImage):
  • platform/graphics/GraphicsContext.h: Added.
  • platform/graphics/GraphicsTypes.cpp: Added. (WebCore::): (WebCore::parseCompositeOperator): (WebCore::compositeOperatorName): (WebCore::parseLineCap): (WebCore::lineCapName): (WebCore::parseLineJoin): (WebCore::lineJoinName):
  • platform/graphics/GraphicsTypes.h: Added. (WebCore::):
  • platform/graphics/svg/SVGResource.cpp: Added. (WebCore::SVGResource::SVGResource): (WebCore::SVGResource::~SVGResource): (WebCore::SVGResource::invalidate): (WebCore::SVGResource::addClient): (WebCore::SVGResource::clients): (WebCore::SVGResource::idInRegistry): (WebCore::SVGResource::setIdInRegistry): (WebCore::SVGResource::externalRepresentation): (WebCore::getResourceById): (WebCore::getPaintServerById): (WebCore::operator<<):
  • platform/graphics/svg/SVGResource.h: Added. (WebCore::): (WebCore::SVGResource::isPaintServer): (WebCore::SVGResource::isFilter): (WebCore::SVGResource::isClipper): (WebCore::SVGResource::isMarker): (WebCore::SVGResource::isMasker): (WebCore::SVGResourceListener::~SVGResourceListener):
  • platform/graphics/svg/SVGResourceClipper.cpp: Added. (WebCore::SVGResourceClipper::SVGResourceClipper): (WebCore::SVGResourceClipper::~SVGResourceClipper): (WebCore::SVGResourceClipper::resetClipData): (WebCore::SVGResourceClipper::addClipData): (WebCore::SVGResourceClipper::clipData): (WebCore::SVGResourceClipper::externalRepresentation): (WebCore::operator<<): (WebCore::getClipperById):
  • platform/graphics/svg/SVGResourceClipper.h: Added. (WebCore::ClipDataList::addPath): (WebCore::SVGResourceClipper::isClipper):
  • platform/graphics/svg/SVGResourceImage.h: Added.
  • platform/graphics/svg/SVGResourceMarker.cpp: Added. (WebCore::SVGResourceMarker::SVGResourceMarker): (WebCore::SVGResourceMarker::~SVGResourceMarker): (WebCore::SVGResourceMarker::setMarker): (WebCore::SVGResourceMarker::setRef): (WebCore::SVGResourceMarker::draw): (WebCore::SVGResourceMarker::externalRepresentation): (WebCore::getMarkerById):
  • platform/graphics/svg/SVGResourceMarker.h: Added. (WebCore::SVGResourceMarker::refX): (WebCore::SVGResourceMarker::refY): (WebCore::SVGResourceMarker::setAngle): (WebCore::SVGResourceMarker::setAutoAngle): (WebCore::SVGResourceMarker::angle): (WebCore::SVGResourceMarker::setUseStrokeWidth): (WebCore::SVGResourceMarker::useStrokeWidth): (WebCore::SVGResourceMarker::isMarker):
  • platform/graphics/svg/SVGResourceMasker.cpp: Added. (WebCore::SVGResourceMasker::SVGResourceMasker): (WebCore::SVGResourceMasker::~SVGResourceMasker): (WebCore::SVGResourceMasker::setMask): (WebCore::SVGResourceMasker::mask): (WebCore::SVGResourceMasker::externalRepresentation): (WebCore::getMaskerById):
  • platform/graphics/svg/SVGResourceMasker.h: Added. (WebCore::SVGResourceMasker::isMasker):
  • platform/graphics/svg/cg/SVGResourceClipperCg.mm: Added. (WebCore::SVGResourceClipper::applyClip):
  • platform/graphics/svg/cg/SVGResourceImageCg.mm: Added. (WebCore::SVGResourceImage::SVGResourceImage): (WebCore::SVGResourceImage::~SVGResourceImage): (WebCore::SVGResourceImage::init): (WebCore::SVGResourceImage::size): (WebCore::SVGResourceImage::cgLayer): (WebCore::SVGResourceImage::setCGLayer):
  • platform/graphics/svg/cg/SVGResourceMaskerCg.mm: Added. (WebCore::applyLuminanceToAlphaFilter): (WebCore::applyExpandAlphatoGrayscaleFilter): (WebCore::transformImageIntoGrayscaleMask): (WebCore::SVGResourceMasker::applyMask):
  • platform/graphics/svg/qt/SVGResourceClipperQt.cpp: Added. (WebCore::SVGResourceClipper::applyClip):
  • platform/graphics/svg/qt/SVGResourceImageQt.cpp: Added. (WebCore::SVGResourceImage::init): (WebCore::SVGResourceImage::size):
  • platform/graphics/svg/qt/SVGResourceMaskerQt.cpp: Added. (WebCore::SVGResourceMasker::applyMask):
  • platform/qt/GraphicsContextQt.cpp:
  • rendering/RenderPath.cpp: (WebCore::RenderPath::paint): (WebCore::DrawMarkersData::DrawMarkersData): (WebCore::RenderPath::drawMarkersIfNeeded):
  • rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::paint):
  • rendering/RenderSVGImage.cpp: (WebCore::RenderSVGImage::paint):
  • rendering/SVGInlineFlowBox.cpp: (WebCore::paintSVGInlineFlow):
5:44 PM Changeset in webkit [17460] by oliver
  • 2 edits in trunk/WebKitTools

2006-10-31 Nikolas Zimmermann <​zimmermann@kde.org>

Reviewed by Oliver.

Add new platform/graphics include directory.

  • DumpRenderTree/DumpRenderTree.qtproj/CMakeLists.txt:
5:35 PM Changeset in webkit [17459] by thatcher
  • 2 edits in trunk

Reviewed by Brady.

Make the universal build return non-zero when module make fails.

  • Makefile:
5:33 PM Changeset in webkit [17458] by thatcher
  • 2 edits in trunk/WebKitTools

Reviewed by Stephanie.

Add a special case for the Internal makefiles, so it can find the OepnSource.

  • Scripts/webkitdirs.pm:
5:25 PM Changeset in webkit [17457] by sullivan
  • 10 edits in trunk

WebCore:

Reviewed by Geoff Garen.


Moved spelling-related methods from bridge to EditorClient. Added one not-yet-used
grammar-related method.

  • bridge/EditorClient.h: declare isContinuousSpellCheckingEnabled(), spellCheckerDocumentTag(), and new isGrammarCheckingEnabled()


  • bridge/mac/WebCoreFrameBridge.h: removed bridge equivalents
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::advanceToNextMisspelling): convert bridge-using code to editor()->client()-using code (WebCore::FrameMac::markMisspellingsInAdjacentWords): ditto (WebCore::FrameMac::markMisspellings): ditto (WebCore::FrameMac::respondToChangedSelection): ditto


  • editing/Editor.h:
  • editing/Editor.cpp: (WebCore::Editor::client): new method, returns EditorClient pointer. In an ideal world all the code that needed to access the EditorClient would be in Editor.cpp, and we wouldn't need this accessor. But for now it's too tricky to extricate the spelling-related code from FrameMac.mm.

WebKit:

Reviewed by Geoff Garen.


Moved spelling-related methods from bridge to EditorClient. Added one not-yet-used
grammar-related method.

  • WebCoreSupport/WebEditorClient.h: declare overrides of isContinuousSpellCheckingEnabled(), spellCheckerDocumentTag(), and new isGrammarCheckingEnabled()
  • WebCoreSupport/WebEditorClient.mm: (WebEditorClient::isContinuousSpellCheckingEnabled): implement by calling through to WebView (WebEditorClient::isGrammarCheckingEnabled): ditto (WebEditorClient::spellCheckerDocumentTag): ditto


  • WebCoreSupport/WebFrameBridge.mm: removed bridge equivalents of these methods
5:21 PM Changeset in webkit [17456] by ggaren
  • 2 edits in trunk/WebKit

Reviewed by Beth.


Fixed nil-deref crash that I saw while using TOT (not sure how to repro,
but the debugger confirmed the cause).

  • WebView/WebFrame.mm: (core): Added check for NULL bridge.
5:01 PM Changeset in webkit [17455] by ggaren
  • 32 edits
    2 adds in trunk

WebCore:

Reviewed by Darin.

Removed a number of editing and selection methods from the bridge.


I moved cross-platform editing and selection code into Editor and
SelectionController, respecitvely.

I moved ObjC and AppKit stuff up into WebKit, so I ended up exporting
everything + the kitchen sink.


Specific comments below for interesting things.


  • bindings/objc/DOMInternal.h: Moved exception handling helper methods into a new file so that WebKit can use them, too. Added a helper method for handling exceptions when selecting a Range.
  • bridge/mac/FrameMac.h: Moved attributedString creation and helper functions into WebKit, since they have to do with creating an NSAttributedString for API consumption, and not much to do with general Frame functionality.
  • editing/Editor.h: Moved lastEditCommand tracking (at least the data, for now) into the Editor. Eventually, the Frame will not have to notify the Editor of what the lastEditCommand was, since the Editor will perform all EditCommands.
  • editing/SelectionController.h: The code here is just stuff moved from the bridge.
  • editing/SelectionController.cpp: (WebCore::SelectionController::setSelectedRange): We now explicitly check from DOM exceptions and return them. The bridge method to select a DOM range did this implicitly, since all ObjC DOM operations handle DOM exceptions by throwing them as ObjC exceptions.
  • editing/TextAffinity.h: Added helper functions for conversion to NSSelectionAffinity. The two enumerations are numerically identical, but that's an implementation detail of TextAffinity that clients shouldn't be required to know about.

WebKit:

Reviewed by Darin.

Removed a number of editing and selection methods from the bridge.


I moved cross-platform editing and selection code into WebCore::Editor and
WebCore::SelectionController, respecitvely.


All of the seemingly new code here is just code grabbed from WebCore or
merged from WebCoreFrameBridge.


I changed one piece of internal API: we now pass around Ranges in places
where we used to pass around broken out components of Ranges.


I also added WebCore XPATH_SUPPORT AND SVG_SUPPORT #defines to the project.
Since we now include WebCore headers that depend on these #defines, we need
to keep in sync with them, to avoid binary incompatibility.

4:56 PM Changeset in webkit [17454] by thatcher
  • 2 edits in tags/WebCore-521.29.3/WebCore

Versioning

4:55 PM Changeset in webkit [17453] by thatcher
  • 2 edits in tags/WebCore-521.29.3/WebCore

Merging fix.

4:55 PM Changeset in webkit [17452] by thatcher
  • 2 edits in tags/WebCore-521.29.3/WebCore

Merging fix.

4:52 PM Changeset in webkit [17451] by thatcher
  • 1 copy in tags/WebCore-521.29.3

new tag.

4:49 PM Changeset in webkit [17450] by thatcher
  • 2 edits in trunk/WebCore

Rolling out the following change because this crash:
<rdar://problem/4806705> REGRESSION: Crash occurs at WebCore::Font::lineSpacing() when loading site (​http://www.photoplusexpo.com/ppe/index.jsp)

And this regression:
<rdar://problem/4728514> REGRESSION: Safari applies the wrong font to BODY element at ​http://www.apple.com/downloads/dashboard/

2006-09-06 David Harrison <​harrison@apple.com>

Reviewed and tweaked by Tim H.

<rdar://problem/4564955> WebKit doesn't trigger Auto Font Activation

If we don't find the font in the available fonts list, call [NSFont fontWithName:size:]
to trigger a search that will include auto activation. No PLT or iBench perf impact.
No layout tests affected. Not testable in an automated way that will work on all systems.

  • platform/mac/WebFontCache.mm: (+[WebFontCache fontWithFamily:traits:size:]):
4:48 PM Changeset in webkit [22445] by sfalken
  • 5 edits in branches/WindowsMerge/WebKitWin

2006-10-29 Steve Falkenburg <​sfalken@apple.com>

Reviewed by Adam, Ada.

Expose printing methods in IWebViewPrivate and removed LPARAM-based dispatching.
Call StretchDIBits instead of BitBlt.
Add a maximum DPI (currently set at 300) to speed up printing.
Add support needed for print preview.

  • Interfaces/IWebViewPrivate.idl: added startPrintJob, endPrintJob, getPrintedPageCount, printPage
  • WebView.cpp: Added MAXIMUM_DPI setting to limit the size of print jobs for performance reasons (getPrintRects): Added. Factored out of print, added maximum DPI handling. (WebView::startPrintJob): Factored out of print (WebView::endPrintJob): Factored out of print (WebView::getPrintedPageCount): Factored out of print (WebView::printPage): Factored out of print. Replaced call to BitBlt with StretchDIBits (WebViewWndProc): Re-add support for printing a window (used by bug reporter code)
  • WebView.h: added startPrintJob, endPrintJob, getPrintedPageCount, printPage
3:25 PM Changeset in webkit [17449] by justing
  • 2 edits in trunk/WebCore

Reviewed by harrison

<rdar://problem/4808375>
REGRESSION: TextIterator slowed down, affecting Find on Page & Copy (etc.) (11460)

  • editing/TextIterator.cpp: (WebCore::TextIterator::exitNode): Only create VisiblePositions when we're going to use them to create a range for an emitted character. We should further speed TextIterators up by avoiding creating VisiblePositions for TIs that are only used for the characters they emit (like the one that plainText uses).
2:41 PM Changeset in webkit [17448] by weinig
  • 46 edits in trunk/WebCore

Reviewed by Mitz.

Fix for ​http://bugs.webkit.org/show_bug.cgi?id=11441
More rendering code cleaning

  • WebCore.xcodeproj/project.pbxproj:
  • rendering/RenderApplet.cpp: (WebCore::RenderApplet::RenderApplet): (WebCore::RenderApplet::createWidgetIfNecessary):
  • rendering/RenderApplet.h:
  • rendering/RenderArena.cpp: (WebCore::): (WebCore::RenderArena::RenderArena): (WebCore::RenderArena::allocate): (WebCore::RenderArena::free):
  • rendering/RenderArena.h:
  • rendering/RenderBR.cpp: (WebCore::RenderBR::RenderBR): (WebCore::RenderBR::baselinePosition): (WebCore::RenderBR::lineHeight): (WebCore::RenderBR::setStyle): (WebCore::RenderBR::caretMinOffset): (WebCore::RenderBR::positionForCoordinates): (WebCore::RenderBR::inlineBox):
  • rendering/RenderBR.h:
  • rendering/RenderBlock.cpp:
  • rendering/RenderBlock.h: (WebCore::RenderBlock::maxTopMargin): (WebCore::RenderBlock::maxBottomMargin): (WebCore::RenderBlock::initMaxMarginValues): (WebCore::RenderBlock::containsFloats): (WebCore::RenderBlock::setHasMarkupTruncation): (WebCore::RenderBlock::BlockSelectionInfo::BlockSelectionInfo): (WebCore::RenderBlock::BlockSelectionInfo::block): (WebCore::RenderBlock::BlockSelectionInfo::state): (WebCore::RenderBlock::FloatingObject::FloatingObject): (WebCore::RenderBlock::CompactInfo::clear):
  • rendering/RenderButton.cpp: (WebCore::RenderButton::removeChild): (WebCore::RenderButton::paintObject):
  • rendering/RenderButton.h: (WebCore::RenderButton::renderName): (WebCore::RenderButton::removeLeftoverAnonymousBoxes):
  • rendering/RenderContainer.cpp: (WebCore::RenderContainer::RenderContainer):
  • rendering/RenderContainer.h: (WebCore::RenderContainer::firstChild): (WebCore::RenderContainer::lastChild): (WebCore::RenderContainer::calcMinMaxWidth):
  • rendering/RenderCounter.cpp: (WebCore::RenderCounter::RenderCounter): (WebCore::toRoman): (WebCore::toHebrew): (WebCore::RenderCounter::calcMinMaxWidth):
  • rendering/RenderCounter.h:
  • rendering/RenderFieldset.cpp: (WebCore::RenderFieldset::paintBoxDecorations): (WebCore::RenderFieldset::paintBorderMinusLegend): (WebCore::RenderFieldset::setStyle):
  • rendering/RenderFileUploadControl.cpp: (WebCore::RenderFileUploadControl::~RenderFileUploadControl): (WebCore::RenderFileUploadControl::setStyle): (WebCore::RenderFileUploadControl::paintObject): (WebCore::RenderFileUploadControl::calcMinMaxWidth):
  • rendering/RenderFileUploadControl.h: (WebCore::RenderFileUploadControl::renderName):
  • rendering/RenderFlexibleBox.h:
  • rendering/RenderFlow.cpp:
  • rendering/RenderFlow.h:
  • rendering/RenderForeignObject.cpp: (WebCore::RenderForeignObject::RenderForeignObject): (WebCore::RenderForeignObject::paint): (WebCore::RenderForeignObject::computeAbsoluteRepaintRect): (WebCore::RenderForeignObject::layout): (WebCore::RenderForeignObject::nodeAtPoint):
  • rendering/RenderForeignObject.h: (WebCore::RenderForeignObject::renderName):
  • rendering/RenderFormElement.cpp: (WebCore::RenderFormElement::setStyle): (WebCore::RenderFormElement::layout): (WebCore::RenderFormElement::textAlignment):
  • rendering/RenderFormElement.h:
  • rendering/RenderFrame.cpp:
  • rendering/RenderFrame.h: (WebCore::RenderFrame::element):
  • rendering/RenderFrameSet.cpp:
  • rendering/RenderFrameSet.h: (WebCore::RenderFrameSet::element):
  • rendering/RenderHTMLCanvas.cpp: (WebCore::RenderHTMLCanvas::RenderHTMLCanvas): (WebCore::RenderHTMLCanvas::paint):
  • rendering/RenderHTMLCanvas.h: (WebCore::RenderHTMLCanvas::renderName):
  • rendering/RenderImage.cpp: (WebCore::RenderImage::RenderImage): (WebCore::RenderImage::setStyle): (WebCore::RenderImage::setContentObject): (WebCore::RenderImage::setCachedImage): (WebCore::RenderImage::imageChanged): (WebCore::RenderImage::paint): (WebCore::RenderImage::layout): (WebCore::RenderImage::updateAltText):
  • rendering/RenderImage.h: (WebCore::RenderImage::element):
  • rendering/RenderInline.cpp: (WebCore::RenderInline::RenderInline): (WebCore::RenderInline::~RenderInline): (WebCore::RenderInline::setStyle): (WebCore::RenderInline::addChildToFlow): (WebCore::RenderInline::cloneInline): (WebCore::RenderInline::splitInlines): (WebCore::RenderInline::splitFlow): (WebCore::RenderInline::paint): (WebCore::RenderInline::absoluteRects): (WebCore::RenderInline::calcMinMaxWidth): (WebCore::RenderInline::requiresLayer): (WebCore::RenderInline::width): (WebCore::RenderInline::height): (WebCore::RenderInline::renderName): (WebCore::RenderInline::nodeAtPoint): (WebCore::RenderInline::positionForCoordinates):
  • rendering/RenderInline.h: (WebCore::RenderInline::layout):
  • rendering/RenderLayer.cpp:
  • rendering/RenderLayer.h: (WebCore::ClipRects::ClipRects): (WebCore::RenderLayer::nextSibling): (WebCore::RenderLayer::root): (WebCore::RenderLayer::setPos): (WebCore::RenderLayer::height): (WebCore::RenderLayer::relativePositionOffset):
  • rendering/RenderLineEdit.cpp: (WebCore::RenderLineEdit::setStyle): (WebCore::RenderLineEdit::updateFromElement): (WebCore::RenderLineEdit::selectionStart): (WebCore::RenderLineEdit::selectionEnd): (WebCore::RenderLineEdit::setSelectionStart): (WebCore::RenderLineEdit::setSelectionEnd): (WebCore::RenderLineEdit::setSelectionRange):
  • rendering/RenderLineEdit.h:
  • rendering/RenderListBox.cpp:
  • rendering/RenderListBox.h: (WebCore::RenderListBox::renderName):
  • rendering/RenderListItem.cpp: (WebCore::RenderListItem::setStyle): (WebCore::getParentOfFirstLineBox): (WebCore::RenderListItem::updateMarkerLocation): (WebCore::RenderListItem::positionListMarker): (WebCore::RenderListItem::paint):
  • rendering/RenderListItem.h:
  • rendering/RenderListMarker.cpp:
  • rendering/RenderListMarker.h: (WebCore::RenderListMarker::renderName): (WebCore::RenderListMarker::isListMarker):
2:35 PM Changeset in webkit [17447] by slewis
  • 5 edits in trunk

Reviewed by Darin.

Change Makefiles to return non-zero when module make fails.

  • Makefile:

WebKitTools:

Change Makefiles to return non-zero when module make fails.
Fix bug where if xcode options are not set, modules can build in the
wrong directory.

  • Makefile:
  • Scripts/webkitdirs.pm:
2:29 PM Changeset in webkit [22444] by andersca
  • 9 edits
    1 add in branches/WindowsMerge

WebCoreWin:

2006-10-30 Anders Carlsson <​acarlsson@apple.com>

Reviewed by Adam.

If there's no plug-in for a given mime type, or if the plug-in fails
to initialize, draw the "missing plug-in" icon.

  • plugins/win/PluginDatabaseWin.cpp: (WebCore::PluginDatabaseWin::createPluginView): If no plug-in can be found, create a null plugin view.


  • plugins/win/PluginViewWin.cpp: (WebCore::PluginViewWin::paint): If the plugin hasn't been started, paint the missing plugin image.


(WebCore::PluginViewWin::setNPWindowSize):
Return early if the plugin hasn't been started.


(WebCore::PluginViewWin::start):
Return if the plugin couldn't be created.


(WebCore::PluginViewWin::createNullPluginView):
New function which just creates a plugin view and doesn't try to start its plugin
(because there is none).


(WebCore::PluginViewWin::PluginViewWin):
New constructor for the null plugin view.


WebKitWin:

2006-10-30 Anders Carlsson <​acarlsson@apple.com>

Reviewed by Adam.

Add "null plugin" image resource.


2:25 PM Changeset in webkit [17446] by thatcher
  • 1 edit in trunk/WebCore/editing/DeleteButtonController.cpp

Build fix.

2:19 PM Changeset in webkit [17445] by thatcher
  • 2 edits in trunk/WebCore

Reviewed by Hyatt.

<rdar://problem/4478625> Basic table editing and culling

Refined the criteria for deletable elements, rely on the renderer more.
Corrected the interface positioning for elements that have borders.

  • editing/DeleteButtonController.cpp: (WebCore::isDeletableElement): (WebCore::DeleteButtonController::show):
1:34 PM Changeset in webkit [17444] by kmccullo
  • 3 edits in trunk/JavaScriptCore

Reviewed by Adam.

  • Fix some timezone issues and JavaScriptCore date tests. Addresses bugzilla 4930.
  • kjs/DateMath.h: (KJS::GregorianDateTime::GregorianDateTime): Here's the fix, to add parenthesis for order of precedence.
  • kjs/date_object.cpp: (KJS::DateProtoFunc::callAsFunction): (KJS::DateObjectImp::construct): memset not needed as GregorianDateTime initializes itself.
12:48 PM Changeset in webkit [17443] by darin
  • 2 edits in trunk/WebCore
  • fix build
  • loader/ResourceLoader.h: Added header guard.
12:08 PM Changeset in webkit [22443] by darin
  • 2 edits in branches/WindowsMerge/WebCore

Reviewed by Adam.

  • WebCore.vcproj/WebCore.vcproj: Added new source files from the loader directory.
12:05 PM Changeset in webkit [17442] by darin
  • 15 edits in trunk/WebCore

Reviewed by Adam.

  • fix FrameLoader files so they can be built on non-Mac platforms
  • page/FramePrivate.h: (WebCore::FramePrivate::FramePrivate): Added code to create the frame loader. (WebCore::FramePrivate::~FramePrivate): Added code to delete the frame loader.
  • page/Page.h:
  • page/Page.cpp: (WebCore::Page::setDefersLoading): Removed the #if PLATFORM(MAC) that Adam added as a stop-gap when I did this wrong the other day.
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::FrameMac): Removed code to create the frame loader. (WebCore::FrameMac::~FrameMac): Removed code to delete the frame loader.
  • loader/FrameLoader.h: Changed import to include, added #if PLATFORM(MAC) around the PolicyCheck definition.
  • loader/FormState.cpp: Changed import to include.
  • loader/FrameLoader.cpp: Ditto.
  • loader/MainResourceLoader.h: Ditto.
  • loader/FrameLoaderTypes.h: Added a header guard.
  • loader/NetscapePlugInStreamLoader.h: Changed import to include. Added #if PLATFORM(MAC) around Mac-specific details.
  • loader/SubresourceLoader.h: Ditto.
  • page/Frame.cpp: (WebCore::Frame::loader): Updated name of loader to m_loader from m_frameLoader.
  • CMakeLists.txt: Updated.
  • WebCoreSources.bkl: Updated.
11:53 AM Changeset in webkit [17441] by andersca
  • 2 edits in trunk/WebCore

2006-10-30 Nikolas Zimmermann <​zimmermann@kde.org>

Reviewed and landed by Anders.

Patch by Zack rusin to fix:
​http://bugs.webkit.org/show_bug.cgi?id=11429

Fix startup crash.

  • platform/qt/FrameQtClient.cpp: (WebCore::FrameQtClientDefault::openURL):
11:38 AM Changeset in webkit [17440] by andersca
  • 4 edits in trunk/WebCore

2006-10-30 Zach Rusin <​zack@kde.org>

Reviewed and landed by Anders.

  • platform/network/qt/ResourceHandleQt.cpp: (WebCore::ResourceHandle::receivedResponse):
  • platform/qt/FrameQtClient.cpp: (WebCore::FrameQtClientDefault::FrameQtClientDefault):
  • platform/qt/LoaderFunctionsQt.cpp: (WebCore::ServeSynchronousRequest): More build fixes.

2006-10-30 Simon Hausmann <​hausmann@kde.org>

Reviewed and landed by Anders.


  • fix Qt build
  • platform/qt/FrameQtClient.cpp: ResourceLoaderClient -> ResourceHandleClient
11:32 AM Changeset in webkit [17439] by darin
  • 2 edits in trunk/LayoutTests

Requested by Maciej.

  • fast/dom/xmlhttprequest-get-expected.txt: Generated a new result. Maciej says that the new behavior of returning empty string instead of undefined is correct and matches Firefox, so the new result is better than the old.
9:20 AM Changeset in webkit [17438] by darin
  • 25 edits
    3 adds in trunk

WebCore:

Reviewed by John Sullivan.

  • work toward removing Objective-C data types from FrameLoader.h: removed NSDate, NSString, WebCorePageState, WebCoreResourceLoader, and WebCoreResourceHandle
  • moved bodyBackgroundColor function from Frame to WebFrame in WebKit
  • page/PageState.h:
  • page/PageState.cpp: Added, C++ class with much of what WebCorePageState had.
  • bridge/mac/FrameMac.h: Removed uneeeded declarations and bodyBackgroundColor().
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::startRedirectionTimer): Use a double for the date instead of an NSDate.
  • bridge/mac/WebCoreFrameBridge.h:
  • bridge/mac/WebCoreFrameBridge.mm: Removed invalidatePageCache and saveDocumentToPageCache methods.
  • bridge/mac/WebCorePageState.h:
  • bridge/mac/WebCorePageState.mm: Removed most of the class and turned it into an Objective-C wrapper for PageState.
  • loader/FrameLoader.h: Removed unused Objective-C types, eliminated use of NSDate and WebCorePageState.
  • loader/FrameLoaderClient.h: Changed NSDate to double.
  • loader/mac/FrameLoaderMac.mm: (WebCore::FrameLoader::receivedMainResourceError): Moved call to setInPageCache here that used to be in invalidateCurrentItemPageCache on the WebKit side. (WebCore::FrameLoader::clientRedirected): Changed NSDate to double for date. (WebCore::FrameLoader::open): Moved one of the open functions that was part of commitProvisionalLoad inside the commitProvisionalLoad function, so that it can extract the parameters from the response. Changed the open function that takes a PageState to use the new C++ PageState. (WebCore::FrameLoader::commitProvisionalLoad): Added the code from the open function; tightened up logic, removing a few cases that could never happen, as well as the unused reload boolean.
  • ForwardingHeaders/kjs/property_map.h: Added.
  • WebCore.exp: Updated.
  • WebCore.xcodeproj/project.pbxproj: Updated.

WebKit:

Reviewed by John Sullivan.

  • work toward removing Objective-C data types from FrameLoader.h: removed NSDate, NSString, WebCorePageState, WebCoreResourceLoader, and WebCoreResourceHandle
  • moved bodyBackgroundColor function here from Frame
  • History/WebHistoryItem.m: (+[WebHistoryItem _closeObjectsInPendingPageCaches]): Updated for change in WebCorePageState.
  • WebCoreSupport/WebFrameBridge.mm: Removed saveDocumentToPageCache method.
  • WebCoreSupport/WebFrameLoaderClient.h: Changed NSDate to double.
  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::invalidateCurrentItemPageCache): Moved the code that was formerly in invalidatePageCache: on the bridge here. (WebFrameLoaderClient::dispatchWillPerformClientRedirect): Added code to make the NSDate here. (WebFrameLoaderClient::createPageCache): Restructured code to create the WebCorePageState object directly instead of calling saveDocumentToPageCache on the bridge.
  • WebView/WebFrame.mm: (-[WebFrame _bodyBackgroundColor]): Rewrote this to work directly with the DOM and renderers rather than using a function on Frame.
9:18 AM Changeset in webkit [17437] by darin
  • 3 edits in trunk/JavaScriptCore

Reviewed by John Sullivan.

  • kjs/SavedBuiltins.h: Added needed include.
  • wtf/OwnPtr.h: (WTF::OwnPtr::set): Fixed mistake in assertion.
8:27 AM Changeset in webkit [17436] by thatcher
  • 5 edits in trunk/WebKitTools

Reviewed by Tim H.

Fix for ​http://bugs.webkit.org/show_bug.cgi?id=10468
[Drosera] The Console toolbar button should bring the console window to the front.

The console now gets focus when its activated but already open. Command + L also
now activates/focuses the console.

  • Drosera/English.lproj/MainMenu.nib/classes.nib:
  • Drosera/English.lproj/MainMenu.nib/info.nib:
  • Drosera/English.lproj/MainMenu.nib/keyedobjects.nib:
  • Drosera/debugger.js:
3:12 AM Changeset in webkit [22442] by mjs
  • 8 edits in branches/WindowsMerge

WebCoreWin:

Reviewed by Anders.

  • added ResourceResponse class and didReceiveResponse delegate call
  • WebCore.vcproj/WebCore.vcproj: Added new files.
  • platform/win/TemporaryLinkStubs.cpp: (WebCore::ServeSynchronousRequest): Changed prototype to match new version.

WebKitWin:

Reviewed by Anders.

  • added ResourceResponse class and didReceiveResponse delegate call
  • WebFrame.cpp: (WebFrame::didReceiveResponse): Updated from receivedResponse.
  • WebFrame.h:
  • WebURLResponse.cpp: (WebURLResponse::WebURLResponse): Reimplemented to work in terms of WebCore::ResourceResponse (ditto for below). Uses BString to convert strings as needed. (WebURLResponse::~WebURLResponse): (WebURLResponse::createInstance): (WebURLResponse::expectedContentLength): (WebURLResponse::initWithURL): (WebURLResponse::MIMEType): (WebURLResponse::suggestedFilename): (WebURLResponse::textEncodingName): (WebURLResponse::URL): (WebURLResponse::suggestedFileExtension):
  • WebURLResponse.h:
3:08 AM Changeset in webkit [17435] by mjs
  • 25 edits
    5 adds in trunk

WebCore:

Reviewed by Anders.

  • added ResourceResponse class and didReceiveResponse delegate call


  • WebCore.xcodeproj/project.pbxproj: Added new files.
  • bridge/mac/WebCoreFrameBridge.h: Removed no longer needed expiresTimeForResponse: method.
  • dom/XMLTokenizer.cpp: (WebCore::openFunc): Updated to use ResourceResponse.
  • loader/FrameLoader.h: Update loadResourceSynchronously to use ResourceResponse.
  • loader/LoaderFunctions.h:
  • loader/icon/IconLoader.cpp: (IconLoader::didReceiveResponse): Updated from receivedResponse and made cross-platform.
  • loader/icon/IconLoader.h: updated for above change
  • loader/loader.cpp: (WebCore::Loader::receivedResponse): reduced to just saving the PlatformResponse (ideally we'd phase out use of this). (WebCore::Loader::didReceivedResponse): most of the ReceivedResponse logic.
  • loader/loader.h:
  • loader/mac/FrameLoaderMac.mm: (WebCore::FrameLoader::loadResourceSynchronously): Return results as ResourceResponse.
  • loader/mac/IconLoaderMac.mm: remove receivedResponse.
  • loader/mac/LoaderFunctionsMac.mm: (WebCore::ServeSynchronousRequest): take ResourceResponse as argument.
  • platform/network/ResourceHandle.cpp:
  • platform/network/ResourceHandle.h: Remove calls for broken out response pieces.
  • platform/network/ResourceHandleClient.h: (WebCore::ResourceHandleClient::didReceiveResponse): New delegate method.
  • platform/network/ResourceHandleInternal.h: (WebCore::ResourceHandleInternal::ResourceHandleInternal): Remove storage of response bits, we're not gonna return them seprately any more.
  • platform/network/ResourceResponse.h: Added. (WebCore::ResourceResponse::ResourceResponse): Freshly implemented. Same for below. (WebCore::ResourceResponse::url): (WebCore::ResourceResponse::mimeType): (WebCore::ResourceResponse::expectedContentLength): (WebCore::ResourceResponse::textEncodingName): (WebCore::ResourceResponse::suggestedFilename): (WebCore::ResourceResponse::httpStatusCode): (WebCore::ResourceResponse::setHTTPStatusCode): (WebCore::ResourceResponse::httpHeaderField): (WebCore::ResourceResponse::httpHeaderFields): (WebCore::ResourceResponse::isMultipart): (WebCore::ResourceResponse::setExpirationDate): (WebCore::ResourceResponse::expirationDate):
  • platform/network/cf/ResourceHandleCFNet.cpp: (WebCore::didReceiveResponse): Send didReceiveResponse client method.
  • platform/network/cf/ResourceResponseCFNet.h: Added.
  • platform/network/cf/ResourceResponseCFNet.mm: Added. (getResourceResponse): Helper function to get a ResourceResponse from CFURLResponseRef.
  • platform/network/mac/ResourceHandleMac.mm: (WebCore::ResourceHandleInternal::~ResourceHandleInternal): Don't bother cleaning up fields we no longer have. (WebCore::ResourceHandle::receivedResponse): Send didReceiveResponse client method.
  • platform/network/mac/ResourceResponseMac.h: Added.
  • platform/network/mac/ResourceResponseMac.mm: Added. (getResourceResponse): Helper function to get a ResourceResponse from an NSURLResponse*.
  • rendering/HitTestResult.cpp:
  • xml/XSLTProcessor.cpp: (WebCore::docLoaderFunc): Prepare for use of ResourceResponse.
  • xml/xmlhttprequest.cpp: (WebCore::XMLHttpRequest::getResponseText): Store http response info in the form of a ResourceResponse, change all methods to account for this. (WebCore::XMLHttpRequest::getResponseXML): (WebCore::XMLHttpRequest::XMLHttpRequest): (WebCore::XMLHttpRequest::open): (WebCore::XMLHttpRequest::send): (WebCore::XMLHttpRequest::getAllResponseHeaders): (WebCore::XMLHttpRequest::getResponseHeader): (WebCore::XMLHttpRequest::getStatus): (WebCore::XMLHttpRequest::getStatusText): (WebCore::XMLHttpRequest::processSyncLoadResults): (WebCore::XMLHttpRequest::didFinishLoading): (WebCore::XMLHttpRequest::didReceiveResponse): (WebCore::XMLHttpRequest::didReceiveData):
  • xml/xmlhttprequest.h:

WebKit:

Reviewed by Anders.

  • added ResourceResponse class and didReceiveResponse delegate call
  • WebCoreSupport/WebFrameBridge.mm: Removed no longer needed expiresTimeForResponse: method.
Note: See TracTimeline for information about the timeline view.