Timeline


and

08/01/07:

23:24 Changeset [24816] by aroben

When DRT crashes, record stderr and restart DRT

This prevents a DRT crash from causing the next few hundred tests to
"fail" because DRT is no longer running.

I also changed the terminology that run-webkit-tests uses in its
output a bit, so that crashing tests are referred to as "crashes"
instead of "failures".

Reviewed by Mark.

  • Scripts/run-webkit-tests: Detect a crash and record it as a tool failure. (sub openDumpTool): Use open3 so that we can access stderr. (sub dumpToolCrashed): Added. (sub printFailureMessageForTest): Added. (sub htmlForExpectedAndActualResults): Added. (sub deleteExpectedAndActualResults): Added. (sub recordActualResultsAndDiff): Added.
21:21 Changeset [24815] by treat

Add an interface to manage global history for clients

19:24 Changeset [24814] by treat

Do not call update or repaint from inside a paint event.

18:43 Changeset [24813] by andersca

Fix build.


  • Misc/WebNSURLExtras.mm: (+[NSURL _web_URLWithUserTypedString:relativeToURL:]): (-[NSURL _web_userVisibleString]): (-[NSURL _web_URLWithLowercasedScheme]): (-[NSURL _web_dataForURLComponentType:]):
18:18 Changeset [24812] by aliceli1

Reviewed by .

Making WebNSURLExtras objc++

  • Misc/WebNSURLExtras.m: Removed.
  • Misc/WebNSURLExtras.mm: Copied from WebKit/Misc/WebNSURLExtras.m.
  • WebKit.xcodeproj/project.pbxproj:
18:15 Changeset [24811] by darin

Reviewed by Anders Carlsson and Kevin Decker.

  • fix <rdar://problem/5377432> Removal of MakeDataExecutable from 64-bit breaks WebKit build

The trick was to ifdef out more of the code that's only needed to support CFM, which exists
only for 32-bit PowerPC.

  • Plugins/WebNetscapePluginPackage.h: Define a SUPPORT_CFM symbol in this internal header when we support CFM. We support it only on 32-bit PowerPC. Only define the isBundle, isCFM, and connID fields when SUPPORT_CFM is on. Also use ResFileRefNum instead of SInt16.
  • Plugins/WebNetscapePluginPackage.m: Only compile the function pointer and transition vector functions when SUPPORT_CFM is on. (-[WebNetscapePluginPackage openResourceFile]): Put the non-bundle case inside a SUPPORT_CFM ifdef, since all non-CFM plug-ins are bundles. (-[WebNetscapePluginPackage closeResourceFile:]): Ditto. (-[WebNetscapePluginPackage _initWithPath:]): Use SUPPORT_CFM to compile out the code for non-bundle and bundle-based CFM plug-ins, and code that sets isBundle and isCFM. (-[WebNetscapePluginPackage executableType]): Put the CFM case inside SUPPORT_CFM. (-[WebNetscapePluginPackage load]): Put the non-bundle and CFM cases inside SUPPORT_CFM. There was a bit of dead code here. (-[WebNetscapePluginPackage _unloadWithShutdown:]): Put the non-bundle case inside SUPPORT_CFM.
17:35 Changeset [24810] by thatcher

Reviewed by Justin.

<rdar://problem/5376156> Mail crash in DeleteButtonController::hide() when dropping selected image on DIV's border

Add the container element back so the selection can not touch the deletion UI nodes. The container
has style to prevent user selection, user drag and user modification.

  • editing/DeleteButtonController.cpp: (WebCore::DeleteButtonController::show): Make the container node, and append the button and outline elements. (WebCore::DeleteButtonController::hide): Remove the container elements and null out the other nodes.
  • editing/DeleteButtonController.h:
17:30 Changeset [24809] by darin

Reviewed by Kevin McCullough.

  • fix <rdar://problem/5375186> pointers to pieces of class definition passed to JSClassCreate should all be const
  • API/JSObjectRef.h: Added const.
  • API/JSClassRef.cpp: (OpaqueJSClass::OpaqueJSClass): Added const. (OpaqueJSClass::create): Added const.
  • API/JSObjectRef.cpp: (JSClassCreate): Added const.
14:14 Changeset [24808] by sfalken

Build mod: Fix sln to match configs in vcproj.


Reviewed by Adam.

12:52 Changeset [24807] by zbujtas

2007-07-31 yadavall <sriram.yadavalli@nokia.com>

Reviewed by Zalan Bujtas (zbujtas@gmail.com).
DESC: Fixes for Menu API

12:47 Changeset [24806] by zbujtas

rathnasa, reviewed by zbujtas

DESC: TSW Id: ELTA-756J4Z When user focused the different links, the other sections are not hidden.
http://bugs.webkit.org/show_bug.cgi?id=14814

12:47 BuildingGdk edited by treat@kde.org
The two linux ports do not share the same dependencies. (diff)
12:45 Changeset [24805] by zbujtas

