Timeline



Nov 2, 2006:

10:41 PM Changeset in webkit [17562] by ggaren
  • 418 edits in trunk

LayoutTests:

Reviewed by Justin Garcia.


Used a PERL script to add calls to opt into dumping of editing callbacks
in all editing tests that didn't do so already. This is required now that
dumping of editing callbacks is opt-in.


Updated editing.js so that any test that calls runEditingTest() opt into
dumping of editing callbacks.


Updated results in all non-editing tests to remove dumping of editing callbacks.


Updated results in all editing tests to reflect minor DOM changes due to new
<script> elements.


[ many tests and results modified ]

WebKitTools:

Reviewed by Justin Garcia.


Made dumping of editing callbacks opt-in, so that editing spew doesn't
cloud non-editing tests.

  • DumpRenderTree/DumpRenderTree.m: (+[LayoutTestController isSelectorExcludedFromWebScript:]): (-[LayoutTestController shouldDumpEditingCallbacks]): (runTest):
  • DumpRenderTree/EditingDelegate.m: (-[EditingDelegate webView:shouldBeginEditingInDOMRange:]): (-[EditingDelegate webView:shouldEndEditingInDOMRange:]): (-[EditingDelegate webView:shouldInsertNode:replacingDOMRange:givenAction:]): (-[EditingDelegate webView:shouldInsertText:replacingDOMRange:givenAction:]): (-[EditingDelegate webView:shouldDeleteDOMRange:]): (-[EditingDelegate webView:shouldChangeSelectedDOMRange:toDOMRange:affinity:stillSelecting:]): (-[EditingDelegate webView:shouldApplyStyle:toElementsInDOMRange:]): (-[EditingDelegate webView:shouldChangeTypingStyle:toStyle:]): (-[EditingDelegate webViewDidBeginEditing:]): (-[EditingDelegate webViewDidChange:]): (-[EditingDelegate webViewDidEndEditing:]): (-[EditingDelegate webViewDidChangeTypingStyle:]): (-[EditingDelegate webViewDidChangeSelection:]):
10:02 PM Changeset in webkit [17561] by ggaren
  • 2 edits in trunk/LayoutTests

Updated results because they were incorrect when checked in. (We never
passed this test.)


  • fast/js/regexp-unicode-handling-expected.txt:
9:07 PM Nice Bugzilla queries edited by anonymous
(diff)
9:05 PM WebKit plus SVG edited by trac
(diff)
9:03 PM WebKit plus SVG edited by trac
(diff)
9:02 PM WebKit plus SVG edited by trac
(diff)
8:54 PM WebKit Team edited by trac
(diff)
8:54 PM WebKit plus SVG edited by trac
(diff)
8:48 PM Nice Bugzilla queries edited by trac
(diff)
8:42 PM WebKit plus SVG edited by trac
(diff)
8:39 PM WebKit plus SVG edited by trac
(diff)
8:39 PM WebKit Team edited by trac
(diff)
8:39 PM Applications using WebKit edited by trac
(diff)
8:39 PM Known incompatibilities between open-source WebKit and Safari edited by trac
(diff)
8:35 PM WebKit plus SVG created by trac
8:35 PM WebKit Team created by trac
8:34 PM Known incompatibilities between open-source WebKit and Safari created by trac
8:34 PM Nice Bugzilla queries created by trac
8:34 PM Nightly Builds created by trac
8:34 PM Applications using WebKit created by trac
8:29 PM Changeset in webkit [17560] by adele
  • 3 edits
    2 adds in trunk

LayoutTests:

Reviewed by Anders.

Test for http://bugs.webkit.org/show_bug.cgi?id=9581
REGRESSION: The new NativeTextArea scrolls to the top when the control is unfocused.

  • fast/forms/textarea-no-scroll-on-blur-expected.txt: Added.
  • fast/forms/textarea-no-scroll-on-blur.html: Added.

WebCore:

Reviewed by Anders.

Test: fast/forms/textarea-no-scroll-on-blur.html

  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::forwardEvent): On blur, only scroll back to the start for text fields.
7:37 PM Changeset in webkit [17559] by andersca
  • 2 edits in trunk/WebCore

Another build fix.


  • rendering/RenderFileUploadControl.cpp: (WebCore::RenderFileUploadControl::paintObject):
7:33 PM Changeset in webkit [17558] by andersca
  • 2 edits in trunk/WebCore

Build fix.


  • page/FramePrivate.h: (WebCore::FramePrivate::FramePrivate):
7:27 PM Changeset in webkit [17557] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Geoff, landed by Anders.


Fix dependency problems caused when running move-js-headers.sh


  • move-js-headers.sh: Use cp -p instead of just cp so that the modification time of the original files are used.
7:23 PM Changeset in webkit [17556] by andersca
  • 1 edit
    6 adds in trunk/LayoutTests

2006-11-02 W. Andy Carrel <wac@google.com>

Reviewed by Maciej, landed by Anders.


Adding tests for Unicode RegExp behavior to match IE and Firefox.

  • fast/js/regexp-unicode-handling-expected.txt: Added.
  • fast/js/regexp-unicode-handling.html: Added.
  • fast/js/resources/regexp-unicode-handling.js: Added.

2006-11-02 Alexey Proskuryakov <ap@nypop.com>

Reviewed by Maciej, landed by Anders.

Test for bugs 7253 and 7445, based on reductions made by Parag Shah and W. Andy Carrel.

  • fast/js/gmail-re-re-expected.txt: Added.
  • fast/js/gmail-re-re.html: Added.
  • fast/js/resources/gmail-re-re.js: Added.
7:17 PM Changeset in webkit [17555] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Tim H, landed by Anders.

Fixes http://bugzilla.opendarwin.org/show_bug.cgi?id=10840
REGRESSION: Shadow of file upload button is clipped

  • rendering/RenderFileUploadControl.cpp: (WebCore::RenderFileUploadControl::paintObject): Add 2px to the clip height to keep from clipping in the shadow
7:16 PM Changeset in webkit [17554] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Maciej, landed by Anders.

  • platform/qt/ScrollViewQt.cpp: Fix translation of coordinates between content and window. ScrollViewQt already receives correct content coordinates from QScrollArea. (WebCore::ScrollView::contentsToWindow): (WebCore::ScrollView::windowToContents):
7:14 PM Changeset in webkit [17553] by andersca
  • 7 edits in trunk

JavaScriptCore:

Reviewed by Maciej, landed by Anders.

  • CMakeLists.txt: Make KDE support optional.

WebKitQt:

Reviewed by Maciej, landed by Anders.

  • QtLauncher/CMakeLists.txt: Make linkage against kde libraries optional
  • QtLauncher/main.cpp: Make it compile without KDE. (main):
7:11 PM Changeset in webkit [17552] by andersca
  • 6 edits in trunk/WebCore

Reviewed by Maciej, landed by Anders

  • CMakeLists.txt: Make linkage against KDE libraries conditional
  • platform/network/qt/ResourceHandleManager.cpp: Added a simple Qt base resource handler that supports only requests to the local filesystem. Used when compiling without KDE support. ResourceHandleManager.cpp/h are to be split up into ResourceHandleManagerKDE and ResourceHandleManagerQt in the future, as well as QtJob.cpp/h. (WebCore::QtJob::QtJob): (WebCore::QtJob::timerEvent): (WebCore::ResourceHandleManager::ResourceHandleManager): (WebCore::ResourceHandleManager::~ResourceHandleManager): (WebCore::ResourceHandleManager::self): (WebCore::ResourceHandleManager::remove): (WebCore::ResourceHandleManager::add): (WebCore::ResourceHandleManager::cancel): (WebCore::ResourceHandleManager::deliverJobData):
  • platform/network/qt/ResourceHandleManager.h:
  • platform/qt/FrameQtClient.cpp: (WebCore::FrameQtClientDefault::runJavaScriptAlert): (WebCore::FrameQtClientDefault::runJavaScriptConfirm): (WebCore::FrameQtClientDefault::runJavaScriptPrompt):
  • platform/qt/LoaderFunctionsQt.cpp: Use the Qt messagebox and input dialog functions when compiling without KDE support (WebCore::ServeSynchronousRequest):
7:09 PM Changeset in webkit [17551] by andersca
  • 3 edits in trunk/WebKitTools

2006-11-02 Alexey Proskuryakov <ap@nypop.com>

Reviewed by Maciej, landed by Anders.

  • Scripts/webkitdirs.pm: Make Windows build work with Visual C++ Express.
  • Scripts/install-win-extras: Make setx.exe actually run.
7:06 PM Changeset in webkit [17550] by andersca
  • 2 edits in trunk/WebCore

2006-11-02 David Carson <dacarson@gmail.com>

Reviewed by Geoff, landed by Anders.

Fix for: http://bugs.webkit.org/show_bug.cgi?id=11471
Initializing variable in constructor.

  • page/FramePrivate.h: (WebCore::FramePrivate::FramePrivate):