2007-07-30 rathnasa <sornalatha.rathnasamy@nokia.com>

Reviewed by zbujtas.
DESC: TSW Id: ELTA-756J4Z When user focused the different links, the other sections are not hidden.
http://bugs.webkit.org/show_bug.cgi?id=14814

12:43 BuildingOnLinux edited by treat@kde.org
Leave the buidling instructions for the pages specifically intended for … (diff)
12:05 WikiStart edited by treat@kde.org
Go directly to the port's specific content page. We are talking about … (diff)
11:54 BuildingGdk edited by alp@atoker.com
Port is now maintained (diff)
09:53 Changeset [24804] by yongjzha

w3liu, reviewed by <yongjun.zhang@nokia.com>

DESC: FPER-74HDN2: Strange character are shown instead of Chinese on www.sina.com.cn
http://bugs.webkit.org/show_bug.cgi?id=14830

WARNING: NO TEST CASES ADDED OR CHANGED

  • khtml/khtml_part.cpp: (KHTMLPart::write):
07:22 BuildingOnLinux edited by samuel.verstraete@gmail.com
just fixed some "typo's" (diff)
07:16 BuildingOnLinux edited by hausmann@webkit.org
(diff)

07/31/07:

22:42 BuildingOnLinux edited by vprajan@gmail.com
(diff)
22:40 BuildingOnLinux edited by vprajan@gmail.com
(diff)
22:40 WikiStart edited by vprajan@gmail.com
(diff)
22:39 BuildingOnLinux edited by vprajan@gmail.com
(diff)
21:30 Changeset [24803] by oliver

fast/encoding/char-encoding.html no longer needs to be in the Leopard skiplist

  • mac/leopard/Skipped:
17:40 Changeset [24802] by harrison

Reviewed by Justin.

<rdar://problem/5362659> CrashTracer: [USER] 11 crashes in Mail at WebCore::InsertLineBreakCommand::doApply()

The problem was that deleting with the X control leaves the selection inside the fragment that was deleted.

  • editing/DeleteButtonController.cpp: (WebCore::DeleteButtonController::deleteTarget): Because the deletion UI only appears when the selection is entirely within the target, we unconditionally update the selection to be a caret where the target had been.
17:36 Changeset [24801] by adele

LayoutTests:

Reviewed by Hyatt.

Updated tests for <rdar://problem/5339395> REGRESSION:http://sudokucraving.com does not render grid correctly

  • fast/forms/input-appearance-minWidth.html: Removed. This test is covered by fast/replaced/width100percent-textfield.html.
  • fast/forms/input-appearance-minWidth-expected.checksum: Removed.
  • fast/forms/input-appearance-minWidth-expected.png: Removed.
  • fast/forms/input-appearance-minWidth-expected.txt: Removed.
  • fast/forms/input-table-expected.checksum:
  • fast/forms/input-table-expected.png:
  • fast/forms/input-table-expected.txt:
  • fast/replaced/width100percent-menulist-expected.checksum:
  • fast/replaced/width100percent-menulist-expected.png:
  • fast/replaced/width100percent-menulist-expected.txt:
  • fast/replaced/width100percent-searchfield-expected.checksum: Added.
  • fast/replaced/width100percent-searchfield-expected.png: Added.
  • fast/replaced/width100percent-searchfield-expected.txt: Added.
  • fast/replaced/width100percent-searchfield.html: Added.
  • fast/replaced/width100percent-textarea-expected.checksum:
  • fast/replaced/width100percent-textarea-expected.png:
  • fast/replaced/width100percent-textarea-expected.txt:
  • fast/replaced/width100percent-textfield-expected.checksum:
  • fast/replaced/width100percent-textfield-expected.png:
  • fast/replaced/width100percent-textfield-expected.txt:

WebCore:

Reviewed by Hyatt.

Fix for <rdar://problem/5339395> REGRESSION:http://sudokucraving.com does not render grid correctly

  • rendering/RenderTextControl.h: (WebCore::RenderTextControl::hasControlClip): Clip for search fields.
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::controlClipRect): Added a control clip so the search field's cancel button and magnifier glass never draw outside the control's bounds. (WebCore::RenderTextControl::calcPrefWidths): Only include the inner box's padding when calculating the min/max width without using calcContentBoxWidth. Our old behavior was causing that inner padding to get counted twice. Also, no need to add in the border for an inner box that can't be controlled from outside this class.
  • rendering/RenderMenuList.cpp: (WebCore::RenderMenuList::calcPrefWidths): ditto.
17:20 Changeset [24800] by andersca

Reviewed by Geoff.

Speculative fix for <rdar://problem/5359695>
REGRESSION (Tiger Beta): Multiple crashes in WebCore::Widget::getView() const + 6


  • page/EventHandler.cpp: (WebCore::EventHandler::updateDragAndDrop): Null check the frame view.
14:17 Changeset [24799] by thatcher

WebCore:

Reviewed by Oliver and Beth.

<rdar://problem/5211271> ADOBE Leopard 9A410: At the first Launching InDesign after deactivate, EULA page gets blanked.

Rename needsAcrobatFrameReloadingQuirk to needsAdobeFrameReloadingQuirk, since this now applies to more Adobe applications.

  • WebCore.exp:
  • page/Settings.cpp: (WebCore::Settings::Settings): (WebCore::Settings::setNeedsAdobeFrameReloadingQuirk):
  • page/Settings.h: (WebCore::Settings::needsAcrobatFrameReloadingQuirk):

WebKit:

Reviewed by Oliver and Beth.

<rdar://problem/5211271> ADOBE Leopard 9A410: At the first Launching InDesign after deactivate, EULA page gets blanked.

Check for more Adobe applications that need the frame reload quirk. Also cache the answer
so the version check dosen't happen more than once.

  • WebView/WebView.mm: (-[WebView _needsAdobeFrameReloadingQuirk]): (-[WebView _updateWebCoreSettingsFromPreferences:]):
13:29 Changeset [24798] by beidson

LayoutTests:

Reviewed by Brady

Test for http://bugs.webkit.org/show_bug.cgi?id=14757
HTMLTokenizer::processingData implementation is incorrect

  • http/tests/loading/slow-parsing-subframe-expected.txt: Added.
  • http/tests/loading/slow-parsing-subframe.html: Added.

WebCore:

Reviewed by Brady and Darin, tweaked by Brady, landed by Brady

Fix for http://bugs.webkit.org/show_bug.cgi?id=14757 and <rdar://problem/5364692>
HTMLTokenizer::processingData implementation is incorrect

  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::processingData): Made it also return true if the HTMLTokenizer was inside the write() call.
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::FrameLoader): (WebCore::FrameLoader::checkLoadCompleteTimerFired): (WebCore::FrameLoader::scheduleCheckLoadComplete): (WebCore::FrameLoader::stopForUserCancel): Changed a call to checkLoadComplete to be asynchronous, since stopForUserCancel can be called while parsing.
  • loader/FrameLoader.h:
12:54 Changeset [24797] by adele

Reviewed by Oliver.

Fix for <rdar://problem/5308020> REGRESSION: Command-N with Dvorak-Qwerty keyboard layout stopped working inside web page text fields

  • WebView/WebHTMLView.mm: (-[WebHTMLView _handleStyleKeyEquivalent:]): The input method may have modified the character we get, so don't use charactersIgnoringModifiers to interpret the character we get.
12:08 Changeset [24796] by andersca

Reviewed by Geoff.

<rdar://problem/5371582>
REGRESSION: PLT .5% slower due to r24451 (copying HTMLCollection objects)


Make the hash maps store CollectionInfo pointers to reduce amount of copying when
inserting/rehashing etc.


  • dom/Document.cpp: (WebCore::Document::~Document): (WebCore::Document::nameCollectionInfo):
  • dom/Document.h:
11:38 Changeset [24795] by adele

Reviewed by Beth.

Updated this test to force display to happen earlier. This works around a bug in DRT that draws the scrollbars in a different way from Safari.

  • fast/replaced/width100percent-textarea-expected.checksum:
  • fast/replaced/width100percent-textarea-expected.png:
  • fast/replaced/width100percent-textarea.html:
11:27 Changeset [24794] by adele

Reviewed by Sam.

Breaking up fast/replaced/width100percent.html into separate tests for the different elements.

  • fast/replaced/width100percent-button-expected.checksum: Added.
  • fast/replaced/width100percent-button-expected.png: Added.
  • fast/replaced/width100percent-button-expected.txt: Added.
  • fast/replaced/width100percent-button.html: Added.
  • fast/replaced/width100percent-checkbox-expected.checksum: Added.
  • fast/replaced/width100percent-checkbox-expected.png: Added.
  • fast/replaced/width100percent-checkbox-expected.txt: Added.
  • fast/replaced/width100percent-checkbox.html: Added.
  • fast/replaced/width100percent-image-expected.checksum: Added.
  • fast/replaced/width100percent-image-expected.png: Added.
  • fast/replaced/width100percent-image-expected.txt: Added.
  • fast/replaced/width100percent-image.html: Added.
  • fast/replaced/width100percent-menulist-expected.checksum: Added.
  • fast/replaced/width100percent-menulist-expected.png: Added.
  • fast/replaced/width100percent-menulist-expected.txt: Added.
  • fast/replaced/width100percent-menulist.html: Added.
  • fast/replaced/width100percent-radio-expected.checksum: Added.
  • fast/replaced/width100percent-radio-expected.png: Added.
  • fast/replaced/width100percent-radio-expected.txt: Added.
  • fast/replaced/width100percent-radio.html: Added.
  • fast/replaced/width100percent-textarea-expected.checksum: Added.
  • fast/replaced/width100percent-textarea-expected.png: Added.
  • fast/replaced/width100percent-textarea-expected.txt: Added.
  • fast/replaced/width100percent-textarea.html: Added.
  • fast/replaced/width100percent-textfield-expected.checksum: Added.
  • fast/replaced/width100percent-textfield-expected.png: Added.
  • fast/replaced/width100percent-textfield-expected.txt: Added.
  • fast/replaced/width100percent-textfield.html: Added.
  • fast/replaced/width100percent.html: Removed.
  • fast/replaced/width100percent-expected.checksum: Removed.
  • fast/replaced/width100percent-expected.png: Removed.
  • fast/replaced/width100percent-expected.txt: Removed.