5:30 PM Changeset in webkit [17549] by andersca
  • 2 edits in trunk/WebCore

2006-11-02 Anders Carlsson <acarlsson@apple.com>

Reviewed by Geoff.

Use CFMutableURLRequestRef instead of CFHTTPMessageRef since not all URL requests are http requests.


  • platform/network/cf/ResourceHandleCFNet.cpp: (WebCore::addHeadersFromHashMap): Don't set all headers at once since that clears any previous headers set.


(WebCore::ResourceHandle::start):

4:27 PM Changeset in webkit [17548] by thatcher
  • 1 edit in trunk/WebKit/WebView/WebViewPrivate.h

Build fix.

4:12 PM Changeset in webkit [17547] by thatcher
  • 4 edits in trunk/WebKit

Reviewed by Brady.

Adding outdent to the WebView responder forwarding list.
Also add outdent to WebHTMLView.h and WebViewPrivate.h.

  • WebView/WebHTMLView.h:
  • WebView/WebViewPrivate.h:
  • WebView/WebView.mm:
3:00 PM Changeset in webkit [22462] by andersca
  • 4 edits in branches/WindowsMerge/WebCore

2006-11-02 Anders Carlsson <acarlsson@apple.com>

Reviewed by Oliver.

  • WebCore.vcproj/WebCore.vcproj: Use the forked CursorWin.cpp


  • platform/win/CursorWin.cpp: (WebCore::moveCursor): Implement this.


  • platform/win/TemporaryLinkStubs.cpp: Get rid of it here.
2:24 PM Changeset in webkit [22461] by lamadio
  • 1 add in branches/WindowsMerge/WebCore/platform/network/win/CookieJarWin.cpp

mirroring CookieJar - porting to CF in next change.

2:12 PM Changeset in webkit [17546] by spadma
  • 3 edits in S60/branches/3.1m/WebKit

2006-11-02 sareen <shyam.sareen@nokia.com>

Reviewed by Sachin Padma.
DESC:
http://bugzilla.opendarwin.org/show_bug.cgi?id=11473

  • ResourceLoader/src/HttpDownloadObserver.cpp: (CHttpDownloadObserver::InitDownloadMgrL):
  • ResourceLoader/src/HttpLoaderAcceptHeader.cpp: (CHttpLoaderAcceptHeader::ReadMimeTypesL):
2:08 PM Changeset in webkit [17545] by ggaren
  • 2 edits in trunk/LayoutTests

Updated results to fix failure seen when running layout tests. Justin
Garcia says the new results look more correct.


  • editing/selection/drag-to-contenteditable-iframe-expected.txt:
1:47 PM Changeset in webkit [17544] by spadma
  • 2 edits in S60/branches/3.1m/WebKit

2006-10-31 zhalu <zhaosong.lu@nokia.com>

Reviewed by Yongjun.
DESC: Browser defaults font family to "Nokia Sans S60" which is not supported under variant 7 build. The system has to default to another font family. If the phone language is English, the system defaults the font family to "Series 60 Hindi" which contains fonts for Greek. If the phone language is set to Greek, the system default the font to "Series 60 ZDigi" which does not contains fonts for Greek. That is why the same variant 7 build supports Greek under English and Automatic Language settings while it does not under Greek language setting. My fix is setting the font family to "Series 60 Hindi" if the phone language is set to Geek language.


http://bugzilla.opendarwin.org/show_bug.cgi?id=11469

  • BrowserCore/Renderers/src/TextRendererFactory.cpp: (CTextRendererFactory::DeviceDefaultFontFamilies):


Merge from ccb revision 17537

1:37 PM Changeset in webkit [17543] by adele
  • 5 edits
    4 adds in trunk

LayoutTests:

Reviewed by Mitz and Geoff.

Test for <rdar://problem/4650271> REGRESSION(NativeTextArea): Textareas don't get scrollbars when text gets too big for content area (10105)

  • fast/forms/textarea-scrollbar-expected.checksum: Added.
  • fast/forms/textarea-scrollbar-expected.png: Added.
  • fast/forms/textarea-scrollbar-expected.txt: Added.
  • fast/forms/textarea-scrollbar.html: Added.

WebCore:

Reviewed by Mitz and Geoff.

  • Fix for: <rdar://problem/4650271> REGRESSION(NativeTextArea): Textareas don't get scrollbars when text gets too big for content area (10105) <rdar://problem/4650813> REGRESSION(tiger-leopard): typing in a textarea in Safari is extremely slow (sample shows focus ring drawing) <rdar://problem/4658779> REGRESSION: Text selection is weird in textareas in Trac wiki editing pages

Test: fast/forms/textarea-scrollbar.html

To avoid some of our flexbox bugs for textareas, we're moving the text controls back to RenderBlock. This should make them
a lot more stable. In the future, when we've worked out more of the flexbox kinks, we may want to consider moving them back.

  • rendering/RenderTextControl.h: Convert text controls back to RenderBlock instead of RenderFlexibleBox.
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::RenderTextControl): (WebCore::RenderTextControl::setStyle): (WebCore::RenderTextControl::createDivStyle): (WebCore::RenderTextControl::updateFromElement): (WebCore::RenderTextControl::calcHeight): (WebCore::RenderTextControl::baselinePosition): (WebCore::RenderTextControl::nodeAtPoint): (WebCore::RenderTextControl::layout): Set the inner div's height explicitly before doing a normal layout. (WebCore::RenderTextControl::scrollWidth): (WebCore::RenderTextControl::scrollHeight): (WebCore::RenderTextControl::scrollLeft): (WebCore::RenderTextControl::scrollTop):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::markContainingBlocksForLayout): Make subtree layout optimization work for textareas.
1:36 PM Changeset in webkit [22460] by andersca
  • 1 edit
    1 add in branches/WindowsMerge/WebCore

2006-11-02 Anders Carlsson <acarlsson@apple.com>

Reviewed by Oliver.

Add forked copy of CursorWin.cpp


  • platform/win/CursorWin.cpp: Added.
10:50 AM Changeset in webkit [17542] by thatcher
  • 2 edits in tags/WebKit-521.29.2/WebKit

Versioning

10:48 AM Changeset in webkit [17541] by thatcher
  • 2 edits in tags/WebKit-521.29.2/WebKit

merging fix for <rdar://problem/4801351>

10:43 AM Changeset in webkit [17540] by thatcher
  • 1 copy in tags/WebKit-521.29.2

New tag.

10:12 AM Changeset in webkit [17539] by sullivan
  • 2 edits in trunk/WebCore

Reviewed by Geoff Garen

  • editing/Selection.cpp: (WebCore::Selection::showTreeForThis): this debugging method now displays start offset and end offset, at Darin's suggestion
9:44 AM Changeset in webkit [17538] by kmccullo
  • 3 edits in trunk/LayoutTests

Reviewed by Maciej.

  • Changed the test to take into account the JavaScript standard's interesting take on DST. According to the standard DST should not reflect historically accurate information but should rather interpolate from a comparable year. This test used to show historically accurate information but now reflects the results of interpolating.
  • fast/js/date-big-setdate-expected.txt:
  • fast/js/resources/date-big-setdate.js:
8:38 AM Changeset in webkit [17537] by yongjzha
  • 2 edits in S60/trunk/WebKit

2006-10-31 zhalu <zhaosong.lu@nokia.com>

Reviewed by <yongjun.zhang@nokia.com>
DESC: Browser defaults font family changes for Greek build


http://bugzilla.opendarwin.org/show_bug.cgi?id=11469

  • BrowserCore/Renderers/src/TextRendererFactory.cpp: (CTextRendererFactory::DeviceDefaultFontFamilies):
8:13 AM Changeset in webkit [17536] by spadma
  • 3 edits in S60/branches/3.1m/WebCore

2006-11-01 w3liu <wei.liu@nokia.com>

Reviewed by <yongjun.zhang@nokia.com>
DESC: Thai text rendering fails when using HTTPS.
http://bugs.webkit.org/show_bug.cgi?id=11486
WARNING: NO TEST CASES ADDED OR CHANGED

  • ChangeLog:
  • kwq/KWQTextCodec.cpp: (KWQTextDecoder::KWQTextDecoder): (KWQTextDecoder::~KWQTextDecoder): (QTextCodec::~QTextCodec): (QTextCodec::makeDecoder): (QTextCodec::toUnicode): (KWQTextDecoder::toUnicode): (QTextCodec::PrepareForExit):
  • kwq/KWQTextCodec.h:


Merge from r17535 from main branch

8:04 AM Changeset in webkit [17535] by yongjzha
  • 3 edits in S60/trunk/WebCore

2006-11-01 w3liu <wei.liu@nokia.com>

Reviewed by <yongjun.zhang@nokia.com>
DESC: handle truncated text fragments as input string
http://bugs.webkit.org/show_bug.cgi?id=11486
Thai text rendering fails when using HTTPS