10:58 Changeset [24793] by weinig

LayoutTests:

Reviewed by Mitz.

Updated tests for http://bugs.webkit.org/show_bug.cgi?id=14825
Non-integer hsl() colours are ignored

  • fast/css/hsl-color-expected.checksum:
  • fast/css/hsl-color-expected.png:
  • fast/css/hsl-color-expected.txt:
  • fast/css/hsl-color.html: Add non-integer hue tests
  • fast/css/hsla-color-expected.checksum:
  • fast/css/hsla-color-expected.png:
  • fast/css/hsla-color-expected.txt:
  • fast/css/hsla-color.html: Add non-integer hue tests

WebCore:

Reviewed by Mitz.

Fix for http://bugs.webkit.org/show_bug.cgi?id=14825
Non-integer hsl() colours are ignored

  • css/CSSParser.cpp: (WebCore::CSSParser::parseHSLParameters): Parse hue as a Number, not an Integer.
01:32 BuildingQtOnWindows edited by hausmann@webkit.org
(diff)

07/30/07:

20:54 Changeset [24792] by justing

LayoutTests:

Reviewed by Darin.


<rdar://problem/5369009>
Crash due to infinite recursion in moveParagraphs deleting line from a Mail message

  • editing/deleting/5369009-expected.checksum: Added.
  • editing/deleting/5369009-expected.png: Added.
  • editing/deleting/5369009-expected.txt: Added.
  • editing/deleting/5369009.html: Added.

WebCore:

Reviewed by Darin.

<rdar://problem/5369009> Crash due to infinite recursion in moveParagraphs on delete


After the delete, we moved content into the previous block, and a style
rule turned a style span in the moved content into a block, throwing
moveParagraphs into infinite recursion, as it continually tried and failed
to get the style span into the same paragraph as the content just before it.


Added a method to ReplaceSelectionCommand to keep inserted style spans
from turning into blocks because of style rules. Will add code to prevent
other kinds of content from changing appearance because of style rules,
post-Leopard (5371536). Also added a bool to ReplaceSelectionCommand's
constructor to guard against infinite recursion.

  • editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::moveParagraphs):
  • editing/JSEditor.cpp:
  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::ReplaceSelectionCommand): (WebCore::ReplaceSelectionCommand::negateStyleRulesThatEffectAppearance): (WebCore::ReplaceSelectionCommand::doApply):
  • editing/ReplaceSelectionCommand.h:
20:44 Changeset [24791] by treat

Fix build.

20:18 Changeset [24790] by sullivan

Reviewed by Darin


  • fixed <rdar://problem/5216176> Need WebKit SPI to allow clients using embedded WebViews to avoid clipping ends of some printed pages


This provides Mail, and other clients that print views that embed WebViews, a way to ensure that
the HTML is laid out for printing before pagination occurs.

  • WebView/WebHTMLViewPrivate.h: new SPI method -_layoutForPrinting


  • WebView/WebHTMLView.mm: (-[WebHTMLView _web_setPrintingModeRecursiveAndAdjustViewSize]): new method, just like existing _web_setPrintingModeRecursive except passes YES for adjustViewSize (-[WebHTMLView _layoutForPrinting]): new SPI method, sets printing mode temporarily to adjust the view size for printing (-[NSView _web_setPrintingModeRecursiveAndAdjustViewSize]): new helper method to do the recursion
18:05 JS Core Garbage Collector edited by john@johnmoe.com
(diff)
17:48 Changeset [24789] by andersca

Reviewed by Brady.

<rdar://problem/5371515>
BitmapImage::getTIFFRepresentation() bug results in favicons not working at many sites

Don't bail out if a single frame is invalid.


  • platform/graphics/mac/ImageMac.mm: (WebCore::BitmapImage::getTIFFRepresentation):
17:29 Changeset [24788] by darin

Fix a bug number on an old check-in.

17:09 Changeset [24787] by adele

Reviewed by Darin.

Fix for <rdar://problem/5367919> A crash occurs at WebCore::Frame::isFrameSet() when attempting to print a iframe before it loads at http://www.monster.com/

  • WebView/WebHTMLView.mm: (-[WebHTMLView knowsPageRange:]): Nil check for frame.
16:56 Changeset [24786] by adele

Reviewed by Darin.

Fix for <rdar://problem/5274937> Safari 3 breaks Missing Sync's WebKit code by crashing

  • loader/MainResourceLoader.cpp: (WebCore::MainResourceLoader::continueAfterContentPolicy): Nil check for the FrameLoader.
16:24 Changeset [24785] by andersca

WebCore:

Reviewed by Darin.

<rdar://problem/5370710>
REGRESSION: After switching from Bookmark view, the Find Banner won't appear while displaying a PDF file

Add a hasHTMLView to FrameLoaderClient. This is only useful for clients who can show different views for different
MIME types and URL schemes.


  • loader/FrameLoader.cpp: (WebCore::FrameLoader::transitionToCommitted): Call delegate methods here if the current view is not a HTML view.


  • loader/FrameLoaderClient.h: (WebCore::FrameLoaderClient::hasHTMLView):

WebKit:

Reviewed by Darin.

<rdar://problem/5370710>
REGRESSION: After switching from Bookmark view, the Find Banner won't appear while displaying a PDF file

Implement hasHTMLView.


  • WebCoreSupport/WebFrameLoaderClient.h:
  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::hasHTMLView):
16:05 Changeset [24784] by darin

Hatcher!

16:05 Changeset [24783] by darin

Reviewed by Tim Matcher.

  • DumpRenderTree/DumpRenderTree.m: (dump): Fix dumping for documents that include null characters. This turned out not to be needed for the test case that motivated me to do it, but it's nice to have this for the future.
15:29 Changeset [24782] by justing

Fixed a typo in my last checkin.

11:29 Changeset [24781] by weinig

LayoutTests:

Reviewed by Maciej.

Update tests for <rdar://problem/5363896>
REGRESSION: Setting document.domain does not override port-based cross-frame security checks -- breaks SAP NetWeaver

  • Also changes frame access test to use shared resource iframes.
  • http/tests/security/cross-frame-access-port-expected.txt:
  • http/tests/security/cross-frame-access-port-explicit-domain-expected.txt:
  • http/tests/security/cross-frame-access-port-explicit-domain.html:
  • http/tests/security/cross-frame-access-port.html:
  • http/tests/security/cross-frame-access-protocol-expected.txt:
  • http/tests/security/cross-frame-access-protocol-explicit-domain-expected.txt:
  • http/tests/security/cross-frame-access-protocol-explicit-domain.html:
  • http/tests/security/cross-frame-access-protocol.html:
  • http/tests/security/resources/cross-frame-iframe-for-port-explicit-domain-test.html: Removed.
  • http/tests/security/resources/cross-frame-iframe-for-port-test.html: Removed.
  • http/tests/security/resources/cross-frame-iframe-for-protocol-explicit-domain-test.html: Removed.
  • http/tests/security/resources/cross-frame-iframe-for-protocol-test.html: Removed.
  • http/tests/security/resources/cross-frame-iframe-with-explicit-domain-set.html: Added.
  • http/tests/security/resources/cross-frame-iframe.html: Added.

WebCore:

Reviewed by Maciej.

Fix for <rdar://problem/5363896>
REGRESSION: Setting document.domain does not override port-based cross-frame security checks -- breaks SAP NetWeaver

Updates the results for:

http/tests/security/cross-frame-access-port-explicit-domain.html
http/tests/security/cross-frame-access-protocol-explicit-domain.html

  • bindings/js/kjs_window.cpp: (KJS::createWindow): Use the new setDomainInternal method. (KJS::Window::isSafeScript): Don't set check the port or protocol if both documents have explicitly set document.domain in the DOM and the those domains are equal.
  • dom/Document.cpp: (WebCore::Document::Document): (WebCore::Document::setDomain): Remove force parameter and change all calls that relied on it use the new setDomainInternal method which does the same thing. (WebCore::Document::setDomainInternal):
  • dom/Document.h: Add new m_domainWasSetInDOM variable and accessor. (WebCore::Document::domainWasSetInDOM):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::checkCallImplicitClose): Use the new setDomainInternal method.
11:25 Changeset [24780] by justing

WebCore:

Reviewed by Tristan.


<rdar://problem/5098931> Attachments are lost when they are moved into a ToDo after a delete


Added an editor client method for asking whether or not it is
OK to merge content after a delete.

  • bridge/EditorClient.h:
  • editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::mergeParagraphs): Ask if the merge is allowed.
  • platform/gdk/EditorClientGdk.cpp: (WebCore::EditorClientGdk::shouldMoveRangeAfterDelete): Added a method stub.
  • platform/gdk/EditorClientGdk.h:
  • platform/graphics/svg/SVGImageEmptyClients.h: (WebCore::SVGEmptyEditorClient::shouldMoveRangeAfterDelete): Ditto.

WebKit:

Reviewed by Tristan.


<rdar://problem/5098931> Attachments are lost when they are moved into a ToDo after a delete