WARNING: NO TEST CASES ADDED OR CHANGED

  • kwq/KWQTextCodec.cpp: (KWQTextDecoder::KWQTextDecoder): (QTextCodec::~QTextCodec): (QTextCodec::makeDecoder): (QTextCodec::toUnicode): (KWQTextDecoder::toUnicode):
  • kwq/KWQTextCodec.h:

Nov 1, 2006:

7:59 PM Changeset in webkit [17534] by slewis
  • 3 edits in trunk/WebKitTools

2006-11-01 Stephanie Lewis <Stephanie Lewis>

Reviewed by Kevin.


Fixed bug where additional arguments got fed to xcode and jsdriver.pl
Since testkjs is being built before the tests are run, we don't need
to build it with webkit.


Fixes bugs
http://bugs.webkit.org/show_bug.cgi?id=11462
http://bugs.webkit.org/show_bug.cgi?id=6168

  • Scripts/build-webkit:
  • Scripts/run-javascriptcore-tests:
7:56 PM Changeset in webkit [17533] by thatcher
  • 2 edits in tags/WebCore-521.29.4/WebCore

Versioning

7:52 PM Changeset in webkit [17532] by justing
  • 10 edits
    16 adds in trunk

LayoutTests:

Reviewed by darin


<rdar://problem/4062865>
Copy/paste of a select element fails to include the options

Updated to reflect fix:

  • editing/pasteboard/4641033-expected.checksum:
  • editing/pasteboard/4641033-expected.png:
  • editing/pasteboard/4641033-expected.txt:
  • editing/pasteboard/4641033.html:


Added:

  • editing/inserting/before-after-input-element-expected.checksum: Added.
  • editing/inserting/before-after-input-element-expected.png: Added.
  • editing/inserting/before-after-input-element-expected.txt: Added.
  • editing/inserting/before-after-input-element.html: Added.
  • editing/pasteboard/input-field-1-expected.checksum: Added.
  • editing/pasteboard/input-field-1-expected.png: Added.
  • editing/pasteboard/input-field-1-expected.txt: Added.
  • editing/pasteboard/input-field-1.html: Added.
  • editing/pasteboard/select-element-1-expected.checksum: Added.
  • editing/pasteboard/select-element-1-expected.png: Added.
  • editing/pasteboard/select-element-1-expected.txt: Added.
  • editing/pasteboard/select-element-1.html: Added.
  • editing/selection/select-element-paragraph-boundary-expected.checksum: Added.
  • editing/selection/select-element-paragraph-boundary-expected.png: Added.
  • editing/selection/select-element-paragraph-boundary-expected.txt: Added.
  • editing/selection/select-element-paragraph-boundary.html: Added.

WebCore:

Reviewed by darin

<rdar://problem/4062865>
Copy/paste of a select element fails to include the options


  • editing/SelectionController.cpp: (WebCore::SelectionController::modify): Added paragraphBoundary.
  • editing/htmlediting.cpp: (WebCore::canHaveChildrenForEditing): Added checks for input elements and textareas. Insertion operations would fail when performed just before/after one of these elements b/c the content would be put inside the element. (WebCore::enclosingNodeWithTag): Fixed a problem when calling these in non editable content, and made the code to stop at an root faster (don't check isDescendantOf on every iteration). (WebCore::enclosingNodeOfType): Ditto. (WebCore::enclosingList): Ditto. (WebCore::enclosingListChild): Ditto. Added a FIXME, this function seems inappropriately named.
  • editing/markup.cpp: (WebCore::startMarkup): Use the text node's value instead of its rendered content for text nodes inside select elements. One might also turn off annotation when createMarkup enters a select element, but createMarkup is iterative, not recursive, so doing so would be complicated. (WebCore::createMarkup): Add markup for unrendered nodes if they are descendants of a select element.
  • editing/visible_units.cpp: (WebCore::startOfParagraph): Migrate to isBlock/enclosingBlock. Fixes bug where various replaced elements can't be copied when they are the only thing selected. (WebCore::endOfParagraph): Ditto.
7:36 PM Changeset in webkit [17531] by thatcher
  • 3 edits in tags/WebCore-521.29.4/WebCore

Merging change.

7:26 PM Changeset in webkit [17530] by thatcher
  • 1 copy in tags/WebCore-521.29.4

new tag.

6:07 PM Changeset in webkit [22459] by adachan
  • 6 edits in branches/WindowsMerge/WebKitWin
5:46 PM Changeset in webkit [22458] by oliver
  • 3 edits
    8 adds in branches/WindowsMerge/WebCore

2006-11-01 Oliver Hunt <oliver@apple.com>

Reviewed by Adam.

Adding SVG_SUPPORT to Boomer.

  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.vcproj/build-generated-files.sh:
  • kcanvas/device/quartz/KCanvasFilterQuartz.cpp: Added. (WebCore::KCanvasFilterQuartz::KCanvasFilterQuartz): (WebCore::KCanvasFilterQuartz::~KCanvasFilterQuartz): (WebCore::KCanvasFilterQuartz::prepareFilter): (WebCore::KCanvasFilterQuartz::applyFilter): (WebCore::KCanvasFilterQuartz::imageForName): (WebCore::KCanvasFilterQuartz::setImageForName): (WebCore::KCanvasFilterQuartz::setOutputImage): (WebCore::KCanvasFilterQuartz::inputImage):
  • platform/graphics/svg/cg/SVGResourceMaskerCg.cpp: Added. (WebCore::SVGResourceMasker::applyMask):
5:41 PM Changeset in webkit [17529] by oliver
  • 19 edits
    2 moves in trunk/WebCore

2006-11-01 Oliver Hunt <oliver@apple.com>

Reviewed by Adam.

Converting more or kcanvas/quartz from Obj-C to C++
Minor SVG updates
compatibility fixes

  • WebCore.xcodeproj/project.pbxproj:
  • kcanvas/device/KRenderingPaintServer.h:
  • kcanvas/device/quartz/KRenderingPaintServerGradientQuartz.cpp: Added. (WebCore::KRenderingPaintServerGradientQuartz::updateQuartzGradientCache):
  • kcanvas/device/quartz/KRenderingPaintServerGradientQuartz.mm: Removed.
  • kcanvas/device/quartz/KRenderingPaintServerQuartz.cpp: Added.
  • kcanvas/device/quartz/KRenderingPaintServerQuartz.mm: Removed.
  • kcanvas/device/quartz/QuartzSupport.h:
  • ksvg2/css/SVGCSSStyleSelector.cpp:
  • ksvg2/misc/KCanvasRenderingStyle.h:
  • ksvg2/scripts/cssmakeprops:
  • ksvg2/scripts/cssmakevalues:
  • ksvg2/scripts/make_names.pl:
  • ksvg2/svg/SVGAnimateColorElement.cpp:
  • ksvg2/svg/SVGAnimateTransformElement.cpp:
  • ksvg2/svg/SVGAnimationElement.cpp:
  • ksvg2/svg/SVGMaskElement.cpp:
  • ksvg2/svg/SVGPatternElement.cpp:
  • ksvg2/svg/svgpathparser.cpp:
  • platform/Path.cpp:
  • platform/graphics/svg/SVGResourceImage.h:
  • rendering/RenderPath.cpp:
  • rendering/SVGRenderAsText.cpp:
5:02 PM Changeset in webkit [17528] by kmccullo
  • 2 edits in trunk/LayoutTests
  • temporarily passing a test so that it does not interfere with others testing.
  • fast/js/date-big-setdate-expected.txt:
4:56 PM Changeset in webkit [17527] by beidson
  • 2 edits in trunk/WebKitTools

Reviewed by Tim Hatcher

Added accessor to get the source directory for use in client scripts

  • Scripts/webkitdirs.pm:
4:43 PM Changeset in webkit [17526] by adele
  • 2 edits in trunk/WebCore

Reviewed by Adele.

  • Fix a regression from r17521: painting of and crash caused by tables with collapsed borders
  • rendering/RenderTable.cpp: (WebCore::RenderTable::paint): Changed 'paintInfo' to 'info'. Prior to r17521, 'paintInfo' was the local variable. Now 'info' is the local variable and 'paintInfo' is the parameter, which we were accidently modifying.
3:34 PM Changeset in webkit [17525] by adele
  • 4 edits in trunk/LayoutTests

Fixing typo. Changed repaintTest to runRepaintTest.

  • fast/repaint/overflow-outline-repaint-expected.checksum:
  • fast/repaint/overflow-outline-repaint-expected.png:
  • fast/repaint/overflow-outline-repaint.html:
3:28 PM Changeset in webkit [17524] by adele
  • 4 edits
    12 adds in trunk

LayoutTests:

Reviewed by Mitz.

Tests for http://bugs.webkit.org/show_bug.cgi?id=11277
REGRESSION: Incomplete repaint of overflow areas when deleting

  • fast/repaint/overflow-delete-line-expected.checksum: Added.
  • fast/repaint/overflow-delete-line-expected.png: Added.
  • fast/repaint/overflow-delete-line-expected.txt: Added.
  • fast/repaint/overflow-delete-line.html: Added.
  • fast/repaint/overflow-scroll-delete-expected.checksum: Added.
  • fast/repaint/overflow-scroll-delete-expected.png: Added.
  • fast/repaint/overflow-scroll-delete-expected.txt: Added.
  • fast/repaint/overflow-scroll-delete.html: Added.
  • fast/repaint/overflow-outline-repaint-expected.checksum: Added.
  • fast/repaint/overflow-outline-repaint-expected.png: Added.
  • fast/repaint/overflow-outline-repaint-expected.txt: Added.
  • fast/repaint/overflow-outline-repaint.html: Added.

WebCore:

Reviewed by Mitz.

Restrict the repaint rect for overflow blocks after the height has been fully computed.
Also, adjust the repaint rect coordinates for the scroll offset.

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::layoutBlock):
  • rendering/bidi.cpp: (WebCore::RenderBlock::layoutInlineChildren):