Mail needs to be asked if it is OK to do the content movement that happens
after a deleting in a situation like this one:
<div contenteditable="plaintext-only">foo</div><div>bar</div>
so that they can prevent the move or so that they can save content that
will be stripped by the move. This could have been done with shouldInsertNode
and a new WebViewInsertAction for "moves", but WebKit clients like Mail and
DashCode think that a shouldInsert* means that the user pasted something and
perform actions only appropriate for pastes. This change is less risky because
it won't require those clients to change their code.

  • DefaultDelegates/WebDefaultEditingDelegate.m: (-[WebDefaultEditingDelegate webView:shouldMoveRangeAfterDelete:replacingRange:]):
  • WebCoreSupport/WebEditorClient.h:
  • WebCoreSupport/WebEditorClient.mm: (WebEditorClient::shouldMoveRangeAfterDelete):
  • WebView/WebEditingDelegatePrivate.h:

win:

Reviewed by Tristan.


<rdar://problem/5098931> Attachments are lost when they are moved into a ToDo after a delete

  • WebEditorClient.cpp: (WebEditorClient::shouldMoveRangeAfterDelete): Method stub.
  • WebEditorClient.h:
11:15 Changeset [24779] by andersca

LayoutTests:

Reviewed by Anders.

<rdar://problem/5352526>
http://bugs.webkit.org/show_bug.cgi?id=14704
REGRESSION: sync XMLHttpRequest.send() raises an exception for failed authentication

  • http/tests/xmlhttprequest/failed-auth-expected.txt: Added.
  • http/tests/xmlhttprequest/failed-auth.html: Added. This test has other cases from bug 13075 commented out, as they still fail.
  • http/tests/xmlhttprequest/xmlhttprequest-missing-file-exception-expected.txt: Added.
  • http/tests/xmlhttprequest/xmlhttprequest-missing-file-exception.html: Added. This test already passed, but I think it's useful to cover this case, too.

WebCore:

Reviewed by Anders.

<rdar://problem/5352526>
http://bugs.webkit.org/show_bug.cgi?id=14704
REGRESSION: sync XMLHttpRequest.send() raises an exception for failed authentication

Test: http/tests/xmlhttprequest/failed-auth.html

  • platform/network/mac/ResourceHandleMac.mm: (WebCore::ResourceHandle::loadResourceSynchronously): Make the fake response better match the real one in case of failed authentication. Unfortunately, NSURLConnection doesn't give us the real response.
  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::send): If the response has an HTTP code in it, then the error wasn't a network one, and an exception shouldn't be raised.
10:48 Changeset [24778] by tristan

Updating layout test results with new expected results.

10:28 Changeset [24777] by andersca

Reviewed by Darin.

<rdar://problem/4942372> REGRESSION: Anchor links in Mail don't work anymore


Make it possible for KURL to handle addig fragments to non-hierarchical URLs
such as mailto: and cid:.


  • platform/KURL.cpp: (WebCore::KURL::init): If the base URL is not hierarchical but the relative URL is a fragment, then allow parsing it.


(WebCore::KURL::parse):
If the URL is not hierarchical, set the fragment start and end positions correctly.

09:10 Changeset [24776] by hausmann

On Windows make sure the dll also ends up in $$OUTPUT_DIR/bin, so that QtLauncher can find it.

09:10 Changeset [24775] by hausmann

Link QtLauncher into $$OUTPUT_DIR/bin

08:30 Changeset [24774] by hausmann

Removed the BUILDING_QT ifdef in JSStringRef.h and changed UChar for the Qt build to use wchar_t on Windows.

08:30 Changeset [24773] by hausmann

Don't link against libxml/libxslt unless we really need it.

06:57 Changeset [24772] by zimmermann

Reviewed by Oliver & Rob.

Enable SVG textPath selection. Actually build chunks for them instead of
ignoring them - was easier than I thought. Respect per-character transformation
when calculating the chunk boundaries for the selection rectangles. Also fixes
selection rect drawing of rotated characters.

Next thing todo is to unify the glyph size calculations, as per-glyph transformations
are not respected when calculating flow box sizes in SVGRootInlineBox.

06:51 BuildingQtOnWindows edited by hausmann@webkit.org
(diff)
06:46 BuildingQtOnWindows edited by hausmann@webkit.org
(diff)
06:39 BuildingQtOnWindows edited by hausmann@webkit.org
(diff)
06:34 BuildingQtOnWindows edited by hausmann@webkit.org
(diff)
06:31 BuildingQtOnWindows edited by hausmann@webkit.org
(diff)

07/29/07:

23:33 Changeset [24771] by tristan

Reviewed by Oliver Hunt.

<rdar://problem/5369282> REGRESSION: DOMHTMLInput throwing exception due to missing _isEdited method

  • bindings/objc/DOMHTML.mm: (-[DOMHTMLInputElement _isEdited]): (-[DOMHTMLTextAreaElement _isEdited]):
  • bindings/objc/DOMPrivate.h: Renamed _isUserEdited to _isEdited.
21:46 Changeset [24770] by alp

2007-07-30 Diego Escalante Urrelo <diegoe@gnome.org>

Reviewed by Adam.

http://bugs.webkit.org/show_bug.cgi?id=14806
Implement can_go_backward and can_go_forward in webkitgtkpage.cpp

  • gtk/Api/webkitgtkpage.cpp: Implement webkit_gtk_page_can_go_backward() and webkit_gtk_page_can_go_forward() functions.
21:07 Changeset [24769] by alp

2007-07-30 Diego Escalante Urrelo <diegoe@gnome.org>

Reviewed by Alp Toker.

http://bugs.webkit.org/show_bug.cgi?id=14810
Bad n_params on load_finished's g_signal_new in webkitgtkpage.cpp

This is the proper fix for #14810.

  • gtk/Api/webkitgtkframe.cpp:
  • gtk/Api/webkitgtkpage.cpp: Fix the marshaller type of the "load_finished" signal and correct the class_offset parameters.
20:12 Changeset [24768] by alp

2007-07-29 Diego Escalante Urrelo <diegoe@gnome.org>

Reviewed by Alp Toker.

http://bugs.webkit.org/show_bug.cgi?id=14810
Bad n_params on load_finished's g_signal_new in webkitgtkpage.cpp

  • gtk/Api/webkitgtkpage.cpp: Use a correct n_params value and the corresponding types for them in the "load_finished" signal.
19:40 Changeset [24767] by treat

Paint and scrolling performance fixes and improvements.

16:25 Changeset [24766] by adele

WebCore:

Reviewed by John.

WebCore part of fix for <rdar://problem/5102522> REGRESSION: Can't tab to webview that doesn't have editable content (affects Safari, preview pane in Mail)

  • WebCore.exp:
  • page/FocusController.cpp: (WebCore::FocusController::setInitialFocus): Added. (WebCore::FocusController::advanceFocus): Don't give the focus back to the chrome if this is the initial focus.
  • page/FocusController.h:

WebKit:

Reviewed by John.

WebKit part of fix for <rdar://problem/5102522> REGRESSION: Can't tab to webview that doesn't have editable content

  • WebView/WebHTMLView.mm: (-[WebHTMLView becomeFirstResponder]): Call new setInitialFocus method instead of advanceFocus.
14:51 Changeset [24765] by zecke

2007-07-29 Brian Mastenbrook <brian@mastenbrook.net>

Build fix for WebKit/Gtk to compile on Fedora 7 as described in
http://bugs.webkit.org/show_bug.cgi?id=14557

  • WebKit.pri: cairo-ft is used by the WebKit/Gtk port
08:18 Changeset [24764] by treat

Respect the margins when creating frames.
Set the scroll mode to always off like we did before the rendered
frames patch and the other ports do now.

07:42 Changeset [24763] by treat

WebCore::Widget::setParent should be virtual.

07:22 Changeset [24762] by treat

We actually pass the mac versions expected results here.

07/28/07:

19:34 Changeset [24761] by weinig

Reviewed by Mark Rowe.

  • Rename test for clarity and consistency.
  • Move resource from ssl/resources to security/resources as it is only being used by the security tests.
  • http/tests/security/cross-frame-access-port-domain-expected.txt: Removed.
  • http/tests/security/cross-frame-access-port-domain.html: Removed.
  • http/tests/security/cross-frame-access-port-explicit-domain-expected.txt: Copied from http/tests/security/cross-frame-access-port-domain-expected.txt.
  • http/tests/security/cross-frame-access-port-explicit-domain.html: Copied from http/tests/security/cross-frame-access-port-domain.html.
  • http/tests/security/cross-frame-access-protocol-expected.txt:
  • http/tests/security/cross-frame-access-protocol.html:
  • http/tests/security/resources/cross-frame-iframe-for-port-domain-test.html: Removed.
  • http/tests/security/resources/cross-frame-iframe-for-port-explicit-domain-test.html: Copied from http/tests/security/resources/cross-frame-iframe-for-port-domain-test.html.
  • http/tests/security/resources/cross-frame-iframe-for-protocol-test.html: Copied from http/tests/ssl/resources/cross-frame-access-protocol-iframe.html.
  • http/tests/ssl/resources: Removed.
  • http/tests/ssl/resources/cross-frame-access-protocol-iframe.html: Removed.
19:15 Changeset [24760] by weinig

Reviewed by Oliver Hunt.

  • Factor access testing code used in protocol and port tests to cross-frame-access.js.
  • Add cross-frame-iframe-for-protocol-explicit-domain-test.html which is parallel test to cross-frame-iframe-for-port-domain-test.html but for different protocols.
  • http/tests/security/cross-frame-access-port-domain-expected.txt:
  • http/tests/security/cross-frame-access-port-domain.html:
  • http/tests/security/cross-frame-access-port.html:
  • http/tests/security/cross-frame-access-protocol-explicit-domain-expected.txt: Added.
  • http/tests/security/cross-frame-access-protocol-explicit-domain.html: Added.
  • http/tests/security/cross-frame-access-protocol.html:
  • http/tests/security/resources/cross-frame-access.js:
  • http/tests/security/resources/cross-frame-iframe-for-port-domain-test.html:
  • http/tests/security/resources/cross-frame-iframe-for-port-test.html:
  • http/tests/security/resources/cross-frame-iframe-for-protocol-explicit-domain-test.html: Added.