3:26 PM Changeset in webkit [22457] by bdakin
  • 1 edit in branches/WindowsMerge/WebCore/ChangeLog

Reviewed by Adam.

Build fix from Sam Weinig's check-in. i.p in the PaintInfo is now
i.context

  • rendering/RenderThemeWin.cpp: (WebCore::RenderThemeWin::paintButton): (WebCore::RenderThemeWin::paintTextField): (WebCore::RenderThemeWin::paintMenuList): (WebCore::RenderThemeWin::paintMenuListButton):
3:22 PM Changeset in webkit [17523] by sullivan
  • 2 edits in trunk/WebKit

Reviewed by Adam Roben


  • fixed <rdar://problem/4801351> Crash reloading PDF file in new Safari (or closing a window containing a PDF file)
  • WebView/WebPDFView.mm: (-[WebPDFView initWithFrame:]): Retain the PDFSubview in the code path where we just obtain it by asking the PDFPreviewView for it. We were unconditionally releasing it in dealloc, but only retaining it in one of the two code paths.
2:42 PM Changeset in webkit [17522] by spadma
  • 2 edits in S60/trunk/WebKit

2006-11-01 sareen <shyam.sareen@nokia.com>

Reviewed by Sachin.
DESC: Cenrep UID define change in HttpDownloadObserver.cpp.
http://bugs.webkit.org/show_bug.cgi?id=11488

  • ResourceLoader/src/HttpDownloadObserver.cpp: (CHttpDownloadObserver::InitDownloadMgrL):
1:49 PM Changeset in webkit [17521] by weinig
  • 51 edits in trunk/WebCore

Reviewed by Mitz.

Fix for http://bugs.webkit.org/show_bug.cgi?id=11474
Rename the "p" member variable of the PaintInfo struct to "context"

  • Renames the 'p' and 'r' member variables of the PaintInfo struct to 'context' and 'rect' respectively.
  • Assorted surrounding cleanups.
  • rendering/EllipsisBox.cpp: (WebCore::EllipsisBox::paint):
  • rendering/EllipsisBox.h:
  • rendering/InlineBox.cpp: (WebCore::InlineBox::paint):
  • rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::paint): (WebCore::InlineFlowBox::paintBackground): (WebCore::InlineFlowBox::paintBackgroundAndBorder): (WebCore::InlineFlowBox::paintDecorations):
  • rendering/InlineFlowBox.h:
  • rendering/InlineRunBox.h:
  • rendering/InlineTextBox.cpp: (WebCore::InlineTextBox::paint):
  • rendering/InlineTextBox.h:
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::paint): (WebCore::RenderBlock::paintChildren): (WebCore::RenderBlock::paintCaret): (WebCore::RenderBlock::paintObject): (WebCore::RenderBlock::paintFloats): (WebCore::RenderBlock::paintEllipsisBoxes): (WebCore::RenderBlock::paintSelection): (WebCore::RenderBlock::fillSelectionGaps): (WebCore::RenderBlock::fillInlineSelectionGaps): (WebCore::RenderBlock::fillBlockSelectionGaps): (WebCore::RenderBlock::fillHorizontalSelectionGap): (WebCore::RenderBlock::fillVerticalSelectionGap): (WebCore::RenderBlock::fillLeftSelectionGap): (WebCore::RenderBlock::fillRightSelectionGap):
  • rendering/RenderBox.cpp: (WebCore::RenderBox::paint): (WebCore::RenderBox::paintRootBoxDecorations): (WebCore::RenderBox::paintBoxDecorations):
  • rendering/RenderBox.h:
  • rendering/RenderButton.cpp: (WebCore::RenderButton::paintObject):
  • rendering/RenderFieldset.cpp: (WebCore::RenderFieldset::paintBoxDecorations):
  • rendering/RenderFileUploadControl.cpp: (WebCore::RenderFileUploadControl::paintObject):
  • rendering/RenderFlow.cpp: (WebCore::RenderFlow::continuationBefore): (WebCore::RenderFlow::addChildWithContinuation): (WebCore::RenderFlow::addChild): (WebCore::RenderFlow::attachLineBox): (WebCore::RenderFlow::destroy): (WebCore::RenderFlow::dirtyLinesFromChangedChild): (WebCore::RenderFlow::dirtyLineBoxes): (WebCore::RenderFlow::createInlineBox): (WebCore::RenderFlow::paintLines): (WebCore::RenderFlow::getAbsoluteRepaintRect): (WebCore::RenderFlow::lowestPosition): (WebCore::RenderFlow::rightmostPosition): (WebCore::RenderFlow::leftmostPosition): (WebCore::RenderFlow::caretRect): (WebCore::RenderFlow::addFocusRingRects): (WebCore::RenderFlow::paintOutline): (WebCore::RenderFlow::paintOutlineForLine):
  • rendering/RenderForeignObject.cpp: (WebCore::RenderForeignObject::paint):
  • rendering/RenderHTMLCanvas.cpp: (WebCore::RenderHTMLCanvas::paint):
  • rendering/RenderImage.cpp: (WebCore::RenderImage::paint):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintLayer):
  • rendering/RenderListBox.cpp: (WebCore::RenderListBox::paintObject): (WebCore::RenderListBox::paintScrollbar): (WebCore::RenderListBox::paintItemForeground): (WebCore::RenderListBox::paintItemBackground):
  • rendering/RenderListMarker.cpp: (WebCore::RenderListMarker::paint):
  • rendering/RenderMenuList.cpp: (WebCore::RenderMenuList::paintObject):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::paint):
  • rendering/RenderObject.h: (WebCore::RenderObject::PaintInfo::PaintInfo): (WebCore::RenderObject::paintBoxDecorations): (WebCore::RenderObject::paintingRootForChildren): (WebCore::RenderObject::shouldPaintWithinRoot): (WebCore::RenderObject::printBoxDecorations):
  • rendering/RenderPath.cpp: (WebCore::RenderPath::paint):
  • rendering/RenderReplaced.cpp: (WebCore::RenderReplaced::shouldPaint):
  • rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::paint):
  • rendering/RenderSVGContainer.h:
  • rendering/RenderSVGImage.cpp: (WebCore::RenderSVGImage::paint):
  • rendering/RenderSVGText.cpp: (WebCore::RenderSVGText::paint):
  • rendering/RenderSVGText.h: (WebCore::RenderSVGText::renderName):
  • rendering/RenderTable.cpp: (WebCore::RenderTable::paint): (WebCore::RenderTable::paintBoxDecorations):
  • rendering/RenderTable.h:
  • rendering/RenderTableCell.cpp: (WebCore::RenderTableCell::paint): (WebCore::RenderTableCell::paintBackgroundsBehindCell): (WebCore::RenderTableCell::paintBoxDecorations):
  • rendering/RenderTableCell.h:
  • rendering/RenderTableRow.cpp: (WebCore::RenderTableRow::paint):
  • rendering/RenderTableRow.h:
  • rendering/RenderTableSection.cpp: (WebCore::RenderTableSection::paint):
  • rendering/RenderTableSection.h: (WebCore::RenderTableSection::getBaseline): (WebCore::RenderTableSection::setNeedCellRecalc):
  • rendering/RenderText.h: (WebCore::RenderText::renderName): (WebCore::RenderText::paint): (WebCore::RenderText::element):
  • rendering/RenderTheme.cpp: (WebCore::RenderTheme::paint): (WebCore::RenderTheme::paintBorderOnly): (WebCore::RenderTheme::paintDecorations):
  • rendering/RenderTheme.h: (WebCore::RenderTheme::RenderTheme): (WebCore::RenderTheme::~RenderTheme): (WebCore::RenderTheme::controlSupportsTints): (WebCore::RenderTheme::adjustRepaintRect): (WebCore::RenderTheme::themeChanged): (WebCore::RenderTheme::supportsHover): (WebCore::RenderTheme::paintCheckbox): (WebCore::RenderTheme::setCheckboxSize): (WebCore::RenderTheme::paintRadio): (WebCore::RenderTheme::setRadioSize): (WebCore::RenderTheme::paintButton): (WebCore::RenderTheme::setButtonSize): (WebCore::RenderTheme::paintTextField): (WebCore::RenderTheme::paintTextArea): (WebCore::RenderTheme::paintMenuList): (WebCore::RenderTheme::paintMenuListButton):
  • rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::paintCheckbox): (WebCore::RenderThemeMac::paintRadio): (WebCore::RenderThemeMac::paintButton): (WebCore::RenderThemeMac::paintTextField): (WebCore::RenderThemeMac::paintTextArea): (WebCore::RenderThemeMac::paintMenuList): (WebCore::RenderThemeMac::paintMenuListButtonGradients): (WebCore::RenderThemeMac::paintMenuListButton):
  • rendering/RenderView.cpp: (WebCore::RenderView::paint): (WebCore::RenderView::paintBoxDecorations):
  • rendering/RenderView.h:
  • rendering/RenderWidget.cpp: (WebCore::RenderWidget::paint):
  • rendering/RenderWidget.h: (WebCore::RenderWidget::isWidget):
  • rendering/RootInlineBox.cpp: (WebCore::RootInlineBox::paintEllipsisBox): (WebCore::RootInlineBox::paintCustomHighlight): (WebCore::RootInlineBox::paint): (WebCore::RootInlineBox::fillLineSelectionGap):
  • rendering/RootInlineBox.h:
  • rendering/SVGInlineFlowBox.cpp: (WebCore::paintSVGInlineFlow):
1:31 PM Changeset in webkit [22456] by andersca
  • 4 edits in branches/WindowsMerge/WebCore

2006-11-01 Anders Carlsson <acarlsson@apple.com>

Reviewed by Geoff.

  • bridge/win/FrameWin.cpp: (WebCore::FrameWin::createJavaAppletWidget): Ask the plugin database to create a java applet plugin.
  • platform/win/TemporaryLinkStubs.cpp: Remove FrameWin::createJavaAppletWidget.


  • plugins/win/PluginDatabaseWin.cpp: (WebCore::PluginDatabaseWin::defaultPluginPaths): Don't look for the Sun Java plugin anymore.
12:34 PM Changeset in webkit [17520] by kmccullo
  • 2 edits in trunk/JavaScriptCore

Reviewed by Brady.

  • Fixes many JavaScriptCore tests in other timezones. The root problem is that on mac localtime() returns historically accurate information for DST, but the JavaScript spec explicitly states to not take into account historical information but rather to interpolate from valid years.
  • kjs/DateMath.cpp: (KJS::equivalentYearForDST): (KJS::getDSTOffsetSimple): (KJS::getDSTOffset):
12:07 PM Changeset in webkit [22455] by andersca
  • 2 edits in branches/WindowsMerge/WebCore

2006-11-01 Anders Carlsson <acarlsson@apple.com>

  • plugins/win/PluginStreamWin.cpp: (WebCore::PluginStreamWin::startStream): Build fix.
12:03 PM Changeset in webkit [22454] by andersca
  • 4 edits in branches/WindowsMerge/WebCore

2006-11-01 Anders Carlsson <acarlsson@apple.com>

Reviewed by Adam.

Convert plugin code over to using ResourceResponse.


  • plugins/win/PluginStreamWin.cpp: (WebCore::PluginStreamWin::startStream): (WebCore::PluginStreamWin::didReceiveResponse): (WebCore::PluginStreamWin::didFinishLoading):
  • plugins/win/PluginStreamWin.h:
  • plugins/win/PluginViewWin.cpp: (WebCore::PluginViewWin::performRequest):
11:43 AM Changeset in webkit [17519] by andersca
  • 3 edits in trunk/WebCore

2006-11-01 Anders Carlsson <acarlsson@apple.com>

Reviewed by Oliver, Brady.

  • platform/network/ResourceResponse.h: (WebCore::ResourceResponse::setLastModifiedDate): (WebCore::ResourceResponse::lastModifiedDate): Add getters and setters for last modified dadte.


  • platform/network/cf/ResourceResponseCFNet.cpp: (WebCore::getResourceResponse): Fetch the last modified date. Add correct offset to expired date.
11:25 AM Changeset in webkit [17518] by thatcher
  • 2 edits in trunk/WebCore

Build fix for 64-bit.

8:35 AM Changeset in webkit [17517] by yongjzha
  • 5 edits in S60/trunk/WebKit

2006-11-01 sareen <shyam.sareen@nokia.com>

Reviewed by yongjun.zhang@nokia.com.
DESC: add an UI option to disable flash content
http://bugs.webkit.org/show_bug.cgi?id=11473

  • BrowserControl/inc/BrCtlDefs.h: (TBrCtlDefs::):
  • BrowserView/inc/WebKitControl.h: (CWebKitControl::DisableFlashSetting):
  • BrowserView/src/WebKitControl.cpp: (CWebKitControl::CWebKitControl): (CWebKitControl::HandleDisableFlashSetting): (CWebKitControl::SetBrowserSettingL): (CWebKitControl::GetBrowserSettingL):
  • Plugin/src/PluginSkin.cpp: (CPluginSkin::NewL):
3:47 AM Changeset in webkit [17516] by ddkilzer
  • 2 edits in trunk/WebCore

2006-11-01 David Kilzer <ddkilzer@kilzer.net>

Reviewed by Mitz.

Added missing "break;" statement in switch statement from r17493.

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

  • css/CSSComputedStyleDeclaration.cpp: (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):

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.

Oct 29, 2006:

5:31 PM Changeset in webkit [17434] by darin
  • 4 edits in trunk/WebCore
  • fix build
  • loader/DocumentLoader.h: Add declarations so this compiles without ObjC.
  • loader/NetscapePlugInStreamLoader.h: Ditto.
  • loader/ResourceLoader.h: Ditto.
5:16 PM Changeset in webkit [17433] by darin
  • 8 edits
    1 move
    5 adds in trunk

WebCore:

  • one last name change -- WebFormState.h to FormState.h
  • added more empty placeholder platform-independent files for currently-Mac-specific loader classes
  • loader/FormState.cpp: Updated include.
  • loader/FrameLoader.cpp: Updated include.
  • loader/mac/FrameLoaderMac.mm: Updated include.
  • loader/WebFormState.h: Removed.
  • loader/FormState.h: Added.
  • loader/DocumentLoader.cpp: Added.
  • loader/MainResourceLoader.cpp: Added.
  • loader/NetscapePlugInStreamLoader.cpp: Added.
  • loader/ResourceLoader.cpp: Added.
  • loader/SubresourceLoader.cpp: Added.
  • WebCore.xcodeproj/project.pbxproj: Updated for changes above.

WebKit:

  • update for the WebCore rename
  • WebCoreSupport/WebFrameLoaderClient.mm:
  • WebView/WebFrame.mm:
5:01 PM Changeset in webkit [17432] by darin
  • 3 edits in trunk/WebKitTools
  • Scripts/do-file-rename: Renames done, ready for the next round.
  • Scripts/do-webcore-rename: Ditto.
5:00 PM Changeset in webkit [22441] by darin
  • 4 edits in branches/WindowsMerge/WebCore
  • update for the WebCore renames
  • WebCore.vcproj/WebCore.vcproj:
  • bridge/win/FrameWin.cpp: (WebCore::FrameWin::bindingRootObject): (WebCore::FrameWin::windowScriptNPObject):
  • plugins/win/PluginViewWin.cpp: (WebCore::PluginViewWin::load): (WebCore::PluginViewWin::bindingInstance):
4:57 PM Changeset in webkit [17431] by darin
  • 71 edits
    15 moves in trunk

WebCore:

  • did the do-webcore-rename renames, including a number of changes to the names of loader files (and WebResourceLoader -> ResourceLoader), jScript() to scriptProxy(), jScriptEnabled() to javaScriptEnabled(), and improvements to the names of decoder classes and their source files (Decoder -> TextResourceDecoder, StreamingTextDecoder -> TextCodec)

WebKit:

  • update for the WebCore renames
4:43 PM Changeset in webkit [17430] by darin
  • 2 edits
    6 moves in trunk/WebCore

Rubber stamped by Adam (kinda).

  • add Mac suffixes and correct filenames inside loader/mac
  • WebCore.xcodeproj/project.pbxproj: Updated for changes below.
  • loader/mac/FrameLoader.mm: Removed.
  • loader/mac/WebDocumentLoader.mm: Removed.
  • loader/mac/WebLoader.mm: Removed.
  • loader/mac/WebMainResourceLoader.mm: Removed.
  • loader/mac/WebNetscapePlugInStreamLoader.mm: Removed.
  • loader/mac/WebSubresourceLoader.mm: Removed.
  • loader/mac/FrameLoaderMac.mm: Added.
  • loader/mac/DocumentLoaderMac.mm: Added.
  • loader/mac/ResourceLoaderMac.mm: Added.
  • loader/mac/MainResourceLoaderMac.mm: Added.
  • loader/mac/WebNetscapePlugInStreamLoaderMac.mm: Added.
  • loader/mac/SubresourceLoaderMac.mm: Removed.