19:02 Changeset [24759] by alp

Fix up reviewer name typo in ChangeLog

18:30 Changeset [24758] by alp

2007-07-29 Alp Toker <alp.toker@collabora.co.uk>

Reviewed by Brian Dash.

http://bugs.webkit.org/show_bug.cgi?id=14711 (revisited)
RenderThemeGdk's buttons are state-agnostic (pressed, hovered)

Generalize shadow state into a function.
use a GtkHBox to work around a theme bug (thanks to Nigel Tao).

  • platform/gdk/RenderThemeGdk.cpp: (WebCore::RenderThemeGdk::determineShadow): (WebCore::RenderThemeGdk::paintCheckbox): (WebCore::RenderThemeGdk::paintRadio): (WebCore::RenderThemeGdk::paintButton): (WebCore::RenderThemeGdk::gtkWindowContainer):
  • platform/gdk/RenderThemeGdk.h:
18:18 Changeset [24757] by bdash

2007-07-28 Mark Rowe <mrowe@apple.com>

Reviewed by Darin and John.

<rdar://problem/5229705> Thai characters render differently between leopard and tiger

Add Leopard-specific test results for tests containing Thai characters.

  • mac/leopard/Skipped:
  • mac/leopard/fast/text: Added.
  • mac/leopard/fast/text/atsui-multiple-renderers-expected.txt: Added.
  • mac/leopard/fast/text/international: Added.
  • mac/leopard/fast/text/international/thai-line-breaks-expected.txt: Added.
18:15 Changeset [24756] by bdash

2007-07-29 Mark Rowe <mrowe@apple.com>

Reviewed by Oliver.

<rdar://problem/5228270> SVG test failures due to different metrics for scaled text blocks

Add Leopard-specific results for three SVG tests.

  • mac/leopard/Skipped:
  • mac/leopard/svg: Added.
  • mac/leopard/svg/hixie: Added.
  • mac/leopard/svg/hixie/text: Added.
  • mac/leopard/svg/hixie/text/003-expected.txt: Added.
  • mac/leopard/svg/hixie/text/003a-expected.txt: Added.
  • mac/leopard/svg/hixie/viewbox: Added.
  • mac/leopard/svg/hixie/viewbox/preserveAspectRatio: Added.
  • mac/leopard/svg/hixie/viewbox/preserveAspectRatio/002-expected.txt: Added.
10:37 Changeset [24755] by bdash

Versioning.

10:37 Changeset [24754] by bdash

Merge r24658.

Reviewed by Oliver.

<rdar://problem/5362783>
In Mail, a crash occurs at WTF::fastMalloc() when attempting to create a new HTML message from http://nytimes.com/

If the cache is disabled and there is already a resource with the given URL, return it instead of creating a new one.

  • loader/DocLoader.cpp: (WebCore::DocLoader::requestResource):
10:37 Changeset [24753] by bdash

Merge r24514.

Reviewed by Anders.

  • improve the fix for <rdar://problem/5298816> Crash redirecting message, in -[SharedBufferData initWithSharedBuffer:]; fixes a crash seen on the buildbot
  • loader/DocLoader.cpp: (WebCore::DocLoader::~DocLoader): Call setDocLoader(0) on all the resources so they don't try to use a stale DocLoader pointer later.
10:36 Changeset [24752] by bdash

Merge r24500.

Reviewed by Darin.

If the style sheet requested is a user style sheet, don't store it in the DocLoader's resource map;
user style sheets can outlive their doc loaders.

  • loader/DocLoader.cpp: (WebCore::DocLoader::requestCSSStyleSheet):
10:36 Changeset [24751] by bdash

Merge r24492.

Reviewed by Darin.

<rdar://problem/5298816> Crash redirecting message, in -[SharedBufferData initWithSharedBuffer:]

Have cached resources keep track of what DocLoader they belong to. This is so that they can remove themselves
from the DocLoader's resource map when they're destroyed when the cache is disabled.

  • html/HTMLImageLoader.cpp: (WebCore::HTMLImageLoader::updateFromElement):
  • loader/Cache.cpp: (WebCore::Cache::requestResource):
  • loader/CachedResource.cpp: (WebCore::CachedResource::CachedResource): (WebCore::CachedResource::~CachedResource):
  • loader/CachedResource.h: (WebCore::CachedResource::setDocLoader):
10:20 Changeset [24750] by bdash

New tag.

07:22 BuildingGdk edited by sylvain.pasche@gmail.com
link to BuildingOnLinux (diff)
07:20 BuildingOnLinux edited by sylvain.pasche@gmail.com
Ubuntu package requirements (diff)
Note: See TracTimeline for information about the timeline view.