4:37 PM Changeset in webkit [17429] by darin
  • 12 edits
    1 move
    1 add in trunk/WebCore

Reviewed by Adam.

  • moving towards use of FrameLoader cross-platform; get FrameLoader.h and FrameLoader.cpp mostly ready
  • WebCore.xcodeproj/project.pbxproj: Added FrameLoader.cpp and FormState.cpp. Removed WebFormState.mm.
  • loader/FrameLoader.h: Put in lots of #if statements to see if we can get this header compiling on non-Mac.
  • loader/FrameLoader.cpp: Added. Has non-Mac-specific bits of FrameLoader.
  • loader/mac/FrameLoader.mm: Moved some non-Mac-specific bits out.
  • loader/FrameLoaderClient.h: Added some ifdefs.
  • loader/WebDocumentLoader.h: Ditto.
  • loader/WebLoader.h: Ditto.
  • loader/WebMainResourceLoader.h: Ditto.
  • loader/mac/WebDocumentLoader.mm: (WebCore::DocumentLoader::isLoadingInAPISense): Rewrote to work without bridge, partly to remove the need for ObjC bridge type in the header, and also because we're tearing down the bridge.
  • loader/mac/WebMainResourceLoader.mm: (WebCore::MainResourceLoader::MainResourceLoader): Removed unused m_contentLength and m_bytesReceived. (WebCore::MainResourceLoader::didReceiveResponse): Ditto. (WebCore::MainResourceLoader::didReceiveData): Ditto.
  • bridge/mac/WebCoreFrameBridge.h: Removed doneProcessingData.
  • bridge/mac/WebCoreFrameBridge.mm: Ditto.
  • loader/mac/FormState.mm: Removed.
  • loader/FormState.cpp: Added.
3:38 PM Changeset in webkit [17428] by darin
  • 2 edits
    7 moves in trunk/WebCore

Rubber stamped by Adam.

  • moving towards use of FrameLoader cross-platform; move the headers for the Mac loader from loader/mac into loader, in preparation for making them useful on non-Mac
  • WebCore.xcodeproj/project.pbxproj:
  • loader/mac/FrameLoader.h: Removed.
  • loader/mac/WebDocumentLoader.h: Removed.
  • loader/mac/WebFormState.h: Removed.
  • loader/mac/WebLoader.h: Removed.
  • loader/mac/WebMainResourceLoader.h: Removed.
  • loader/mac/WebNetscapePlugInStreamLoader.h: Removed.
  • loader/mac/WebSubresourceLoader.h: Removed.
  • loader/FrameLoader.h: Added.
  • loader/WebDocumentLoader.h: Added.
  • loader/WebFormState.h: Added.
  • loader/WebLoader.h: Added.
  • loader/WebMainResourceLoader.h: Added.
  • loader/WebNetscapePlugInStreamLoader.h: Added.
  • loader/WebSubresourceLoader.h: Added.
3:01 PM Changeset in webkit [17427] by aroben
  • 2 edits in trunk/WebCore

Reviewed by Geoff.

Finish Darin's clean up of the use of APPLE in WebCore, begun in
r17405.

  • platform/DeprecatedString.h:
2:07 PM Changeset in webkit [17426] by bdakin
  • 35 edits in trunk/WebCore

Rubber stamped by Geoff.

This is a followup to my HitTestResult checkin. Most of the
HitTestResult local variables in the code were named i or info for
when the class was named NodeInfo. This patch renames all of these
local variables to result.

12:48 PM Changeset in webkit [17425] by aroben
  • 3 edits in trunk/WebCore

More semi-fixes.

  • page/Page.cpp: Forgot to wrap the actual definition of setDefersLoading() in PLATFORM(MAC).
  • platform/PlatformWheelEvent.h: Fix up preprocessor directives.
12:32 PM Changeset in webkit [17424] by aroben
  • 3 edits in trunk/WebCore

Build semi-fix.

Move all FrameLoader-related code in Page.{cpp,h} within #if
PLATFORM(MAC), since this class currently only exists on Mac.

  • page/Page.cpp:
  • page/Page.h:
11:15 AM Changeset in webkit [17423] by darin
  • 2 edits in trunk/WebKitTools
  • Scripts/do-file-rename: And again.
11:06 AM Changeset in webkit [17422] by ggaren
  • 3 edits in trunk/WebCore

build fix.


  • platform/PlatformString.h: (WebCore::String::operator NSString*):
11:03 AM Changeset in webkit [17421] by ggaren
  • 2 edits in trunk/WebCore

Rolled out accidental commit.


  • platform/AtomicString.h: (WebCore::AtomicString::AtomicString): (WebCore::AtomicString::operator NSString*):
10:58 AM Changeset in webkit [17420] by ggaren
  • 1 edit
    2 adds in trunk/LayoutTests

Reviewed by Geoff.

Test for http://bugs.webkit.org/show_bug.cgi?id=11444
REGRESSION (r17378): Exception (-[NSCFDictionary setObject:forKey:]: attempt to insert nil value)
when submitting a form with an empty uninitialized field

  • fast/forms/submit-nil-value-field-assert-expected.txt: Added.
  • fast/forms/submit-nil-value-field-assert.html: Added.
10:54 AM Changeset in webkit [17419] by ggaren
  • 4 edits in trunk/WebCore

RS by Darin.

Rolled out change to map NULL Strings to nil NSStrings because it caused
even more AppKit crashes. Maybe we should remove the NSString * operator
entirely, since its use is not straight-forward, and require callers to
specify explicitly whether they tolerate NULL.


  • platform/AtomicString.h:
  • platform/PlatformString.h: (WebCore::String::operator NSString*):
  • platform/mac/TextFieldMac.mm: (-[NSSearchFieldCell _addStringToRecentSearches:]):
10:28 AM Changeset in webkit [17418] by darin
  • 2 edits in trunk/WebKitTools
  • Scripts/do-webcore-rename: Tweaked plans for renaming again.
10:03 AM Changeset in webkit [17417] by darin
  • 1 edit in trunk/WebKit/ChangeLog

Added a bug number to change log.

9:44 AM Changeset in webkit [17416] by darin
  • 2 edits in trunk/WebCore

Reviewed by Geoff.

  • fixed storage leak showing up on the buildbot
  • loader/mac/WebSubresourceLoader.mm: (WebCore::SubresourceLoader::create): Release the NSURLRequest after passing it to load. Also clean up logic a bit.
9:12 AM Changeset in webkit [17415] by darin
  • 9 edits
    1 move in trunk

WebCore:

Rubber stamped by Adam Roben.

  • renamed WebFrameLoaderClient to match the class name inside it
  • loader/mac/WebFrameLoaderClient.h: Removed.
  • loader/FrameLoaderClient.h: Moved it here.
  • WebCore.xcodeproj/project.pbxproj: Updated project.
  • bridge/mac/FrameMac.mm: Updated includes.
  • bridge/mac/WebCoreFrameBridge.mm: Ditto.
  • loader/mac/FrameLoader.mm: Ditto.
  • loader/mac/WebLoader.h: Ditto.

WebKit:

Rubber stamped by Adam Roben.

  • renamed WebCore's WebFrameLoaderClient to match the class name inside it
  • WebCoreSupport/WebFrameBridge.mm: Update include.
  • WebCoreSupport/WebFrameLoaderClient.h: Ditto.
8:50 AM Changeset in webkit [17414] by darin
  • 3 edits in trunk/WebCore

Reviewed by Mitz.

  • fix crash visible on buildbot and also when running the test anchor-image-scrolled-x-y.html under libgmalloc
  • loader/mac/WebSubresourceLoader.h: Renamed m_loader to m_handle now that the class is named ResourceHandle.
  • loader/mac/WebSubresourceLoader.mm: Renaming, plus: (WebCore::SubresourceLoader::didFinishLoading): Get the handle into a local RefPtr before calling finishJobAndHandle, since we might get cancelled inside that function, so we should not rely on m_handle to keep the ResourceHandle alive. (WebCore::SubresourceLoader::didFail): Ditto. (WebCore::SubresourceLoader::didCancel): Ditto.
4:21 AM Changeset in webkit [17413] by bdash
  • 5 edits in trunk/LayoutTests

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

Reviewed by Maciej.

Fix fast/block/basic/014.html to pass when Microsoft Office fonts are present.

  • fast/block/basic/014-expected.checksum:
  • fast/block/basic/014-expected.png:
  • fast/block/basic/014-expected.txt:
  • fast/block/basic/014.html: Always use default sans-serif font rather than preferring Microsoft Office fonts.
3:15 AM Changeset in webkit [17412] by bdash
  • 2 edits in trunk/LayoutTests

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

Reviewed by Mitz.

Fix failure in editing/inserting/insert-3851164-fix if Gill Sans font is present.

  • editing/inserting/insert-3851164-fix.html: Specify sans-serif to match expected results generated without Gill Sans present.
2:00 AM Changeset in webkit [17411] by mjs
  • 2 edits in trunk/WebCore

Not reviewed.

  • build fix
  • platform/network/mac/ResourceHandleMac.mm:
1:50 AM Changeset in webkit [22440] by mjs
  • 12 edits
    2 deletes in branches/WindowsMerge

WebCoreWin:

Rubber stamped by Darin.

  • renamed ResourceLoader to ResourceHandle (and same for related classes)
  • WebCore.vcproj/WebCore.vcproj:
  • bridge/win/FrameWin.cpp:
  • platform/network/win/ResourceHandleWin.cpp: Added. (WebCore::): (WebCore::addToOutstandingJobs): (WebCore::lookupResourceHandle): (WebCore::ResourceHandleWndProc): (WebCore::ResourceHandle::onHandleCreated): (WebCore::ResourceHandle::onRequestRedirected): (WebCore::ResourceHandle::onRequestComplete): (WebCore::initializeOffScreenResourceHandleWindow): (WebCore::ResourceHandleInternal::~ResourceHandleInternal): (WebCore::ResourceHandle::~ResourceHandle): (WebCore::ResourceHandle::startHTTPRequest): (WebCore::ResourceHandle::start): (WebCore::ResourceHandle::fileLoadTimer): (WebCore::ResourceHandle::cancel):
  • platform/network/win/ResourceHandleWin.h: Added.
  • platform/network/win/ResourceLoaderWin.cpp: Removed.
  • platform/network/win/ResourceLoaderWin.h: Removed.
  • platform/win/TemporaryLinkStubs.cpp: (WebCore::ResourceHandle::assembleResponseHeaders): (WebCore::ResourceHandle::retrieveResponseEncoding): (IconLoader::receivedResponse):
  • plugins/win/PluginStreamWin.cpp: (WebCore::PluginStreamWin::start): (WebCore::PluginStreamWin::deliverData): (WebCore::PluginStreamWin::receivedResponse): (WebCore::PluginStreamWin::didReceiveData): (WebCore::PluginStreamWin::receivedAllData):
  • plugins/win/PluginStreamWin.h:

WebKitWin:

Rubber stamped by Darin.

  • renamed ResourceLoader to ResourceHandle (and same for related classes)
  • WebFrame.cpp: (WebFrame::loadDataSource): (WebFrame::receivedRedirect): (WebFrame::receivedResponse): (WebFrame::didReceiveData): (WebFrame::didFinishLoading):
  • WebFrame.h:
  • WebURLResponse.cpp: (WebURLResponse::createInstance):
  • WebURLResponse.h:
  • WebView.cpp:
1:44 AM Changeset in webkit [17410] by mjs
  • 31 edits
    14 moves in trunk/WebCore

Rubber stamped by Darin.

  • renamed ResourceLoader to ResourceHandle (and same for related classes)
  • CMakeLists.txt:
  • Projects/gdk/webcore-gdk.bkl:
  • WebCore.vcproj/WebCore/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • bridge/mac/FrameMac.mm:
  • bridge/win/FrameWin.cpp:
  • dom/XMLTokenizer.cpp:
  • loader/Cache.cpp:
  • loader/CachedResource.h:
  • loader/LoaderFunctions.h:
  • loader/icon/IconLoader.cpp: (IconLoader::startLoading): (IconLoader::didReceiveData): (IconLoader::didFinishLoading):
  • loader/icon/IconLoader.h:
  • loader/loader.cpp: (WebCore::Loader::servePendingRequests): (WebCore::Loader::receivedAllData): (WebCore::Loader::receivedResponse): (WebCore::Loader::didReceiveData): (WebCore::Loader::cancelRequests): (WebCore::Loader::jobForRequest):
  • loader/loader.h:
  • loader/mac/IconLoaderMac.mm: (IconLoader::receivedResponse):
  • loader/mac/LoaderFunctionsMac.mm:
  • loader/mac/WebLoader.h:
  • loader/mac/WebSubresourceLoader.h:
  • loader/mac/WebSubresourceLoader.mm: (WebCore::SubresourceLoader::SubresourceLoader): (WebCore::SubresourceLoader::create):
  • platform/gdk/FrameGdk.cpp: (WebCore::FrameGdkClientDefault::FrameGdkClientDefault): (WebCore::FrameGdkClientDefault::openURL): (WebCore::FrameGdkClientDefault::submitForm): (WebCore::FrameGdkClientDefault::receivedResponse): (WebCore::FrameGdkClientDefault::didReceiveData): (WebCore::FrameGdkClientDefault::receivedAllData):
  • platform/gdk/FrameGdk.h:
  • platform/gdk/TemporaryLinkStubs.cpp: (WebCore::ServeSynchronousRequest): (WebCore::ResourceHandle::assembleResponseHeaders): (WebCore::ResourceHandle::retrieveCharset):
  • platform/network/ResourceHandle.cpp: Added. (WebCore::ResourceHandle::ResourceHandle): (WebCore::ResourceHandle::create): (WebCore::ResourceHandle::isErrorPage): (WebCore::ResourceHandle::error): (WebCore::ResourceHandle::setError): (WebCore::ResourceHandle::responseEncoding): (WebCore::ResourceHandle::responseHTTPHeadersAsString): (WebCore::ResourceHandle::kill): (WebCore::ResourceHandle::requestHeaders): (WebCore::ResourceHandle::url): (WebCore::ResourceHandle::postData): (WebCore::ResourceHandle::method): (WebCore::ResourceHandle::client):
  • platform/network/ResourceHandle.h: Added. (WebCore::ResourceHandle::getInternal):
  • platform/network/ResourceHandleClient.h: Added. (WebCore::ResourceHandleClient::~ResourceHandleClient): (WebCore::ResourceHandleClient::didReceiveData): (WebCore::ResourceHandleClient::didFinishLoading): (WebCore::ResourceHandleClient::receivedRedirect): (WebCore::ResourceHandleClient::receivedResponse): (WebCore::ResourceHandleClient::receivedAllData):
  • platform/network/ResourceHandleInternal.h: Added. (WebCore::ResourceHandleInternal::ResourceHandleInternal):
  • platform/network/ResourceLoader.cpp: Removed.
  • platform/network/ResourceLoader.h: Removed.
  • platform/network/ResourceLoaderClient.h: Removed.
  • platform/network/ResourceLoaderInternal.h: Removed.
  • platform/network/cf/ResourceHandleCFNet.cpp: Added. (WebCore::willSendRequest): (WebCore::didReceiveResponse): (WebCore::didReceiveData): (WebCore::didFinishLoading): (WebCore::didFail): (WebCore::willCacheResponse): (WebCore::didReceiveChallenge): (WebCore::ResourceHandleInternal::~ResourceHandleInternal): (WebCore::ResourceHandle::~ResourceHandle): (WebCore::ResourceHandle::start): (WebCore::ResourceHandle::cancel):
  • platform/network/cf/ResourceLoaderCFNet.cpp: Removed.
  • platform/network/gdk/ResourceHandleCurl.cpp: Added. (WebCore::ResourceHandleInternal::~ResourceHandleInternal): (WebCore::ResourceHandle::~ResourceHandle): (WebCore::ResourceHandle::start): (WebCore::ResourceHandle::cancel):
  • platform/network/gdk/ResourceHandleManager.cpp: Added. (WebCore::ResourceHandleManager::ResourceHandleManager): (WebCore::ResourceHandleManager::get): (WebCore::ResourceHandleManager::useSimpleTransfer): (WebCore::writeCallback): (WebCore::ResourceHandleManager::downloadTimerCallback): (WebCore::ResourceHandleManager::remove): (WebCore::ResourceHandleManager::add): (WebCore::ResourceHandleManager::cancel):
  • platform/network/gdk/ResourceHandleManager.h: Added.
  • platform/network/gdk/ResourceLoaderCurl.cpp: Removed.
  • platform/network/gdk/ResourceLoaderManager.cpp: Removed.
  • platform/network/gdk/ResourceLoaderManager.h: Removed.
  • platform/network/mac/ResourceHandleMac.mm: Added. (WebCore::ResourceHandleInternal::~ResourceHandleInternal): (WebCore::ResourceHandle::~ResourceHandle): (WebCore::ResourceHandle::start): (WebCore::ResourceHandle::assembleResponseHeaders): (WebCore::ResourceHandle::retrieveResponseEncoding): (WebCore::ResourceHandle::receivedResponse): (WebCore::ResourceHandle::cancel): (WebCore::ResourceHandle::redirectedToURL): (WebCore::ResourceHandle::addData): (WebCore::ResourceHandle::finishJobAndHandle): (WebCore::ResourceHandle::reportError):
  • platform/network/mac/ResourceLoaderMac.mm: Removed.
  • platform/network/qt/ResourceHandleManager.cpp: Added. (WebCore::ResourceHandleManager::ResourceHandleManager): (WebCore::ResourceHandleManager::~ResourceHandleManager): (WebCore::ResourceHandleManager::self): (WebCore::ResourceHandleManager::slotData): (WebCore::ResourceHandleManager::slotMimetype): (WebCore::ResourceHandleManager::slotResult): (WebCore::ResourceHandleManager::remove): (WebCore::ResourceHandleManager::add): (WebCore::ResourceHandleManager::cancel):
  • platform/network/qt/ResourceHandleManager.h: Added.
  • platform/network/qt/ResourceHandleQt.cpp: Added. (WebCore::ResourceHandleInternal::~ResourceHandleInternal): (WebCore::ResourceHandle::~ResourceHandle): (WebCore::ResourceHandle::start): (WebCore::ResourceHandle::cancel): (WebCore::ResourceHandle::assembleResponseHeaders): (WebCore::ResourceHandle::retrieveResponseEncoding): (WebCore::ResourceHandle::receivedResponse): (WebCore::ResourceHandle::extractCharsetFromHeaders):
  • platform/network/qt/ResourceLoaderManager.cpp: Removed.
  • platform/network/qt/ResourceLoaderManager.h: Removed.
  • platform/network/qt/ResourceLoaderQt.cpp: Removed.
  • platform/network/win/ResourceHandleWin.cpp: Added. (WebCore::): (WebCore::addToOutstandingJobs): (WebCore::lookupResourceHandle): (WebCore::ResourceHandleWndProc): (WebCore::initializeOffScreenResourceHandleWindow): (WebCore::ResourceHandleInternal::~ResourceHandleInternal): (WebCore::ResourceHandle::~ResourceHandle): (WebCore::ResourceHandle::onHandleCreated): (WebCore::ResourceHandle::onRequestRedirected): (WebCore::ResourceHandle::onRequestComplete): (WebCore::ResourceHandle::start): (WebCore::ResourceHandle::fileLoadTimer): (WebCore::ResourceHandle::cancel): (WebCore::ResourceHandle::setHasReceivedResponse): (WebCore::ResourceHandle::hasReceivedResponse):
  • platform/network/win/ResourceHandleWin.h: Added.
  • platform/network/win/ResourceLoaderWin.cpp: Removed.
  • platform/network/win/ResourceLoaderWin.h: Removed.
  • platform/qt/FrameQt.cpp:
  • platform/qt/FrameQtClient.cpp: (WebCore::FrameQtClientDefault::openURL): (WebCore::FrameQtClientDefault::submitForm): (WebCore::FrameQtClientDefault::receivedResponse): (WebCore::FrameQtClientDefault::didReceiveData): (WebCore::FrameQtClientDefault::receivedAllData):
  • platform/qt/FrameQtClient.h:
  • platform/win/TemporaryLinkStubs.cpp: (WebCore::ResourceHandle::assembleResponseHeaders): (WebCore::ResourceHandle::retrieveResponseEncoding): (WebCore::ServeSynchronousRequest):
  • xml/XSLTProcessor.cpp:
  • xml/xmlhttprequest.cpp: (WebCore::XMLHttpRequest::send): (WebCore::XMLHttpRequest::didFinishLoading): (WebCore::XMLHttpRequest::receivedRedirect): (WebCore::XMLHttpRequest::didReceiveData):
  • xml/xmlhttprequest.h:
1:36 AM Changeset in webkit [17409] by darin
  • 5 edits in trunk

LayoutTests:

  • fast/events/js-keyboard-event-creation-expected.txt: Update test that is affected in a minor way by the change to DumpRenderTree.

WebKitTools:

Reviewed by Mitz.

  • DumpRenderTree/DumpRenderTree.m: (-[DumpRenderTreeWindow keyDown:]): Added. Does nothing, which prevents a beep.
  • Scripts/do-webcore-rename: Tweaked plans for renaming a bit.
1:47 AM Changeset in webkit [17408] by darin
  • 21 edits
    2 deletes in trunk

WebCore:

Reviewed by Maciej.

  • eliminate use of NSArray to carry form data around
  • WebCore.exp:
  • WebCore.xcodeproj/project.pbxproj:
  • bridge/mac/FormDataMac.h: Removed.
  • bridge/mac/FormDataMac.mm: Removed.
  • bridge/mac/FrameMac.h:
  • bridge/mac/WebCoreFrameBridge.h:
  • bridge/mac/WebCoreFrameBridge.mm:
  • html/HTMLFormElement.cpp: (WebCore::HTMLFormElement::submit):
  • loader/FormData.h: (WebCore::FormData::isEmpty):
  • loader/mac/FrameLoader.h:
  • loader/mac/FrameLoader.mm: (WebCore::FrameLoader::load): (WebCore::FrameLoader::post): (WebCore::FrameLoader::loadEmptyDocumentSynchronously): (WebCore::FrameLoader::loadResourceSynchronously):
  • loader/mac/LoaderFunctionsMac.mm: (WebCore::ServeSynchronousRequest):
  • loader/mac/WebFormDataStream.h:
  • loader/mac/WebFormDataStream.m: (WebCore::pairRetain): (WebCore::pairRelease): (WebCore::pairEqual): (WebCore::pairHash): (WebCore::closeCurrentStream): (WebCore::scheduleWithPair): (WebCore::advanceCurrentStream): (WebCore::openNextStream): (WebCore::formCreate): (WebCore::formFinalize): (WebCore::formOpen): (WebCore::formRead): (WebCore::formCanRead): (WebCore::formClose): (WebCore::formSchedule): (WebCore::formUnschedule): (WebCore::formEventCallback): (WebCore::setHTTPBody):
  • loader/mac/WebSubresourceLoader.h:
  • loader/mac/WebSubresourceLoader.mm: (WebCore::SubresourceLoader::create):
  • platform/network/mac/ResourceLoaderMac.mm:

WebKit:

Reviewed by Maciej.

  • eliminate use of NSArray to carry form data around (the code in this framework was actually using the NSArray to hold a single

NSData anyway, so I just went back to an NSData for now)

  • History/WebHistoryItem.m: (-[WebHistoryItem _setFormInfoFromRequest:]): (-[WebHistoryItem formData]):
  • History/WebHistoryItemPrivate.h:
  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::dispatchWillSubmitForm):
  • WebView/WebFrame.mm: (-[WebFrame _loadItem:withLoadType:]):
1:36 AM Changeset in webkit [17407] by mjs
  • 4 edits in trunk/WebCore

Reviewed by Darin.

  • create SubresourceLoader with a ResourceRequest, not broken out request bits
  • loader/mac/WebSubresourceLoader.h:
  • loader/mac/WebSubresourceLoader.mm: (WebCore::SubresourceLoader::create):
  • platform/network/mac/ResourceLoaderMac.mm: (WebCore::ResourceLoader::start):
12:30 AM Changeset in webkit [17406] by mjs
  • 12 edits
    3 deletes in trunk/WebCore

Reviewed by Darin.

  • removed ObjC glue between ResourceLoader and SubresourceLoader, then now talk to each other directly

Removed WebCoreResourceLoader, WebCoreResourceHandle and WebCoreResourceLoaderImp


  • WebCore.xcodeproj/project.pbxproj:
  • bridge/mac/WebCoreFrameBridge.h:
  • loader/mac/FrameLoader.h:
  • loader/mac/FrameLoader.mm:
  • loader/mac/LoaderFunctionsMac.mm:
  • loader/mac/WebLoader.h:
  • loader/mac/WebSubresourceLoader.h:
  • loader/mac/WebSubresourceLoader.mm: (WebCore::SubresourceLoader::SubresourceLoader): (WebCore::SubresourceLoader::create): (WebCore::SubresourceLoader::willSendRequest): (WebCore::SubresourceLoader::didReceiveResponse): (WebCore::SubresourceLoader::didReceiveData): (WebCore::SubresourceLoader::didFinishLoading): (WebCore::SubresourceLoader::didFail): (WebCore::SubresourceLoader::didCancel):
  • platform/network/ResourceLoader.h:
  • platform/network/ResourceLoaderInternal.h:
  • platform/network/mac/ResourceLoaderMac.mm: (WebCore::ResourceLoader::~ResourceLoader): (WebCore::ResourceLoader::start): (WebCore::ResourceLoader::receivedResponse): (WebCore::ResourceLoader::cancel): (WebCore::ResourceLoader::redirectedToURL): (WebCore::ResourceLoader::addData): (WebCore::ResourceLoader::finishJobAndHandle): (WebCore::ResourceLoader::reportError):
  • platform/network/mac/WebCoreResourceLoader.h: Removed.
  • platform/network/mac/WebCoreResourceLoaderImp.h: Removed.
  • platform/network/mac/WebCoreResourceLoaderImp.mm: Removed.
Note: See TracTimeline for information about the timeline view.